Как вывести значения одной строки PDO?

Question

[dimaweyder]

if(isset($_GET['edit_id'])){
			$id = $_GET['edit_id'];
			$find = $db->query("SELECT * FROM products WHERE id='$id' ");

		}

При клике на ссылку такого вида:
http://localhost/shop/admin/edit.php?edit_id=3
Передается edit_id методом GET.
Как мне вывести значение товара по переданному id.
Значение товара:
name, description, price,.

Answer 1

[green_goo]

$id = $_GET['edit_id'];
$stmt = $db->prepare("SELECT * FROM products WHERE id=?");
$result = $db->execute([$id]);
if (($data = $result->fetch()) !== false) {
    var_dump($data);
}

Comments

[Антон Р.]

Если не добавить PDO::FETCH_ASSOC то ключи и значения задублируются, будет: [0]=>'id' и тут же ['id]=>'id' например

[FanatPHP]

Антон Р., это можно сделать один раз при соединении. И не писать потом в каждом запросе

Answer 2

[Антон Р.]

if(isset($_GET['edit_id'])){
      $id = $_GET['edit_id'];
      $find = $db->query("SELECT * FROM products WHERE id='$id' ");
      $result = $find->fetch(PDO::FETCH_ASSOC);
    }

В результате в переменной $result получишь ассоциативный массив где наименование столбцов это индексы а значения это значения из ячеек.

Comments

[green_goo]

Или false, если такой строки нет.

[Антон Р.]

green_goo, ну да. Я кстати сам встрял с аналогичным вопросом когда надо было вывести ОДНУ строку через PDO

[FanatPHP]

Это очень плохой ответ.

Должно быть написано "В результате получишь SQL инъекцию"

[Антон Р.]

FanatPHP, тут же вопрос не про безопасность, а так да, надо ->bindvalue

[FanatPHP]

Когда ты в аптеке покупаешь антибиотик от сифилиса, вопрос тоже не про безопасность, а про таблетки. Но провизор тебе обязан сказать, что водку пить с антибиотиком нельзя. Это называется профессионализм. То есть если ты взялся отвечать, то или делай так, чтобы твой ответ не нанес вреда, или стой молча.

[Антон Р.]

FanatPHP, задача была написать ответ на Конкретный вопрос, а не переписать код Полностью со всеми проверками и валидациями, до которых человек должен сам дойти и изучить. Если так капитально всем тут "жопу подтирать" - жизни не хватит.
Суть вопроса => Суть ответа, а не "он мне слово, я ему - сто".

[FanatPHP]

Ишь, разошелся. Себе сначала жопу подтирать научись, учитель

[Антон Р.]

FanatPHP, а ты бы лучше ответы писал а не критиковал чужие, учитель.

[FanatPHP]

Молодец, применил детсадовский дискуссионный прием. Выставил себя ещё большим посмешищем.

Когда (если) вырастешь, то поймешь, что умным ты выглядишь не когда оставил за собой последнее слово, а когда научишься признавать и исправлять свои ошибки

[Антон Р.]

FanatPHP, сочувствую тем кто работает с вами.

[FanatPHP]

Со мной работают либо взрослые, либо молодежь, которая не пытается никого учить. И мы отлично ладим :)

Answer 3

[ThunderCat]

Зачем пдо если вы все равно лепите на нем инъекции?

if(!empty($_GET['edit_id']) && !empty(intval($_GET['edit_id']))){
      $id = intval($_GET['edit_id']);
      $res = $db->prepare("SELECT * FROM `products` WHERE id = :id ");
      $res->execute(['id'=>$id]);
      var_dump($res->fetch(PDO::FETCH_ASSOC));
 }

Comments

[FanatPHP]

Какой-то переусложненный код с недовалидацией.

Хотя конечно основная мысль верная

[ThunderCat]

FanatPHP, какой сурс такой и код. Если сейчас начну еще филтервар сюда писать чувак вообще потонет.

[FanatPHP]

В сурсе ничего этого нет :)
И оно там не нужно.

Надо или добавлять нормальную валидацию, или не добавлять ничего. А вот этот вот бессмысленный карго культ код смысла не имеет.

Валидацию вообще должна быть отделена от работы с бд. и давать осмысленный фидбек. А в том виде, в котором оно здесь сейчас, то лучше совсем убрать

[ThunderCat]

FanatPHP, не согласен, ну да ладно. Если я фигачу что-то из скриптов на 100 строк все равно худо-бедно валидирую все что приходит из пользовательского ввода.

Валидацию вообще должна быть отделена от работы с бд. и давать осмысленный фидбек.

Кто спорит, просто тут про какую либо архитектуру вообще ни сном ни духом, хоть как то приучать к валидации.