Problem z wyswietlaniem komunikatu

pytanie zadane 10 marca 2016 w PHP przez MlodyJavaS Użytkownik (500 p.)

<html>
<?php
    session_start();
?>
    <head>
<meta http-equiv="Content-Type" content="text/html;charset=UTF-8">
<link href="style/arkusz.css" rel="stylesheet" type="text/css">
<title>Prosty CMS - Logowanie</title>

</head>
<body>
<div class="center">

<div id="panel">
    <form action="index.php" method="post">
        <label for="username">Nazwa użytkownika:</label>
        <input type="text" id="username" name="login">
        <label for="password">Hasło:</label>
        <input type="password" id="password" name="haslo">
        <div id="lower">
        <input type="submit" value="Login" name="wyslij">
        </div>
    </form>
    <?php
        
        if(isset($_SESSION['blad']))
         echo $_SESSION['blad'];
        
    ?>
</div>

</div>
</body>
</html>

<?php
session_start();
    
    
    $login=$_POST['login'];
    $haslo=$_POST['haslo'];
    
    $fp = fopen("login.txt", "r");
    $login_plik = file_get_contents("login.txt");
    
    fclose($fp);
    
    $fp = fopen("haslo.txt", "r");
    $haslo_plik = file_get_contents("haslo.txt");
    
    fclose($fp);
    
    if(($login==$login_plik) || ($haslo==$haslo_plik))
        {   
            
            echo "Witaj <b>$login</b> !";
            unset($_SESSION['blad']);
        
        }
    else
    {  
            $_SESSION['blad']='<span style="color:red">'
            . 'Nieprawidlowy login lub haslo</span>';
             header('Location:loguj.php');
    }
    
?>

Prosze o pomoc w rozwiazaniu kwestii wyswietlania sie napisu o nieprawidolowym loginie i hasle. Chodzi mi o to ze, napis ten wyswietla sie przed kliknieciem przycisku wyslij. PS login i haslo to admin, 123 ( sa one zawarte w plikach .txt)

2 odpowiedzi

odpowiedź 10 marca 2016 przez Mateusz11 Pasjonat (22,910 p.)
edycja 10 marca 2016 przez Mateusz11

<html>
<?php
    session_start();
?>

session_start() daje się na samym początku strony, przed wszystkimi tagami php i html

  if(($login==$login_plik) || ($haslo==$haslo_plik))
        {   
             
            echo "Witaj <b>$login</b> !";
            unset($_SESSION['blad']);
         
        }

Sprawdzasz czy login lub hasło są poprawne!? Żeby się zalogować potrzeba wiedzieć czy i login i hasło są prawidłowe (Zamiast || użyj &&)

   $_SESSION['blad']='<span style="color:red">'

brak średnika

            . 'Nieprawidlowy login lub haslo</span>';

Po co tu kropka?

Ja bym jeszcze dał sprawdzenie czy login i hasło nie są puste i czy przyszły postem ;)

Jak robisz operacje na plikach, to tak samo mógłbyś zdefiniować login i hasło w pliku php

Ogólnie mało skuteczne :(

Polecam skorzystać z bazy danych :)

PS.

   $fp = fopen("login.txt", "r");
    $login_plik = file_get_contents("login.txt");

Po co ta operacja na górze?

<meta http-equiv="Content-Type" content="text/html;charset=UTF-8">       //Wytłumacz mi tą linijke

Wytłumacz mi tą linijke

EDIT:

<title>Prosty CMS - Logowanie</title>

Jak za 10 lat tak będą wyglądały CMS`y...

Pozdrawiam

1	komentarz 10 marca 2016 przez efiku Szeryf (75,160 p.) Dajesz operatora kropki jezeli w stringu chcesz miec zmienna: echo 'Witaj <b>'.$login.'</b> !'; Dlaczego?