Как написать код проверки данных из бд?

Question

[rewqqwer]

Привет, возникла курьезная проблема в моей простой формочке для регистрации, способ (если это так можно назвать) по проверке на существования в бд введеного логина/почты не оправдал себя, т.к. он попросту не работает. Не подскажете как реализовать эту проверку, желательно с помощью библ PDO, чтобы сохранять лаконичность или же укажите, пожалуйста, ошибку.

$sth = $conn->prepare("SELECT COUNT(*) as count FROM users WHERE mail =". $data['mail']);
		$sth->execute();
		$members = $sth->fetchColumn();
		//if ($members=$conn->query("SELECT COUNT(*) as count FROM users WHERE mail =". $data['mail'])->fetchColumn()) {
		if ($members) {
			$data['mail'] = '##';
			echo "privet";
			throw new Exception('Пользватель с таким email уже существует');
		}
		$sth = $conn->prepare("SELECT COUNT(*) as count FROM users WHERE login =". $data['login']);
		$sth->execute();
		$members = $sth->fetchColumn();
		if ($members) {
			$data['login'] = '##';
			throw new Exception('Пользватель с таким логином уже существует');
		}
    	$sql = "INSERT INTO users ( login, mail, pass, proj ) VALUES ( :login, :mail, :pass, :proj )";
    	$st = $conn->prepare ( $sql );
    	$st->bindValue( ":login", $data['login'], PDO::PARAM_STR );
    	$st->bindValue( ":mail", $data['mail'], PDO::PARAM_STR );
    	$st->bindValue( ":pass", password_hash($data['pass'], PASSWORD_DEFAULT), PDO::PARAM_STR );
    	//$st->bindValue( ":id", $data['id'], PDO::PARAM_INT );
    	$st->bindValue(":proj", '##', PDO::PARAM_STR);
    	$st->execute();
    	$data['id'] = $conn->lastInsertId();
		$user = new user($data);
    	$conn = null;
		$_SESSION['user'] = $user;

Comments

[vyrkmod]

Никогда не пишите запросы в базу так, а то с ней что-то нехорошее сделают.

Answer 1

[morricone85]

Зачем делать несколько запросов? Можно так:
SELECT login, mail from users WHERE mail =". $data['mail'] AND login=". $data['login'] LIMIT 1
Eсли значения есть - выводишь ошибку, а если нет - то, добавляешь пользователя.

Comments

[ArcadyZherdev]

Я дико извиняюсь но вы разве не видите что это другая проверка? То есть возможна ситуация когда пользователь уже выбрал этот логин, но почта у него другая. А вы ищете именно совпадение пары логин+имейл. :)

Answer 2

[Максим Тимофеев]

$nRows = $pdo->query('select count(*) FROM users WHERE login =". $data['login']')->fetchColumn(); 
if($nRows > 0){
echo 'такой логин занят'
}

Answer 3

[ArcadyZherdev]

Я бы начал с того что использовал bind переменные в проверяющих запросах:

$sth = $conn->prepare("SELECT COUNT(*) as count FROM users WHERE mail = :mail");
$sth->bindValue(":mail", $data['mail']);
$sth->execute();
if ($members = $sth->fetchColumn()) {
    throw new Exception('Пользватель с таким email уже существует');
}