0
        public function search(Request $request) 
            {
                
                $prefixLinks = [
                    'Page' => '',
                    'Painter' => 'avtomalyar/',
                    'Post' => 'post/',
                ];
                
                $query = $request->input('query');
                
                $links = [];
                
                $pages = Page::where('title', 'LIKE', '%' . $query . '%')
                        ->orWhere('body', 'LIKE', '%' . $query . '%')
                        ->get();
                $painters = Painter::where('company', 'LIKE', '%' . $query . '%')
                        ->orWhere('about', 'LIKE', '%' . $query . '%')
                        ->get();
                $posts = Post::where('title', 'LIKE', '%' . $query . '%')
                        ->orWhere('body', 'LIKE', '%' . $query . '%')
                        ->get();
                
                foreach ($pages as $page){
                    $linksRow['slug'] = $prefixLinks['Page'] . $page->slug;
                    $linksRow['title'] = $page->title;
                    array_push($links, $linksRow);
                }
                
                foreach ($painters as $painter){
                    $linksRow['slug'] = $prefixLinks['Painter'] . $painter->id;
                    $linksRow['title'] = $painter->company;
                    array_push($links, $linksRow);
                }
                
                foreach ($posts as $post){
                    $linksRow['slug'] = $prefixLinks['Post'] . $post->slug;
                    $linksRow['title'] = $post->title;
                    array_push($links, $linksRow);
                }
                
    
    //отображаем VIEW
                foreach ($links as $link){
                    echo '<a href="' . url($link['slug']). '">'. $link['title'] . '</a><br>'; 
                }
            }


//Вариант 2 переделал
$request->validate([
        'query' => 'string',
    ]);
    
    $query = $request->input('query');
    
    $links = [];
    
    //Модели по каким производить поиск
    $modelsSearch = [
        
        'Page' => [
            'options' =>[
                'prefixLink' => '/',
                'getLink' => 'slug',
                'getLinkText' => 'title',
                'searchBy' => 'title',
            ],
        ],
        'Painter' => [
            'options' =>[
                
                'prefixLink' => 'avtomalyar/',
                'getLink' => 'id',
                'getLinkText' => 'company',
                'searchBy' => 'company',
            ],
        ],
        'Post' => [
            'options' =>[     
                'prefixLink' => 'post/',
                'getLink' => 'slug',
                'getLinkText' => 'title',
                'searchBy' => 'title',
            ],
        ],
    ];
    
    foreach ($modelsSearch as $modelName => $setting){
         
        $model = 'App\\Models\\'. $modelName;
        
        $searchLists = $model::where($setting['options']['searchBy'], 
                'LIKE', '%' . $query . '%')->get();
        
        foreach ($searchLists as $searchList){
            $linksRow['link'] = $setting['options']['prefixLink'] . $searchList[$setting['options']['getLink']];
            $linksRow['linkText'] = $searchList[$setting['options']['getLinkText']];
            array_push($links, $linksRow);
        }
    }
    
    $collectLinks = collect($links);
    
    // convert array to collection with pagination
    $results = (new Collection($collectLinks))->paginate(self::DEFAULT_PAGINATE);
    
    return view('search', compact('results')); 
1

1 ответ 1

0

Честно говоря - то что вы выложили это очень плохой код, и это очень мягко говоря. Советы:


Валидация входных данных. Создайте Request, провалидируйте данные, сделайте исключения и т.д.


Реализуйте логику не по необходимости, а по смыслу и предназначению. Например создайте репозиторий, реализуйте в нём метод searchPagesByQuery($query) и уже этот метод через контракт используйте в вашем методе search()


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


Не используйте полнотекстовый поиск SQL. В перспективе это в лучшем случае - рефакторинг. я бы посоветовал использовать ElasticSearch

7
  • спасибо, посмотрите переделал 2 вариант
    – junior
    Commented 18 июн. 2021 в 12:22
  • @junior, гораздо лучше, но всё равно так не пишут. Если в реализации метода контроллера более 10 строчек - что то пошло не так)
    – CoderYooda
    Commented 18 июн. 2021 в 14:06
  • вынести в другой метод часть то не проблема уже)
    – junior
    Commented 19 июн. 2021 в 7:25
  • а что вы имели ввиду Реализуйте логику не по необходимости, а по смыслу и предназначению. Например создайте репозиторий, реализуйте в нём метод searchPagesByQuery($query) и уже этот метод через контракт используйте в вашем методе search() ///////////////////Вопрос////////// а почему не создать прям в контроллере searchPagesByQuery($query) и тут его и вызвать? self::searchPagesByQuery($query)?
    – junior
    Commented 19 июн. 2021 в 8:13
  • Потому что этот метод вам пригодится и в других контроллерах с вероятностью 80+ процентов и всё равно придется выносить логику в репозиторий
    – CoderYooda
    Commented 20 июн. 2021 в 9:25

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.