Upload macht in einem best. Fall (IE 6) Probleme...

[Der Martin]

Hallo,

if ( ! (
( ( ( $_FILES [ 'grafik' ] [ 'type' ] == 'image/jpeg' ) || ( $_FILES [ 'grafik' ] [ 'type' ] == 'image/pjpeg' ) ) && ( $grafik_typ == 2 ) )
                     ||
( ( $_FILES [ 'grafik' ] [ 'type' ] == 'image/gif' ) && ( $grafik_typ == 1 ) )
                  )
         )

Wenn ihr als Programmierer nach Bytes bezahlt werdet, dann habe ich einen Tipp für euch: Setzt auch in die konstanten Stringliterale zu Anfang und Ende noch Leerzeichen, und fügt die trim()-Funktion drum herum ein.

netter Vorschlag, das würde den Zeichensalat vielleicht noch etwas beleben.

Ansonsten solltet ihr euch mal ein paar Styleguides ansehen, wie man Codes lesbarer macht ...

Ja. Günstig gesetzte Leerzeichen und/oder Zeilenumbrüche können die Lesbarkeit von Quelltexten kolossal verbessern. Hier wird aber (auch meiner Ansicht nach) durch Übertreibung genau das Gegenteil erreicht.

Ciao,
 Martin

PS: Michi, du solltest das Ergebnis von getimagesize() noch in deine Bewertung, ob eine gültige Grafikdatei vorliegt, einbeziehen. Dann hat auch virus.exe keine Chance mehr. ;-)