Как правильно сформировать запрос SQL?

Question

[vlad1753]

Есть запрос:

$sql_update = "UPDATE `numbers_book` SET `name`='$num' WHERE `id`= ".$id;

Вылазит ошибка:

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

Comments

[Дмитрий]

Так покажите строку на которую ругается, сейчас видно только выражение. мы не знаем что в $id и поэтому не можем знать какое получится выражение.

[vlad1753]

Дмитрий, Если вывести через echo эту переменную получается такое:

UPDATE `numbers_book` SET `name`='p16ex3' WHERE `id`= 34097

$id - числовое значение

Answer 1

[FanatPHP]

переменная id у тебя пустая.
чтобы переменные не вызывали ошибок в запросах, их надо подставлять не напрямую, а через плейсхолдеры, а сами запросы выполнять через подготовленные выражения

Comments

[vlad1753]

Попробывал PDO.

$charset = 'utf8';
$options = [PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION];


try{
	$pdo = new PDO("mysql:localhost;dbname=copyhomework;charset=$charset", 'root', '', $options);
}catch(PDOException $e){
	exit('Не удалось подключиться к базе данных!');
}

$from = 427;

$to = 436;


for ($i = $from; $i <= $to; $i++) { 
	replacement($i);
}


function replacement($topic_id)
{
	echo $topic_id.'<br>';
	
	$sql = 'SELECT * FROM `numbers_book` WHERE `topic_id` = :topic_id'; 

	$stmt = $pdo->prepare($sql);

	$stmt->execute( [':topic_id' => $topic_id] );



	while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
		echo $row['num'];
	}
}

Выдает ошибку:

Fatal error: Uncaught Error: Call to a member function prepare() on null in

[FanatPHP]

ты проделал очень неплохую работу!
ошибка эта мелкая - внутри функций переменные снаружи не видны.
поэтому $pdo надо передавать в функцию тоже:

function replacement($pdo, $topic_id)

и вызывать так же

replacement($pdo, $i);

из поправок ещё - код соединения с адо совсем не годится. надо взять отсюда: https://phpdelusions.net/pdo_examples/connect_to_mysql

[vlad1753]

FanatPHP, Спасибо, вам, огромное за помощь.
Я исправил ту ошибку, но вылезла ещё одна...

Fatal error: Uncaught PDOException: SQLSTATE[3D000]: Invalid catalog name: 1046 No database selected in F:\OSPanel\domains\gdzcom\parser\numbers.php:35 Stack trace: #0 F:\OSPanel\domains\gdzcom\parser\numbers.php(35): PDOStatement->execute(Array) #1 F:\OSPanel\domains\gdzcom\parser\numbers.php(23): replacement(Object(PDO), 427) #2 {main} thrown in F:\OSPanel\domains\gdzcom\parser\numbers.php on line 35

Строка 35:

$stmt->execute( [':topic_id' => $topic_id] );

[vlad1753]

vlad1753, Решил, вместо:

$sql = 'SELECT * FROM `numbers_book ` WHERE `topic_id` = :topic_id';

Надо:

$sql = 'SELECT * FROM copyhomework.numbers_book WHERE `topic_id` = :topic_id';

[FanatPHP]

надо проверить что copyhomework действительно прописано правильно при соединении