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

Gdy funkcja cos zwraca to wypadaloby to odebrac...

$wynik = CheckUser(...);
echo $wynik;

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.

Widzisz takie proste a ja spędziłem nad tym ponad godzine głowiąc się co może być nie tak i próbując co chwile coś zmieniać.
Ostatecznie rozwiązałem to póki co za pomocą zapisu wyniku w zmiennej sesji, ale już zmieniłem tylko zapis w index.php na

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

i działa aż miło

Dzięki za pomoc.

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

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.



W sumie masz rację. Już to sobie zmieniłem

pisz nazwy funkcji z malych liter, a nazwy klas z duzych... poczytaj o coding styles na http://www.php-fig.org/