Как быть с контроллером?

Question

[Николай Громов]

Здравствуйте! При переходе по ссылке с главной страницы на страницу с какой-то конкретной категории, у меня задействуется CategoryController который является дочерним (по маршрутам) от модуля Shop, вот конфигурационный файл где это описано:

'router' => [
        'routes' => [
            'shop' => [
                'type'    => Literal::class,
                'options' => [
                    // Change this to something specific to your module
                    'route'    => '/shop/',
                    'defaults' => [
                        'controller'    => Controller\IndexController::class,
                        'action'        => 'index',
                    ],
                ],
                'may_terminate' => true,
                'child_routes' => [
                    'category' => [
                        'type'    => Segment::class,
                        'options' => [
                            'route'    => 'category/[:action/][:id/]',
                            'constraints' => [
                                'action' => '[a-zA-Z][a-zA-Z0-9_-]*',
                                'id' => '[0-9]+',
                            ],
                            'defaults' => [
                                'controller'    => Controller\CategoryController::class,
                                'action'        => 'index',
                            ]
                        ]
                    ],
                    // You can place additional routes that match under the
                    // route defined above here.
                ],
            ],
        ],
    ],

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

zblog.local/shop/category/1

Когда работаешь с пагинатором адрес изменяется на

zblog.local/shop/category/?page=2

, но id должен оставаться так как на нем завязана работа контроллера. Код который отображает пагинатор на странице:

<?= $this->paginationControl($products,
            'Sliding',
            'application/partial/paginator', 
            ['route' => 'shop/category']); ?>

Я так понял, что бы пагинатор работал правильно в route нужно дописать id, но что бы это сделать, нужно для category прописать дочерний маршрут в конфигурационном файле. Я просто не знаю как правильно это сделать, нужно что бы id обязательно был цифрами. Подскажите кто знает!

Comments

[German Jet]

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

[Николай Громов]

German Jet: скажем так, это что то вроде хобби. Я не планирую на нем работать, по крайней мере пока. А то что сообщество мертвое, для меня в какой-то мере даже хорошо, чем меньше мне подсказывают тем больше я учусь!)))

[German Jet]

Николай Громов: о, молодец! Читали книжку Кришны Шасанкара?

[Николай Громов]

German Jet: Если честно, даже не знаю кто это такой, и о чем книга про которую вы говорите!))) Но судя по имени и форуму, это наверно индус-программист!))) На досуге посмотрю что за книга, если будет полезна почитаю!

[German Jet]

Николай Громов: я говорю о книге «Zend Framework 2.0. Разработка веб- приложений»)

[Николай Громов]

German Jet: Надо будет посмотреть, может поможет разобраться с маршрутами для пагинатора

Answer 1

[novrm]

Значит так - пиши такой маршрут.
А об пагинаторе я тебе отписал дальше.

'router' => [
    'routes' => [

        /** Configuration of the 'shop' routes. */
        'shop' => [
            'type'    => 'Literal',
            'options' => [
                'route'    => '/shop/',
                'defaults' => [
                    'controller' => Controller\IndexController::class,
                    'action'     => 'index',
                ],
            ],
            'may_terminate' => true,
            'child_routes' => [

                /** Configuration of the 'shop/category' routes. */
                'category' => [
                    'type'    => 'Literal',
                    'options' => [
                        'route'    => 'category/',
                        'defaults' => [
                            'controller' => Controller\CategoryController::class,
                            'action'     => 'index',
                        ],
                    ],
                    'may_terminate' => true,
                    'child_routes' => [

                        /** Configuration of the 'shop/category/action' routes. */
                        'action' => [
                            'type' => 'Segment',
                            'options' => [
                                'route' => '[:action]',
                                'constraints' => [
                                    'action' => '(create)',
                                ],
                            ],
                        ],

                        /** Configuration of the 'shop/category/category' routes. */
                        'category' => [
                            'type' => 'Segment',
                            'options' => [
                                'route' => '[:id/]',
                                'constraints' => [
                                    'id' => '[0-9]+',
                                ],
                                'defaults' => [
                                    'id' => null,
                                ],
                            ],
                            'may_terminate' => true,
                            'child_routes' => [

                                /** Configuration of the 'shop/category/category/action' routes. */
                                'action' => [
                                    'type' => 'Segment',
                                    'options' => [
                                        'route' => '[:action]',
                                        'constraints' => [
                                            'action' => '(update|delete)',
                                        ],
                                    ],
                                ],
                            ],
                        ],
                    ],
                ],
            ],
        ],
    ],
],

Comments

[Николай Громов]

Что означает 'action' => '(create)', 'action' => '(update|delete)'. данные маршруты не работают. В indexcontroller не отображаются фотографии. Вот код во вьюхе который вызывает изображение

<?php foreach($files as $file): ?>  
                <?php if ($product->getImage() == $file):?>
                    <div class="img-thumbnail">
                        <img src="<?= $this->url('shop/category', ['action'=>'file'], 
                                ['query'=>['name'=>$file], 'thumbnail'=>true]); ?>" width="100">
                    </div>
                    <div class="caption">                
                        <p>
                            <a target="_blank" href="<?= $this->url('shop/category', ['action'=>'file'], 
                                ['query'=>['name'=>$file]]); ?>" 
                                class="btn btn-default" role="button">Увеличить изображение</a>
                        </p>
                    </div>
                <?php endif;?>
            <?php endforeach;?>

при переходе по ссылке в categorycontroller вообще ничего нет, ни пагинатора, ни таблички товара. Вот что пишется в адресной строке

zblog.local/shop/category

так как нет id то соответственно нет и карточки товара, потому что она выдергивается из базы по id. Кстати в этом участке коде разве не нужен "/"

'action' => [
                            'type' => 'Segment',
                            'options' => [
                                'route' => '[:action???/]',
                                'constraints' => [
                                    'action' => '(create)',
                                ],
                            ],
                        ],

[novrm]

Николай Громов: action - create, update, delete - соответственно создать, редактировать, удалить категорию.
Для этого нужно создать соответствующие action ну и виды.
Я ведь не знаю каким образом вы манипулируете категориями.

Относительно слеша - это по вкусу. Можете добавлять.

Извините немного трудно вам объяснить не зная уровня вашего познания ZF3.

Теперь относительно вашего action 'file'.
Как я понимаю у вас вместо defaults action index - нужно указать defaults action file.

---------------

В общем, как я понимаю, у вас - есть 'category' и 'goods'.
Вы собираетесь манипулировать этими объектами?
Если да - то вам нужен CRUD - create, read, update, delete интерфейс.

Честно говоря не совсем понимаю для чего вы пытаетесь через action вызвать изображение?
Вы сохраняете изображения в БД? Это достаточно затратная по времени процедура.
Почему просто не написать:
<img src="/img/my_picture.jpg">

Чем вы взаимодействуете с БД?

Честно говоря - пагинатор это одновременно и легко и тяжело.
В двух словах не опишешь.

[Николай Громов]

novrm: Начнем по порядку.
Мой уровень познания ZF3, в принципе как и PHP, самый нижний из начинающих.
Нет у меня defaults action index.
CRUD у меня уже создан для управления category и products, только в другом модуле.
В базе данных у меня сохраняется только название файла изображения, сам файл сохраняется в папке img. Из-за этого у меня и идет это условие<?php if ($product->getImage() == $file):?>
С БД я взаимодействую через Doctrine, кстати сам пагинатор тоже ее, только обертка ZF3. Наверно я лучше покажу вам код с контроллера, может тогда вам легче понять меня будет.

<?php

namespace Shop\Controller;

use Zend\Mvc\Controller\AbstractActionController;
use Shop\Entity\Category;
use Shop\Entity\Products;
use Zend\View\Model\ViewModel;

use DoctrineORMModule\Paginator\Adapter\DoctrinePaginator as DoctrineAdapter;
use Doctrine\ORM\Tools\Pagination\Paginator as ORMPaginator;
use Zend\Paginator\Paginator;

class CategoryController extends AbstractActionController
{
    /**
    * Менеджер сущностей.
    * @var Doctrine\ORM\EntityManager
    */
    protected $entityManager;
    
    /**
     * Менеджер продуктов.
     * @var Shop\Service\ProductManager 
     */
    protected $productManager;
  
    // Метод конструктора, используемый для внедрения зависимостей в контроллер.
    public function __construct($entityManager, $productManager) 
    {
        $this->entityManager = $entityManager;
        $this->productManager = $productManager;
    }
    
    public function indexAction()
    {       
        $categories = $this->entityManager->getRepository(Category::class)
                    ->getAllMainCatsWithChildren();
        
        // Получаем ID категории.         
        $categoryId = $this->params()->fromRoute('id', -1);
        
        // Находим существующую категорию в базе данных.    
        $cat = $this->entityManager->getRepository(Category::class)
                ->ChildrenCategory($categoryId);          
        if ($cat == null) {
            $queries = $this->entityManager->getRepository(Products::class)
                          ->getProductsById($categoryId);                        
        } else {
            $ids = [];            
            foreach ($cat as $c) {
                $ids[] = $c->getId(); 
            } 
            $queries = $this->entityManager->getRepository(Products::class)
            ->getProductsByCategoryIds($ids);   
        }
        
        // Получаем список уже сохраненных файлов.
        $files = $this->productManager->getSavedFiles();         
        $page = $this->params()->fromRoute('page', 1);
        
        $adapter = new DoctrineAdapter(new ORMPaginator($queries, false));
        $paginator = new Paginator($adapter);        
        $paginator->setDefaultItemCountPerPage(1);        
        $paginator->setCurrentPageNumber($page);
        // Визуализируем шаблон представления.       
        return new ViewModel([
            'categoryId'    => $categoryId,
            'categories'    => $categories,
            'products'      => $paginator,
            'files'         => $files,
        ]);        
    }
    
    public function fileAction() 
    {
        // Получаем имя файла из GET-переменной.
        $fileName = $this->params()->fromQuery('name', '');

        // Проверяем, что нужно пользователю: изображение в полном размере или миниатюра.
        $isThumbnail = (bool)$this->params()->fromQuery('thumbnail', false);
    
        // Получаем путь к файлу изображения.
        $fileName = $this->productManager->getImagePathByName($fileName);
        
        if($isThumbnail) {
        
            // Изменяем размер изображения.
            $fileName = $this->productManager->resizeImage($fileName);
        }
                
        // Получаем информацию файла изображения (размер и MIME-тип).
        $fileInfo = $this->productManager->getImageFileInfo($fileName);        
        if ($fileInfo===false) {
            // Устанавливаем код состояния 404 Not Found
            $this->getResponse()->setStatusCode(404);            
            return;
        }
                
        // Запсиываем HTTP-заголовки.
        $response = $this->getResponse();
        $headers = $response->getHeaders();
        $headers->addHeaderLine("Content-type: " . $fileInfo['type']);        
        $headers->addHeaderLine("Content-length: " . $fileInfo['size']);
            
        // Записываем содержимое файла.
        $fileContent = $this->productManager->getImageFileContent($fileName);
        if($fileContent!==false) {                
            $response->setContent($fileContent);
        } else {        
            // Устанавливаем код состояния 500 Server Error.
            $this->getResponse()->setStatusCode(500);
            return;
        }
        
        if($isThumbnail) {
            // Удаляем временный файл миниатюры изображения.
            unlink($fileName);
        }
        
        // Возвращаем экземпляр Response, чтобы избежать визуализации представления по умолчанию.
        return $this->getResponse();
    }    
}

p.s. Кстати по моему из-за того что

'may_terminate' => true,
            'child_routes' => [

                /** Configuration of the 'shop/category' routes. */
                'category' => [
                    'type'    => 'Literal',
                    'options' => [
                        'route'    => 'category/',
                        'defaults' => [
                            'controller' => Controller\CategoryController::class,
                            'action'     => 'index',
                        ],
                    ],

литеральный, не получается запустить fileaction, у меня уже было такое и мне пришлось переделать его в сегмент, правда у меня после него не было дочерних маршрутов.

[novrm]

Николай Громов: Здравствуйте.
Насколько я понял - вы начинающий разработчик на ZF3.
Позвольте тогда вам несколько советов, которые сильно упростят ваши разработки.

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

С конструктором в контроллере я бы не возился - это лишнее. Используйте геттеры/сеттеры.

К примеру возьмем пример блога на ZF3, где есть только два объекта "post" и "tag". В вашем случае - "post" это "product", "tag" - '"category".
Значит для отображения списка постов с применением пагинации - есть контроллер PostsController.

namespace Blog\Controller;

use Zend\Mvc\Controller\AbstractActionController;
use Zend\View\Model\ViewModel;

/**
 * PostsController controller.
 *
 * Handles the PostsController requests for the Blog module.
 *
 * @package Blog\Controller
 */
class PostsController extends AbstractActionController
{

    /**
     * Traits.
     */
    use \Fork\Zend\ServiceManager\Traits\ServiceManagerAwareTrait;
    use \Blog\View\Traits\PostsViewAwareTrait;

    /**
     * indexAction action.
     *
     * @return mixed
     */
    public function indexAction()
    {                                   /** Retrieve services and plugins. */
        $postsView = $this->getPostsView();
                                        /** Creates and settings view model. */
        return (new ViewModel())
        ->setVariables($postsView->getListVariables())
        ->setVariables($postsView->getPaginationVariables())
        ; 
    }

}

Для отображения отдельно взятого поста - другой контроллер PostController.

namespace Blog\Controller;

use Zend\Mvc\Controller\AbstractActionController;
use Zend\View\Model\ViewModel;

/**
 * PostController controller.
 *
 * Handles the PostController requests for the Blog module.
 *
 * @package Blog\Controller
 */
class PostController extends AbstractActionController
{

    /**
     * Traits.
     */
    use \Fork\Zend\ServiceManager\Traits\ServiceManagerAwareTrait;
    use \Blog\View\Traits\PostViewAwareTrait;

    /**
     * indexAction action.
     *
     * @return mixed
     */
    public function indexAction()
    {                                   /** Retrieve services and plugins. */
        $postView = $this->getPostView();
                                        /** Retrieve request parameters. */
        if (!empty($this->params('slugPost'))) {
                                        /** Fetches entity corresponding request parameters. */
            if (!empty($postViewVariables = $postView->getVariables())) {
                return new ViewModel($postViewVariables);
            } else {                    /** If entity not exists - return page "404". */
                 return $this->CreateHttpNotFoundModel($this->getResponse());
            };
        } else {                        /** Redirect to posts controller. */
            return $this->redirect()->toRoute('posts');
        };
    }

}

Логику разного рода манипуляций, проверок, извлечения, работы с БД переносите в специализированные сервисы. Даже работу с переменными вида можете закинуть в отдельный сервис.

Теперь дальше. Если работает с Doctrine - вам вполне будет достаточно и родного пагинатора для ZF3. Однако адаптер к пагинатору вам все же нужно будет написать свой.

Дело в том, что снова же - настроить пагинацию - это тонны кода.

Потом я бы вам рекомендовал все же перейти на шаблонизатор twig вместо родного php. twig легок в изучении и гораздо гибче. Это сэкономит вам уйму времени.

--------------
Еще вопрос. Извините, не совсем понимаю смысл ваших манипуляций с изображениями при помощи php на стороне сервера.
Почему это нельзя сделать JavaScript на стороне клиента?
Как я понимаю - это галерея? Тогда только JavaScript.
--------------

Если желаете продолжить диалог - пишите мне на почту [e-mail]

Answer 2

[Vladimir Zhosan]

Мне одному маршруты в этом фреймворке кажутся убогими?

Comments

[Николай Громов]

Может мы их просто не достаточно хорошо понимаем?

[Vladimir Zhosan]

Николай Громов: в Laravel и Codeigniter по-моему мнению больше читабельны роуты

Answer 3

[OrlandoST]

Передавайте id в параметрах.
Вот пример:
<?= $this->paginationControl($products,
'Sliding',
'application/partial/paginator',
['route' => 'shop/category', 'params' => array('id' =>1')]); ?>

Comments

[Николай Громов]

Я не могу передать конкретный id, мне его надо вытащить из объекта пагинатора $products. Есть две таблицы продукты и категории и они связаны между собой. Получается нужно через таблицу продуктов вытащить id категории. Как это можно сделать?

[OrlandoST]

Николай Громов: а что нельзя во вьюху передать id категории? Что мешает сделать это в контроллере?

[Николай Громов]

OrlandoST: Когда я предаю во вьюху id взятую из маршрута

$categoryId = $this->params()->fromRoute('id', -1);
return new ViewModel([
            'categoryId'    => $categoryId
        ]);

и подставляю ее в отображение виджета пагинатора

<?php 
    $this->paginationControl($products, 
            'Sliding',
            'application/partial/paginator', 
            ['route' => 'shop/category', 'params'=>['id'=>$categoryId]]); 
?>

виджет не отображается!

[OrlandoST]

Надо писать $this->categoryId

[Николай Громов]

Так все равно не отображается
'params'=>['id'=>$this->$categoryId]
и так тоже
'params'=>['id'=>$this->categoryId]

[Николай Громов]

OrlandoST: var_dump($categoryId) в контроллере выдает такой результат string(2) "13" может это быть из-за того что это строка а не число, хотя другие запросы в контроллере с этой переменной работают.

[OrlandoST]

Николай Громов: посмотри есть ли переменная в шаблоне. Сделай vardump из шаблона

[OrlandoST]

Попробуй указать все параметры маршрута, включая экшен и контроллер

[Николай Громов]

OrlandoST: var_dump($categoryId) из шаблона показывает string(1) "1" т.е. тот id который я выбираю. А на счет экшена и контроллера, они же и так у меня указаны,

['route' => 'shop/category', 'params'=>['id'=>$categoryId]]);

или вы имеете ввиду в params дополнительно их прописать?

[Николай Громов]

OrlandoST: написал так

['route' => 'shop', 'params'=>['controller'=>'category', 'action'=>'index', 'id'=>$categoryId]]);

всеравно не отображается!

[Николай Громов]

OrlandoST: и 'id'=>$this->$categoryId
тоже не работает

[OrlandoST]

Николай Громов: сделай без переменных сначала, передай просто статические значения

[OrlandoST]

Николай Громов: это и не будет работать. Неверный синтаксис

[Николай Громов]

OrlandoST: Можете подсказать где неверный синтаксис, не среда разработки не интерпритатор php ничего не говорят!

[Николай Громов]

OrlandoST: статика тоже эффекта не дает

['route' => 'shop/category', 'params'=>['id'=>'1']]);

[Николай Громов]

OrlandoST: Видать ошибку синтаксиса нашел, раз виджет появился, но при переходе на другую страницу id из адреса пропадает.

[Николай Громов]

OrlandoST: Тут в другом посте мне посоветовали сделать такvar_dump($this->params);в application/partial/paginator
вот что мне показал var_dump при

['route' => 'shop/category', 'params'=>['id'=>$categoryId]]

array(1) { ["id"]=> string(1) "1" }

,
при переходе на другую страницу var_dump показывает

array(1) { ["id"]=> int(-1) }

т.е. согласно этой строки

$categoryId = $this->params()->fromRoute('id', -1);

никакой id не передается в маршрут.
а вот что он же показал при

['route' => 'shop/category', 'params'=>['id'=>$this->$categoryId]

array(1) { ["id"]=> NULL }

Я так понимаю надо как то либо закольцевать передачу id пока пользователь прыгает по страницам пагинатора, либо где то отдельно сохранять id и его уже передавать в вьюху. Можете подсказать как правильно это сделать?

[OrlandoST]

Николай Громов: пришли код экшена из контроллера и покажи как ты id в шаблон отдаешь

[Николай Громов]

public function indexAction()
    {    
        // Создаем меню  
        $categories = $this->entityManager->getRepository(Category::class)
                    ->getAllMainCatsWithChildren();
        
        // Получаем ID категории.         
        $categoryId = $this->params()->fromRoute('id', -1);
        
        // Находим существующую категорию в базе данных.    
        $cat = $this->entityManager->getRepository(Category::class)
                ->ChildrenCategory($categoryId);          
        if ($cat == null) {
            $queries = $this->entityManager->getRepository(Products::class)
                          ->getProductsById($categoryId);                        
        } else {
            $ids = [];            
            foreach ($cat as $c) {
                $ids[] = $c->getId(); 
            } 
            $queries = $this->entityManager->getRepository(Products::class)
            ->getProductsByCategoryIds($ids);   
        }
        
        // Получаем список уже сохраненных файлов.
        $files = $this->productManager->getSavedFiles();         
        $page = $this->params()->fromRoute('page', 1);
        
        $adapter = new DoctrineAdapter(new ORMPaginator($queries, false));
        $paginator = new Paginator($adapter);        
        $paginator->setDefaultItemCountPerPage(1);        
        $paginator->setCurrentPageNumber($page);
        // Визуализируем шаблон представления.       
        return new ViewModel([
            'categoryId'    => $categoryId,
            'categories'    => $categories,
            'products'      => $paginator,
            'files'         => $files,
        ]);        
    }