Подготавливаемый запрос PHP

Здравствуйте, Александр.

Помогите разобраться с корректностью записи кода.
1. весь ли код ООП?
2. там ли htmlspecialchars и он ли должен быть для экранирования?
3. синтаксические ошибки в написанном prepare()?

function insert($name, $desc, $year, $rating, $poster, $category_id) {
	$mysqli = new mysqli('localhost', 'root', '', 'kinomonster');
	if ($mysqli->connect_errno) {
		printf('Connect failed: ' . $mysqli->connect_error);
	    exit();
	}

/*	instead 5,6,7 strings
if ($mysqli->connect_errno) {
    die('Connect failed: ' . $mysqli->connect_errno);
}*/

	$mysqli->set_charset('utf8');

	/*$query = "INSERT INTO movie VALUES(null, '$name', '$desc', '$year', '$rating', '$poster', Now(), '$category_id')";*/
	 $stmt = $mysqli->prepare("INSERT INTO movie (null, name, desc, year, rating, poster, Now(), category_id) VALUES (?, ?, ?, ?, ?, ?)");
	$result = false;
	if($mysqli->query($query)) {
		$result  = true;
	}
	return $result;
}

$xml = simplexml_load_file("xml/movies.xml") or die("Error: Cannot create object");

/*echo count($xml);*/

$title = null;
$description = null;
$post = null;
$rating = null;
$year = null;

<cut/> foreach ($xml as $movie_key => $movie) {
 	$title = $movie->title_russian;
 	$description = $movie->description;
 	$year = $movie->year;

 	foreach ($movie->poster->big->attributes() as $poster_key => $poster) {
 		$post = $poster;
 	}

 	if($movie->imdb) {
 		$rating = $movie->imdb->attributes()['rating'];
 	} else {
 		$rating = null;
 	}

    $stmt -> bind_param('ssidsi', null, '$name', '$desc', '$year', '$rating', '$poster', Now(), '$category_id');
	$stmt -> execute();

   insert($title, $description, $year, $rating, $post, 1);
 }

echo "<pre>";
print_r ($xml);
echo "</pre>";

СПАСИБО!

Комментарии: 4

Sergey
2020-04-27 16:17:01
Спасибо, Александр за такой обширный ответ. Вопросов еще больше стало — но, в первую очередь, буду разбираться с синтаксисом ООП.

Прошу прощения, относительно htmlspecialchars, то строка выглядит следующим образом
```
$stmt = htmlspecialchars($mysqli->prepare('INSERT INTO movie (null, name, desc, year, rating, poster, Now(), category_id) VALUES (?,?,?,?,?,?)'));
```
Сам синтаксис вероятно неправильный функции prepare! Возможно, еще по этому поводу пару замечаний?!

СПАСИБО!

Александр Мальцев
2020-04-28 13:18:12
Так htmlspecialchars использовать нельзя. Если вы вставляете в какое-то поле текст и вам необходимо заменить в нём все специальные символы на HTML сущности, то эту функцию необходимо использовать именно для него.
Например:
```
$description = htmlspecialchars($movie->description);
```
Если, например, вставляемые данные должны быть числом, то так:
```
$year = (int) $movie->year;
```
Кроме этого, после вставки данных (в данном случае после foreach) запросы рекомендуется явно закрыть:
```
$stmt->close();
```