Выборка из бд по id и get запросы?

Question

[valitskiydmitriy]

Имеется сайт, некое подобие интернет магазина. На главной странице есть список товаров, у каждого товара свой id. Пытаюсь передать его на страницу покупки, не получается, в ручную вписываю в url ?id=2 и тоже не работает, на странице покупки такой код: (Что не так, куда копать?)

$products_id = isset($_GET['id']) ? $_GET['id'] : 0;
  $stmt = $db->prepare('SELECT * FROM rm_products WHERE id=$id' );
  $stmt->execute();

  $print = '';
     while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
      $print .= '<div class="product">';
И,Т,Д

Comments

[Александр Евгеньевич]

Зачем выполнять запрос к БД, если id не существует в GET массиве ? Возвращайте NotFound.

Answer 1

[Виталий Инчин ☢]

Вам что не говорили, что переменные между апострофами PHP не видит?

$stmt = $db->prepare('SELECT * FROM rm_products WHERE id='.$id);
//Или
$stmt = $db->prepare("SELECT * FROM rm_products WHERE id=$id");

Comments

[valitskiydmitriy]

Виталий Инчин Попробовал оба варианта, не работает, ничего не выводится, может ли быть ошибка где то здесь: while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) {

[Виталий Инчин ☢]

valitskiydmitriy: Лучше так:

$stmt->setFetchMode(PDO::FETCH_ASSOC);  
while($row = $stmt->fetch()).......

[Равшан Абдулаев]

valitskiydmitriy: может... если $db - объект PDO, что требуется еще уточнить с вашей стороны. То:
1) параметры в функции ->prepare, лучше использовать как в Примере 1 по ссылке: php.net/manual/ru/pdo.prepare.php
2) обратите внимание что функция ->execute() возвращает bool, и в случае ошибки не плохо было бы добавить обработчик по типу trigger_error('код ошибки:'.$db->errorCode(), E_USER_ERROR)

... что делаете дальше в принципе по мануалу...

[Равшан Абдулаев]

valitskiydmitriy: хм... вместо пункта 2) можно использовать настройку PDO:
$db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
php.net/manual/ru/pdo.error-handling.php
в рамках тестирования можно включать данную опцию

[Alexander Litvinenko]

а может раз использован prepare то параметры по нормальному передавать?

Answer 2

[Алексей Уколов]

$products_id = isset($_GET['id']) ? $_GET['id'] : 0;
$stmt = $db->prepare('SELECT * FROM rm_products WHERE id=$id' );

Вы сохраняете id в переменную $products_id, а в запрос передаете $id.
Ну и как правильно указал Виталий Инчин, в одинарных кавычках переменная не интерполируется.

Answer 3

[Евгений Елчев]

На самом деле из вашего примера ничего не ясно. Попробуйте пройтись дебагером и посмотреть соответсьвет ли алгоритм логике. Если не работаете с дебагером, то то расставьте флаги на каждом этапе. вроде такого:

echo $_GET['id'];
$products_id = isset($_GET['id']) ? $_GET['id'] : 0;
echo $products_id;
echo 'SELECT * FROM rm_products WHERE id='.$id;

ну и так далее, вы увидите где ошибка.

Comments

[valitskiydmitriy]

Евгений Елчев Увидел что id передается коректно. Теперь просто с бд инфа не передается

[Виталий Инчин ☢]

valitskiydmitriy: проверьте подключение к БД, авось ошибка?

[Евгений Елчев]

Тогда выкладывайте полный листинг.

[valitskiydmitriy]

Евгений Елчев:

<? 
require 'template.php'; 

  $db = new PDO('mysql:host=localhost;dbname=admin','admin','admin');
  $id = $_GET['id'];
  echo $id;
  $stmt = $db->prepare("SELECT * FROM rm_products WHERE id=$id");
  $stmt->execute();
  $print = '';
     $stmt->setFetchMode(PDO::FETCH_ASSOC);  
     while($stmt->fetch()){
              echo $row['link'];
      $print .= '<div class="product">';
    //Создаем div - p-image
      $print .= '<div class="p-image">'
    //В таблице находим данные в столбцах "link" и "img"
        .'<a href="' . $row['link'] . '"> <img src="/_media/' . $row['media'] . '.png"/></a>';
    //Закрываем div
      $print .= '</div>';
      $print .= '<div class="p-title">'
        .'<a href="' . $row['link'] . '"> ' . $row['title'] . ' </a>';
      $print .= '</div>';
      $print .= '<div class="p-description">'
        .'<p> ' . $row['description'] . ' </p>';
      $print .= '</div>';
    //Закрываем div - Product
    $print .= '</div>';
     }
?>

[Евгений Елчев]

Во первых в цикле Wile вы не присваиваете результат извлечения в переменную Row while($row = $stmt->fetch()){
Во вторых сделайте счетчик в цикле и проверьте сколько раз он выполняется
В третьих ваша таблица содержит данные?
В четверых не мешайте логику и вывод, это ужасно. Поищите статьи про mvc.