MM
Переслано от Mr. Maklaren
Насколько плохо, что такая логика - в акшене контроллера?
Size: a a a
2020 September 07
MM
Но узнал, очень полезную штуку
Gt
На самом деле, всё затевалось, чтобы стриггерить на то, что в сёрвис не перенёс)
много ходов очка
мн
Читать сложно такое кол-во ифов, если return юзать в них, то можно избавиться от излишней вложенности, вместо этих else
MM
Читать сложно такое кол-во ифов, если return юзать в них, то можно избавиться от излишней вложенности, вместо этих else
Ну, я всегда читаю по названиям методов. Сразу видна логическая цепочка
мн
У тебя тут 2 состояния в итоге только получается, а мест где ты их проставляешь аж 4
мн
Инфа сотка это можно до двух ифов свести
мн
Вот эти два, сначала первый набор правил одним ифом, внутри хоть свой эко, а после него return, затем во втором также сразу эко и сразу ретерн, а ответ с false после этих двух ифов сразу экоешь. Грубо говоря так
мн
В итоге код читается гораздо легче, да и можно уже пересмотреть более логично что ты там ифать и проверять должен
мн
Вот эти два, сначала первый набор правил одним ифом, внутри хоть свой эко, а после него return, затем во втором также сразу эко и сразу ретерн, а ответ с false после этих двух ифов сразу экоешь. Грубо говоря так
Хотя я вижу это кжается вообще в один можно перекинуть, но мне лень разбираться в коде, я спать
A
$result = $service-sendMessage();
if(!$result) $error = ‘Error’
if(!$result) $error = ‘Error’
A
Переписал в две строки
мн
$result = $service-sendMessage();
if(!$result) $error = ‘Error’
if(!$result) $error = ‘Error’
Затрем плохой код под ковер, так сказать)
Gt
$result = $service-sendMessage();
if(!$result) $error = ‘Error’
if(!$result) $error = ‘Error’
мн
Чтобы никто не увидел
A
Затрем плохой код под ковер, так сказать)
Нет. Разделил код на два, каждый из которых свою ответственность реализует
мн
Нет. Разделил код на два, каждый из которых свою ответственность реализует
Я к тому, что в сервисе останется то же самое, только вместо простановки этих параметров ретерны булевых значений
A
Я к тому, что в сервисе останется то же самое, только вместо простановки этих параметров ретерны булевых значений
Да. Но более важно, что мы отделили бизнес логику от логики формирования ответа
a
ктото может поделиться открытым проектом с хорошим покрытием тестами ?