Как решить проблему Undefined index: id?

Question

[Mini MVC]

У меня не выводиться новость на странице. Выводиться ошибка

Notice: Undefined index: id in D:\OpenServer\domains\mvc-oop\mvc-oop\www\controllers\News.php on line 21
Notice: Undefined offset: 0 in D:\OpenServer\domains\mvc-oop\mvc-oop\www\classes\AbstractModel.php on line 40
Warning: Invalid argument supplied for foreach() in D:\OpenServer\domains\mvc-oop\mvc-oop\www\views\news\one.php on line 1

В controllers/News.php есть функция actionOne()

<?php

namespace Application\Controllers;

use Application\Models\News as NewsModel;

class News{

    public function actionAll()
    {

        $news = NewsModel::findAll();
        $view = new \View();
        $view->items = $news;
        $view->display('news/all.php');

    }

    public function actionOne()
    {
        $id = $_GET['id'];
        $newsById = NewsModel::findOneByPk($id);
        $view = new \View();
        $view->items = $newsById;
        $view->display('news/one.php');
    }

}

А в AbstractModel есть:

public static function findOneByPk($id)
    {
        $class = get_called_class();
        $sql = 'SELECT * FROM ' . static::$table . ' WHERE id=:id';
        $db = new DB();
        $db->setClassName($class);
        return $db->query($sql, [':id' => $id])[0];
    }

а в самом view/one.php

<?php foreach ($items as $item): ?>
    <h1><?php echo $item->title; ?></h1>
    <div><?php echo $item->author; ?></div>
<?php endforeach; ?>

Не пойму что делаю не так, почему у меня не показывает новость по id?

Answer 1

[Максим Федоров]

$id = $_GET['id'];
у Вас нет гет-параметра id. Вообще в таких случаях нужно делать проверки на наличие индекса id в глобальном массиве $_GET

Answer 2

[Максим Тимофеев]

судя по всему findOneByPk() null, а для foreach должен быть массив или объект.
Мне кажется что ошибка в архитектуре так как foreach - это перебор массива, а findOneByPk подразумевает одну запись или null, но решить именно эту ошибку можно так:

<?php 
if(isset($items) and is_array($items)){
  foreach ($items as $item){
    echo $item->title;
    .....
  }
}

Comments

[Mini MVC]

Так тоже не идёт

[Максим Тимофеев]

Mr Web: где ошибка? В какой строке?

[Mini MVC]

Максим Тимофеев: никаких изменений небыло всё также

[Максим Тимофеев]

Mr Web: В Вашем коде нет нумерации строк. Поэтому понять на какую сроку жалуется "Notice: Undefined index: id in D:\OpenServer\domains\mvc-oop\mvc-oop\www\controllers\News.php on line 21" сложно, сделайте комменты хотя бы

[Mini MVC]

А как тут сделать так чтобы было видно нумерацию кода? Могу сделать скриншот

[Максим Тимофеев]

Mr Web: можно просто цифру вставить. Кнопочки такие, второй ряд клавиатуры сверху. Все не надо. Обозначьте только те строки что в ошибках.

[Mini MVC]

Максим Тимофеев:

Notice: Undefined index: id in D:\OpenServer\domains\mvc-oop\mvc-oop\www\controllers\News.php on line 21

public function actionOne()
    {
        <b>$id = $_GET['id'];</b> // 21 stroka
        $item = NewsModel::findOneByPk($id);
        $view = new \View();
        $view->assign('item', $item);
        $view->display('news/one.php');
    }

Notice: Undefined offset: 0 in D:\OpenServer\domains\mvc-oop\mvc-oop\www\classes\AbstractModel.php on line 40

public static function findOneByPk($id)
    {
        $class = get_called_class();
        $sql = 'SELECT * FROM ' . static::$table . ' WHERE id=:id';
        $db = new DB();
        $db->setClassName($class);
        <b>return $db->query($sql, [':id' => $id])[0];</b> //40 stroka
    }

Warning: Invalid argument supplied for foreach() in D:\OpenServer\domains\mvc-oop\mvc-oop\www\views\news\one.php on line 1

<?php foreach <b>($items as $item)</b>: ?> // 1 stroka
    <h1><?php echo $item->title; ?></h1>
    <div><?php echo $item->author; ?></div>
<?php endforeach; ?>

[Максим Тимофеев]

Mr Web: Ну так не передан id в GET? Остальные следствие.

[Mini MVC]

Максим Тимофеев: Где кето не переданно? Можешь пакозать как и куда передать ато я вообще заблудился

[Максим Тимофеев]

запрос к серверу должен содержать id, например mysite.com?id=2 или mysite.com/2 смотря как настроена маршрутизация, может и так mysite.com/news/2.html где "2" это параметр, который ты получаешь через $_GET['id']. Собственно ошибка именно о том, что его нет. Как следствие это влечет ошибки далее.

[Mini MVC]

Максим Тимофеев: Есть он у меня вот так и я вызываю его mvc-oop/product/index/2 всё равно пишет : Warning: Invalid argument supplied for foreach() in D:\OpenServer\domains\mvc-oop\mvc-oop\www\views\default\product.php on line 1
В контроллере у меня:

class Product{

    public function actionIndex()
    {

        $productId = isset($_GET['id']) ? $_GET['id'] : null;
        //if($productId == null) exit;

        $news = ProductModel::findOneByPk($productId);
        $view = new \View();
        $view->items = $news;
        $view->display('default/product.php');

    }

}

а в default\product.php

<?php foreach ($items as $item): ?>//stroka 1

<h3><?php echo $item->name; ?></h3>
<img src="/images/product/<?php echo $item->image; ?>" alt="">

<?php endforeach; ?>

Не пойму что я делаю не так......

[Mini MVC]

а вот абстракт модель:

public static function findOneByPk($id)
{
$class = get_called_class();
$sql = 'SELECT * FROM ' . static::$table . ' WHERE id=:id';
$db = new DB();
$db->setClassName($class);
return $db->query($sql, [':id' => $id])[0];
}

[Максим Тимофеев]

Mr Web: Видимо нет логики для парсинга параметров урл или она работает не верно. Наличие id в url не гарантирует его попадания в экшен контроллера.

[Mini MVC]

Максим Тимофеев: если я тебе скину свой проект мог бы пасмотреть где у меня ошибка? И конечно написатъ тут где именно была ошибка, буду очень благодарен, ато сам не могу вообще разобраться, что я толъко не пробывал ни чего не получаеться у меня.

[Максим Тимофеев]

Mr Web: Наверное нет. Вопервых реально нет времени, а во вторых не понимаю зачем Вы в эпоху отличных framework-ов пишите свой. Единственный вариант - это опыт и самообразование, но тогда нет смысла помогать точно.

[Mini MVC]

Максим Тимофеев: Для того чтобы понять любой другой framework мне нужно сперва самому фундамент создать узнать как всё работает, вот для этого я и создаю свой чтобы понятъ как laravel, symphone... все работают. Но я уже сам разобрался, прошлось много попотеть но зато не зря, думал тут мне быстро смогут ответить так как у меня самого времени мало бывает, семья работ....

[Максим Тимофеев]

Mr Web: Все отлично учится на примерах framework

Answer 3

[Дмитрий]

Добрый день.
Начните со следующего:
$id = isset($_GET['id']) ? $_GET['id'] : null;
А дальше посмотрите.
И если findOneByPk выбирает одну запись, то зачем Вам цикл в view/one.php?