Coding Kritiken und Anregungen bitte!

Asmodis

Pain Elemental
1. eindruck, hat bei mir EWIG zum laden gebraucht.
Outfit gefaellt mir, code soll jemand beurteilen, der davon ahnung hat
 

Asmodis

Pain Elemental
Hatte sich auf beide bezogen, aber jetzt laed er schneller, lag vlt. an meinem pc. Ich sehe grade dass du frames verwendest. das ist nicht gut...

*edit* und bei link 1 unter links :gaga: kommt das
Code:
Warning: main(content/links.txt): failed to open stream: No such file or directory in /Platz/www/docs/user/saatkamp/home.php on line 55

Warning: main(content/links.txt): failed to open stream: No such file or directory in /Platz/www/docs/user/saatkamp/home.php on line 55

Warning: main(): Failed opening 'content/links.txt' for inclusion (include_path='.:/usr/share/php') in /Platz/www/docs/user/saatkamp/home.php on line 55
aber hast ja selbst gesagt, dass es noch "development" ist, aber du solltest versuchen, das ganze ohne frames zu realisieren
 

LordSavage

Revenant
die fehler kommen daher, dass da einfach noch keine dateien liegen, könnte da ne leere txt hinpacken und der fehler wäre weg. is mir aber zu aufwendig :D.

hab auch schon darüber nachgedacht, dass statt mit frames einfach eine tabelle drum zu machen. die sind ja nur da, damit das ganze zentriert ist. das zweite ist btw. ohne frames.
 

LordSavage

Revenant
öhm, ich glaube du hast da was falsch verstanden :).
natürlich nimmt man da tabellen. das kannst du mir glauben. die einzige wirkliche alternative ist das ganze per css zu positionieren aber naja. darüber will ich jetzt mal nicht reden.
 
jo die seite is verdammt groß!
solltest auf keinenfall die PNG dateien benutzen, sondern statt dessen lieber Jpegs oder gifs (musste bei GIFs aufpassen, hatte schonmal bilder gehabt, die waren in GIF größer als in JPG

hgestalterisch ganz nett, aber is halt inner dateimenge zu groß (lerne die ISDN nutzer lieben)

http://www.webmasterplan.com/cgi-local/loa...gn.com/&popup=1

http://www.webmasterplan.com/cgi-local/loa...m/user/&popup=1

schulnote 6

meine seite:

http://www.webmasterplan.com/cgi-local/loa...loch.de&popup=1

kannste mal vergleichen, hab fast nur gifs genutzt...
 

LordSavage

Revenant
hmm, seltsame fehlermeldungen auf validator.w3.org

also topmargin, leftmargin, rightmargin etc. sind vllt keine standart html befehle aber aber sie funktionieren in allen browsern erstaunlich gut :D.

edit: background klappt auch sehr gut wie ihr seht und bei jedem bild alt anzugeben ist auch käse, weil dann sobald man die maus darüber bewegt, der blöde schriftzug kommt.
die einzig hilfreiche meldung ist eigentlich, dass kein document typ deklariert ist.

edit2: @Manhunter: was kümmern mich isdn user ;). entweder perfekte bildquali und etwas warten oder gar nix :D
 

McGunn0r

Chthon
Originally posted by LordSavage@26.08.05, 14:56
edit2: @Manhunter: was kümmern mich isdn user ;). entweder perfekte bildquali und etwas warten oder gar nix :D
glaubst du alle werden mit DSL im "Das Ding, aus dem die Snickers kommen" gebohrn?! Es gibt auch Leute die kein dsl kriegen, sei es wegen Geographischer Lage, Glasfaser vorm haus, oder was weiss ich....

Aber wenn es deine Meinung is, Geh und steck dir dein DSL, und dein "was kümmern mich ISDN user" sonstwohin....

naja, das war jetzt etwas hart, Offtopic und was weiss ich noch was, aber bei sonem, geistigen Dünschiss geh ich schneller auf 180 als du kucken kannst...

MFG Gun
 

LordSavage

Revenant
ich zwing ja keine isdn user sich die seite anzugucken. wenn sie es gerne möchten warten sie halt ein wenig länger und wenn nicht dann eben nicht. hatte bis vor 2 jahren noch nen 56k modem. ich weiß wohl wie das ist.
geh mal auf die seiten wie audi.de oder bmw.de was meinste wie lange du da laden muß, mit den ganzen flash files :).

ich hatte mir auch eigentlich mehr kommentare zum layout etc gewünscht und nicht zu den technischen aspekten :).
 

McGunn0r

Chthon
es geht umd die einstellung,und du must überlegen:

rund ide Hälfte aller Inet User haben noch ISDN, und Teilweise Modem, ergo:

==> fallen eine Menge besucher weg...

MFG Gun
 

Itchy

Revenant
Originally posted by Manhunter@26.08.05, 16:26
jo die seite is verdammt groß!
solltest auf keinenfall die PNG dateien benutzen, sondern statt dessen lieber Jpegs oder gifs (musste bei GIFs aufpassen, hatte schonmal bilder gehabt, die waren in GIF größer als in JPG

hgestalterisch ganz nett, aber is halt inner dateimenge zu groß (lerne die ISDN nutzer lieben)

http://www.webmasterplan.com/cgi-local/loa...gn.com/&popup=1

http://www.webmasterplan.com/cgi-local/loa...m/user/&popup=1

schulnote 6

meine seite:

http://www.webmasterplan.com/cgi-local/loa...loch.de&popup=1

kannste mal vergleichen, hab fast nur gifs genutzt...
Habe zwar 0 Ahnung vom Website bauen, aber ist das nicht etwas komisch. Bei allen stehen Fileangaben nur bei dir steht immer "Size unkown". Kein Wunder, dass du immer ne Note 1 hast, wenn der keine Filegröße ausliest. :fuck:


Klick mich

Na so berauschend ist das Ergebnis da auch nicht. Note 4 und das von einem, ich zitiere: "einen wirklich guten Webdesigner namens Manhunter (Nickname)." *tztztz* :peace:


Aber nun zum eigentlich Thema:

Mir gefallen beide Seiten sehr gut und schön gestaltet. Und Perfomance-Probs habe ich auch nicht. Also ich würde den Site eher ne 1 als ne 6 geben :P
 
Originally posted by DDnator@26.08.05, 19:18

Klick mich

Na so berauschend ist das Ergebnis da auch nicht. Note 4 und das von einem, ich zitiere: "einen wirklich guten Webdesigner namens Manhunter (Nickname)." *tztztz* :peace:
liegt daran, dass das teil irgendwie kagge rechnet, läd auch grafiken, die im klappentext sind, die eigendlich erst geladen werden, wenn man auf "mehr" klickt, is mir schonmal aufgefallen, warum da nu unknown steht weiss ich net, aber meine seite is nur 113 kb groß oder so (ok is nen bischen zu viel) aber auf der seite von lorsavage haben schon manche bilder 60 kb auf der startseite, wenn man ne gallery mit fotos hat, is das normal, aber PNGs sind verdammt groß

http://www.masse-design.com/user/

hier is die raupe schon über 100 kb groß

designerisch gefällt mir die seite auch sehr, aber is halt son problem mit den ISDN nutzer und das sind noch verdammt viele, das sind verlorene besucher, ich weiss noch bei meiner ersten seite, da hatte ich auch verdammt große dateimengen an bildern gehabt, weil ich die immer auf maximaler quali gespeichert habe.

und jetz wo ich den beitrag geschrieben hab, weiss ich warum da unkown steht, weil meine .de domain ne weiterleitung is

http://www.webmasterplan.com/cgi-local/loa...drat.de&popup=1

1 bleibt 1 :stolz:

bei gaminggentlemen hab ich verdammt viel verhunzt, wird sich im neuen design ändern :)
die bilder hab ich z.b. alle in jpeg gemacht, in gif sind die glaube ich noch kleiner
 

Burzum

Fiend
Kritik zu: http://www.masse-design.com

Tausch die Seiten, das weiße nach links das schwarze nach rechts. Wotan sei dank leben wir noch in einer westlichen Kultur und hier beginnt man normal mit einem Blick in die linke obere Ecke eines Textes bzw. Bildes. Solltest du mal in die Verlegenheit kommen eine Seite für asiatisches oder islamisches Publikum zu erstellen, solltest du natürlich deren Schreibweise beachten. Von rechts oben nach links oder unten oder wie auch immer.

Das "Masse - Art & Design" würde ich daher auch links oben plazieren. Ich hab das jetzt erst beim 3. Betrachten wirklich bemerkt! Außerdem würde ich Art gegen Kunst austauschen da es noch genügend Leute gibt die mit dem Begriff nix anzufangen wissen mangels Englischkentnissen. Und irgendwie klingt "Kunst & Design" für mich sowieso besser.

Der HTML-Code

Dein HTML-Code ist, sorry, absolut grottig.

Kein Doctype und kein Charset deklariert!
Setzen 6 und nochmal HTML von Anfang an RICHTIG lernen.

Dir fehlen wichtige Attribute wie z.B. alt="text" im <img> Tag.

Die URLs enthalten unescapte &'s die gehören durch &amp; escaped welches dem & entspricht.

Wofür sind die Frames? Die erfüllen keinen Sinn und bringen mehr Probleme als Nutzen im Zweifelsfall. Falls das ein plumper Versuch ist damit die horizontale UND vertikale Zentrierung zu erzielen ist das eine sehr schlechte Lösung. Das funktioniert mit CSS und EINER Tabelle mit 9 Felder sehr gut und erspart uns die ollen Frames. Hier muß man leider eine Tabelle mißbrauchen, da der behinderte IE sonst Probleme bekäme.

Das Tabellenkonstrukt für die ganze Seite kannst du auch in die Tonne treten und das ganze noch mal mit CSS neu machen. Tabellen sind KEINE Designelemente (!!!) sondern zur darstellung von tabellarischen Daten gedacht! Im übrigen ist height="" KEIN erlaubtes Attribut einer Tabelle!

Wieso ist das Rollover des Menüs (das man ebenfalls schlecht als solches gleich erkennt) mit JS realisiert? CSS funktioniert auch hier wunderbar und man braucht kein JS. Rund 18% der User im Web sind ohne JS unterwegs bzw. blocken es und es werden mehr. Von barierefreiem Webdesign mal ganz zu schweigen.

Eine der goldenen Regeln des guten Webdesigns:
NIEMALS JS oder Flash für eine wichtige Navigation benutzen!

Fragt sich eh welche Navigatio NICHT wichtig ist. Imo ist jede wichtig.

Code:
Object not found!

Erm... das ist auch besch...eiden und sollte um JEDEN PREIS vermieden werden!

Das Rollover auf den einzelnen schwarzen Bildern ist mir gerade nur durch Zufall aufgefallen. Das Rollover ist sicher auch mit JS gemacht (ich spar mir jetzt mal die Mühe das im Source zu überprüfen) und funktioniert bei mir nicht korrekt. Erst wenn ich einmal auf ein Bild geklickt habe wird der Rollovertext angezeigt! Auch hier -> CSS statt JS nehmen!

Die PHP-Fehler

Code:
Warning: main(content/news.txt): failed to open stream: No such file or directory in /Platz/www/docs/user/saatkamp/home.php on line 55

Wenn du vor hast mit der Seite für dich zu bewerben solltest du zusehen das solche Fehler a) vermieden werden B) SAUBER abgefangen werden, sprich nicht einfach mit einem @ kaschiert werden sondern korrekt mit is_file() geprüft werden ob die Datei überhaupt existiert und dann entsprechend etwas dagegen unternehmen!

die Ladezeit

Die Ladezeit ließe sich dramatisch reduzieren, wenn man CSS anstatt den fetten Tabellen benutzen würde. Ich hab keine Lust wieder das Rechenexampel auszugraben, frag McGunnor nach dem Thread wo er eine Kritik von mir erhalten hat.

PNG's mit Alphachannel und CSS-Hack für eine korrekte Darstellung im grottigsten Browser der Welt - ja, dem IE! Alternativ auch transparente Gifs. Spart noch mal EINIGE Kb bei den Bildern.

--------------------------------------------------------------------

Kritik zu: http://www.masse-design.com/user/

Technisch genau die gleiche Kritik wie oben.

Zum Design und Layout: Auch hier versteckt sich das Logo wieder klein und unscheinbar in der untersten linken Ecke und die Raupe dominiert die Seite. "Hä, watt? Worum gehts hier?" ist meine, und sicher auch die Reaktion der meisten User bei dem ersten Eindruck der Seite. Kein Mensch hat Bock lange den Sinn/Inhalt einer Seite zu begreifen. Auswahl gibts genug und so wird er wohl recht fix weiterziehen.

So fern es sich nicht um Omma Kätes und Progamerlamers k1ll0Rs Cheatseite handelt ist die allerwichtigste Aufgabe einer Website Informationen schnell und eindeutig zu vermitteln. Erst danach kommt das Design! Es dürfen NIEMALS Abstriche am Layout wegen irgendwelcher JS, Flash oder generell Grafikspielereien gemacht werden. Gerade bei Firmenseiten ist das wichtig.

Das Layout ist aber auch nicht pralle. Die Abstände sind durchweg anders, besonders bei dem Textblock fällt das auf. Der Textblock erscheint obendrein viel zu schmal mit ~40 Zeichen Länge. Außerdem wird die Lesbarkeit durch fehlende Absätze gemindert. Versuch den Text auf 60 Zeichen Länge und 4-6 Zeilen pro Absatz zu formatieren. Den Text könnte man im ganzen auch noch leicht andersfarbig hinterlegen um ihn besser vom Hintergrund abzuheben.

Wo wir gerade bei Hintergründen sind: Der Hintergrund der Seite ist weiß, der Teil der Seite der den Content enthält hat auch einen weißen Hintergrund. Das ist reichlich suboptimal. Auch hier würde ich den Hintergrund durch ein LEICHTES, SCHLICHTES Muster oder eine LEICHTE Farbänderung des Hintergrundes abheben.

---------------------------------------------------------------------

Abschließendes Fazit:

Falls du vor hast das ganze beruflich zu machen, mach es richtig. Pfuscher gibts schon mehr als genug:
http://www.heise.de/newsticker/meldung/58041

96,1% aller Webdesigner haben nämlich offenbar keine Ahnung von dem was sie tun. Aber Webdesigner ist ja heute eh jeder der Frontpage halbwegs beklicken ehhh bedienen kann...

Deine Designs und Layouts haben einen guten Ansatz, sind relativ sauber aber es gibt noch reichlich Optimierungspotential. Ich denke mal da solltest du auch relatisch schnell lernen wie man es besser macht. Ansich gibt es nur ein paar Grundregeln einzuhalten und ein Auge dafür zu entwickeln. Das kommt aber mit der Zeit.

Technisch gesehen solltest du UNBEDINGT etwas tun. Die Fehler die du im Code gemacht haste verraten mir, dass du keinen Plan hast was da eigentlich passiert und wieso und wofür so manches überhaupt gemacht wird.

Du solltest dir Selfthml -> http://www.selfhtml.net noch mal vom Anfang her ansehen oder am besten die Dokus auf http://www.w3.org durchlesen. Apropos W3. Wenn du schon mal da bist hilft es auch deine Seite dort mit dem Validator gleich zu validieren. Der verrät dir wo die Fehler stecken http://validator.w3.org

Weiterhin kann ich die Tidy-Extension für Firefox empfehlen und Tidy selbst auch. Tidy ist ein programm das HTML-Code validieren und bedingt korrigieren kann. Google einfach danach. Tidy gibts für Linux, Windows, MacOS und als Firefoxextension.

Dann würde ich noch die Webdeveloper-Toolbar für Firefox empfehlen und "Live HTTP Headers", eine Extension die die Headerinformationen live anzeigt. Sehr nützlich u.a. für die php-Entwicklung.

Abschließend schau dir noch http://www.css4you.de an und verwirf die Tabellen! Die haben nix als Designelemente in einer Seite verloren!
 

LordSavage

Revenant
Ich baue seit 5 Jahren html Seiten aus Tabellen zusammen und das hat bisher noch nie Probleme gegeben, warum sollte ich jetzt plötzlich divs nehmen?

Zitat: "Falls du vor hast das ganze beruflich zu machen, mach es richtig."

Ich mache das beruflich, also naja so mehr oder weniger, neben bei zum Studium arbeite ich in einer Werbeagentur.

Die Seite haben wir zuletzt erstellt. Und ja, da wurde mit divs statt mit Tabellen gearbeitet aber die habe ich auch nicht programmiert, sondern nur gestaltet.

www.winkhaus.de


Edit: Ach, ich hätte den Thread garnicht öffnen sollen, jetzt muß ich mir hier 1000 technische Sachen anhören :D.

Edit2: Nochmal, es geht mir hier nur um die Gestaltung und nicht um die Technik.

Edit3: Hab mir gerade mal son paar Quelltexte von großen Seiten angeschaut. Seit wann nutzen die denn alle divs? Hab ich irgendwas verpennt oder so?
 

Burzum

Fiend
Ich baue seit 5 Jahren html Seiten aus Tabellen zusammen und das hat bisher noch nie Probleme gegeben, warum sollte ich jetzt plötzlich divs nehmen?

Dann hinkst du mindestens 2 Jahre der aktuellen Entwicklung hinterher.

Warum? Hab ich das nicht ausreichend erklärt? Es bringt nur Vorteile, für dich (wenn du es denn einmal korrekt anwenden kannst) und für die User vorallem.

Zitat: "Falls du vor hast das ganze beruflich zu machen, mach es richtig."

Ich mache das beruflich, also naja so mehr oder weniger, neben bei zum Studium arbeite ich in einer Werbeagentur.

Glückwunsch, dann gehörst du, sorry, zu den Pfuschern.

Die Seite haben wir zuletzt erstellt. Und ja, da wurde mit divs statt mit Tabellen gearbeitet aber die habe ich auch nicht programmiert, sondern nur gestaltet.

Dann bleib bei der Gestaltung. In großen Agenturen ist es durchaus üblich quasi eine Produktionskette zu haben Design -> HTML-Template -> Anpassungs an ein CMS und/oder Programmierung bei der eine Arbeitsteilung herrscht. Allerdings dürfte das auch nur bei wirklich großen Agenturen der Fall sein. Presönlich erwarte ich von jemanden der sich Webdesigner schimpft, dass er in der Lage ist eigenständig ein sauberes Template mit HTML/CSS zu bauen UND das Design vorher zu erstellen. php oder sonstige Programmiersprachen (HTML ist KEINE Programmiersprache) erwarte ich dabei keinesfalls.

Edit: Ach, ich hätte den Thread garnicht öffnen sollen, jetzt muß ich mir hier 1000 technische Sachen anhören :D.

Wie schon erwähnt, das gehört dazu.

Edit2: Nochmal, es geht mir hier nur um die Gestaltung und nicht um die Technik.

Siehe oben. Aber ein netter Versuch sich um den mangelhaften Code zu drücken ;).

Edit3: Hab mir gerade mal son paar Quelltexte von großen Seiten angeschaut. Seit wann nutzen die denn alle divs? Hab ich irgendwas verpennt oder so?

Japp, du hast definitiv etwas verpennt. Z.b. das alle öffentlichen Seiten (Bund, Länder, Kommunen) behindertengerecht sein müssen bis, ich glaub 2006 war es. Das geht wunderbar mit CSS ohne das die Website an grafischer Qualität einbüßen müßte. CSS ermöglicht es eine Website (sagen wir mal mit z.B. nur 300 Seiten) global im Design zu verändern ohne eine Templateengine dafür zu benutzen. CSS verkürzt die Ladezeiten, besonders bei größeren Seiten dramatisch. Die CSS-File (außer man plaziert den CSS Code im HTML-Dokument, dann bringts gar nix) wird 1x geladen, gecached und für alle folgenden Seiten wird nur noch der tabellenfreie und damit sehr kurze HTML-Code geladen. Stöber einfach auf css4your.de rum, ich glaub da findet sich einiges zu dem Thema.

Ich kann dir auch gern einen Edonkey-Link zukommen lassen von einem Vortrag (mp3) zu dem Thema wenn du willst. http://www.roletschek.de/ dieser Typ hier hält eine ansich sehr gute Rede (über 1-2 Dinge könnte ich mich mit ihm streiten ;)) zum Thema barrierefreies Webdesign. Der Vortrag ist vom CCC (Chaos Computerclub) Treffen 2004 und frei. Aber da hier manche Deppen ja eine Paranoia schieben bezüglich Edonkey :wall:, werde ich den Link hier besser nicht posten. Mail mich an wenn du das File haben willst, ich schiebs dann auf meinen Server. Oder Mail den Typen direkt an ;).
 
Top