Size: a a a

Laravel для начинающих

2020 September 07

MM

Mr. Maklaren in Laravel для начинающих
Переслано от Mr. Maklaren
Насколько плохо, что такая логика - в акшене контроллера?
источник

MM

Mr. Maklaren in Laravel для начинающих
Но узнал, очень полезную штуку
источник

MM

Mr. Maklaren in Laravel для начинающих
источник

Gt

Galèriarch the Koban... in Laravel для начинающих
Mr. Maklaren
На самом деле, всё затевалось, чтобы стриггерить на то, что в сёрвис не перенёс)
много ходов очка
источник

мн

мистер никитос... in Laravel для начинающих
Mr. Maklaren
Переслано от Mr. Maklaren
Насколько плохо, что такая логика - в акшене контроллера?
Читать сложно такое кол-во ифов, если return юзать в них, то можно избавиться от излишней вложенности, вместо этих else
источник

MM

Mr. Maklaren in Laravel для начинающих
мистер никитос
Читать сложно такое кол-во ифов, если return юзать в них, то можно избавиться от излишней вложенности, вместо этих else
Ну, я всегда читаю по названиям методов. Сразу видна логическая цепочка
источник

мн

мистер никитос... in Laravel для начинающих
У тебя тут 2 состояния в итоге только получается, а мест где ты их проставляешь аж 4
источник

мн

мистер никитос... in Laravel для начинающих
Инфа сотка это можно до двух ифов свести
источник

мн

мистер никитос... in Laravel для начинающих
Вот эти два, сначала первый набор правил одним ифом, внутри хоть свой эко, а после него return, затем во втором также сразу эко и сразу ретерн, а ответ с false после этих двух ифов сразу экоешь. Грубо говоря так
источник

мн

мистер никитос... in Laravel для начинающих
В итоге код читается гораздо легче, да и можно уже пересмотреть более логично что ты там ифать и проверять должен
источник

мн

мистер никитос... in Laravel для начинающих
мистер никитос
Вот эти два, сначала первый набор правил одним ифом, внутри хоть свой эко, а после него return, затем во втором также сразу эко и сразу ретерн, а ответ с false после этих двух ифов сразу экоешь. Грубо говоря так
Хотя я вижу это кжается вообще в один можно перекинуть, но мне лень разбираться в коде, я спать
источник

A

Adel in Laravel для начинающих
$result = $service-sendMessage();
if(!$result) $error = ‘Error’
источник

A

Adel in Laravel для начинающих
Переписал в две строки
источник

мн

мистер никитос... in Laravel для начинающих
Adel
$result = $service-sendMessage();
if(!$result) $error = ‘Error’
Затрем плохой код под ковер, так сказать)
источник

Gt

Galèriarch the Koban... in Laravel для начинающих
Adel
$result = $service-sendMessage();
if(!$result) $error = ‘Error’
источник

мн

мистер никитос... in Laravel для начинающих
Чтобы никто не увидел
источник

A

Adel in Laravel для начинающих
мистер никитос
Затрем плохой код под ковер, так сказать)
Нет. Разделил код на два, каждый из которых свою ответственность реализует
источник

мн

мистер никитос... in Laravel для начинающих
Adel
Нет. Разделил код на два, каждый из которых свою ответственность реализует
Я к тому, что в сервисе останется то же самое, только вместо простановки этих параметров ретерны булевых значений
источник

A

Adel in Laravel для начинающих
мистер никитос
Я к тому, что в сервисе останется то же самое, только вместо простановки этих параметров ретерны булевых значений
Да. Но более важно, что мы отделили бизнес логику от логики формирования ответа
источник

a

aden in Laravel для начинающих
ктото может поделиться открытым проектом с хорошим покрытием тестами ?
источник