Как избавиться от if else?

Question

[MrDickson]

Можно ли как-то избавиться от уродливых if else, чтобы не нести их через весь код?
Или единственной альтернативой является перед if писать предполагаемую ошибку? что тоже не есть гуд

public function post(Request $request, Response $response, array $args)
    {
        if(!$this->container->session->has('user'))
        {
            if (Security::PassEmail($request->getParam('email')))
            {
                $boss = Boss::where(['email'], '=', $request->getParam('email'))->first();
                if ($boss !== null)
                {
                    if (password_verify($request->getParam('password'), $boss->password))
                    {
                        $this->container->session->set(['user' => $boss->email,
                            'auth_hash' => sha1($boss->email)]);
                        $this->setError('lang:post_auth_successful_auth');
                    } else
                    {
                        $this->setError('lang:post_auth_bad_password');
                    }
                } else
                {
                    $this->setError('lang:post_auth_no_user_found');
                }
            } else
            {
                $this->setError('lang:post_auth_bad_email');
            }
        } else
        {
            $this->setRedirect('/dashboard');
        }

        return AuthResponder::respond($response, $this->response_status);
    }

Comments

[ThunderCat]

SmInc, Справедливости ради стоит заметить что санация/проверка данных пользователя через механизм исключений так себе идея.
В целом, последовательно проверять данные наиболее верный путь. Копить ошибки или возвращать только первую - вопрос чисто юзеркейса.

[Stalker_RED]

SmInc, или собираем все ошибки в массив и возвращаем на клиент, чтобы показать уведомления.

[Stalker_RED]

SmInc, ок

Answer 1

[alex-1917]

На порядок более длинных простынях прекрасно ориентируюсь. Подсветка кода все такое...

1. Уберите никому ненужные переносы строки перед } else
кол-во строк сразу в 2 раза меньше
2. Уберите перенос строки здесь:

$this->container->session->set(['user' => $boss->email,
                            'auth_hash' => sha1($boss->email)]);

ЗАЧЕМ вы все это делаете?? Прекратите кодить на калькуляторе и купите уже 27-монитор, там вообще не будет таких проблем...

Comments

[Roman K]

Ах, как забавно читать людей, восхваляющих увеличение цикломатической сложности

[alex-1917]

Roman Kitaev, если честно, я и не собирался вникать в код...))
ТС озвучил в вопросе слово "уродливые", человеку самостоятельному сразу становится ясно, что в его коде торчат очевидные костыли в виде переносов. ТС кодит на калькуляторе, поэтому СНАЧАЛА ему необходимо оказать первую медицинскую в виде покупки монитора, а дальше уже удел вас, подающих. Разжуйте ему и в рот положите, да не оскудеет рука ...
Ну а человеку, сидящему на мамкиных борщах, хочется сразу блеснуть псевдознаниями и оные рвутся в бой, услужливо переписывая подкинутый ТС говнокод...

[Roman K]

alex-1917, 2 года пишу на ноуте 13.3 дюйма, прекрасно себя чувствую.

[Ilnur Davletbaev]

Видимо кто-то не очень знаком с общепринятыми правилами написания кода...

[Сергей]

У меня 2 по 27 и в строке не бывает больше 100 символов.

[alex-1917]

Сергей, вот тебе запросик!!!)))

$sql = 'SELECT al.order_id, MIN(al.x) as x1, MIN(al.y) as y1, MAX(al.x)+1 as x2, MAX(al.y)+1 as y2, (MAX(al.x)+1-MIN(al.x))*(MAX(al.y)+1-MIN(al.y)) as sumpix, mi.text as text, mi.link as link, mi.img_id as img_id, mi.img_type as img_type, mi.name as name, mi.email as email, mi.phone as phone, inet_ntoa(mi.user_ip) as user_ip, mi.mill_date as mill_date, mi.mill_id as mill_id, (SELECT COUNT(*) FROM mill_like WHERE mill_like.like_order_id = mi.mill_id) AS like_count, (SELECT COUNT(*) FROM mill_like WHERE mill_like.like_order_id = mi.mill_id AND mill_like.like_ip=INET_ATON("'.$ip.'")) AS like_ip FROM all_items al, mill_items mi WHERE order_id>0 AND s=2 AND al.order_id=mi.mill_id AND mi.text LIKE ? group by order_id ORDER BY order_id ASC LIMIT '.$start.', '.$num;

[Сергей]

alex-1917, И удобно вам читать такое?