Правильно ли написан php-код?

Question

[Роман]

На внутренней странице для вывода одной строки:

$id=1;
$views = $pdo->prepare('UPDATE post SET views=views+1 WHERE id = ?');
$views->execute([$id]);
$post = $pdo->prepare('SELECT * FROM post WHERE id = ?');
$post->execute([$id]);
$game = $post->fetch();

А внизу вывод комментариев:

$q = $pdo->prepare('SELECT * FROM comment WHERE app_id = "'.$comm_id.'" ORDER BY id');
	$q->execute();
	while ($comm = $q->fetch()){
}

Нормально? Не вытаскиваю ли я лишних данных? А то хостер каждый день ставит мне ограничения по ресурсам т.к. мои скрипты сильно грузят процессор.

Comments

[krypt3r]

По какой-то причине вы не используете prepared statement в выводе комментариев. Советую поправить

Answer 1

[Владлен Грачев]

Если комментарии без пагинации, то вроде ничего лишнего. А от хостера такого надо уходить. Либо к другому хостеру, если ничего нагруженного не держите, и он просто наглеет, либо на сервер с мощностями, соответствующими требованиям.

Comments

[Роман]

нагрузна на цп каждый день доходит до 80-100%, не понимаю из-за чего...

Answer 2

[IceJOKER]

php.net/manual/en/mysqli.quickstart.prepared-state...
Не закрываете statement

Comments

[Роман]

оригинал не понимаю, слишком сложно, можно на примере моего кода или же одну две строчки свои для примера

[Роман]

$post = null;
так?

[shell_execute]

Роман: /* explicit close recommended */
$stmt->close();

Answer 3

[Константин Б.]

Я думаю тебе нужно привести полный код. То что ты показал это наипростейшие два запоса.

for($i=1; $i<100; $i++){
			$array_id = $PDO->query('SELECT id_board_post FROM board_post');
			foreach ($array_id as $id){
				$id = $id['id_board_post'];
				$views = $PDO->prepare('UPDATE board_post SET count=count+1 WHERE id_board_post = :id');
				$views->execute(array('id' => $id));
				$post = $PDO->prepare('SELECT * FROM marka, board_post,  model 
									WHERE id_board_post= :id 
									AND model.id_model=board_post.id_model 
									AND board_post.id_marka=marka.id_marka LIMIT 1');
				$post->execute(array('id' => $id));
				$game = $post->fetch(PDO::FETCH_ASSOC);
				echo $game['brand'];
			}
		}

Вот такой запрос у меня не вызывает никаких проблем. Вероятнее всего у тебя конфликт в другом месте где-то, он и тормозит и грузит код.

Answer 4

[bigl]

А индексы в базе по полям из where у вас есть?

Comments

[Роман]

о чем вы?)