если нормально переписать

Ну не будь так строг)

Код избыточен.

Не соответствует стандартам

Что за эпитеты)

говнокод)

говнокод)

настрочил тут 5 сообщений вместо одного сообщения емкого. человек не любит мусолить, поэтому так и пишу

То что фореч в форече плохо я понял, пошел смотреть примеры join на ларе

хорошее начинание

всем хороших выходных

всем хороших выходных

вам тоже)

Тур два

public function lists($url){ $sections=SectionSite::where('url',$url)->first(); if(is_null($sections)) return abort(404); $subSections=SectionSite::where('parent_id',$sections->id)->pluck('id')->all(); $arrayCompanysIds=CompanySection::where('section_id',$sections->id)->orWhereIn('section_id',$subSections)->pluck('company_id')->all(); $arrayCompanys=Company::WhereIn('id',$arrayCompanysIds)->get(); return view('company.list')->with('Sections',$sections)->with('arrayCompanys',$arrayCompanys); }

Какие еще будут советы по улучшению?

Какие еще будут советы по улучшению?

я тебе выше про связи писал

я тебе выше про связи писал

что такое связь><

и насчет "наглядности и так понятности типов данных" $arrayCompanys=Company::WhereIn('id',$arrayCompanysIds)->get(); Вот это тебе не массив вернет

что такое связь><

https://laravel.com/docs/5.4/eloquent-relationships

я про one-to-one, one-to-many, many-to-many и тп

Какие еще будут советы по улучшению?

для начала давай на gist с подсветкой.тяжело же считать

В другом чате писал —- и получилось что-то типа $arrayCompanys = $section->companys()->get()->toArray()

для начала давай на gist с подсветкой.тяжело же считать

да, согласен сюда кидать код было не самой удачной идеей

для начала давай на gist с подсветкой.тяжело же считать

2 е. Если уже не знаете анг. пишите транслитом по русски

$sections=SectionSite::where('url',$url)->first(); if(is_null($sections)) есть firstOrFail или как-то так

$sections=SectionSite::where('url',$url)->first(); if(is_null($sections)) есть firstOrFail или как-то так

занятно что у ларика куча крутых методов...

Я же сторался чтобы реализованный модуль был отказаустойчивым

https://github.com/Folleah/clementine/blob/master/app/Http/Controllers/MenuController.php у вас просто говнокода нормального не было

кекеке

Вообщем моментов интересных много, но они все связаны с самой ларой...На чистом php не так то просто было бы строчки выкидывть)

$sections=SectionSite::where('url',$url)->first(); if(is_null($sections)) есть firstOrFail или как-то так

спасибо за fistrorfail -2 строчки

На чистом пхп было бы едва ли не проще. Вы слишком полагаетесь на фреймворк

Например на чистом пхп вы бы написали может и кривой запрос к базе, но один вынимающий все и сразу, оставалось бы только отсортировать

На ларавеле заюзали оверхед в виде элоквента чтобы загнать в форич

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

кекеке

father ахаха)

не ну а шо)

сексизм в коде

Например на чистом пхп вы бы написали может и кривой запрос к базе, но один вынимающий все и сразу, оставалось бы только отсортировать

тут скорее сказыватеся непривычность. согласен что на sql я бы заджоинил в 1 строку...длинную...Но это пройдет буквально скоро.

Гг

Вообще это не вина фреимворка

я не нашел способа лучше отсортировать категории

сексизм в коде

Не подходишь под новую фичу гитхаба с опенсурной инициативой. Сердечко которое в настройках добавили

что бы на фронте их можно было просто выводить

сексизм в коде

Ты в Ваней то договорился до чего-то?

Ты в Ваней то договорился до чего-то?

мне сказали в пятницу-понедельник ответят

тишина...

пока что вообще ничего не ясно, даже тестовое не дал

тишина...

Ты сам напомни ему

да навязываться как то... Не этично)

У них часто запара бывает большая и забывают

я подожду до понедельника

да навязываться как то... Не этично)

Тебе деньги и работа не нужны?

нужны конешн

но он сказал пятница-понедельник

Тебе деньги и работа не нужны?

и мне нужны. что там у тебя за деньги ?

раньше срока то че спрашивать

Ну так напомни ему в телеге

и мне нужны. что там у тебя за деньги ?

кыш кыш кыш

фу кака

?

Раз напомнили про напоминалки... блин, а этот алкаш свалил отсюда

и мне нужны. что там у тебя за деньги ?

Да вот фолли сосватал прежнему работодателю, а он стесняется о себе напоминать

Кинул на 200 ре короче((00

ERROR: S client not available

все все

ща напомню :D

Да вот фолли сосватал прежнему работодателю, а он стесняется о себе напоминать

давай ник. я сам о себе напомню с рекламой себя ?

давай ник. я сам о себе напомню с рекламой себя ?

Ну, у тебя же есть работа, а фолли без дела мается) надо же помогать людям)

Если увидите @matero в чатах, или он уже есть в общих с вами - гоните его и насмехайтесь над ним. Алкаш занявший 200 ре на два дня, в итоге пропавший на три месяца ?

Ну, у тебя же есть работа, а фолли без дела мается) надо же помогать людям)

у мен нет денег =(

бля, да ты охуел

бля, да ты охуел

Ахахаха

у мен нет денег =(

Какой прайс?

у мен нет денег =(

у тебя батя бизнесмен, живешь в праге

работаешь

А то может в след месяце надо будет поискать людей

еще и у меня тут хочешь единственный шанс на работу отнять

А то может в след месяце надо будет поискать людей

Вот фолли мается без дела

Какой прайс?

от 25 евро/ч

в минуту

от 25 евро/ч

Нормально так

от 25 евро/ч

Ок, запомню

Нормально так

Обычная относительно московская зп

Обычная в смысле обычная для нормального спеца

у вас спецы 250 тыщ все получают ?

Обычная относительно московская зп

286к в месяц?

286к в месяц?

Для сениора?

Это где столько обычным платят?

Там где вас нет очевидно)

?

охуеть ставочка

Там где вас нет очевидно)

Я подчеркиваю - обычным нормальным спецам, а не сеньерам))

А сеньеры необычные спецы?

Волшебные?)

Мы же уровни обсуждаем, а не словоблудием занимаемся?)