[www] Do oceny dwa kodowania Opcje

[Artu123]

Witam!

Mam do oceny dwa moje kodowania .

http://artu123.pl/projekty/blast/

http://artu123.pl/projekty/sk8s/

w validatorze w 2 kodowania są po 3 błędy, ale dlatego, że hosting na końcu strony dodaje coś takiego:

Kod

<!-- www.000webhost.com Analytics Code -->
<script type="text/javascript" src="http://analytics.hosting24.com/count.php"></script>
<noscript><a href="http://www.hosting24.com/"><img src="http://analytics.hosting24.com/count.php" alt="web hosting" /></a></noscript>
<!-- End Of Code --

Prosiłbym o jak najbardziej zjechanie moich prac, co chciałbym się dowiedzieć co jeszcze źle robię (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Grafikę nie ja wykonywałem przy obu pracach (IMG:http://forum.php.pl/style_emoticons/default/exclamation.gif) (IMG:http://forum.php.pl/style_emoticons/default/exclamation.gif)

Ten post edytował Artu123 27.02.2009, 14:42:43

[Fantazyn]

Style ładuj do osobnych plików .css - nie pisz ich w nagłówku strony ani (przynajmniej się staraj) nie dopisuj ich inline.
Odstępy między elementami ustawiamy wg potrzeby albo padding albo margin, a nie znakami <br /> czy &nbsp; .
Tworząc menu korzystaj z listy <ul> i <li>.
Radzę dopisać meta tagi.

Ps. A layout mi się podoba... nie musisz się go wyrzekać : ).

[phpion]

Ogólnie miernota. Straszny bałagan, brak wcięć w kodzie, style inline, szalona divomania, inwazyjny JS, &nbsp;, brak jakiejkolwiek semantyki (gdzie chociażby nagłówki?). Prosisz o ocenę kodowania: kupa.

[webber]

1. http://witia.jogger.pl/2005/12/20/divitis-classitis/

Projekt 1:

style pisane inline to zły pomysł, teraz wyobraź sobie jakbyś miał kilkanaście podstron i wszędzie musiał zmieniać jeden element o_O

to iż strona się waliduje nie znaczy iż jest zgodna ze standardami (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

wydaj te parę dolców na to zdjęcie z topu zamiast pokazywać "stock photo" (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

bardzo ładny estetyczny wygląd, nie podoba mi się tylko to iż tekst na tym obrazku w topie jest mało widoczny,

błędów w kodzie jest na prawdę mnóstwo, postaram się podać kilka przykładów:

Kod

<img src="images/sql3_08.jpg" alt="" /><img src="images/sql3_101.jpg" alt="" /><img src="images/sql3_121.jpg" alt="" /><img src="images/sql3_141.jpg" alt="" /><img src="images/sql3_tll.jpg" alt="" /><img src="images/tlo.png" alt="" /><img src="images/tlo1.png" alt="" /><img src="images/home.jpg" alt="" /><img src="images/userzy.jpg" alt="" /><img src="images/wiadomosci.jpg" alt="" /><img src="images/sql3_181.jpg" alt="" />

Kod

<div id="menu1">
<div style="float:left;"><img src="images/sql3_03.jpg" alt="" /></div><div style="float:left;"><a href="index.html" class="link"><img src="images/sql3_081.jpg"  onmouseover="this.src='images/sql3_08.jpg'" onmouseout="this.src='images/sql3_081.jpg'" alt="" style="border:none;" /></a></div>
<div style="float:left;"><img src="images/sql3_03.jpg" alt="" /></div><div style="float:left;"><a href="index.html" class="link"><img src="images/sql3_10.jpg"  onmouseover="this.src='images/sql3_101.jpg'" onmouseout="this.src='images/sql3_10.jpg'" alt="" style="border:none;" /></a></div>
<div style="float:left;"><img src="images/sql3_03.jpg" alt="" /></div><div style="float:left;"><a href="index.html" class="link"><img src="images/sql3_12.jpg"  onmouseover="this.src='images/sql3_121.jpg'" onmouseout="this.src='images/sql3_12.jpg'" alt="" style="border:none;" /></a></div>
<div style="float:left;"><img src="images/sql3_03.jpg" alt="" /></div><div style="float:left;"><a href="index.html" class="link"><img src="images/sql3_14.jpg"  onmouseover="this.src='images/sql3_141.jpg'" onmouseout="this.src='images/sql3_14.jpg'" alt="" style="border:none;" /></a></div>
<div style="float:left;"><img src="images/sql3_03.jpg" alt="" /></div><div style="float:left;"><a href="index.html" class="link"><img src="images/sql3_16.jpg" alt="" style="border:none;" /></a></div>
<div style="float:left;"><img src="images/sql3_03.jpg" alt="" /></div><div style="float:left;"><a href="index.html" class="link1"><img src="images/sql3_18.jpg"  onmouseover="this.src='images/sql3_181.jpg'" onmouseout="this.src='images/sql3_18.jpg'" alt="" style="border:none;" /></a></div>
</div>

Po to wstawiasz tam alt by opisać ten element, szczególnie w menu...

Kod

<a href="index.html">Contrary to popular</a>&nbsp;&nbsp;&nbsp;|&nbsp;&nbsp;&nbsp;<a href="index.html">belief, Lorem Ipsum is not simply</a>&nbsp;&nbsp;&nbsp;|&nbsp;&nbsp;&nbsp;<a href="index.html">It has roots in a piece</a>&nbsp;&nbsp;&nbsp;|&nbsp;&nbsp;&nbsp;<a href="index.html">Latin professor at Hampden-Sydney</a>    </div></div>

&nbsp;? oj...

Kod

<a href="index.html" class="linkmenu" style="padding-left:43px;padding-top:11px;">Struktura szkoły</a>
<a href="index.html" class="linkmenu" style="padding-left:43px;padding-top:11px;">Dyrekcja szkoły</a>
<a href="index.html" class="linkmenu" style="padding-left:43px;padding-top:11px;">Nauczyciele</a>
<a href="index.html" class="linkmenu" style="padding-left:43px;padding-top:11px;">Rada rodziców</a>
[...]

wystarczyło dodać padding-left: 43px; padding-top:11px; do klasy .linkmenu (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

W drugim projekcie te same błędy. Myślę że parę godzin i to załatasz (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ogólnie rzecz biorąc sporo pracy przed Tobą, jak i przed nami wszystkimi w sumie... (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) powodzenia (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[Artu123]

1. http://witia.jogger.pl/2005/12/20/divitis-classitis/

Projekt 1:

style pisane inline to zły pomysł, teraz wyobraź sobie jakbyś miał kilkanaście podstron i wszędzie musiał zmieniać jeden element o_O

to iż strona się waliduje nie znaczy iż jest zgodna ze standardami (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

wydaj te parę dolców na to zdjęcie z topu zamiast pokazywać "stock photo" (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

bardzo ładny estetyczny wygląd, nie podoba mi się tylko to iż tekst na tym obrazku w topie jest mało widoczny,

błędów w kodzie jest na prawdę mnóstwo, postaram się podać kilka przykładów:

Kod

<img src="images/sql3_08.jpg" alt="" /><img src="images/sql3_101.jpg" alt="" /><img src="images/sql3_121.jpg" alt="" /><img src="images/sql3_141.jpg" alt="" /><img src="images/sql3_tll.jpg" alt="" /><img src="images/tlo.png" alt="" /><img src="images/tlo1.png" alt="" /><img src="images/home.jpg" alt="" /><img src="images/userzy.jpg" alt="" /><img src="images/wiadomosci.jpg" alt="" /><img src="images/sql3_181.jpg" alt="" />

Kod

<div id="menu1">
<div style="float:left;"><img src="images/sql3_03.jpg" alt="" /></div><div style="float:left;"><a href="index.html" class="link"><img src="images/sql3_081.jpg"  onmouseover="this.src='images/sql3_08.jpg'" onmouseout="this.src='images/sql3_081.jpg'" alt="" style="border:none;" /></a></div>
<div style="float:left;"><img src="images/sql3_03.jpg" alt="" /></div><div style="float:left;"><a href="index.html" class="link"><img src="images/sql3_10.jpg"  onmouseover="this.src='images/sql3_101.jpg'" onmouseout="this.src='images/sql3_10.jpg'" alt="" style="border:none;" /></a></div>
<div style="float:left;"><img src="images/sql3_03.jpg" alt="" /></div><div style="float:left;"><a href="index.html" class="link"><img src="images/sql3_12.jpg"  onmouseover="this.src='images/sql3_121.jpg'" onmouseout="this.src='images/sql3_12.jpg'" alt="" style="border:none;" /></a></div>
<div style="float:left;"><img src="images/sql3_03.jpg" alt="" /></div><div style="float:left;"><a href="index.html" class="link"><img src="images/sql3_14.jpg"  onmouseover="this.src='images/sql3_141.jpg'" onmouseout="this.src='images/sql3_14.jpg'" alt="" style="border:none;" /></a></div>
<div style="float:left;"><img src="images/sql3_03.jpg" alt="" /></div><div style="float:left;"><a href="index.html" class="link"><img src="images/sql3_16.jpg" alt="" style="border:none;" /></a></div>
<div style="float:left;"><img src="images/sql3_03.jpg" alt="" /></div><div style="float:left;"><a href="index.html" class="link1"><img src="images/sql3_18.jpg"  onmouseover="this.src='images/sql3_181.jpg'" onmouseout="this.src='images/sql3_18.jpg'" alt="" style="border:none;" /></a></div>
</div>

Po to wstawiasz tam alt by opisać ten element, szczególnie w menu...

Kod

<a href="index.html">Contrary to popular</a>&nbsp;&nbsp;&nbsp;|&nbsp;&nbsp;&nbsp;<a href="index.html">belief, Lorem Ipsum is not simply</a>&nbsp;&nbsp;&nbsp;|&nbsp;&nbsp;&nbsp;<a href="index.html">It has roots in a piece</a>&nbsp;&nbsp;&nbsp;|&nbsp;&nbsp;&nbsp;<a href="index.html">Latin professor at Hampden-Sydney</a>    </div></div>

&nbsp;? oj...

Kod

<a href="index.html" class="linkmenu" style="padding-left:43px;padding-top:11px;">Struktura szkoły</a>
<a href="index.html" class="linkmenu" style="padding-left:43px;padding-top:11px;">Dyrekcja szkoły</a>
<a href="index.html" class="linkmenu" style="padding-left:43px;padding-top:11px;">Nauczyciele</a>
<a href="index.html" class="linkmenu" style="padding-left:43px;padding-top:11px;">Rada rodziców</a>
[...]

wystarczyło dodać padding-left: 43px; padding-top:11px; do klasy .linkmenu (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

W drugim projekcie te same błędy. Myślę że parę godzin i to załatasz (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ogólnie rzecz biorąc sporo pracy przed Tobą, jak i przed nami wszystkimi w sumie... (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) powodzenia (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Czytaj uważnie ! Nie ja lay robiłem i nie będę wydawał kasy na głupi stock, bo i tak dla treningu kodowałem (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) A co do tych błędów to dzięki wszystkim . Postaram się następnym razem nie popełniać tych błędów .