[www] typer Opcje

[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]

(IMG:http://sowa-mat.pl/bet-zone.net.png)

z-index (IMG:style_emoticons/default/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)

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 (IMG:style_emoticons/default/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]

Ten post edytował toaspzoo 12.08.2012, 15:36:12

[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 (IMG:style_emoticons/default/smile.gif)

offtop: Ładne liczby (IMG:style_emoticons/default/smile.gif)
(IMG:http://sowa-mat.pl/toaspzoo.png)

Ten post edytował toaspzoo 12.08.2012, 16:35:41

[xeroxxxx89]

Można pisać wiadomości do siebie samego (IMG:style_emoticons/default/smile.gif)

offtop: Ładne liczby (IMG:style_emoticons/default/smile.gif)
(IMG:http://sowa-mat.pl/toaspzoo.png)

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

[PHP] pobierz, plaintext 
$_GET['id'] !=$_SESSION['id']
[PHP] pobierz, plaintext 

[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 (IMG:style_emoticons/default/smile.gif)

Update:
Dalej można (IMG:style_emoticons/default/smile.gif) 20:51 / 12-08

Ten post edytował toaspzoo 12.08.2012, 17:00:05

[pyro]

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 (IMG:style_emoticons/default/smile.gif)

Update:
Dalej można (IMG:style_emoticons/default/smile.gif) 20:51 / 12-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]

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]

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' (IMG:style_emoticons/default/smile.gif)

[Monter08]

http://validator.w3.org/check?uri=http%3A%...ine&group=0
Ściana płaczu? (IMG:style_emoticons/default/biggrin.gif)

[pyro]

Już się nie da

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

[Usunięte na życzenie autora tematu poprzez PW]

Ten post edytował pyro 13.08.2012, 15:36:35

[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