Sitecheck

[Stefan]

Hallo,
bitte Sitecheck:
www.shirtmagazin.de
Habe erst vor 3 Wochen mit Datenbanken angefangen, dürften also
einige Kritikpunkte zusammen kommen.
Bis jetzt aufgefallen:

• Fehlendes Logo
• Mageres Layout
• Keine Cookie-Sperre
  Gruß

[Hansel]

www.shirtmagazin.de

Der längste Text auf den Seiten ist der Nutzungsbedingungsbandwurm über eineinhalb Seiten. Denk mal drüber nach.

[Tom]

Hello,

Du hast da noch wahnsinnige Fehler in der Programmlogik

Uploads sollte man erst überürpfen, bevor man sie weiterverarbeitet.

Warning: getimagesize() [function.getimagesize]: Read error! in /mnt/ja2/03/347/00000019/htdocs/create_new.php on line 48

Harzliche Grüße vom Berg
http://www.annerschbarrich.de

Tom

[wahsaga]

hi,

Du hast da noch wahnsinnige Fehler in der Programmlogik

Uploads sollte man erst überürpfen, bevor man sie weiterverarbeitet.

Und andere Werte, die vom Client kommen, genauso - bspw. die Werte, die für die Checkbuttons bei der Abstimmung übergeben werden, sollten auf den definierten Wertebereich geprüft werden.

Wenn man jetzt nämlich mal unter den "Topshirts" nachschaut, haben wir da die ersten beiden mit (aktuell)
Platz 1  Punkte: 13703.42 (73 Votes)
Platz 2  Punkte: 24.85 (33 Votes)

Hoppla, für einen Abstimmungsbereich von 1 bis 10 Punkten sind das jetzt aber ziemlich hohe Durchschnittswerte geworden, hm?

gruß,
wahsaga

[Robert Bienert]

Moin!

www.shirtmagazin.de

Aha, wusste gar nicht, dass es ein „self Schirt“ gibt (da stimmt übrigens der Link nicht).

Habe erst vor 3 Wochen mit Datenbanken angefangen, dürften also einige Kritikpunkte zusammen kommen.

Und wie lange programmierst du schon in PHP, seit wann setzt du dich mit HTML auseinander? Das sind auch wichtige Fragen.

Bis jetzt aufgefallen:

• Fehlendes Logo

Aufgefallen?

• Mageres Layout

Schön schlicht.

• Keine Cookie-Sperre

Was ist denn das?

Was mir noch aufgefallen ist:

• Die Webseite besteht nur aus einer großen Tabelle, wie wärs denn mit so Dingen wie einer Überschrift?
 • Beim Hochladen eines Shirts hast du gleich ein ganzes Fass an XSS-Lücken aufgemacht, d.h. man kann in jedes Feld HTML- und Javascript-Code eingeben.
 • Die Bewertung eines Shirts wird nich hinreichend überprüft: Mit Erweiterungen wie Firebug kann ich die Formularwerte verändern und somit auch mehr als 10 Punkte vergeben.

HTH, Robert