Witam,


napisałem sobie prosty skrypt jako że postanowiłem zacząć naukę OOP. Skrypt to w zasadzie szkielet bez zabezpieczeń, jednak napotkałem pewien problem.

Po wywołaniu poniższej klasy wyskakuje komunikat 'Proszę uzupełnić wszystkie dane!', wnioskując parametry nie są w ogóle pobierane.

[PHP] pobierz, plaintext 
<?php
 
    class Register {
 
	        public $login;
		public $password;
		public $mail;
 
		public function check_register( $login, $password, $mail){
 
		    if( isset($login) && isset($password) && isset($mail)){ 
 
		        if( $this->login == '' or $this->password == '' or $this->mail == ''){
 
			        echo 'Proszę uzupełnić wszystkie dane!';
 
			    }
			    else{
 
			        $this->query = mysql_query("SELECT * FROM `baza`.`uzytkownicy` WHERE `login` = '$this->login'");
 
				    if( mysql_num_rows( $this->query) > 1){
 
				        echo 'Podana nazwa użytkownika jest już zajęta!';
 
				    }
				    else{
 
				        $this->query = mysql_query("INSERT INTO `baza`.`uzytkownicy` VALUES ('', '$this->login', '$this->password', '$this->mail', '')");
 
					    echo 'Konto zostało utworzone poprawnie!';
 
				    }
 
			    }
 
		    }
 
	    }   
 
	}
 
	$obiekt = new Register;
 
	$obiekt-> check_register( 'Piotrek', 'haslo', 'mail');
 
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
if( $this->login == '' or $this->password == '' or $this->mail == ''){
[PHP] pobierz, plaintext 

To jest źle, sprawdzasz nieustawione parametry powinno być, nie ustawiłeś jeszcze ich

[PHP] pobierz, plaintext 
if( $login == '' or $password == '' or $mail == ''){
[PHP] pobierz, plaintext 

i w elsie przypisać te zmienne do ich odpowiedników.

Poprawiłem kod, jednak to nadal nic nie zmienia. Poza tym parametry zostały wysłane, patrz ostatnia linia, z tym że nie wiem czy zrobiłem to prawidłowo.

Wysłałeś, ale nie przypisałeś.

[PHP] pobierz, plaintext 
$this->login != $login;
[PHP] pobierz, plaintext 

W momencie sprawdzania

[PHP] pobierz, plaintext 
if( $this->login == '' or $this->password == '' or $this->mail == ''){
[PHP] pobierz, plaintext 

Te zmienne są puste.

Możesz zamienić tak jak mówił kolega wyżej, lub wywołać

[PHP] pobierz, plaintext 
__construct
[PHP] pobierz, plaintext 

i tam przypisać wszystko do

[PHP] pobierz, plaintext 
$this->login // itp.
[PHP] pobierz, plaintext 

Faktycznie, problem rozwiązany.