[MySQL][PHP]Skrypt logowania - proszę o pomoc w naprawie Opcje

[neneth]

Witam, mam problem ze skryptem logowania. Gdy wpisuję login i hasło, zamiast przekierowania na stronę główną, pojawia się nowe okno z białym tłem i przekierowaniem do pliku login.php . Siedzię nad ty już jakiś czas, proszę więc o pomoc. Oto te pliki:

login.php

[PHP] pobierz, plaintext 
session_start();
ob_start();
$_SESSION['login']=$konto; 
if ($_SESSION['login'] > 0){
echo "Nie jesteĹ_ zalogowany...";
header("Location: strona_glowna.php");
exit;
}
 
session_start();
 
if (isset($_POST['konto']) and isset($_POST['password']) )
 
{
 
require('conn.php');
 
$konto=mysql_real_escape_string(trim($_POST['konto']));
 
$password=mysql_real_escape_string(trim($_POST['password']));
 
if ($konto!="" and $password!="")
 
{
 
$password = sha1(md5($password));
 
$zapytanie="SELECT id FROM user WHERE login='$konto' and password ='$password'";
 
$temp=mysql_query($zapytanie) or die("WystÄ_piĹ_ bĹ_Ä_d");
 
$ile=mysql_num_rows($temp);
 
$temp=mysql_fetch_array($temp);
 
$id=$temp['id'];
 
 
 
if ($ile==1)
 
{
 
$_SESSION['user_id']=$id;
 
$_SESSION['login']=$konto;
 
echo('ZostaĹ_eĹ_ zalogowany. ');
 
}
 
else echo ('PodaĹ_eĹ_ zĹ_e dane. Kliknij wstecz, aby sprĂłbowaÄ_ ponownie.');
 
}
 
}
 
 
 
ob_end_flush();
 
[PHP] pobierz, plaintext 

conn.php

[PHP] pobierz, plaintext 
 
session_start();
ob_start();
$_SESSION['login']=$konto;
if ($_SESSION['login'] > 0)
 {
echo "Nie jesteś zalogowany...";
header("Location: strona_glowna.php");
exit;
}
    $host='localhost'; //Tu moĹźliwe Ĺźe bÄ_dziesz musiaĹ_ wpisaÄ_ innego hosta ale czÄ_sto jest to wĹ_aĹ_nie localhost
    $login='...............................';
    $haslo='...................................';
    $nazwaBazy='..................................';
    $lacz=mysql_connect($host, $login, $haslo);
    if ($lacz==false) {
 
    die('Brak polaczenia z serwerem');
 
    }
 
    mysql_select_db($..........................................) or die(mysql_error());
 
    	mysql_close();
	ob_end_flush();
[PHP] pobierz, plaintext 

Czy mógłby mi ktoś skorygować błędy? Bardzo proszę, to dla mnie ważne.

Ten post edytował neneth 11.10.2010, 16:09:44

[bmL]

Otwierasz sesję 3 razy session_start(); wystarczy jeden raz na samym początku... Uruchamiasz bufor wyjścia 2 razy (w pliku conn i w pliku login) to też powinno być wykonane raz.

if ($_SESSION['login'] > 0){
echo "Nie jesteĹ_ zalogowany...";
header("Location: strona_glowna.php");
exit;
}

yyy, jeżeli login jest większy od zera (użytkownik jest zalogowany?) to otrzymuje komunikat który mówi iż nie jest zalogowany? Domyślam się że miało być że jest już zalogowany.

$_SESSION['login']=$konto; co ma robić ta linijka na początku? Skąd zawartość zmiennej $konto się bierze?

To tyle na dobry początek.

[neneth]

Witam, linijki poprawiłem i dalej nie działa, więc już nie mam pojęcia, dlaczego. Zamieszczam poprawiony kod jeszcze raz

plik conn.php

[PHP] pobierz, plaintext 
 
session_start();
ob_start();
 
if ($_SESSION['login'] > 0)
 {
echo "Jesteś zalogowany...";
header("Location: strona_glowna.php");
exit;
}
    $host='localhost'; //Tu moĹźliwe Ĺźe bÄ_dziesz musiaĹ_ wpisaÄ_ innego hosta ale czÄ_sto jest to wĹ_aĹ_nie localhost
    $login='...........................';
    $haslo='.............................';
    $nazwaBazy='...................................';
    $lacz=mysql_connect($host, $login, $haslo);
    if ($lacz==false) {
 
    die('Brak polaczenia z serwerem');
 
    }
 
    mysql_select_db($............................) or die(mysql_error());
 
    	mysql_close();
	ob_end_flush();
 
[PHP] pobierz, plaintext 

plik login.php

[PHP] pobierz, plaintext 
 
if (isset($_POST['konto']) and isset($_POST['password']) )
 
{
 
require('conn.php');
 
$konto=mysql_real_escape_string(trim($_POST['konto']));
 
$password=mysql_real_escape_string(trim($_POST['password']));
 
if ($konto!="" and $password!="")
 
{
 
$password = sha1(md5($password));
 
$zapytanie="SELECT id FROM user WHERE login='$konto' and password ='$password'";
 
$temp=mysql_query($zapytanie) or die("WystÄ_piĹ_ bĹ_Ä_d");
 
$ile=mysql_num_rows($temp);
 
$temp=mysql_fetch_array($temp);
 
$id=$temp['id'];
 
 
 
if ($ile==1)
 
{
 
$_SESSION['user_id']=$id;
 
$_SESSION['login']=$konto;
 
echo('ZostaĹ_eĹ_ zalogowany. ');
 
}
 
else echo ('PodaĹ_eĹ_ zĹ_e dane. Kliknij wstecz, aby sprĂłbowaÄ_ ponownie.');
 
}
 
}
 
 
 
ob_end_flush();
 
 
[PHP] pobierz, plaintext 

Co teraz jest źle? Ciągle wyskakuje biała karta po zalogowaniu i nie ma żadnego komunikatu.

[bmL]

Podaj więcej informacji, po wysłaniu formularza pojawia się biała strona na górze strony wpasku adresu widnieje... login.php?
Po tej białej stronie użytkownik zostaje zalogowany czy nie?

[neneth]

Po zalogowaniu nie chce mnie przenieść na stronę index.php , więc nie mam pojęcia o c teraz chodzi, a siedzę nad tym już 3 dni.

Podam teraz pliki po modyfikakcji:

plik login.php

[PHP] pobierz, plaintext 
 
if (isset($_POST['konto']) and isset($_POST['password']) )
 
{
 
require('conn.php');
 
$konto=mysql_real_escape_string(trim($_POST['konto']));
 
$password=mysql_real_escape_string(trim($_POST['password']));
 
if ($konto!="" and $password!="")
 
{
 
$password = sha1(md5($password));
 
$zapytanie="SELECT id FROM user WHERE login='$konto' and password ='$password'";
 
$temp=mysql_query($zapytanie) or die("Wystąpił błąd");
 
$ile=mysql_num_rows($temp);
 
$temp=mysql_fetch_array($temp);
 
$id=$temp['id'];
 
 
 
if ($ile==1)
 
{
 
$_SESSION['user_id']=$id;
 
$_SESSION['login']=$konto;
 
echo('Zostałeś zalogowany. ');
 
}
 
else echo ('Podałeś złe dane. Kliknij wstecz, aby spróbować ponownie.');
 
}
 
}
 
 
 
ob_end_flush();
 
?>
 
 
[PHP] pobierz, plaintext 

i plik conn.php

[PHP] pobierz, plaintext 
 
session_start();
ob_start();
 
if ($_SESSION['login'] < 0)
 {
echo "Jesteś zalogowany...";
header("Location: index.php");
exit;
}
    $host='localhost'; //Tu moĹźliwe Ĺźe bÄ_dziesz musiaĹ_ wpisaÄ_ innego hosta ale czÄ_sto jest to wĹ_aĹ_nie localhost
    $login='................................';
    $haslo='...................................';
    $nazwaBazy='.................................';
    $lacz=mysql_connect($host, $login, $haslo);
    if ($lacz==false) {
 
    die('Brak polaczenia z serwerem');
 
    }
 
    mysql_select_db($..........................) or die(mysql_error());
 
    	mysql_close();
	ob_end_flush();
 
[PHP] pobierz, plaintext 

Ten post edytował neneth 12.10.2010, 14:00:34

[bmL]

ob_start wrzuć na początek login.php bo nie będziesz mógł zrobić przekierowania w tym pliku. Usuń z pliku conn.php ob_start i ob_end_flush bo Ci zatrzyma bufor w połowie drogi...
#
echo('Zostałeś zalogowany. ');
#
Logicznie rzecz biorą to tutaj powinno się znaleźć przekierowanie tak więc Header('Location: index.php');

[neneth]

Dziękuję za podpowiedź. Naprawiłam pliki, ale teraz pojawił się błąd

Parse error: syntax error, unexpected T_ELSE in /login.php on line 52

Moje pliki wyglądają teraz tak:

plik conn.php

[PHP] pobierz, plaintext 
 
session_start();
 
if ($_SESSION['login'] < 0)
 {
echo "Jesteś zalogowany...";
 
exit;
}
    $host='localhost'; //Tu moĹźliwe Ĺźe bÄ_dziesz musiaĹ_ wpisaÄ_ innego hosta ale czÄ_sto jest to wĹ_aĹ_nie localhost
    $login='..............................';
    $haslo='..........................';
    $nazwaBazy='....................................';
    $lacz=mysql_connect($host, $login, $haslo);
    if ($lacz==false) {
 
    die('Brak polaczenia z serwerem');
 
    }
 
    mysql_select_db($...................................) or die(mysql_error());
 
    	mysql_close();
 
[PHP] pobierz, plaintext 

plik login.php

[PHP] pobierz, plaintext 
 
ob_start();
if (isset($_POST['konto']) and isset($_POST['password']) )
 
{
 
require('conn.php');
 
$konto=mysql_real_escape_string(trim($_POST['konto']));
 
$password=mysql_real_escape_string(trim($_POST['password']));
 
if ($konto!="" and $password!="")
 
{
 
$password = sha1(md5($password));
 
$zapytanie="SELECT id FROM user WHERE login='$konto' and password ='$password'";
 
$temp=mysql_query($zapytanie) or die("Wystąpił błąd");
 
$ile=mysql_num_rows($temp);
 
$temp=mysql_fetch_array($temp);
 
$id=$temp['id'];
 
 
 
if ($ile==1)
 
{
 
$_SESSION['user_id']=$id;
 
$_SESSION['login']=$konto;
 
echo('Zostałeś zalogowany. ');
 
}
header('Location: index.php'); 
else echo ('Podałeś złe dane. Kliknij wstecz, aby spróbować ponownie.');
 
}
 
}
 
 
 
ob_end_flush();
 
 
[PHP] pobierz, plaintext 

Teraz to ja już nie wiem, co może być jeszcze źle. Proszę o wskazanie błędów.

[Ulysess]

pewności nie mam ale jeśli używasz else musisz użyć klamr czyli od kąd do kąd obejmuje

[bmL]

Nie rozumiem neneth, nie znasz podstaw php a za takie rzeczy się chwytasz.

[PHP] pobierz, plaintext 
ob_start();
if (isset($_POST['konto']) and isset($_POST['password'])) {
 
    require('conn.php');
 
    $konto = mysql_real_escape_string(trim($_POST['konto']));
 
    $password = mysql_real_escape_string(trim($_POST['password']));
 
    if ($konto != "" and $password != "") {
 
        $password = sha1(md5($password));
 
        $zapytanie = "SELECT id FROM user WHERE login='$konto' and password ='$password'";
 
        $temp = mysql_query($zapytanie) or die("Wystąpił błąd");
 
        $ile = mysql_num_rows($temp);
 
        $temp = mysql_fetch_array($temp);
 
        $id = $temp['id'];
 
 
 
        if ($ile == 1) {
 
            $_SESSION['user_id'] = $id;
 
            $_SESSION['login'] = $konto;
 
            echo('Zostałeś zalogowany. ');
        } else {
            echo ('Podałeś złe dane. Kliknij wstecz, aby spróbować ponownie.');
            header('Location: index.php');
        }
    }
}
 
 
 
ob_end_flush();
 
[PHP] pobierz, plaintext 

Ten post edytował bmL 12.10.2010, 20:51:20

[neneth]

Witam, poprawiłem pliki zgodnie ze wskazówkami. Oto one:

plik conn.php

[PHP] pobierz, plaintext 
 
session_start();
 
if ($_SESSION['login'] < 0)
 {
echo "Jesteś zalogowany...";
 
exit;
}
    $host='localhost'; //Tu moĹźliwe Ĺźe bÄ_dziesz musiaĹ_ wpisaÄ_ innego hosta ale czÄ_sto jest to wĹ_aĹ_nie localhost
    $login='....................................';
    $haslo='........................';
    $nazwaBazy='..........................';
    $lacz=mysql_connect($host, $login, $haslo);
    if ($lacz==false) {
 
    die('Brak polaczenia z serwerem');
 
    }
 
    mysql_select_db($...................................) or die(mysql_error());
 
    	mysql_close();
 
 
[PHP] pobierz, plaintext 

plik login.php

[PHP] pobierz, plaintext 
 
<?php
 
      ob_start();
 
      if (isset($_POST['konto']) and isset($_POST['password'])) {
 
      require('conn.php');
 
      $konto = mysql_real_escape_string(trim($_POST['konto']));
 
      $password = mysql_real_escape_string(trim($_POST['password']));
 
      if ($konto != "" and $password != "") {
 
      $password = sha1(md5($password));
 
      $zapytanie = "SELECT id FROM user WHERE login='$konto' and password ='$password'";
 
      $temp = mysql_query($zapytanie) or die("Wystąpił błąd");
 
      $ile = mysql_num_rows($temp);
 
      $temp = mysql_fetch_array($temp);
 
      $id = $temp['id'];
 
 
      if ($ile == 1) {
 
 
 
      $_SESSION['user_id'] = $id;
 
 
 
      $_SESSION['login'] = $konto;
 
 
 
      echo('Zostałeś zalogowany. ');
 
      } else {
 
      echo ('Podałeś złe dane. Kliknij wstecz, aby spróbować ponownie.');
 
      header('Location: index.php');
 
      }
 
      }
 
      }
 
      ob_end_flush();
 
 
 
[PHP] pobierz, plaintext 

Nie wiem, co tym razem jest źle. Proszę o odpowiedź pełną i konkretnie co gdzie mam wstawić, bo ja dopiero się uczę i chciałbym wiedzieć, jakie błędy popełniam. To co do tej pory poprawiłem zostało mi wyjaśnione na innych forach (m. in.).

[nospor]

A czymże objawia się tym razem to "źle"?
Lektura obowiązkowa dla Ciebie:
Temat: Jak poprawnie zada pytanie