Ocena niekompletnej stronki

Question

Witam,

robię projekt do szkoły i prosiłbym o ocenkę strony. Bardziej chodzi o błędy w kodzie - PHP jeszcze nie działa i nie ma strony do logowania i rejestracji. Chodzi o samą stronę główną.

http://serwispytan.16mb.com/

Pozdrawiam, BS

Answer 1

W systemie istnieje też cos takiego jak ranking użytkowników

Cieszę się, że zdecydowałeś się skorzystać z szkolnej platformy pytań powtórkowych do, na przykład, sprawdzianów, kartkówek oraz egzaminu gimnazjalnego

Nie logiczne xD 

	<h1 id="h1"> Rejestracja </h1>

Po co nadałeś id?

<center><b>Drogi uczniu!</b></center>
	<p id="margin"> Cieszę się, że zdecydowałeś się skorzystać z szkolnej platformy pytań powtórkowych do, na przykład, sprawdzianów, kartkówek oraz egzaminu gimnazjalnego. Baza pytań jest wzbogacana przez nauczycieli szkoły co jakiś czas. Tylko jedna odpowiedź spośród podanych jest prawidłowa. Masz możliwość wylosowania zestawów po dziesięć pytań. Za każdą prawidłową odpowiedź ortrzymujesz 1 punkt. W systemie istnieje też cos takiego jak ranking użytkowników. Pokazuje on pierwszych, dziesięciorga uczniów o największej liczbie punktów. Zadania nie są ograniczone czasowo, masz na ich wykonanie tyle czasu ile Ci będzie potrzebne. Nie ma też ograniczonej liczby wejść na stronę - wchodzisz tyle razy ile zechcesz. Aby odpowiadać na pytania - zaloguj się, jeśli nie masz u nas jeszcze konta - zarejestruj się. Zrobisz to pod tym wpisem. </br>
	<div id="podpis" style="text-align: right; margin-right: 10px;">Życzę Ci przyjemnego korzystania z serwisu </br>Borys Strzeboński</div></p>

Tego już zazwyczaj się nie używa.

 

Pozdrawiam

Answer 2

Tego <center> to nawet nie powinno być, ale okej.

-> Polecam: http://www.creamsoft.com/blog/nowe-znaczniki-html-5-i-usuniete-elementy-z-html-4-01.html

Poczytaj o HTML5 (semantyka) - commander nawet ma gdzieś poradnik o tym, poszukaj :)

Co do designu, mówiąc delikatnie średnio to wygląda.

Answer 3

Jedno pytanie.
Dlaczego ktoś daje do oceny niekompletną pracę? Jeżeli nie jest skończona po co ją dajesz do oceny?
Ja mogę dać niekompletną odpowiedz bo po co się rozpisywać.
- HTML5

Answer 4

Zanim dojdziemy do PHPa jest dużo błędów w składni html:

• brak doctype -> utworzyłeś komentarz zawierający doctype html, ale nie zadeklarowałeś go.
• lepiej linkować google font powyżej własnych stylów - strona jest czytana od góry do dołu, idąc tym tokiem załadujesz czcionkę, która jeszcze nie jest dostępna,
• masz zbędny tag style w head
• strona nie jest semantyczna,
• używanie id do stylizowania - to jest sprawa dyskusyjna, ponieważ ID jest do stylizowania niepowtarzalnej rzeczy, natomiast klasa - do wielu. Powszechnie całość webdevowej społeczności dzieli się na dwa obozy: wg jednych powinno pisać się w CSSie zarówno klasy jak i ID, wg drugich ID powinno się używać w JSie jako uchwyt. Notabene robisz to zbędnie, ponieważ zapis w css #container > h1 by wystarczył
• nie znam atrybutu aglin dla znacznika hr, a znacznik width możesz określić za pomocą CSS,
• tag center jest przestarzały,
• diva o id podpis stylujesz wewnątrz tagu oraz za pomocą zewnętrznego pliku CSS. Stylizować tagi wewnętrznie nie powinno się wręcz wcale, a Ty jeszcze to komplikujesz.
• znacznik font wogóle istnieje? XD Osobiście umieściłbym to w spanie o jakiejś klasie,
• używasz br do tworzenia przestrzeni pomiędzy jednym a drugim elementem - od tego jest margin lub padding,
• divy "Zaloguj się" oraz "Zarejestruj się" powinny być przyciskami,
• powtarzasz id "pole_log"
• warto dodawać artybut title dla znacznika a - chociażby w przypadkach gdzie odwiedzający nie rozumie/ nie jest pewien tego co widzi,
• tagujesz tło z zewnętrznego serwera, co jeśli on by padł? To kolejna sprawa dyskusyjna - serwer warto odciążyć, ale strona to nie wielki kombajn, więc najlepiej po prostu pobrać tło i wrzucić je do katalogu strony,
• brak outline'u,

Poza tym, strona jest nieczytelna graficznie. Strona do poprawy ;) Pozdrawiam.

Answer 5

<hr aglin="center" width="50%" color="#514F59">

powinno być align, a nie aglin.

<center>
	<div id="content">
	<center><b>Drogi uczniu!</b></center>

To jest bez sensu.

• containera nie używaj na szerokość 100% bo wszystko się rozciąga. Daj np. max-width: 1200px
• nie używaj <center>
• nie używaj <font>
• nie używaj </br> - a tak w ogóle to nie </br> tylko <br/>
• używaj <label>
• zapoznaj się z typografią
• same <div>'y http://tutorials.comandeer.pl/html5-blog.html

Answer 6

Dopiszę, bo chyba nikt jeszcze o tym nie napisał:
-> ​przycisk "zarejestruj się" ma inne wymiary od pozostałych (jest trochę rozciągnięty na lewo)... nie wiem jak innych ale mnie to strasznie irytuje... :(