Wie findet ihr diesen Code?

[LeGaN]

Hi,

warum verwendest du

$config[$key]

wenn du in der foreach-Schleife

$key=>$entry

hast. Dann kannst du gleich

$entry

anstatt

$config[$key]

schreiben.

[ghgh]

Nö.

PHP arbeitet normalerweise mit Kopien und nicht mit Referenzen

[hustbaer]

1. verfrickelt ist kein Wort
2. der Code ist zum Kotzen
3. der Code ist fehlerhaft (nicht "threadsafe")
   ...

[Headhunter]

Hallo,

den Code wuerde ich auch als frickelig bezeichnen. Trifft zwar auf 99% aller PHP Codes zu, auf den geposteten aber besonders

Das Speichern von Daten ohne Datenbank ist nicht sauber realisiert worden. Wie gesagt - nicht threadsave. Da muessen nur mal zwei User in kurzen zeitlichen Abstaenden darauf zugreifen und die Datei wird irgendwie ueberschrieben.
Zusatzlich ist deine Schleife irgendwie falsch .. ich denke damit kann man u.U. Daten falsch beschreiben. Habe mir das aber nicht im Detail angeschaut.

Auch das Benutzen von "|" als Delimiter ist boeeeehse... da muss nur mal einer ne Pipe reinschummeln und dein ganzer Datensatz ist schrott.

[letters from iwo]

Kann man in PHP überhaupt nicht-frickeligen Code schreiben? Bei diesen ganzen $ allein wird mir schon übel

[sdfdsfds]

@Threadersteller: net böse sein, aber der code ist kacke

[DEvent]

Es würde doch reichen die Datei zu locken, damit kein anderer Benutzer auf die Datei zugreifen kann, wenn ein anderer die Datei geöffnet hat.

[LeGaN]

ghgh schrieb:

Nö.

PHP arbeitet normalerweise mit Kopien und nicht mit Referenzen

Das ist klar, aber für z.B.

explode("|", $config[$key]);

kann man gleich

$entry

schreiben.

[Programmierer des Scripts]

Mach dir mal über das evtl. unset zuvor Gedanken!

Außerdem würde ich das schon alleine wegen der Einheitlichkeit nicht schreiben!

Und der Code ist nicht frickelig, ihr seid nur nicht genial genug um ihn zu verstehen. Danke.

[sdfdsfdsf]

Hat dir das PHP zu Kopf geschlagen oder was?