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