Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [www] Do oceny dwa kodowania
Forum PHP.pl > Inne > Oceny
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ę smile.gif

Grafikę nie ja wykonywałem przy obu pracach exclamation.gif exclamation.gif
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 sad.gif

wydaj te parę dolców na to zdjęcie z topu zamiast pokazywać "stock photo" 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 sad.gif

W drugim projekcie te same błędy. Myślę że parę godzin i to załatasz smile.gif

Ogólnie rzecz biorąc sporo pracy przed Tobą, jak i przed nami wszystkimi w sumie... winksmiley.jpg powodzenia smile.gif
Artu123
Cytat(webber @ 28.02.2009, 12:56:35 ) *
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 sad.gif

wydaj te parę dolców na to zdjęcie z topu zamiast pokazywać "stock photo" 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 sad.gif

W drugim projekcie te same błędy. Myślę że parę godzin i to załatasz smile.gif

Ogólnie rzecz biorąc sporo pracy przed Tobą, jak i przed nami wszystkimi w sumie... winksmiley.jpg powodzenia 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 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 .
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.