Mailinglisten-Archive |
Hallo, als erstes solltes Du mal was an Deiner Struktur ändern, sonst wirst Du immer wieder Fehler einbauen und dann vergeblich suchen. Ein Beispiel: --code.Start-- $pic=mysql_query("SELECT file FROM results WHERE id = '$_GET[edit]'"); while($zeile=mysql_fetch_array($pic)): $file=$zeile["file"]; If ($file != NULL) echo " <table> <tr> <td width=342><a href='$file' target='_blank'><img src='$file' width='120'></a></td> </tr> </table>"; endwhile; --code.End-- Du liest hier ein Feld aus einer Zeile (ich nehme mal an die ID ist eindeutig) wozu brauchst Du "while"? Besser: alle 3 Felder mit einem Select auslesen, dann sind auch Schleifen für die Bearbeitung oder Ausgabe sinnvoll. Als zweites: legst du bei deinem Uploadformularen, die natürlich wieder alle 3 einzeln (Fehlerquelle!!!!) in deinem Code zu suchen sind folgende Get-Parameter fest: file1 edit=edit1&id=$_GET[edit] file2 edit=$_GET[file2] file3 edit=$_GET[file3] nebenbei werden auch hier wieder die daten einzeln aus der DB gelesen, während Du aber im Formular selbst Dich auf '{$row['file3']}' beziehst. Unabhängig davon, dass Du hier unterschiedliche Parameter übergibst, solltest Du diese besser in ein Hidden-Feld packen.... Mein Vorschlag: - schreib Dir auf, was genau dein Script machen soll - versuche Die eigentlichen Aufgaben vom HTML-Code zu trennen (z.B. Daten auslesen, bearbeiten, etc dann erst Erzeugen des HTMLs) - baue Dir sinnvolle Teilabschnitte, z.B. Funktionen - ... da gibt es sicher noch mehr Dinge viel Erfolg Jan
php::bar PHP Wiki - Listenarchive