[MySQL][PHP]Znowu o logowaniu, Proszę, spójrzcie tu Opcje

[andrzejlechniak]

Hej, mam taką sprawę. Mam skrypt logowania. I tu się pojawiają dwa problemy. Jak się loguję, to jest ok. Ale kiedy nie podam któregoś z pól, to wyskakuje pusta strona zamiast komunikatu, że jest błąd. Drugi to ważniejsza sprawa. Mam inną stronę i chcę przenieść się do strefy dla zalogowanych. Gdy klikam np. 'przejdź' to owszem jestem w tej strefie, ale od razu jestem zalogowany, mimo iż wcześniej się nie logowałem. Możecie na to spojrzeć, PROSZĘ:

Skrypt logowania:

[PHP] pobierz, plaintext 
if($_GET["wyloguj"] == '1') $_SESSION["logged"] = "nie";
 
							if($_SESSION["logged"] != "tak") 
							{
								if (empty($_POST)) {
 
								 echo '<form name="form1" action="user.php?id=konto" method="Post" >
										<div class="log_main">
										<div>&rsaquo; LOGOWANIE DO SERWISU</div>
										<div>	
											<div class="log_form">Użytkownik: <input name="myusername" type="text" id="myusername"></div>
											<div class="log_form">Hasło: <input name="mypassword" type="password" id="mypassword"></div>
											<div class="log_form">&nbsp;
												<input type="submit" name="Submit" value="Zaloguj" title="zaloguj się">
											</div>
										</div>
										</div>
									   </form>';
 
								}
 
								// username and password sent from form
								$myusername=$_POST['myusername'];
								$mypassword=$_POST['mypassword'];
 
								// To protect MySQL injection
								$myusername = stripslashes($myusername);
								$mypassword = stripslashes($mypassword);
								$myusername = mysql_real_escape_string($myusername);
								$mypassword = mysql_real_escape_string($mypassword);
 
								$sql="SELECT * FROM log WHERE login='$myusername' and password='$mypassword'";
								$result=mysql_query($sql);
 
								$count=mysql_num_rows($result);
 
								if($count==1){
									$_SESSION["logged"] = "tak";
									$_SESSION["username"] = $myusername;
								}
								else {
									return "Błędny login lub hasło";
								}
							}
 
							if($_SESSION["logged"] == "tak") {
 
										echo '<div>Witaj, jesteś w strefie dla zalogowanych użytkowników! Jesteś zalogowany jako: 
											  <strong>' .$_SESSION['username']. '</strong> | <a href="user.php?id=konto&wyloguj=1" title="Wyloguj się z serwisu">Wyloguj się</a><br />';
										echo '</div>';
 
							}
[PHP] pobierz, plaintext 

i sam skrypt, który powinien być na stronie dla zalogowanych użytkowników. Nie wiem czemu nie działa:

[PHP] pobierz, plaintext 
if($_SESSION["logged"] == "tak") {
 
										echo '<div>Witaj, jesteś w strefie dla zalogowanych użytkowników! Jesteś zalogowany jako: 
											  <strong>' .$_SESSION['username']. '</strong> | <a href="user.php?id=konto&wyloguj=1" title="Wyloguj się z serwisu">Wyloguj się</a><br />';
										echo '</div>';
 
							}
							else echo 'Musisz się zalogować';
[PHP] pobierz, plaintext 

[sadistic_son]

Pusta strona pojawia sie dlatego ze sprawdzasz czy POST jest pusty, wystarczy wstawic w jakiekolwiek pole cokolwiek, lub nawet tylko nadusic submit aby warunek byl spelniony. Wtedy nie pojawi sie nic na stronie. Kod z linijki 5 musisz przerobic w taki sposob:

[PHP] pobierz, plaintext 
if (!isset($_POST['myusername']) || $_POST['myusername']=="" || !isset($_POST['mypassword']) || $_POST['mypassword']=="") {
[PHP] pobierz, plaintext 

Nastepnie kod od linijki 22 do 43 wlacznie opatrz w

[PHP] pobierz, plaintext 
if (isset($_POST['myusername']) && $_POST['myusername']!="" && isset($_POST['mypassword']) && $_POST['mypassword']!="") {
//kod od 22 do 43
}
[PHP] pobierz, plaintext 

Wylogowanie powinno wygladac tak:

[PHP] pobierz, plaintext 
if($_GET["wyloguj"] == '1'){
session_destroy();
unset($_SESSION["logged"]);
}
[PHP] pobierz, plaintext 

Ten post edytował sadistic_son 19.11.2010, 11:30:28

[andrzejlechniak]

Dzięki, działa