[php] błąd w klasie Opcje

[Lejto]

Mam taki fragment index.php

[PHP] pobierz, plaintext 
include("config.php");
include("libs/db.class.php");
$db = new DB();
include("libs/core.class.php");
$c = new core($db);
include("function.inc.php");
$f = new functions();
[PHP] pobierz, plaintext 

wszystko chodzi
wchodzę na stronę gdzie rejestruje usera
połączenie z bazą jest wywołuje funkcje sprawdzającą

[PHP] pobierz, plaintext 
global $db;
global $page;
global $f;
 
include("libs/user.class.php");
$u = new User($db); 
 
        $user =  $u->getByUsername($_POST['username']);
[PHP] pobierz, plaintext 

połączenie z bazą w tym pliku jest ale nie ma już jego w klasie user.class.php jak juz wcześniej wymieniłem plik code.class.php (tam wszystko działa)
user.class.php

[PHP] pobierz, plaintext 
class User
{ 
	public $uid;
	public $fields = array();	
 
	public function __construct($sql)
	{
    	$this->sql = $sql;
		$this->uid = null;
		$this->fields = array('username' => '',
							  'password' => '',
							  'emailAddr' => '',
							  'isActive' => false);
 	}
	public static function getByUsername($username)
	{
		echo 'aa';
  		$db = $this->sql;
	 echo 'aa';
		$sql = $db->query('select * from users where username = $username');
 
	}
[PHP] pobierz, plaintext 

db.class.php

[PHP] pobierz, plaintext 
class DB{
 
	public $_lacz;
 
	public function __construct()
  	{
    	$this->_lacz = new mysqli(DB_HOST, DB_USER, DB_PASS, DB_NAME);
		if(mysqli_connect_errno() != 0)
		{
			echo '<p>Wystąpił błąd połączenia: ' . mysqli_connect_error(). ', przepraszamy.';
		}
	}
[PHP] pobierz, plaintext 

dlaczego nie mam połączenie w klasie user?
reszta klas działa normalnie

[darko]

Gdzie masz pole $sql w klasie User ?

[Lejto]

jak pole $sql?
w core.class mam

[PHP] pobierz, plaintext 
class core{	
 
	public function __construct($sql)
	{
    	$this->sql = $sql;
	}
	////////////
}
[PHP] pobierz, plaintext 

i jest ok

[darko]

Nie zauważyłem że public static function getByUsername($username) Ok w takim razie - wiesz, że w metodach statycznych nie korzysta się z $this, a z self:: ? Poza tym źle wywołujesz tę metodę, powinno być: $u::getByUsername($_POST['username']);

Ten post edytował darko 4.01.2010, 20:54:28

[skowron-line]

[PHP] pobierz, plaintext 
public [url="http://www.php.net/static"]static[/url] function getByUsername($username)
[PHP] pobierz, plaintext 

usuń static.

[darko]

Jak usuniesz static, to dodaj jeszcze w klasie User pole
protected $sql;

[Lejto]

usunąłem ale nadal nie chce wykonać zapytania var_dump($db); zwraca NULL

[darko]

a dodałeś to pole protected $sql; ?

Ten post edytował darko 4.01.2010, 21:01:53

[Lejto]

jakie protected?
mam normalną prywatność dla tego jak w innych klasach

ok już wiem o co chodzi
dodałem i poprawnie wykonuję te funkcje ale teraz zatrzymuje się na wywołaniu $user->userId

[PHP] pobierz, plaintext 
 $user =  $u->getByUsername($_POST['username']);
        if ($user->userId)
        {
          	echo '<p><strong>Przepraszamy, ' .
                'takie konto już istnieje.</strong></p> <p>Prosimy podać ' .
                'inną nazwę użytkownika.</p>';
 
        }
[PHP] pobierz, plaintext 

klasa (część)

[PHP] pobierz, plaintext 
<?
class User
{ 
	public $uid;
	public $fields = array();	
 	protected $sql;
 
	public function __construct($sql)
	{
    	$this->sql = $sql;
		$this->uid = null;
		$this->fields = array('username' => '',
							  'password' => '',
							  'emailAddr' => '',
							  'isActive' => false);
 	}
	public function __get($field)
	{
		if($field == 'userId')
		{
			return $this->uid;
		}
		else
		{
			return $this->fields[$field];
		}
	}
	public function __set($field, $value)
	{
		if(array_key_exists($field, $this->fields))
		{
			$this->fields[$field] = $value;
		}
	}
	public static function validateUserName($username)
	{
		return preg_match('/^[A-Z0-9]{2.20}$/i', $username);
 
	}
	public static function validateEmailAddr($email)
	{
		return filter_var($email, FILTER_VALIDATE_EMAIL);
	}
	public static function getById($uid)
	{ 		
 		$db = $this->sql;
		$u = new User($db);
		$sql =  $db->query('select username, password, email_addr, is_active from users where userid = $uid');
 
		if($sql->num_rows)
		{
			$row = $sql->fetch_array();
			$u->username = $row['username'];
			$u->password = $row['password'];
			$u->emailAddr = $row['email_addr'];
			$u->isActive = $row['is_active'];
			$u->uid = $uid;
		}
		return $u;
	}
 
	public  function getByUsername($username)
	{
   		$db = $this->sql;
 		$sql = $db->query('select * from users where username = $username');
 	}
 
 
 
}
?>
[PHP] pobierz, plaintext 

[darko]

Metoda getByUsername nie operuje na polu userId

[Lejto]

ale mam to w __get
no to jest mam to poprawić?

[darko]

Wypełnij pola klasy User analogicznie, jak w metodzie getById tylko nie statycznie, a kontekstowo z użyciem wskaźnika $this. W metodzie getByUsername wyciągasz dane z bazy, ale nigdzie ich nie przypisujesz do pól klasy User, chodzi o to, że powinieneś nadpisać wartości pól zapisane po wywołaniu konstruktora.

Ten post edytował darko 5.01.2010, 15:57:28

[Lejto]

nie za bardzo cię rozumiem, poprawiłem klase, rejestruje ale nie sprawdza czy nazwa użytkownika się powtarza ;/

[PHP] pobierz, plaintext 
<?
class User extends functions
{ 
	public $uid;
	public $fields = array();	
 	protected $sql;
 
	public function __construct($sql)
	{
    	$this->sql = $sql;
		$this->uid = null;
		$this->fields = array('username' => '',
							  'password' => '',
							  'emailAddr' => '',
							  'isActive' => false);
 	}
	public function __get($field)
	{
		if($field == 'userId')
		{
			return $this->uid;
		}
		else
		{
			return $this->fields[$field];
		}
	}
	public function __set($field, $value)
	{
		if(array_key_exists($field, $this->fields))
		{
			$this->fields[$field] = $value;
		}
	}
	public  function validateUserName($username)
	{
  		if(strlen($username)<3)
		{
			echo 'Nazwa użytkownika musi zawierać co najmniej 3 znaki';
		}
		else
		{
			return true;
		}
		if(strlen($username)>20)
		{
			echo 'Za długa nazwa użytkownika max 20 znaków';
		}
		else
		{
			return true;
		}
	}
	public static function validateEmailAddr($email)
	{
		return filter_var($email, FILTER_VALIDATE_EMAIL);
	}
	public function getById($uid)
	{ 	
 		$db = $this->sql;
		$u = new User($db);
		$sql =  $db->query('select username, password, email_addr, is_active from users where userid = $uid');
 
		if($sql->num_rows)
		{
			$row = $sql->fetch_array();
			$u->username = $row['username'];
			$u->password = $row['password'];
			$u->emailAddr = $row['email_addr'];
			$u->isActive = $row['is_active'];
			$u->uid = $uid;
		}
		return $u;
	}
 
	public function getByUsername($username)
	{ 
		$db = $this->sql;
   		$u = new User($db);
 
		$sql = $db->query('select user_id, password, email_addr, is_active from USERS where username = $username');
 
		if($sql->num_rows)
		{
			$row = $sql->fetch_array();
			echo $row['username'];
			echo 'aa';
			$u->password = $row['password'];
			$u->emailAddr = $row['email_addr'];
			$u->isActive = $row['is_active'];
			$u->uid = $row['user_id'];
		}
		return $u;
 	}
 
	public function save()
	{
 
 
 		$db = $this->sql;
 		if($this->uid)
		{
			$sql = $db->query('update USERS set username = $this->username, password =  $this->password,
							  email_addr = $this->emailAddr, is_active = $this->isActive where user_id = $this->userId');
 		}
		else
		{
			 $sql = $db->query('insert into users (username, password, email_addr, is_active)
												  values ("'.$this->username.'", "'.$this->password.'", "'.$this->emailAddr.'", "'.$this->isActive.'")');
			 if(!$sql)
			 {
				 var_dump($this->password);
			 }
 		}
	}
 
	public function setInactive()
	{
		$db = $this->sql;
		$this->isActive = false;
		$this->save();
 
		$token = $this->random_text(5);
		$sql = $db->query('insert into users_pending  (user_id, token) values ($this->uid, $token)');
 
		return $token;
	} 
 
	public function setActive($token)
	{
		$db = $this->sql;
 
		$sql = $db->query('select token from users_pending where user_id = $this->uid and token = $token');
 
		if(!$sql->num_rows)
		{
			return false;
		}
		else
		{
			$sql = $db->query('delete from users_pending where user_id = $this->uid and token = $this->token');
			$this->isAdcive = true;
			$this->save();
			return true;
		}
	} 
}
?>
[PHP] pobierz, plaintext 

[Crozin]

To może najpierw kilka uwag:

#2: Klasa 'User' dziedziczy po 'functions' (może jakaś konsekwencja w nazewnictwie?) co wydaje mi się nieco dziwne. W jaki sposób użytkownik jest rozszerzeniem funkjci?
#4,5: Dlaczego te pola są publiczne? Przecież z tego co widzę stworzyłeś dla nich magiczne gettery/settery.
#6: SQL to skrót odnoszący się do języka zapytań, a z tego co widzę to używasz tego jako referencji do czegoś co pełni rolę API do komunikacji z bazą... niepotrzebne nieporozumienia się rodzą w takiej sytuacji
#11,12: Dlaczego nie ustawisz sobie wartości domyślnych odrazu w momencie deklarowania składowych klasy?
#35: Jak do tej pory ta klasa wygląda na taką, która ma reprezentować użytkownika... a tu nagle wyskakuje jakaś metoda do walidacji nazwy użytkownika. Powinieneś wydzielić te (i podobne) metody do innego obiektu będącego walidatorem dla użytkownika używanym podczas rejestracji. Poza tym kod tej metody jest bez sensu... najpierw sprawdzasz czy długość nazwy jest mniejsza od 3-ech. Jeśli tak to wyświetlasz komunikat (co już jest błędem, bo obiekt niepotrzebnie bierze na swoje barki dodatkową kwestię jaką jest prezentowanie treści użytkownikowi). Jeżeli nazwa jest dłuższa lub równa podanym 3-em znakom to metoda kończy działanie zwracając TRUE. To po jakie licho jest jeszcze później ten drugi warunek? Przecież jeżeli nazwa jest krótsza od tych 3-ech znaków to już nie będzie dłuższa niż 20, a sytuacja gdy nazwa jest dłuższa niż 2 znaki po czym następuje sprawdzenie czy aby przypadkiem nie jest dłuższa niż 20 znaków nie może nigdy zajść...
#58: Tutaj znów obiekt reprezentujący użytkownika pełni rolę kolekcji użytkowników... pobieranie użytkowników wydziel do osobnego obiektu odpowiedzialnego za to konkretne zadanie (tj. reprezentację kolekcji i operacji na niej)
#60: W jakim celu każdorazowo robisz: $db = $this->sql? Po co ta dodatkowa referencja?
#76: Zauważyłeś, że ta metoda robi dokładnie to co wcześniejsza tylko różni się drobnym fragmentem wykorzystywanego zapytania? Google: DRY. Innymi słowy: wydziel sobie jakąś prywatną metodę pobierającą użytkowników i z poziomu getById, getByUsername zwracaj jej wynik (jako argument przekazując coś w stylu:
"id = 3" lub "username = 'abc'"
#103: Nie widzę żadnej filtracji danych przed wprowadzeniem ich do treści zapytania. Co więcej te zapytania nie mają prawa działać - w ich treści masz:
...username = abc my username...
Przeczytaj w dokumentacji jak w MySQL (bo chyba z tego korzystasz) reprezentuje się tekst - podobnie jak w kodzie PHP podpowiem.
#124: Poczytaj o różnicy pomiędzy: "blah blah $variable blah blah", a 'blah blah $variable blah blah'

rejestruje ale nie sprawdza czy nazwa użytkownika się powtarza

1) Nie pokazujesz jak używasz obiektu tej klasy
2) Nie widzę też w tym kodzie niczego co by mogło sprawdzać unkalność nazwy użytkownika...

[Lejto]

dziedziczę jeszcze jeszcze functions bo jest tam funkcja którą potrzebuję do wylosowania tekstu
#4,5 ok zmienię
#6 mógłbyś wyjaśnić?
#11,12 to też (ucze się cały czas oop) nie wiem jeszcze wszystkiego...
#35 i #58 to jak bym miał to poprawić?
#60 no właśnie o tym już kiedyś pisałem potrzebuje tego aby było połączenie, przyjrzyj się klasie łączącej
#124 jakieś namiary? nie wiem za bardzo o co chodzi

dzięki za wskazówki

Ten post edytował Lejto 5.01.2010, 22:25:56

[Crozin]

dziedziczę jeszcze jeszcze functions bo jest tam funkcja którą potrzebuję do wylosowania tekstu

Nie o to chodzi w dziedziczeniu. Jest sporo materiałów dotyczących obiektówki - traktujących o teorii - poczytaj jeszcze zanim zabierzesz się za pisanie.

#6 mógłbyś wyjaśnić?

Po prostu zwróciłem uwagę na to, że źle sobie nazywasz zmienne. Pod $sql umieszczasz sobie obiekt, który z SQLem nie ma wiele wspólnego. Po prostu: zła praktyka wg mnie.

11,12 to też (ucze się cały czas oop) nie wiem jeszcze wszystkiego...

Zastanawiałem się dlaczego nie zrobisz po prostu:

[PHP] pobierz, plaintext 
<?php
 
class ABC {
  protected $xyz = array('...' => '...', '...' => '...');
 
  //...
}
[PHP] pobierz, plaintext 

#35 i #58 to jak bym miał to poprawić?

Utworzyć kilka nowych klas, gdzie każda odpowiedzialna jest za jedną konkretną rzecz i doprowadzić do współpracy różnych obiektów tych klas między sobą, a nie tak jak teraz masz... jeden obiekt robiący wszystko co się tylko da - ma on za duży zakres obowiązków.

#60 no właśnie o tym już kiedyś pisałem potrzebuje tego aby było połączenie, przyjrzyj się klasie łączącej

Eee... ale Ty po prostu robisz niepotrzebnie kolejną referencję do tego obiektu... Zaśmiecasz kod tylko niepotrzebnymi zmiennymi, nie zyskując nic na tym, bo zarówno $db->doSth('...'); jak i $this->db->doSth('...') są bardzo czytelne.

#124 jakieś namiary? nie wiem za bardzo o co chodzi

Oczywiście manual PHP, rozdział dotyczący typu danych: String (nie pomyl z listą funkcji operujących na tekscie). Ale jeżeli nie masz tak elementarnej wiedzy z zakresu działania PHP (różnica pomiędzy "test", a 'test') to nie wiem czy dobrym pomysłem jest branie się za prog. obiektowe.

[Lejto]

ok dzięki
ale powiedź jeszcze dlaczego nie chce mi działać wywołanie $user->userId

[PHP] pobierz, plaintext 
 
$user = $u->getByUsername($_POST['username']);
if ($user->userId)
 
{
 
echo '<p><strong>Przepraszamy, ' .
 
'takie konto już istnieje.</strong></p> <p>Prosimy podać ' .
 
'inną nazwę użytkownika.</p>';
 
 }
[PHP] pobierz, plaintext 

poradzi ktoś coś?
kod już cały potrzebny chyba podałem.

Ten post edytował Lejto 6.01.2010, 18:15:52

[Crozin]

Wyświetl sobie błędy zapytań MySQL...

[Lejto]

nie wpadł bym na to
problem rozwiązany, błąd był w zapytaniu (myślałem że gdzieś funkcje źle zrobione)