Sven Rautenberg: 2 schleifen ineinander (thread übernahme)

Beitrag lesen

Moin!

ist das script in ordnung ???:

Wohl nicht, wenn es Fehler ausgibt.

Das Problem dürfte darin bestehen, dass du die beiden Directory-Einträge "." und ".." nicht rausfilterst.

Ganz dumm gefragt: Du willst _Files_ aus dem Verzeichnis auslesen. Warum prüfst du dann mit

if (!is_dir($datei)){

ob der Verzeichniseintrag _kein_ Directory ist? Warum prüfst du nicht einfach mit
if (is_file($datei)){
ob der Verzeichniseintrag eine _Datei_ ist - die du dann lesen kannst. Wahlweise kannst du $datei auch auf Identität mit "." und ".." prüfen und diese Einträge dann überspringen.

Ansonsten: Dein Code ist dermaßen mieserabel zu lesen, dass es absolut keine Freude ist. Schon mal was von "Einrücken" gehört? Schon mal was von "Schleifen und IF-Konstrukte _immer_ klammern, auch wenn technisch nicht notwendig!" gehört? Man kriegt dafür zwar nicht mehr Geld, und Arbeit macht es auch - es erleichtert aber die Übersicht über das Skript ganz enorm und verhindert Fehler. Halte dich mal daran und präsentiere bitte nicht wieder so eine Bleiwüste.

- Sven Rautenberg