16-03-2011, 22:03
(Ten post był ostatnio modyfikowany: 16-03-2011, 22:05 przez R_Rafalsky.)
W 40 linijce includujesz ten sam plik co w drugiej (co ciekawe najpierw używasz include_once a później include.
Korzystasz ze zmiennych sesji a samej sesji nie uruchamiasz: session_start(); ob_start();
Za każdym odświeżeniem sprawdzasz w bazie danych użytkownika?
W zapytaniu do bazy niepotrzebny jest LIMIT=1
Zamiast: $t = mysql_fetch_array($z); możesz zrobić $z = mysql_fetch_array($z); zmienna $z nie jest Ci już przecież potrzebna (ale to już czepianie się szczegółów), tym bardziej nie widzę sensu niżej deklarowanie kolejnej zmiennej i przepisywanie wartości z t
if ($upr == user ) - źle powinno być 'user' - do tego zły pomysł z tym ifem, wykorzystałbym bardziej switch case.
<font> jest zdeprecjonowany (szczególnie jak deklarujesz jako XHTML Strict), używaj <span> i wykorzystuj do kolorowania style nie color
Po co to: if(isset($_GET['go'])) { $go=$_GET['go']; } else { $go='false';} ? Po pierwsze jeżeli robimy jedną operację dla if to nie jest wymagany nawias, po drugie wystarczy $go=$_GET['go'] (każda zmienna, która nie istnieje ma wartość false), a po trzecie $go niepotrzebnie wogóle deklarujesz.
Jak zapytanie nie zostanie zapisane (edycja hasła) to i tak wyświetli się komentarz 'Twoje Haslo Zostały Zmienione!!'
Nie <br> tylko <br /> XHTML wymaga by wszystkie znaczniki były zamknięte. Jeżeli nie chcesz tak pilnować to zmień deklarację na HTML
Jeżeli chodzi o projektowanie działania aplikacji to nie chcę się wypowiadać
I jeszcze jedna sprawa: jeżeli używasz znaków diakrytycznych to używaj ich wszędzie, bo raz są raz ich nie ma
Korzystasz ze zmiennych sesji a samej sesji nie uruchamiasz: session_start(); ob_start();
Za każdym odświeżeniem sprawdzasz w bazie danych użytkownika?
W zapytaniu do bazy niepotrzebny jest LIMIT=1
Zamiast: $t = mysql_fetch_array($z); możesz zrobić $z = mysql_fetch_array($z); zmienna $z nie jest Ci już przecież potrzebna (ale to już czepianie się szczegółów), tym bardziej nie widzę sensu niżej deklarowanie kolejnej zmiennej i przepisywanie wartości z t
if ($upr == user ) - źle powinno być 'user' - do tego zły pomysł z tym ifem, wykorzystałbym bardziej switch case.
<font> jest zdeprecjonowany (szczególnie jak deklarujesz jako XHTML Strict), używaj <span> i wykorzystuj do kolorowania style nie color
Po co to: if(isset($_GET['go'])) { $go=$_GET['go']; } else { $go='false';} ? Po pierwsze jeżeli robimy jedną operację dla if to nie jest wymagany nawias, po drugie wystarczy $go=$_GET['go'] (każda zmienna, która nie istnieje ma wartość false), a po trzecie $go niepotrzebnie wogóle deklarujesz.
Jak zapytanie nie zostanie zapisane (edycja hasła) to i tak wyświetli się komentarz 'Twoje Haslo Zostały Zmienione!!'
Nie <br> tylko <br /> XHTML wymaga by wszystkie znaczniki były zamknięte. Jeżeli nie chcesz tak pilnować to zmień deklarację na HTML
Jeżeli chodzi o projektowanie działania aplikacji to nie chcę się wypowiadać
I jeszcze jedna sprawa: jeżeli używasz znaków diakrytycznych to używaj ich wszędzie, bo raz są raz ich nie ma
Jak pomogłem daj '+'. To nie boli, a ja mam satysfakcję
Any sufficiently advanced technology is indistinguishable from magic.
Any sufficiently advanced technology is indistinguishable from magic.
