Что не так с кодом? MySQL запрос не корректен?

Question

[PO6OT]

<?php
const N = "\n";
if(!$mysqli = @mysqli_connect('127.0.0.1', 'root', 'password', 'site'))
	exit('Fatal error: Couldn\'t connect to MySQL DB.');
if(mysqli_num_rows(mysqli_query($mysqli, 'SHOW TABLES LIKE `users`')) == 0) {
	mysqli_query($mysqli,
		'CREATE TABLE `users` ('.N.
		'    `id` int(11) unsigned NOT NULL auto_increment,'.N.
		'    `email` varchar(30) NOT NULL,'.N.
		'    `password` varchar(32) NOT NULL,'.N.
		'    PRIMARY KEY (`id`)'.N.
		') ENGINE=MyISAM DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ;'
	);
}

Почему то mysqli_query()выдаёт false, хотя запрос валидный. Что не так?

Comments

[AlikDex]

ПИ З ДЕ Ц! такой херни я еще не видел =)

CREATE TABLE IF NOT EXISTS `users`
    `id` int(11) unsigned NOT NULL auto_increment,
    `email` varchar(30) NOT NULL,
    `password` varchar(32) NOT NULL,
     PRIMARY KEY (`id`)
 ) ENGINE=MyISAM DEFAULT CHARSET=utf8 AUTO_INCREMENT=1

вот прямо так вставляй в двойных кавычках без всяких

if(mysqli_num_rows(mysqli_query($mysqli, 'SHOW TABLES LIKE `users`'))==0)

[PO6OT]

AlikDex: в таком случае постоянно требуется отправлять большой запрос в MySQL. Почему бы сначала не проверить наличие, а затем отправлять весь запрос?

[AlikDex]

PO6OT: в данном случае это делается 1 раз, а не постоянно. Создание таблицы делается на этапе проектирование базы данных для приложения. Потом пишутся миграции всякие.

[PO6OT]

AlikDex: И зачем в двойных кавычках? Я их не признаю. Я для этого даже все спец символы в константы перевожу и сую их после одинарных кавычек, выше вы можете это наблюдать =)

[AlikDex]

const N = "\n"; замена этому в пхп есть константа PHP_EOL

[AlikDex]

PO6OT: можно и в одинарных =) Двойные все же в пхп иногда используются, т.к. позволяют вставить переменную прямо в строку, минуя конкатенацию.

[PO6OT]

AlikDex: Нет, не совсем. PHP_EOL на разных ОС разный.
>в данном случае это делается 1 раз, а не постоянно. Создание таблицы делается на этапе проектирование базы данных для приложения. Потом пишутся миграции всякие.
Как раз наоборот. Это не установочный скрипт. Этот скрипт - часть формы авторизации. Мне не хочется создавать специальный установочник, почему бы не сделать всё автономным? Зачем проблемы при первом старте? Пусть всё так работает! Если бы мне нужно было, я бы это вообще в PHP не пихал, а сделал бы всё через phpmyadmin

[AlikDex]

PO6OT: ;D

[PO6OT]

AlikDex: Двойные кавычки работают чуть-чуть медленнее так как парсят строку на наличие переменных и спец символов. К тому же в некоторых случаях они непредсказуемы.

[PO6OT]

AlikDex: Ну ок, ваш вариант лучше чем мой неработающий. Но всё-же длинноват и было бы неплохо таки решить проблему с неработающим mysqli_query($mysqli, 'SHOW TABLES LIKE `users`')

[AlikDex]

PO6OT: я думаю 10 конкатенаций для 1 запроса против двойных кавычек проиграет =)

[PO6OT]

AlikDex: Конкатенации тоже убивают время?

[PO6OT]

AlikDex: Я догадывался... В любом случае мне так удобнее)

[AlikDex]

PO6OT: совсем чуть чуть, но да, убивают =) например можно выводить так:

$name = 'Вася';
echo 'Привет ' , $name;

И это будет чуточку быстрее чем

$name = 'Вася';
echo 'Привет ' . $name;

[PO6OT]

AlikDex: У меня всё-равно проблема. Ваш код тоже не работает. Такое ощущение, что дело в кодировках.

[PO6OT]

PO6OT:

if(mysqli_query($mysqli, "
CREATE TABLE IF NOT EXISTS `users` (
    `id` int(11) unsigned NOT NULL auto_increment,
    `email` varchar(30) NOT NULL,
    `password` varchar(32) NOT NULL,
    PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ;
")===false)
    echo 'Error!<br>';
echo mysqli_error($mysqli);

Возвращает:

Error!
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '`users`' at line 1

[AlikDex]

Может надо указать дефолтное значение для NOT NULL полей?

if(mysqli_query($mysqli, '
CREATE TABLE IF NOT EXISTS `users` (
    `id` int(11) unsigned NOT NULL auto_increment,
    `email` varchar(30) NOT NULL DEFAULT '',
    `password` varchar(32) NOT NULL  DEFAULT '',
    PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ;
')===false)
    echo 'Error!<br>';
echo mysqli_error($mysqli);

[PO6OT]

AlikDex: Нет, не помогло.

[PO6OT]

AlikDex: Ещё пара строк, тоже не работает:

if(!$query=@mysqli_query($mysqli, "SELECT COUNT(id) FROM users WHERE email='".mysqli_real_escape_string($mysqli, $_POST['email'])."'"))
		exit('Fatal error: '.mysqli_error($mysqli));
	if(mysqli_num_rows($query)>0)
		$error.='Такой email уже зарегистрирован!'.BR;

Ошибок не выдаёт, пишет что email зарегистрирован, хотя таблица пуста

Answer 1

[Rsa97]

Дык это, телепаты все заняты. mysqli_error что говорит?

Comments

[PO6OT]

Ничего особенного:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '`users`' at line 1

[Rsa97]

Понятно. Не те кавычки в LIKE использованы.

[PO6OT]

Rsa97: А почему это имеет значение? Я поменял кавычки, теперь всё ок. Но что за хрень? Это странно

[PO6OT]

Проблема не совсем решена. Она какая-то массовая.. Опять:

if(!$query=@mysqli_query($mysqli, "SELECT COUNT(id) FROM users WHERE email='".mysqli_real_escape_string($mysqli, $_POST['email'])."'"))
    exit('Fatal error: '.mysqli_error($mysqli));
if(mysqli_num_rows($query)>0)
    echo 'Такой email уже зарегистрирован!<br>';

Выводит "email зарегистрирован" хотя таблица пуста.

[Rsa97]

PO6OT: Кавычки - потому что в обратных кавычках указываются имена таблиц, полей, индексов и т.д. LIKE же принимает аргументом строку.

А с этой ошибкой тоже просто. Такой запрос всегда вернёт строку результата, просто COUNT(id) будет равен нулю.

И ещё, не используйте подавление ошибок через @. Ошибки нужно видеть и устранять, а не скрывать.

[PO6OT]

Rsa97: на счёт LIKE понятно.
На счёт следующей ошибки - получается нужно проверять сразу
mysqli_query()>0 ?

[Rsa97]

PO6OT: Очень просто. возвращается одна строка, в которой указано число выбранных записей - 0.
Ну и я бы просто объявил поле `email` как UNIQUE KEY, давал команду на добавление записи, а потом контролировал mysqli_affected_rows. Следить за уникальностью проще на стороне базы, а не клиента.