Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [www] typer
Forum PHP.pl > Inne > Oceny
xeroxxxx89
Witam, proszę o ocenę typera piłkarskiego napisanego w PHP, od strony technicznej jak i wyglądu layoutu.

http://bet-zone.net

Pozdrawiam.
toaspzoo


z-index aaevil.gif

Style zagnieżdżone bezpośrednio w html :\

Przydałby się jakiś ładny hover przy 'Rejestracja; Zapomniane hasło ?'


Po zalogowaniu bez aktywacji - 'error'
Box z informacją o pomyślnej aktywacji zachodzi na ... tabelę ! (Zakłady)


Cytat
Uwaga: mail nie może być na serwerze hotmail.com!

U mnie na gmailu pojawiła się wiadomość w spamie.

Obrazki Panelu Użytkownika nie są preloadowane, co powoduje znaczne opóźnienie.

Regulamin hmm.. osiedlowego gangu tongue.gif


Z innej beczki:

http://bet-zone.net/index.php?p=profile&id=1'or id=2


Brak paddingu treści od lewej: http://bet-zone.net/index.php?p=faq

Usuwasz całe tagi html zamiast podmieniać znaków, pisane na szybko ?

Brak linkowania w sb

Apostrofy w polach logowania = error,error

div.content włazi na tabelkę z lewej, wszędzie.[/b]
xeroxxxx89
Tak, usuwam całe tagi html funkcją strip_tags, ponieważ nie są one mi do niczego potrzebne, nie tyle mi co innym użytkownikom. Co do tego co jest na obrazku to będę musiał coś z tym zrobić, na szczęście dzieje się to tylko podczas przewijania. Co do "error", to ograniczyłem się tylko do tego w niektórych miejscach, gdyż strona jest ciągle "w budownie". Dzięki za wyczerpującą ocenę.
toaspzoo
Można pisać wiadomości do siebie samego smile.gif

offtop: Ładne liczby smile.gif
xeroxxxx89
Cytat(toaspzoo @ 12.08.2012, 17:36:42 ) *
Można pisać wiadomości do siebie samego smile.gif

offtop: Ładne liczby smile.gif


Hmm, a jak to zrobiłeś? Dziwne, bo mam w skrypcie żeby tylko wysyłało wiadomość, jeśli

  1. $_GET['id'] !=$_SESSION['id']


toaspzoo
http://bet-zone.net/index.php?p=message&am...amp;users_id=36 nie da się :\
http://bet-zone.net/index.php?p=message&am...'or%20id=36 a tutaj już tak smile.gif

Update:
Dalej można smile.gif 20:51 / 12-08
pyro
Cytat(toaspzoo @ 12.08.2012, 20:52:08 ) *



Tak naprawdę wszędzie na stronie jest SQL Injection....
toaspzoo
Hmm, nie tyle sql injection co wywalanie istotnych informacji na stronę - np. błędów MySQL.
xeroxxxx89
Cytat(pyro @ 13.08.2012, 00:01:39 ) *
Tak naprawdę wszędzie na stronie jest SQL Injection....


Już się nie da, chociaż nie mam pewnośći, czy sprawdzanie $_GET['users_id'] funkcją is_numeric to dobry sposób?
toaspzoo
Cytat(xeroxxxx89 @ 13.08.2012, 02:12:33 ) *
Już się nie da, chociaż nie mam pewnośći, czy sprawdzanie $_GET['users_id'] funkcją is_numeric to dobry sposób?



Dobry, zawsze można zastosować rzutowanie, ale ta metoda wydaje się 'lepsza' smile.gif
Monter08
http://validator.w3.org/check?uri=http%3A%...ine&group=0
Ściana płaczu? biggrin.gif
pyro
Cytat(xeroxxxx89 @ 13.08.2012, 02:12:33 ) *
Już się nie da


Skoro tak mówisz... ale jakby co dane administratora:

[Usunięte na życzenie autora tematu poprzez PW]
cv65
Po co używasz tych wszystkich bibliotek ?
jQuery
Prototype
MintAjax
Moo
Wersja bez bibliotek

Zdecyduj się na którąś i używaj jej konsekwentnie
albini
Za dużo bibliotek
cv65
OO a nawet dwa razy wywołujesz jQuery i po co ?
Uporządkuj sobie trochę pliki na serwerze bo zrobisz tam jeden wielki śmietnik
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.