Айбек
1. неиспользуемые переменные
2. move кинет тебе exception если не сможет положить на сервер, а так просто выводи
Timur
Timur
хотел еще вопрос задать
Timur
по стайл гайду вся бизнес логика прописывается в Модели, верно?
invariance
какой еще стайлгайд
invariance
для бизнес логики
invariance
шта
invariance
по стайл гайду вся бизнес логика прописывается в Модели, верно?
тонкие модели, толстые контроллеры
invariance
из контроллеров можешь в сервисы вынести
Айбек
Самый простой способ, создать папку app/Services
и там хранить
Timur
Timur
одни говорят толстые Модели, тонкие контроллеры
Timur
другие наоборот
Timur
непонятно
Timur
в моем методе кроме говнокода что не так? имеется ввиду логику и всякие проверки модели вытащить?
invariance
все со своей колокольни просто смотрят
Timur
читал про mvс паттерн написано вся логика хранится в Модели
Timur
пол года уже пишу, всю логику держу в контроллере
invariance
эти паттерны не являются спецификацией
invariance
просто советы
invariance
которые работают совсем не везде и не всегда
invariance
значит я делаю пока что все правильно?
ну конкретно в твоем методе, сделай вывод ответа в одном месте, плюс юзай связи что бы код сократить
invariance
@as2989 чуть не по делу тут не приветствуется сорян
invariance
спрашивай в чате цссников
Serhio
ок, но фулстеки то тут есть
Timur
invariance
спасибо за помощь
и еще придерживайся правила - один метод не больше 50 строк. Но чем меньше - тем лучше
Timur
и еще придерживайся правила - один метод не больше 50 строк. Но чем меньше - тем лучше
да, вот сел как раз рефакторить код к черту) буду срезать, вытаскивать в отдельные файлы и т.д. Еще раз спасибо
Nik
да, вот сел как раз рефакторить код к черту) буду срезать, вытаскивать в отдельные файлы и т.д. Еще раз спасибо
и да начнется первый круг ада :)
invariance
да, вот сел как раз рефакторить код к черту) буду срезать, вытаскивать в отдельные файлы и т.д. Еще раз спасибо
главное не делай избыточную абстракцию и декомпозицию
Timur
главное не делай избыточную абстракцию и декомпозицию
постараюсь, у меня очень много методов использующих к примеру загрузку файлов одну и ту же. Можно ведь это в сервис провайдер вытащить? или просто хжлпер класс написать?
Владислав
и да начнется первый круг ада :)
потом выяснится, что эти швабры держали потолок. и внезапно выскочит alert с текстом: "оставь надежду, всяк сюда входящий"
invariance
Nik
Nik
потом выяснится, что эти швабры держали потолок. и внезапно выскочит alert с текстом: "оставь надежду, всяк сюда входящий"
и гдето тут начинаешь понимать ценность тестов :)
Айбек
мб отдельный сервис который работает с FIleBag
и тупо $this->service->store($request->files)
invariance
сможешь доказать обратное - валяй)
Nik
сможешь доказать обратное - валяй)
зачем? как правильно сказали - трейты, это легализованная копипаста. в контексте ларки я вообще не вижу места для их применения, кроме моделей, если нужно в них поднасрать
invariance
Nik
то-есть фраза "поднасрать" не смутила? :)
Nik
если тебе нужна копипаста, то сервис ну прям напрашивается
Anton
Вопрос, а допустим у тебя есть Юзер и у Юзера есть релейшен country (country_id). И вот у тебя загруженные User, ты ему меняешь country_id и ->save(), но у тебя до этого был загружен релейшен country с другой страной... внимание вопрос - релейшен сбросится?
Айбек
лучше так
$user->country()->associate($country);
Айбек
а если просто $user->country_id = $id; то нет
Anton
перечитать это все понятно
Nik
я почти что уверен что нет, так как не факт что это вообще country_id :) ты в релейшенах можешь биндить любое на любое
Nik
у меня в бд например вот такие нейминги - ModelId, вместо model_id, и мне многое приходится делать ручками :)
Айбек
когда $user->country_id = $id
то просто смотрит есть ли мутатор, берет его значение
значение кидает в атрибуты
больше ничего вроде
Nik
кто-то юзал какой-то пре-хук чтобы модифицировать версию проекта автоматически при коммите ?
Nik
хочу счетчик обновлять
Nik
https://github.com/antonioribeiro/version - никто не юзал ?
Anonymous
и еще придерживайся правила - один метод не больше 50 строк. Но чем меньше - тем лучше
в книжках пишут за 10ть строк (
invariance
это идеальный несуществующий код
invariance
к нему надо стремиться, но не слепо придерживаться
Maksim (Ellrion)
https://github.com/antonioribeiro/version - никто не юзал ?
я так понял там версия не автоматическая, автоматом только билд вставляется не? просто саму версию можешь только ты корректно проставить (ведь надо решь что это за изменения минор\мажор\патч)
Nik
ну минор-мажор можно и руками, но патч инкрементить было бы приятно
Nik
думал вообще чейнджлог вывернуть наружу по коммитам, но потом понял что это не лучшая идея :D
Maksim (Ellrion)
ну минор-мажор можно и руками, но патч инкрементить было бы приятно
я делал у себя в Kernel константу версии. а саму версию выводил так:
$commit_hash = trim(exec('git log --pretty="%h" -n1 HEAD'));
$commit_date = new \DateTime(trim(exec('git log -n1 --pretty=%ci HEAD')));
$commit_date->setTimezone(new \DateTimeZone('UTC'));
return sprintf(
'v%s.build-%s [%s] (%s)',
Kernel::VERSION,
$commit_hash,
app()->version(),
$commit_date->format('Y-m-d H:m:s')
);
Женя
Гайс, как отключить DB_CONNECTION полностью?
Maksim (Ellrion)
чё?
Женя
чё?
Задача использовать лару как фронтенд оболочку, хочу отключить коннект к базе, что бы крашей при сборке не было.
На сервере нет никаких database драйверов.
Maksim (Ellrion)
чёт я не догоняю. не будешь использовать бд то ничего и не будет. не?
Maksim (Ellrion)
запросишь бд ну оно полезит искать конфиги коннекта. их не будет или они будут некорректные и ты отвалишься с эксепшеном. логично же.
Женя
Вот этот экзепшн в jenkisns делает билд нестабильным
Maksim (Ellrion)
так занчит что то у тебя лезит в бд
Alexander
Так и не используй бд
Женя
Сори, в джобах забыл убрать migrate.
Спасибо :*
Maksim (Ellrion)
открыл тут для себя неплохой сервис, мб я конечно слоупок но https://repl.it/
удобно, самое вкусное это аватокомплит и разные языки в одном месте
invariance
давно уж есть
invariance
мне больше https://3v4l.org нравится, там интересные фичи есть