Как править такой код?(или как написать самый идиотоустойчивый код)?

Question

[Leopandro]

foreach ($params as $param)
        {
            if ($param['label'] != 'name'){
                if ($param['type_value'] == '1') {
                    if ($param['value'] != '')
                        $query->where(['like', $param['label'], $param['value']]);
                }
                else {
                    if(($param['firstcompare'] == '<' || $param['firstcompare'] == '<=' || $param['firstcompare'] == '=') && $param['firstvalue'] != '') {
                        if($param['secondcompare'] != '' && $param['secondvalue'] != ''){
                            $query->where([
                                'or',
                                $param['label'].$param['firstcompare'].$param['firstvalue'],
                                $param['label'].$param['secondcompare'].$param['secondvalue']
                            ]);
                               // ->andWhere([$param['secondcompare'], $param['label'], $param['secondvalue']]);
                        }
                        else{
                            $query->where([$param['firstcompare'], $param['label'], $param['firstvalue']]);
                        }
                    }
                    elseif($param['firstcompare'] == '>' || $param['firstcompare'] == '>=' && $param['firstvalue'] != '')
                    {
                        if($param['secondcompare'] != '' && $param['secondvalue'] != ''){
                            $query->where([
                                'and',
                                $param['label'].$param['firstcompare'].$param['firstvalue'],
                                $param['label'].$param['secondcompare'].$param['secondvalue']
                            ]);
                        }
                        else{
                            $query->where([$param['firstcompare'], $param['label'], $param['firstvalue']]);
                        }
                    }
                    elseif($param['firstcompare'] == '' || $param['firstvalue'] == '') {
                        if ($param['secondcompare'] != '' || $param['secondvalue'] != '')
                            $query->where([$param['secondcompare'], $param['label'], $param['secondvalue']]);
                    };
                }
            }
        }

Это функция, которая парсит это дело:


(Клиент попросил чтобы если первое сравнение < а второе > то был поиск "and" если первое значение больше второго а у меня если < и > применяется поиск по "or" т.к. по моему предположению должно быть < 1 и > 2 чтобы исключить из результата то что находится между 1 и 2, а он вводит первым значением 2 а вторым 1... )

Answer 1

[pibiv]

Javascript который будет разблокировать ввод второго поля только когда заполнено первое, такой вариант не катит?

Answer 2

[Александр Кубинцев]

Для таких случаев давно уже придумали в разных нормальных фреймворках Forms

Comments

[Leopandro]

это форма из динамической формы(модели).... вопрос был в другом

[Александр Кубинцев]

Leopandro: возможно не очень корректно поставлен вопрос. У вас в заголовке их 2 и они разные.
"Как править такой код" - я бы на вашем месте сперва задумался над огромным количество вложенных if-then-else и отрефакторил их.
"Как написать самый идиотоустойчивый код" - писать тесты, как уже порекомендовали.
Если же вопрос из области юзабилити, то это не к программистам.

Answer 3

[Александр Макаров]

Вынести в отдельный класс, написать для него тест. Потом можно причёсывать сколько влезет и смотреть, не сломался ли тест.