[skrypt] rejestracja wystawców Opcje

[qwaq]

Proszę o ocenę skryptu.


opis: Skrypt do rejestracji wystawców podczas nadchodzących targów

chodzi o to że przyszły klient rezerwuje stoiska na hali targowej.

Bardzo proszę o ocenę i wytknięcie wszystkich błędów jakie znajdziecie. Z góry dziękuję

--------------------------------------------------------------------------------
naprawde 0 ocen?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)
proszę wejdzcie i poszukajcie ukrytych błędów działania skryptu.
Jest dla mnie ważna opinia osób trzecich ponieważ posiadając pewne nawyki nie jestem w stanie wykryć wszystkich błędów

Ten post edytował qwaq 17.07.2007, 08:52:23

[phpion]

Kod HTML leży:
- kodowanie windows-1250
- brak DOCTYPE
-

[HTML] pobierz, plaintext 
text="#063E7B" link="#063E7B" vlink="#063E7B" alink="#063E7B"
[HTML] pobierz, plaintext 

- style inline do tego powtarzane dla innych elementów z takim samym stylem (użyj *.css)
- złe użycie tabel
-

[HTML] pobierz, plaintext 
   
[HTML] pobierz, plaintext 

Kodu PHP nie oceniam bo nie mam za bardzo czasu na zabawę z nim. Na pewno dopisz walidację całego formularza (i w PHP i w JS żeby nie trzeba było wysyłać niepotrzebnie formularza).

Jeśli chodzi o wygląd to te grafiki są dla mnie po prostu tandetne... :| Po co jest ta pierwsza skoro do niczego nie służy? Natomiast odnośnie drugiej to zdecydowanie lepiej by wyglądała jako tabela w HTMLu z użyciem CSS i JS. Mógłbyś dodać opcję wyboru stoiska poprzez klikanie w odpowiednie obszary, a nie tylko poprzez <select> Stoiskom wybranym przez innych mógłbyś nadać bardziej wyblakły kolor (może jakaś przezroczystość?), natomiast te wybierane przez aktualnego usera wyróżnić np. czerwoną ramką. Byłoby to i efektowniejsze i efektywniejsze.

To takie moje uwagi.

Ogólnie skrypt sprawia wrażenie raczej kiepskiego... ale to moje zdanie.