Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP]System logowania - problem
Forum PHP.pl > Forum > Przedszkole
cent4
10.03.2010, 11:12:38
Witam.
Mam problem z systemem logowania.
Gdy jest on zewnętrzny (sam system logowania bez zagnieżdżania w stronę) wszystko działa.

Natomiast ja wkładam go w stronę www nie działa. Klikam na zakładkę strefa użytkownika - wpisuje dane i nic - pisze mi, że jestem nie zalogowany, natomiast gdy drugi raz kliknę na strefa użytkownika i wpiszę dane - zaloguję się bez problemu - co jest nie tak?

tak wygląda strona główna:
[PHP] pobierz, plaintext 
  1. <li><a href="index.php?go=home">Home</a></li>
  2. <li><a href="index.php?go=strefa">Strefa uczestnika</a></li>
  3. ...
  4.  
  5. <div id="KONTENER_TLO">
  6.  
  7.                     <?
  8.     					switch ($_GET['go'])
  9.                       	{
  10.                       		case 1:
  11.                       		include("strefa.php");
  12.                       		break;
  13.  
  14.                                 case logout:
  15.                       		include("logout.php");
  16.                       		break;
  17.  
  18.                       		case zalogowany:
  19.                       		include("zalogowany.php");
  20.                       		break;
  21.                         }
  22.                     ?>
  23.  
  24.     	</div>
[PHP] pobierz, plaintext


tak wygląda plik zalogowany.php:
[PHP] pobierz, plaintext 
  1. session_start();
  2.  
  3. if (isset($_SESSION['user_id']) and isset($_SESSION['login']))
  4.  
  5. {
  6. ........
  7.  
  8. }
  9. ?>
[PHP] pobierz, plaintext


a tak wygląda plik strefa.php:
[PHP] pobierz, plaintext 
  1. <?php
  2.    session_start();
  3. ?>
  4.  
  5. .....
  6.  
  7. <form  action="index.php?go=strefa" method="post">
  8.                                      <?php
  9.                  if (isset($_POST['konto']) and isset($_POST['password']) )
  10.  
  11.                   {
  12.  
  13.                   require('conn.php');
  14.  
  15.                   $konto=mysql_real_escape_string(trim($_POST['konto']));
  16.  
  17. ..........
  18.  
  19. }
  20.   else{
  21.  ?>
  22.  TUTAJ MIEJSCE NA LOGIN I HASŁO W HTML
  23. </form>
  24. <?php
  25. }
  26.    ob_end_flush();
  27. ?>
[PHP] pobierz, plaintext
1937817
10.03.2010, 14:22:27
W 2 kodzie pasowało by dopisać na początku <?php
I jeszcze mała edycja 1 kodu:
[PHP] pobierz, plaintext 
  1. <li><a href="index.php?go=home">Home</a></li>
  2. <li><a href="index.php?go=strefa">Strefa uczestnika</a></li>
  3. ...
  4.  
  5. <div id="KONTENER_TLO">
  6.  
  7.                     <?php
  8.     					switch ($_GET['go'])
  9.                       	{
  10.                       		case 1:
  11.                       		include("strefa.php");
  12.                       		break;
  13.  
  14.                                 case logout:
  15.                       		include("logout.php");
  16.                       		break;
  17.  
  18.                       		case zalogowany:
  19.                       		include("zalogowany.php");
  20.                       		break;
  21.                         }
  22.                     ?>
  23.  
  24.     	</div>
  25.  
[PHP] pobierz, plaintext
mortus
10.03.2010, 14:39:27
Skoro w $_GET['go'] przekazujesz nazwę podstrony, to dlaczego masz
[PHP] pobierz, plaintext 
  1. case 1:
  2.     include("strefa.php");
  3.     break;
[PHP] pobierz, plaintext
Powinno być:
[PHP] pobierz, plaintext 
  1. switch($_GET['go']) {
  2.     case 'strefa':
  3.         include("strefa.php");
  4.         break;
  5.     case 'home':
  6.         include("home.php");
  7.         break;
  8.     case 'zalogowany':
  9.         include("zalogowany.php");
  10.         break;
  11.     case 'logout':
  12.         include("logout.php");
  13.         break;
  14.     ...
  15. }
[PHP] pobierz, plaintext
No i gdzie jest sam proces logowania? No i jeszcze w pliku strefa.php sprawdzenie zmiennych $_POST['konto'] i $_POST['password'] oraz samo logowanie powinno się odbyć jeszcze zanim wyślesz do przeglądarki formularz, czyli przed:
[HTML] pobierz, plaintext 
  1. <form action="index.php?go=strefa" method="post">
[HTML] pobierz, plaintext
Oczywiście jeżeli użytkownik nie został jeszcze zalogowany. Podałeś za mało kodu i trudno się domyślić, co jest nie tak.
cent4
10.03.2010, 16:55:24
Problem leży na pewno po stronie pliku zalogowany.php, gdy "//if (isset($_SESSION['user_id']) and isset($_SESSION['login']))" jest wyłączone - wszystko działa prawidłowo, tylko nie pisze, jaki user jest zalogowany (Witaj xxx). Natomiast jak to włączę - muszę logować się 2 razy: 

[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3. ?>
  4.  
  5. ............
  6.  
  7. <?php
  8.  
  9.  
  10. //if (isset($_SESSION['user_id']) and isset($_SESSION['login']))
  11.  
  12. {
  13.  
  14. echo "<CENTER><img src=\"images/otwarte.png\" width=\"90\" height=\"90\" border=\"0\"></CENTER>";
  15.  
  16. echo "Witaj: ";
  17.  
  18. echo $_SESSION['login'];
  19.  
  20. echo "<br />";
  21.  
  22. echo "Jestes zalogowany w strefie użytkownika. ";
  23.  
  24. echo "<br /><br />";
  25.  
  26. echo "
  27.     <TABLE>
  28.         <TR>
  29.             <TD width=\"80px\">
  30.            	    <img src=\"images/pdf-logo.png\" width=\"40\" height=\"40\" border=\"0\">
  31.             </TD>
  32.             <TD>
  33.                 <a href='plik.php?nazwa=test.pdf'>pobierz test</a>
  34.  
  35.             </TD>
  36.         </TR>
  37.         <TR>
  38.             <TD width=\"80px\">
  39.            	    <img src=\"images/pdf-logo.png\" width=\"40\" height=\"40\" border=\"0\">
  40.             </TD>
  41.             <TD>
  42.                 <a href='plik.php?nazwa=test2.pdf'>pobierz test2</a>
  43.             </TD>
  44.         </TR>
  45.     </TABLE>              
  46.     "
  47.     ;
  48.  
  49.  
  50. echo "<br /><br />";
  51.  
  52. echo "<a href=\"index.php?go=logout\">Wyloguj się</a>";
  53.  
  54. }
  55.  
  56. //else
  57.  
  58. //{
  59.  
  60. //echo "Nie jestes zalogowany";
  61.  
  62.  
  63.  
  64. //}
  65. ?>
[PHP] pobierz, plaintext
mortus
10.03.2010, 17:57:46
Czy mi się wydaje, czy Ty w każdym pliku rozpoczynasz sesję (session_start())? Nie możesz tak robić, sesję powinieneś rozpocząć tylko raz, najlepiej w pliku index.php.
cent4
11.03.2010, 07:58:49
Dalej problem ten sam...

Jak wyłączę:

[PHP] pobierz, plaintext 
  1. //   if (isset($_SESSION['user_id']) and isset($_SESSION['login']))
[PHP] pobierz, plaintext


wszystko jest ok...

Co może być problemem?
bobo168
11.03.2010, 08:51:44
Wyświetl sobie te sesje ($_SESSION['user_id'],$_SESSION['login'])) , zobacz czy coś w nich w ogóle jest.
cent4
11.03.2010, 09:01:24
Nie ma nic w tych sesjach za pierwszym razem jak się loguję...

Jak drugi raz się loguję to już mam dane w tych sesjach.

O co może chodzić?
mortus
11.03.2010, 09:04:47
Wrzuć tutaj skrypt logowania, bo wygląda na to, że tam tkwi problem.
cent4
11.03.2010, 09:15:38
Nie wiem czy przypadkiem tutaj nie tkwi problem bo:
1. <form action="index.php?go=strefa" method="post"> - odnosi się do indexu najpierw
2. w skrypcie logowania to samo w linii: header("Location: $domena"."index.php?go=zalogowany");

Na początku jesk kod html - nie ma nic w php w tym pliku oprócz tego co zamieszczam poniżej
sesje php zaczynam na stronie index.php
Oto fragment odpowiedzialny za logowanie:
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3.                                     unset($_SESSION['user_id']);
  4.                                     unset($_SESSION['login']);
  5.                                     echo "sesja user_id: ";
  6.                                     echo $_SESSION['user_id'];
  7.                                     echo "sesja login: ";
  8.                                     echo $_SESSION['login'];
  9.  
  10.  
  11.                                     if (isset($_POST['konto']) and isset($_POST['password']) )
  12.                                     {
  13.                                         require('conn.php');
  14.                                         $konto=mysql_real_escape_string(trim($_POST['konto']));
  15.                                         $password=mysql_real_escape_string(trim($_POST['password']));
  16.                                         if ($konto!="" and $password!="")
  17.                                         {
  18.                                             $password = md5($password);
  19.                                             $zapytanie="SELECT id FROM user WHERE login='$konto' and password ='$password'";
  20.                                             $temp=mysql_query($zapytanie) or die("Wystšpił błšd");
  21.                                             $ile=mysql_num_rows($temp);
  22.                                             $temp=mysql_fetch_array($temp);
  23.                                             $id=$temp['id'];
  24.                                             if ($ile==1)
  25.                                             {
  26.                                                 $_SESSION['user_id']=$id;
  27.                                                 $_SESSION['login']=$konto;
  28.                                                 //echo('Zostales zalogowany. ');
  29.                                                 $domena='http://nawr.pl/';
  30.                                                 header("Location: $domena"."index.php?go=zalogowany"); 
  31.                                                 exit;
  32.                                             }
  33.                                             else echo ('Podales zle dane. Kliknij wstecz aby sprobowac ponownie.');
  34.                                         }
  35.                                     }
  36.                                     else{
  37.                                 ?>
  38.  
  39.  
  40. 								<form  action="index.php?go=strefa" method="post">
  41.  
  42.                                      <table align="center">
  43.                                         <tr>
  44.                                             <td colspan="2" align="center">
  45.                                                 <img src="images/zamkniete.png" width="90" height="90" border="0">    
  46.                                             </td>
  47.                                         </tr>
  48.                                         <tr>
  49.                                             <td>
  50.                                                 <br>
  51.                                                 <br>
  52.                                             </td>
  53.                                         </tr>
  54.                                         <tr>
  55.                                             <td>
  56.                                                 <img src="images/user.png" width="40" height="40" border="0">
  57.                                             </td>
  58.                                             <td>
  59.                                                 <input onclick="this.value=''" name="konto" value="Nazwa konta" type="text" />
  60.                                             </td>
  61.                                         </tr>
  62.                                         <tr>
  63.                                             <td>
  64.                                                 <img src="images/klucze.png" width="40" height="40" border="0">
  65.                                             </td>
  66.                                             <td>
  67.                                                 <input onclick="this.value=''" name="password" value="Hasło" type="text" />
  68.                                             </td>
  69.                                         </tr>
  70.                                         <tr>
  71.                                             <td>
  72.                                             </td>
  73.                                             <td align="center">
  74.                                                 <input type="submit" value="Zaloguj" />
  75.                                             </td>
  76.                                         </tr>
  77.                                     </table>
  78.                                 </form>
  79.  
  80.  
  81.                                 <?php
  82.                                     }
  83. ?>
[PHP] pobierz, plaintext
mortus
11.03.2010, 10:22:51
Z założenia zmienne $_SESSION['user_id'] oraz $_SESSION['login'] nie powinny istnieć, więc po co ten kod:
[PHP] pobierz, plaintext 
  1. unset($_SESSION['user_id']);
  2. unset($_SESSION['login']);
[PHP] pobierz, plaintext
Powinniśmy raczej zrobić coś takiego:
[PHP] pobierz, plaintext 
  1. if(isset($_SESSION['user_id'] && $_SESSION['user_id'] != '' && isset($_SESSION['login']) && $_SESSION['login'] != '') {
  2. echo 'Jesteś już zalogowany!';
  3. // opcjonalnie
  4. // header("Location: index.php?go=zalogowany");
  5. }
[PHP] pobierz, plaintext
Poza tym użyłbym raczej operatora && zamiast and, ponieważ oba te operatory działają inaczej.
W funkcji header() wystarczy podać "Location: index.php?go=zalogowany" .
No i pozostaje jeszcze kwestia tego, czy w pliku index.php nie dochodzi gdzieś do modyfikacji zmiennych $_SESSION['user_id'] i $_SESSION['login']. Najlepiej wrzuć tutaj kod całego pliku index.php.
cent4
15.03.2010, 14:13:20
Mortus wszystko działa pięknie - WIELKIE DZIĘKI guitar.gif

A jak zrobić, żeby w sesji też przekazać imie i nazwisko uzytkownika, tak jak jest to zrobione z "loginem" i "id"?
Oczywiście pola imie i nazwisko mam w tabeli
mortus
15.03.2010, 15:35:44
Zamiast pobierać z bazy danych samo id pobierasz również imię i nazwisko (linia 19)
[PHP] pobierz, plaintext 
  1. $zapytanie = "SELECT id, imie, nazwisko FROM user WHERE login='$konto' AND password='$password'";
[PHP] pobierz, plaintext
i przepisujesz do sesji w bloku
[PHP] pobierz, plaintext 
  1. if($ile == 1) {
  2.     $_SESSION['id'] = $temp['id'];
  3.     $_SESSION['imie'] = $temp['imie'];
  4.     $_SESSION['nazwisko'] = $temp['nazwisko'];
  5.     // reszta operacji
  6. }
[PHP] pobierz, plaintext
cent4
16.03.2010, 07:41:22
mortus You Are The Best - dzienks yahoo.gif
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.