Лучший способ использования, если условие

Question

Мне нужно запустить тестовый скрипт, который имеет более 10 условий для проверки. Если какой-либо шаг завершится неудачно, программа завершится с ошибкой, и дальнейшие шаги не будут выполнены. Текущая, моя программа такова:

boolean status = 0; 

status = function1(param1, param2); 
if (status) { 
    status = function2(param1, param2); 
    if(status) { 
     status = function3(param1, param2); 
     if(status) { 
     status = function4(param1, param2); 
     } 
     : 
     : 
     : 
    } 
} 

return status; 

Я не доволен столькими встроенными предложениями IF. Есть ли лучший способ сделать это?

Answer 1

В переводе на Perl, вы можете написать:

sub testing 
{ 
    my $status; 

    if (($status = function1($param1, %param2)) && 
     ($status = function2($param1, %param2)) && 
     ($status = function3($param1, %param2)) && 
     ($status = function4($param1, %param2))) 
    { 
     # ...do whatever... 
    } 

    return $status; 
} 

Или, учитывая, что $status просто булево:

sub testing 
{ 
    return 0 if !function1($param1, %param2) || 
       !function2($param1, %param2) || 
       !function3($param1, %param2) || 
       !function4($param1, %param2); 

    # ...do whatever... 

    return 1; 
} 

Или:

sub testing 
{ 
    return 0 unless function1($param1, %param2); 
    return 0 unless function2($param1, %param2); 
    return 0 unless function3($param1, %param2); 
    return 0 unless function4($param1, %param2); 
    # ...do whatever... 
    return 1; 
} 

Если вы действительно получил 10 функций с идентичными сигнатурами вызова, тогда вы могли бы даже сделать:

sub testing 
{ 
    my @funcs = (&function1, &function2, &function3, &function4, &function5, 
        &function6, &function7, &function8, &function9, &function10, 
       ); 

    for my $funcref (@funcs) 
    { 
     return 0 unless &$funcref($param1, %param2); 
    } 

    # ...do whatever... 

    return 1; 
} 

Обратите внимание, что первые три альтернативы будут обрабатывать сигнатуры вызова расходящихся функций только с мессианской компоновкой кода; для последнего в значительной степени требуется одна и та же подпись вызова функции для каждой из функций в списке.

Answer 2

я хотел бы отметить еще один вариант:

прошел = истинные

прошло = прошло & & COND1 прошли = прошли & & cond2 ... если (прошло) ...

Answer 3

return function1($param1, %param2) && 
     function2($param1, %param2) && 
     function3($param1, %param2) && 
     function4($param1, %param2); 

Для этого используется short-circuit, чтобы прекратить оценку, как только один из функции возвращают ложное значение.

Answer 4

Помните, что && короткого замыкания, что означает, что в следующем операторе

func1 && func2 

если func1 оценивается как ложное, то func2 никогда не оценивали, так как && никогда не может быть правдой в дальнейшем.

Это означает, что вы можете реплицировать функциональность без каких-либо if пунктов:

return func1 && func2 && func3 ... 

Answer 5

Другой вариант, который позволяет избежать всех вложенности, но это похоже на оригинал:

boolean status = 0; 

status = function1(param1, param2); 
if (status) { 
    status = function2(param1, param2); 
} 
if (status) { 
    status = function3(param1, param2); 
} 
if (status) { 
    status = function4(param1, param2); 
} 

Этот стиль позволяет легко добавлять больше звонков и простого переупорядочения.