0

Есть большой if, нужно избавится от повторений и более грамотно и красиво оформить условие, но совсем не понимаю как это сделать.

    if ($contr_type == 0) {
        if ($contr_doc_id != 4) {
            if ($contr_doc_id == 1 && ($contr_ser_doc =~ /\D/g || $contr_num_doc =~ /\D/g)) {
                if (!$contr_bday) {
                    $personal_data = '1. паспорт и дата рождения';
                } else {
                    $personal_data = '2. паспорт';
                }
            } else {
                if ($contr_ser_doc && $contr_num_doc && $contr_bday) {
                    $personal_data = 'Ок';
                } elsif ((!$contr_ser_doc || !$contr_num_doc) && !$contr_bday) {
                    $personal_data = '1. паспорт и дата рождения';
                } elsif ((!$contr_ser_doc || !$contr_num_doc) && $contr_bday) {
                    $personal_data = '2. паспорт';
                } elsif (($contr_ser_doc && $contr_num_doc) && !$contr_bday) {
                    $personal_data = '3. дата рождения';
                }
            }
        } else {
            if (($contr_ser_doc || $contr_num_doc) && $contr_bday) {
                $personal_data = 'ОК';
            } elsif (!$contr_ser_doc && !$contr_num_doc && !$contr_bday) {
                $personal_data = '1. паспорт и дата рождения';
            } elsif ((!$contr_ser_doc && !$contr_num_doc) && $contr_bday) {
                $personal_data = '2. паспорт';
            } elsif (($contr_ser_doc || $contr_num_doc) && !$contr_bday) {
                $personal_data = '3. дата рождения';
            }
        }
    }
8
  • subroutines - если перл. про пых не знаю. 26 окт 2018 в 11:10
  • А поподробнее ? 26 окт 2018 в 11:26
  • А что у вас в $contr_doc_id, $contr_ser_doc, $contr_num_doc, $contr_bday находится?
    – user285292
    26 окт 2018 в 11:30
  • $contr_doc_id - число, $contr_ser_doc- строка, $contr_num_doc - строка, $contr_bday - строка 26 окт 2018 в 11:35
  • если еще конкретнее, то серия, номер документа и дата рождения. В contr_doc_id просто id-шник 26 окт 2018 в 11:38

1 ответ 1

3

Можно отмечать отдельными битами отсутствие того или иного признака, а потом по их совокупности подготовить нужную фразу. Как нибудь так:

my $data=0;
if ($contr_type == 0) {
        if($contr_doc_id == 1) {
                $data=1 if($contr_ser_doc =~ /\D/g || $contr_num_doc =~ /\D/g);
        } else {
                $data=1 if(!$contr_ser_doc || !$contr_num_doc);
        }
} else {
        $data=1 if(!$contr_ser_doc && !$contr_num_doc);
}
$data |= 2 if(!$contr_bday);

$personal_data = {
        0 => 'Ok',
        1 => '2. паспорт',
        2 => '3. дата рождения',
        3 => '1. паспорт и дата рождения'
}->{$data};
1
  • Спасибо огромное! Все прекрасно работает, и выглядит в разы лучше) 26 окт 2018 в 12:43

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.