Ciągle wyświetlający sie napis "Nieprawidłowy login lub hasło" na własnej stronie

pytanie zadane 26 sierpnia 2015 w PHP przez francus11 Gaduła (3,250 p.)
zamknięte 26 sierpnia 2015 przez francus11

Ostatnio uczę się tworzyc strony WWW według poradników z kanału YT Mirosław Zelent. Gdy doszedłem do momentu, gdzie napisałem skryp PHP odpowiedzialny za wyświetlanie na stronie komunikatu "Nieprawidłowy login lub hasło", on wyświetlał się też przy pierwszym wyświetleniu strony. Myślałem, że problem występuje w moim kodzie, ale wszystko było dobrze. Następnie zaloadowałem do XAMPPa pliki z poradnika, ale problem też występował.

Pytanie rodzi się jedno - co tworzy ten błąd, skoro on występuje nawet w plikach od autora poradnika???

komentarz zamknięcia: Zakończony

8 odpowiedzi

odpowiedź 26 sierpnia 2015 przez Schizohatter Nałogowiec (39,600 p.)

1. Pokaż kod

komentarz 26 sierpnia 2015 przez francus11 Gaduła (3,250 p.)

<?php
	session_start();
?>
<!DOCTYPE HTML>
<html lang="pl">
<head>
	<meta charset="utf-8"/>
	<meta htt-equiv="X-UA-Compatible" content="IE=edge,chrome=1"/>
	<title>Osadnicy - Gra przeglądarkowa</title>
</head>

<body>

	Tylko martwi ujrzeli koniec wojny - Platon<br/><br/>

	<form action="zaloguj.php" method="post">
		
		Login: <br/> <input type="text" name="login"/><br/>
		Hasło: <br/> <input type="password" name="haslo"/><br/>
		<input type="submit" value="Zaloguj się"/>
		
		</form>
<?php
	if (isset($_SESSION['blad'])) echo $_SESSION['blad'];
?>
</body>
</html>

odpowiedź 26 sierpnia 2015 przez Lukasz Bywalec (2,320 p.)

odpowiedź 26 sierpnia 2015 przez francus11 Gaduła (3,250 p.)

<?php
    session_start();
?>
<!DOCTYPE HTML>
<html lang="pl">
<head>
    <meta charset="utf-8"/>
    <meta htt-equiv="X-UA-Compatible" content="IE=edge,chrome=1"/>
    <title>Osadnicy - Gra przeglądarkowa</title>
</head>
 
<body>
 
    Tylko martwi ujrzeli koniec wojny - Platon<br/><br/>
 
    <form action="zaloguj.php" method="post">
         
        Login: <br/> <input type="text" name="login"/><br/>
        Hasło: <br/> <input type="password" name="haslo"/><br/>
        <input type="submit" value="Zaloguj się"/>
         
        </form>
<?php
    if (isset($_SESSION['blad'])) echo $_SESSION['blad'];
?>
</body>
</html>

odpowiedź 26 sierpnia 2015 przez DL TD Nałogowiec (36,710 p.)

Chyba wiem co powoduje ten "błąd" Pan Mirosław nie umieścił w pliku index.php kodu który usuwa zmienną sesyjną blad. W pliku index.php na samym dole gdzie jest if dodaj pod instrukcją echo unset($_SESSION['blad']);

Proste i skuteczne rozwiązanie. Po następnym refresh problem będzie znikał, gdyż zmienna nie będzie istnieć :)

EDIT: Widzę, że dodałeś kod. To super. Podmień ten if na taki:

if (isset($_SESSION['blad'])) {
     echo $_SESSION['blad'];
     unset($_SESSION['blad']);
}

odpowiedź 26 sierpnia 2015 przez francus11 Gaduła (3,250 p.)

index.php

<?php
	session_start();
?>
<!DOCTYPE HTML>
<html lang="pl">
<head>
	<meta charset="utf-8"/>
	<meta htt-equiv="X-UA-Compatible" content="IE=edge,chrome=1"/>
	<title>Osadnicy - Gra przeglądarkowa</title>
</head>

<body>

	Tylko martwi ujrzeli koniec wojny - Platon<br/><br/>

	<form action="zaloguj.php" method="post">
		
		Login: <br/> <input type="text" name="login"/><br/>
		Hasło: <br/> <input type="password" name="haslo"/><br/>
		<input type="submit" value="Zaloguj się"/>
		
		</form>
<?php
	if (isset($_SESSION['blad'])) echo $_SESSION['blad'];
?>
</body>
</html>

zaloguj.php

<?php

   session_start();

   if ((!isset($_POST['login'])) || (!isset($_POST['haslo'])))
   {
       header('Location: index.php');
       exit();
   }

require_once "connect.php";

$polaczenie = @new mysqli($host, $db_user, $db_password, $db_name);

 if ($polaczenie->connect_errno!=0)
 {
 echo "Error: ".$polaczenie->connect_errno;
 }
 else
 {
 $login = $_POST['login'];
 $haslo = $_POST['haslo'];

 $login = htmlentities($login, ENT_QUOTES, "UTF-8");
 $haslo = htmlentities($haslo, ENT_QUOTES, "UTF-8");

 if ($rezultat = @$polaczenie->query(
 sprintf("SELECT * FROM uzytkownicy WHERE user='%s' AND pass='%s'",
 mysqli_real_escape_string($polaczenie,$login),
 mysqli_real_escape_string($polaczenie,$haslo))))
 {
 $ilu_userow = $rezultat->num_rows;
 if($ilu_userow>0)
 {
 $_SESSION['zalogowany'] = true;

 $wiersz = $rezultat->fetch_assoc();
 $_SESSION['id'] = $wiersz['id'];
 $_SESSION['user'] = $wiersz['user'];
 $_SESSION['drewno'] = $wiersz['drewno'];
 $_SESSION['kamien'] = $wiersz['kamien'];
 $_SESSION['zboze'] = $wiersz['zboze'];
 $_SESSION['email'] = $wiersz['email'];
 $_SESSION['dnipremium'] = $wiersz['dnipremium'];

 unset($_SESSION['blad']);
 $rezultat->free_result();
 header('Location: gra.php');

 } else {

 $_SESSION['blad'] = 'Nieprawidłowy login lub hasło!';
 header('Location: index.php');

 }

 }

 $polaczenie->close();
 }

?>

connect.php

<?php

	$host = "localhost";
	$db_user = "root";
	$db_password = "";
	$db_name = "osadnicy";

?>

gra.php

<?php

session_start();

 if (!isset($_SESSION['zalogowany']))
 {
 header('Location: index.php');
 exit();
 }

?>
<!DOCTYPE HTML>
<html lang="pl">
<head>
 <meta charset="utf-8" />
 <meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1" />
 <title>Osadnicy - gra przeglądarkowa</title>
</head>

<body>

<?php

echo "Witaj ".$_SESSION['user'].'! [ <a href="logout.php">Wyloguj się!</a> ]';
 echo "Drewno: ".$_SESSION['drewno'];
 echo " | Kamień: ".$_SESSION['kamien'];
 echo " | Zboże: ".$_SESSION['zboze']."";

 echo "E-mail: ".$_SESSION['email'];
 echo " Dni premium: ".$_SESSION['dnipremium']."";

?>

</body>
</html>

komentarz 26 sierpnia 2015 przez DL TD Nałogowiec (36,710 p.)

Dzięki za kod. Zobacz moją odpowiedź :)

odpowiedź 26 sierpnia 2015 przez francus11 Gaduła (3,250 p.)
przywrócone 26 sierpnia 2015 przez francus11

niestety to nic nie daje

komentarz 26 sierpnia 2015 przez DL TD Nałogowiec (36,710 p.)

Odpisuj konkretnej osobie bo nw komu odpisujesz i co niedziała :)

odpowiedź 26 sierpnia 2015 przez francus11 Gaduła (3,250 p.)

A gdyby przekształcić

if (isset($_SESSION['blad'])) 
echo $_SESSION['blad'];

tak, żeby wyświetlał się dopiero wtedy, gdy plik zaloguj.php odesłał nas z powrotem do index.php, bo jak na razie index.php traktuje ten skrypt jako napis, który wyświetla się jako podstawowa część strony, a nie warunkowa. Masz pomysł jak to przekształcić, bo ja na razie jestem zielony w tym temacie.