Jak zrobić żeby po zalogowaniu znikło w menu opcja..., Nie wiem jak sformułować pytanie w google więc pisze na forum... Opcje

[plastus321]

Nie wiem jak sformułować pytanie w google więc pisze na forum, jeśli więc był taki temat to serdecznie przepraszam.

Jak zrobić by po zalogowaniu znikło z menu np. "Logowanie" i "Rejestracja" a na miejscu tego pojawiło się np. "Profil" i "Wyloguj"?


próbowałem czegoś takiego (uprzedzę że kod jest wycięty z środka pliku .php):

[PHP] pobierz, plaintext 
if(isset($_SESSION['login'])) 
		{
 
echo "<a href='index.php?page=profil'>$login</a> | ";
echo "<a href='index.php?page=wyloguj'>Wyloguj</a><br>";    
 
  		}else{
 
echo "<a href='index.php?page=logowanie'>Logowanie</a> | ";
echo "<a href='index.php?page=rejestracja'>Rejestracja</a><br>";
 
}
[PHP] pobierz, plaintext 

ale niestety... nie wypaliło u mnie

Serdecznie proszę o pomoc jeśli można

[markonix]

Wpisz najpierw:

Kod

<?php print_r($_SESSION);?>

i pokaż przed i po zalogowaniu jak wygląda wynik.

[plastus321]

Takie coś mi wypisało:

[PHP] pobierz, plaintext 
Array ( [zalogowany] => 1 [login] => plastus321 [haslo] => pawlixer1 ) 
[PHP] pobierz, plaintext 

[Kshyhoo]

To:

[PHP] pobierz, plaintext 
if(isset($_SESSION['login'])) 
[PHP] pobierz, plaintext 

zamień na:

[PHP] pobierz, plaintext 
if(isset($_SESSION['zalogowany'])) 
[PHP] pobierz, plaintext 

[plastus321]

Niestety bez zmian...

[toaspzoo]

a startujesz sesje?

if($_SESSION['zalogowany'] <> '')
{
echo 'jesli zalogowany...';
}

[grrizli]

zamień na:

[PHP] pobierz, plaintext 
if($_SESSION['zalogowany']) 
[PHP] pobierz, plaintext 

widocznie jako niezalogowany posiada wartość 0 ta zmienna

[plastus321]

również nici z tego...
a jest jakiś jeszcze inny sposób czy muszę pewnie kod po przestawiać

[sadistic_son]

ale niestety... nie wypaliło u mnie

...bo stało się co? Komp eksplodował? Napisz co nie wypaliło Która z dwóch możliwości warunku zaistniała?


Jeśli sesję tworzysz poprawnie, a print_r pokazuje, że raczej tak, to to co pokazałeś musi działać. Błąd leży najwidoczniej gdzie indziej. Pokaż więcej kodu.

PS. Przechowywanie w sesji hasła zalogowanego usera to bardzo zły pomysł.

Ten post edytował sadistic_son 12.05.2011, 05:02:48

[plastus321]

Oto cały kod:

[PHP] pobierz, plaintext 
<?php
require('conn.php');
session_start;
 
echo '<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2" />';
 
$login=mysql_real_escape_string(trim($_POST['login']));
$email=$_POST['email'];
$haslo=mysql_real_escape_string(trim($_POST['haslo']));
$haslo2=$_POST['haslo2'];
$regulamin=$_POST['regulamin'];
$rejestruj=$_POST['rejestruj'];
 
 
echo "<center>";
if(isset($_SESSION['zalogowany'])) 
		{
 
echo "<a href='index.php?page=profil'>$login</a> | ";
echo "<a href='index.php?page=wyloguj'>Wyloguj</a><br>";    
 
  		}else{
 
echo "<a href='index.php?page=logowanie'>Logowanie</a> | ";
echo "<a href='index.php?page=rejestracja'>Rejestracja</a><br>";
 
}
echo "</center><br />";
 
 
 
 
$page=$_GET['page'];
 
	switch($page)
		{
 
 
			case 'logowanie':
			echo "<fieldset>
			<legend align='center'>Logowanie</legend>
			<form method='post'>
			Login:<br><input name='login' type='text' value='' /><br />
			Hasło:<br><input type='password' name='haslo' value='' /><br />
			</fieldset>						
			<center>
			<input type='submit' name='loguj' value='Loguj' />
			<input type='reset' value='Wyczyść dane' />
			</center>
			</form>";
 
			if(isset($_POST['loguj'])) {
 
 
			   if(mysql_num_rows(mysql_query("SELECT *
			   FROM user WHERE login = '".$_POST['login']."' 
			   && haslo = '".$_POST['haslo']."' ")) > 0) {
 
 
			       if(mysql_num_rows(mysql_query("SELECT * FROM user
			       WHERE login = '".$_POST['login']."' 
			       && haslo = '".$_POST['haslo']."' ")) > 0 ) {
 
 
			           $_SESSION['zalogowany'] = true;
			           $_SESSION['login'] = $_POST['login'];
			           $_SESSION['haslo'] = $_POST['haslo'];
			           header('Location: index.php');
 
 
			       } else { 
 
			   echo "Złe hasło, proszę spróbować ponownie";
			}
			} else { 
			   echo "Nie ma takiego użytkownika";
			}}
 
			break;
 
			case 'rejestracja':
 
 
			$ile=mysql_query("SELECT * FROM `user` WHERE login = '$login'");
			$ile=mysql_num_rows($ile);
 
			$zapytanie = "CREATE TABLE user (id int NOT NULL AUTO_INCREMENT, 
			login varchar(50), email varchar(30), haslo varchar(30), miejscowosc varchar(30), 
			imie varchar(30), nazwisko varchar(30), wiek varchar(30), 
			PRIMARY KEY(id))
			ENGINE = InnoDB CHARACTER SET utf8 COLLATE utf8_polish_ci;";
			$wykonaj=mysql_query($zapytanie);
 
 
		if($rejestruj)
		{
			if(isset($login) and ($email) and ($haslo) and ($haslo2))
			{
 
			      		if($haslo2 == $haslo)
	      						{
 
							if($regulamin)	    			  
	  							{
 
			      						if($ile==0)
			      							{
 
								$haslo = sha1($haslo);
								$dodaj_user= ("INSERT INTO user SET login='$login', email='$email', haslo='$haslo'");
					          		$dodawanie_usera=mysql_query($dodaj_user);
 
								echo "<center><b><font color='green'>Twoje konto $login zostało utworzne!</font></b></center>";
 
										}else{echo "<center><b><font color='red'>Taki użytkownik już istnieje. Wybierz inny login.</b></center></font>";}
 
								}else{echo "<center><b><font color='red'>Musisz akceptować regulamin.</b></center></font>";}	//Sprawdza czy regulamin jest akceptowany	
 
							}else{echo "<center><b><font color='red'>Hasła się nie zgadzają.</b></center></font>";}	  //Sprawdza czy hasła się zgadzają		
 
	  		}else{echo "<center><b><font color='red'>Wypełnij wszystkie pola.</b></center></font>";} //Sprawdza czy wszystkie pola są wypełnione
		}
 
			echo "
			<fieldset>
			<legend align='center'>Rejestracja konta</legend>
			<form method='post'>
			Login:<br><input name='login' type='text' value='' /><br />
			Email:<br><input name='email' type='text' value='' /><br />
			Hasło:<br><input type='password' name='haslo' value='' /><br />
			Powtórz hasło:<br><input type='password' name='haslo2' value='' /><br />
			<br />
			<input type='checkbox' name='regulamin' /> - Akceptuję <a href='index.php?page=regulamin'>regulamin</a>.
			</fieldset>						
			<br /><center>
			<input type='submit' name='rejestruj' value='Rejestruj' />
			<input type='reset' value='Wyczyść dane' />
			</center>
			</form>";
 
			break;
 
 
		default:
		echo "Strona główna";
 
			}
 
?>
[PHP] pobierz, plaintext 

[markonix]

Kod beznadziejny, po co te $login? To na marginesie.

A prosiłem o podanie zawartości tablic w dwóch przypadkach - zalogowany i po wylogowaniu.

[plastus321]

Zawartość tablic wyskakuje tylko w jednym przypadku...

...po zalogowaniu.

[PHP] pobierz, plaintext 
Array ( [zalogowany] => 1 [login] => plastus321 [haslo] => pawlixer1 ) 
[PHP] pobierz, plaintext 

przed zalogowaniem nic nie wyskakuje...

[Kshyhoo]

Poprawnie jest

[PHP] pobierz, plaintext 
session_start();
[PHP] pobierz, plaintext 

Spróbuj zamienić:

[PHP] pobierz, plaintext 
require('conn.php');
session_start();
[PHP] pobierz, plaintext 

na:

[PHP] pobierz, plaintext 
session_start();
require('conn.php');
[PHP] pobierz, plaintext 

lub użyj buforowania.

[plastus321]

Trochę czytałem o tym buforowaniu, ale nie mogę sobie z tym poradzić... sorki, że was tak gnębię, ale dopiero co zacząłem uczyć się PHP.

Macie jakąś inną poradę dla mnie, żeby to zrobić?

Zależy mi najbardziej na tym żeby ten kod pozostał w funkcji switch.

Nie chce rozdzielać tego na kilka plików .php

Z góry i tak dziękuje za chęci

[Rid]

Jak już che Pan poprawić coś w swoim kodzie to należałoby dać:
<form method='post'>

przed

<fieldset>
<legend align='center'>Logowanie</legend>

W kodzie HTML także obowiązuje hierarchizacja.

[plastus321]

A dziękuje za uwagę już to u siebie poprawiłem

Szkoda tylko, że to o co mi chodzi od samego początku nie zależało właśnie od tego błędu

ale mam nadzieję, że znajdzie się ktoś kto zauważy jakiś podstawowy błąd w tym kodzie (pewnie jest ich dużo) który powoduje to,
że nie działa tak jak należy albo nie tak jakbym chciał.

[Rid]

Proszę powiedzieć mi kto tak robi?
$_SESSION['haslo'] = $_POST['haslo'];
Dobrym nawykiem jest nieprzechowywać haseł w sesji.

Druga rzecz:

if(mysql_num_rows(mysql_query("SELECT *
FROM user WHERE login = '".$_POST['login']."'
&& haslo = '".$_POST['haslo']."' ")) > 0)

a do bazy zapisuje pan tak:
$haslo = sha1($haslo);
$dodaj_user= ("INSERT INTO user SET login='$login', email='$email', haslo='$haslo'");
Wniosek:
Porównuje Pan hasło z inputa z zakodowanym hasłem z bazy danych co w rezultacie nie przejdzie.

Niech Pan przeniesie $haslo = sha1($haslo) przed:

if(mysql_num_rows(mysql_query("SELECT *
FROM user WHERE login = '".$_POST['login']."'
&& haslo = '".$_POST['haslo']."' ")) > 0)

,albo dodać

if(mysql_num_rows(mysql_query("SELECT *
FROM user WHERE login = '".$_POST['login']."'
&& haslo = '".sha1($_POST['haslo'])."' ")) > 0)

Ten post edytował Rid 12.05.2011, 23:26:57

[plastus321]

Poprawiłem już to dzięki za uwagę

Może się mylę, ale zdaje mi się, że sesje nie chcą mi działać przez jakiś błąd w logowaniu...

Macie jakiś znany i prosty skrypt logowania który może byłby lepszy od tego??