Не делайте лишний вызов функции

В данном фрагменте php-кода проверяется значение, возвращаемое функцией. При этом функция вызывается дважды, хотя можно было бы сохранить результат выполнения функции в переменную, и уже дальше работать с этой переменной.

Плохо

foreach($rules as $rule) {
    //1-й вызов функции
    if ($this->userMatchRule($user, $rule) === 'match') {
        return true;
    }
    //2-й вызов функции
    if ($this->userMatchRule($user, $rule) === 'notMatch') {
        return false;
    }
}

Хорошо

foreach($rules as $rule) {
    //Сохраняем результат работы ф-ции в переменную
    $matchResult = $this->userMatchRule($user, $rule);
    
    //Далее работаем с переменной
    if ($matchResult === 'match') {
        return true;
    }
    if ($matchResult === 'notMatch') {
        return false;
    }
}