Witam, mam taką oto klasę w php (chciałbym prosić o sprawdzenie jej pod kątem poprawności - coś mi się wydaje, że wyjątki w klasie tworzę źle i powinny one być poza klasą. W każdym razie sama klasa działa, lecz gdy dodaję sprawdzanie w mySQL mam błąd przy linijce:

[PHP] pobierz, plaintext 
	$sql->$this->pdo->bindValue(':login',$login,PDO::PARAM_STR);
[PHP] pobierz, plaintext 

Kod błędu:


Oto całość kodu:

[PHP] pobierz, plaintext 
class Register {
	private $pdo;
	function __construct($pdo) {
		$this->pdo = $pdo;	
	}
 
	function register_player($login, $pass, $repass, $email) {
		try {
			if(strlen($login) < 4) {
				throw new Exception('Login za krótki.');
			} elseif(md5($pass) != md5($repass)) {
				throw new Exception('Wpisane hasła nie zgadzają się.');
			} elseif(!preg_match("/^[a-zA-Z0-9.-]+@[a-zA_Z0-9.-]+[a-zA-Z0-9.-]+$/",$email)) {
				throw new Exception('Błędny adres email.');
			}
			$sql = $this->pdo->prepare("SELECT * as COUNT FROM user WHERE login=:login OR email=:email");
			$sql->$this->pdo->bindValue(':login',$login,PDO::PARAM_STR);
			$sql->$this->pdo->bindValue(':email',$email,PDO::PARAM_STR);
			$zajety = $sql->$this->pdo->execute();
			print_r($zajety);
 
		} catch(Exception $exception) {
			echo 'Wystąpił błąd: '. $exception->getMessage() .'<br />';
		}
	}
}
[PHP] pobierz, plaintext 

Czy mógłby mi ktoś doradzić co robię źle, ewentualnie naprowadzić?

Pozdrawiam.

$sql->$this

Trochę mi to zajęło zanim zrozumiałem co jest źle... w każdym razie działa

[PHP] pobierz, plaintext 
$sql = $this->pdo->prepare("SELECT * FROM user WHERE login=:login OR email=:email");
$sql->bindValue(':login',$login,PDO::PARAM_STR);
$sql->bindValue(':email',$email,PDO::PARAM_STR);
$sql->execute();
$sql->closeCursor();
[PHP] pobierz, plaintext 

A co z poprawnością klasy?

[edit: 15:47]

Przeniosłem wyjątki poza klasę, wszystko działa.
Jednak prosiłbym o ogólną ocenę i uwagi dotyczące tej klasy.

index.php:

[PHP] pobierz, plaintext 
try {
	require_once('./class/class.register.php');	
	$player = new Register($pdo);
	$player->register_player($_POST['login'], $_POST['pass'], $_POST['repass'], $_POST['email']);
	echo 'Użytkownik zarejestrowany poprawnie.<br />';
} catch(Exception $exception) {
	echo 'Wystąpił błąd: '. $exception->getMessage() .'<br />';
}
[PHP] pobierz, plaintext 

class.register.php:

[PHP] pobierz, plaintext 
class Register {
	private $pdo;
	function __construct($pdo) {
		$this->pdo = $pdo;	
	}
 
	private function istnieje($login, $email) {
		$sql = $this->pdo->prepare("SELECT * FROM user WHERE login=:login OR email=:email");
		$sql->bindValue(':login',$login,PDO::PARAM_STR);
		$sql->bindValue(':email',$email,PDO::PARAM_STR);
		$sql->execute();
		$sql->closeCursor();
		if($sql->rowCount() == 0) {
			return false;
		} else {
			return true;
		}
	}
 
	function register_player($login, $pass, $repass, $email) {
		if(strlen($login) < 3) {
			throw new Exception('Login za krótki.');
		} elseif(md5($pass) != md5($repass)) {
			throw new Exception('Wpisane hasła nie zgadzają się.');
		} elseif(!preg_match("/^[a-zA-Z0-9.-]+@[a-zA_Z0-9.-]+[a-zA-Z0-9.-]+$/",$email)) {
			throw new Exception('Błędny adres email.');
		} elseif($this->istnieje($login, $email)) {
			throw new Exception('Podany login lub email znajduje się już w bazie.');
		}
 
		$new = $this->pdo->prepare("INSERT INTO user (login, haslo, email) VALUES (:login, :haslo, :email)");
		$new->bindValue(':login',$login,PDO::PARAM_STR);
		$new->bindValue(':haslo',md5($pass),PDO::PARAM_STR);
		$new->bindValue(':email',$email,PDO::PARAM_STR);
		$new->execute();
		$new->closeCursor();
		return true;
	}
}
[PHP] pobierz, plaintext