[MySQL][PHP] Błędy w skryptach rejestracji i logowania Opcje

[artur2233]

Witam, wziąłem sktypty rejestracji i logowania z tej strony
Skrypt działa poprawnie, tzn. rejetruje userów, loguje itp. Jednak w skrypcie logowania jest błąd:
Notice: Undefined index: name in C:\xampp\htdocs\www\login.php on line 32
Notice: Undefined index: password in C:\xampp\htdocs\www\login.php on line 36
W skrypcie rejestracji są takie same błędy, ale dopasowane do pól tego skryptu.

Wiem, że to jakiś bardzo prosty błąd, jefnak nie mogę ich wyłapać.
Bardzo proszę o pomoc w wyszukaniu tych błędów.

Pozdrawiam

[Kshyhoo]

Pół roku na forum, więc powinieneś był sobie poradzić z Notice. To nie błąd, tylko komunikat. Jaki? Użyj translatora Google to się dowiesz. Potem wróć na Forum i użyj szukajki. Ten temat był wałkowany tysiące razy.

[artur2233]

Użyłem szukajki, jednak polecane wyłączenie wyświetlania tych komunikatów nie jest najlepszym pomysłem pod słońcem.

w linii nr 8

[PHP] pobierz, plaintext 
 if(isset($_POST['name'])) {
[PHP] pobierz, plaintext 

pole "name" jest zissetowane, jednak wciąż komunikaty pojawiają się.

Mógłby ktoś mądry poprawić kod?

Z góry dziękuję za pomoc.

Ten post edytował artur2233 12.12.2012, 16:41:28

[Kshyhoo]

Nie miałeś poszukać, jak wyłączyć komunikaty, ale jak im zapobiegać. Przetłumaczyłeś treść komunikatu?

[artur2233]

Tłumaczenie funkcjonuje ok, ale rozwiązanie problemu już niezbyt.
Sam radziłeś w innym temacie wyłączyć wyświetlanie komunikatów (IMG:style_emoticons/default/wink.gif)

[Sephirus]

Ucinek kodu z tego "skryptu"...

Jest if ze sprawdzaniem czy "name" jest w POST ale po ifie i tak jest echo z odwołaniem do $_POST['name'] i $_POST['password']. Tak więc tu leży błąd.

[PHP] pobierz, plaintext 
if(isset($_POST['name'])) {
        // filtrujemy dane...
        $_POST['name'] = clear($_POST['name']);
        $_POST['password'] = clear($_POST['password']);
        // i kodujemy hasło
        $_POST['password'] = codepass($_POST['password']);
 
        // sprawdzamy prostym zapytaniem sql czy podane dane są prawidłowe
        $result = mysql_query("SELECT `user_id` FROM `users` WHERE `user_name` = '{$_POST['name']}' AND `user_password` = '{$_POST['password']}' LIMIT 1");
        if(mysql_num_rows($result) > 0) {
            // jeśli tak to ustawiamy sesje "logged" na true oraz do sesji "user_id" wstawiamy id usera
            $row = mysql_fetch_assoc($result);
            $_SESSION['logged'] = true;
            $_SESSION['user_id'] = $row['user_id'];
            echo '<p>Zostałeś poprawnie zalogowany! Możesz teraz przejść na <a href="index.php">stronę główną</a>.</p>';
        } else {
            echo '<p>Podany login i/lub hasło jest nieprawidłowe.</p>';
        }
    }
 
    // wyświetlamy komunikat na zalogowanie się
    echo '<form method="post" action="login.php">
        <p>
            Login:<br>
            <input type="text" value="'.$_POST['name'].'" name="name">
        </p>
        <p>
            Hasło:<br>
            <input type="password" value="'.$_POST['password'].'" name="password">
        </p>
        <p>
            <input type="submit" value="Zaloguj">
        </p>
    </form>';
 
[PHP] pobierz, plaintext 

Powinno być na przykłąd tak:

[PHP] pobierz, plaintext 
if(isset($_POST['name'])) {
        // filtrujemy dane...
        $_POST['name'] = clear($_POST['name']);
        $_POST['password'] = clear($_POST['password']);
        // i kodujemy hasło
        $_POST['password'] = codepass($_POST['password']);
 
        // sprawdzamy prostym zapytaniem sql czy podane dane są prawidłowe
        $result = mysql_query("SELECT `user_id` FROM `users` WHERE `user_name` = '{$_POST['name']}' AND `user_password` = '{$_POST['password']}' LIMIT 1");
        if(mysql_num_rows($result) > 0) {
            // jeśli tak to ustawiamy sesje "logged" na true oraz do sesji "user_id" wstawiamy id usera
            $row = mysql_fetch_assoc($result);
            $_SESSION['logged'] = true;
            $_SESSION['user_id'] = $row['user_id'];
            echo '<p>Zostałeś poprawnie zalogowany! Możesz teraz przejść na <a href="index.php">stronę główną</a>.</p>';
        } else {
            echo '<p>Podany login i/lub hasło jest nieprawidłowe.</p>';
        }
    } else {
        $_POST['name'] = '';
        $_POST['password'] = '';
    }
 
    // wyświetlamy komunikat na zalogowanie się
    echo '<form method="post" action="login.php">
        <p>
            Login:<br>
            <input type="text" value="'.$_POST['name'].'" name="name">
        </p>
        <p>
            Hasło:<br>
            <input type="password" value="'.$_POST['password'].'" name="password">
        </p>
        <p>
            <input type="submit" value="Zaloguj">
        </p>
    </form>';
[PHP] pobierz, plaintext 

Że też ludzie którzy umieszczają to w necie nie sprawdzają jak działa z error_reporting(E_ALL) ...

[artur2233]

Dzięki wielkie za wyjaśnienie problemu.

[Kshyhoo]

@Sephirus, lepiej kolekcjonować "Pomógł" niż nauczyć myśleć? Za kilka dni wróci na forum z równie trywialnym problemem.

[Sephirus]

@Kshyhoo, naprawdę uważasz, że robię to dla "pomógł"? To jakiś wyścig szczurów czy coś?

Normalnie skierowałbym na LMGTFY, lecz zraziło mnie w tym konkretnym przypadku to, że autor tematu skorzystał z gotowego skryptu ładnie opisanego w internecie, który był nie do końca przemyślany. Widać na pierwszy rzut oka, że autor jest początkujący. Czemu więc nie miałbym wytknąć niedbalstwa tym, którzy umieszczają takie rzeczy i pomóc komuś przy tym opisując co było źle?

Jeśli autor wróci z banalnym problemem to prędzej czy później czy to ja czy to Ty czy ktokolwiek inny z forum i tak pokaże, że nic nie umie. Nie sądzę bym zmienił bieg historii (IMG:style_emoticons/default/wink.gif)