phpbar.de logo

Mailinglisten-Archive

[php] Fehler im Script

[php] Fehler im Script

Jan Heppe list1 at afvm.de
Don Dez 9 17:03:02 CET 2004


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