[PHP]Nie działa return w funkcji ? PHP Opcje

[bubolski]

Witam,

wziałem się za przerobienie starej strony www na bardziej zaawansowaną. Chcę zrobić ją przy użyciu funkcji , obiektów etc. I o dziwo mam problem na samym początku.

Zacząłem tworzyć panel logowania i jedna funkcja nie zwraca mi zmiennej.

Posiadam jedną funkcje do nawiązania połączenia z bazą danych. Druga natomiast sprawdza, czy użytkownik istnieje w bazie i jak tak to tworzy się zmienna sesji , a jak nie to zwraca odpowiedni komunikat a konkretnie $wynik 1 lub 2 (1 istnieje login , ale niepoprawne hasło ; 2 zapytanie sql zwrociło 0 wierszy więc user nie istnieje). Jednak nie zwraca mi się wartość wynik.

Mam 2 pliki index.php i function.php

Index.php

[PHP] pobierz, plaintext 
<?php
	session_start();
	require_once('function.php');
	if(isset($_POST['textInput'])&&isset($_POST['passwordInput']))
	{
		ConnectKacik();
		CheckUser(html_entity_decode($_POST['textInput']),html_entity_decode($_POST['passwordInput']));
		echo $wynik; // niczego mi to nie wyświetla
 
	}
?>
[PHP] pobierz, plaintext 

a tutaj funkcja CheckUser z function.php

[PHP] pobierz, plaintext 
function CheckUser($login,$password){
 
		$sql = "SELECT * FROM usery WHERE login='$login'";
		$result = mysql_query($sql);		
 
		$row = mysql_fetch_assoc($result);
 
		if($login==$row['login'] && $password==$row['haslo'])
		{
 
		}else
		{
			$wynik = 1;
		}
		if(mysql_num_rows($result)<1)
		{
			$wynik = 2;
		}			
			// jeżeli w tym miejscu dam echo $wynik to wyświetla mi wartość na index.php (bo wyświetla z tego miejsca)	
		return $wynik;
	}
[PHP] pobierz, plaintext 

Czemu mimo return $wynik , jak mam echo $wynik w index.php nie wyświetla mi tego wyniku (IMG:style_emoticons/default/questionmark.gif)

Ten post edytował bubolski 9.08.2016, 17:07:30

[KsaR]

Bo zmienna $wynik jest tylko w funkcji twojej.
mozesz zrobic.

[PHP] pobierz, plaintext 
$wynik = CheckUser(html_entity_decode($_POST['textInput']),html_entity_decode($_POST['passwordInput']));
		echo $wynik;
[PHP] pobierz, plaintext 

A dwa ze zależnie od twojego warunku w funkcji czasem zmienna $wynik nie istnieję. - edit, źle się przyjrzałem.

Ten post edytował KsaR 9.08.2016, 17:12:34

[bubolski]

Bo zmienna $wynik jest tylko w funkcji twojej.
mozesz zrobic.

[PHP] pobierz, plaintext 
$wynik = CheckUser(html_entity_decode($_POST['textInput']),html_entity_decode($_POST['passwordInput']));
		echo $wynik;
[PHP] pobierz, plaintext 

A dwa ze zależnie od twojego warunku w funkcji czasem zmienna $wynik nie istnieję.

To akurat jest specjalny zabieg. Jak nie ma wyniku to nie wyświetla błędu logowania. Jeżeli ktoś się loguje i wpisze złe hasło, lub użytkownik nie istnieje to tylko wtedy otrzyma śliczny czerwony button z opisem błędu. (IMG:style_emoticons/default/smile.gif)

Zrób albo użytkownik istnieje albo nie istnieje. Nie podpowiadaj ewentualnemu włamywaczowi że dany użytkownik istnieje.

W sumie masz rację. Już to sobie zmieniłem (IMG:style_emoticons/default/smile.gif)