Performance MySQL updaten

[j4nk3y]

Hey,

ist das zweite

include('db_connect_function.php');

vor

foreach($data as $id => $set){

nicht überflüssig? Ich meine, die Datei wird doch ganz oben bereits eingebunden?

Das kommt darauf an, ob darin sofort ausführbarer Code drinsteht oder nur beispielsweise Funktionen.

Anscheinend wird darin die Datenbankverbindung aufgebaut. Guter Stil ist das jedenfalls nicht, denn man sieht nicht, was der Aufruf bewirkt. Besser wäre, wenn der Aufbau in einer Funktion geschähe, die man aufruft und das DB-Handle bekommt, à la $db = open_database_connection();. Das würde durch den sprechenden Namen der Funktion besser verdeutlichen, was an der Stelle geschieht.

Ja das hab ich mir auch schon überlegt, aber als ich das geschrieben hab war ich noch nicht so weit (gemeint sind Programmierkentnisse).

Gruß
Jo