Имея более эффективный код для возврата ошибок

Question

я создал главный класс, где все мои основные функции деактивации активации члена являются и всеми другими вещами, связанных с ними, также сделал

это основной класс (наряду с некоторыми основными функциями) вызывается из разных мест, в том числе через завиток

Теперь давайте мою activaton функцию (Одна из основных функций) в классе

activationFunction($data) 
{ 
//use data to generate total, discount etc 

$this->giveAffiliates($total); 
if($this->_error){ return $this->_error;} 

$this->activateOrder($total,$discount,id); 
if($this->_error){ return $this->_error;} 

$this->activatePlan($total,$discount,id); 
if($this->_error){ return $this->_error;} 

//similarily calling various functions which themselves call other functions 

} 

activatePlan() 
{ 

try{ 

//call other functions and do necessary stuff for plan A 

} 
catch(Exception $e) 
{ 

$this->_error.="Error occurred while activating plan A"; 

} 
//for plan B 
try{ 

//call other functions and do necessary stuff for plan B 

} 
catch(Exception $e) 
{ 

$this->_error.="Error occurred while activating plan B"; 

} 

//for other plans similarily 

} 
} 

Теперь вопрос, имеющий if($this->_error){ return $this->_error;} после каждого вызова подфункций. (Total Im имеет около 35 таких похожих строк) Мне это нужно, поскольку мне нужно отправить сообщение об ошибке пользователю и остановить мой код от дальнейшего использования. Но это делает мой код длинным и неэффективным. Как я могу уменьшить все эти возвращения, но показать пользователю ошибку, когда одна из вспомогательных функций не работает, и попытаться сохранить структуру кода как есть. Я должен вызывать различные подфункции из каждой основной функции (это я не могу изменить, в ней будет только один класс и различные функции), и ошибки в основном должны быть пойманы на каждом уровне и возвращены (очень мало простых ошибок не возвращается, и код разрешен для продолжения работы). Должен также иметь в виду, что позже могут быть добавлены различные другие функции, и он должен быть достаточно гибким, чтобы обрабатывать все это позже.

Answer 1

Вы говорите: «это делает мой код длинным и неэффективным». Что именно вы имеете в виду «неэффективно»? Вы говорите, что ваш код медленный?

Если ваш код слишком медленный, вам необходимо прокомментировать ваш код с помощью инструмента, такого как XDebug, чтобы узнать, где именно ваш код медленный. Если вы не измеряете, вы просто догадываетесь о том, в чем проблема.

Answer 2

Я бы подумал, что каждая функция должна обрабатывать свои собственные ошибки/исключения. Чтобы вы не нажимали всю обработку ошибок в каждую бледность, вызываемую функцией. Абоненту функции не нужно будет содержать логику о том, как генерировать уведомления об ошибках вызываемой функции.

Так что скажите, что каждая функция выдает исключение из условия ошибки. Затем вы можете сделать что-то вроде этого:

activationFunction($data) 
{ 
    //use data to generate total, discount etc 
    try { 
    $this->giveAffiliates($total); 
    $this->activateOrder($total,$discount,id); 
    $this->activatePlan($total,$discount,id); 
    } catch (Exception $e) { 
    // throw the exception up the call chain 
    throw $e; 
    } 
}