OOP - problem z przekazywaniem zmiennych Opcje

[trol2]

Mam nastepujacy problem. Napisalem taka klase (dla skrocenia kodu wyrzucilem pare funkcji, ale to nie istotne):

[PHP] pobierz, plaintext 
<?php
class user_manager
{
 
public $login;
public $pass1;
public $pass2;
public $email;
 
public function __get($nazwa)
{
	return $this->$nazwa;
}
 
public function __set($nazwa, $wartosc)
{
	$this->$nazwa = $wartosc;
}
 
public function __construct($login, $pass1, $pass2, $email)
{
require_once('funkcje/header.php');
 
$this->login = $login;
$this->pass1 = $pass1;
$this->pass2 = $pass2;
$this->email = $email;
}
 
public function Exist_data_1()
{
 
if($this->login = '' or $this->pass1 = '' or $this->pass2 = '' or $this->email = '')
{
$_SESSION['blad_rej'] = 1;
$_SESSION['login_r'] = "$this->login";
$_SESSION['mail_r'] = "$this->email";
my_header($do = 'zarejestruj.php', $czas = 0);
exit;
}
 
}
 
public function Login_ok()
{
if(!ctype_alnum($this->login))
{
$_SESSION['blad_rej'] = 1;
$_SESSION['login_r'] = "$this->login";
$_SESSION['mail_r'] = "$this->email";
my_header($do = 'zarejestruj.php', $czas = 0);
exit;
}
}
 
public function Haslo_ok()
{
if($this->pass1 != $this->pass2)
{
$_SESSION['blad_rej'] = 2;
$_SESSION['login_r'] = "$this->login";
$_SESSION['mail_r'] = "$this->email";
my_header($do = 'zarejestruj.php', $czas = 0);
exit;
}
}
 
public function Zarejestruj()
{
$this->Exist_data_1();
$this->Login_ok();
$this->Haslo_ok();
$this->Email_ok();
$this->Zarejestruj_go();
}
 
}
?>
[PHP] pobierz, plaintext 

wywoluje nastepujaca klase tak:

[PHP] pobierz, plaintext 
<?php
require_once('obiekty/user_manager.php');
$usermng = new user_manager($login, $pass1, $pass2, $email);
 
$usermng->Zarejestruj();
?>
[PHP] pobierz, plaintext 

Jednak contruktor nie przypisuje zmiennym wartosci. Tak samo dzieje sie jezeli w kodzie dla kazdej zmiennej dam $usermng->login = $login; Dziala jedynie jak przekazuje niezbedne zmienne do kazdej funkcji, wtedy wywoluje bez $this-> . Co w tym kodzie jest nie tak ze nie przypisuje wartosci zmiennym clasy?

[Morkai]

1. Zdecyduj się na jeden język... polski albo angielski
2. Po co __get()/__set() skoro zmienne w klasie są publiczne?
3. Nie musisz wsadzać zmiennych w cudzysłowy. $_SESSION['login_r'] = $this->login; też zadziała i ładniej wygląda
4. Wrzuć var_dump( $login ); przed i po require w konstruktorze i powiedz co wywala.
5. Co robi plik funkcje/header.php?

Ten post edytował Morkai 1.06.2007, 13:55:23

[trol2]

1. Zdecyduj się na jeden język... polski albo angielski

ok, to pozniej, napewno nie wplywa to na dzialanie skryptu.

2. Po co __get()/__set() skoro zmienne w klasie są publiczne?

usunalem.

3. Nie musisz wsadzać zmiennych w cudzysłowy. $_SESSION['login_r'] = $this->login; też zadziała i ładniej wygląda

cudzyslowia tez usunalem.

4. Wrzuć var_dump( $login ); przed i po require w konstruktorze i powiedz co wywala.

W kodzie przed require i po require jest prawidlowy login, w construktorze tak samo, w dowolnej funkcji (bez przekazywania jej zmiennej $login) po wpisaniu bez this jest tez prawidlowy login, a po wpisaniu z $this->login jest NULL

5. Co robi plik funkcje/header.php?

to jest przeniesienie do innej strony przy pomocy znacznikow META, tam przyjmuje zmienna $do i $czas .

[dr_bonzo]

[PHP] pobierz, plaintext 
<?php
if($this->login = ''
?>
[PHP] pobierz, plaintext 

lol

'=' != '=='

[trol2]

Faktycznie. Operator przypisania zamiast operatora porownania juz powinno byc ok. Nie przykladalem do tego specialnej uwagi, bo kod sprawdzajacy kopiowalem z funkcji, a wczasniej dzialal. Musialem gdzies przy zmianie tych $this-> usunac znak rownosci.