如何避免PHP嵌套如果..还..条件和编写干净、易于理解的代码?

Question

我正在codeigniter中编写一个登录方法，用户将在其中输入电子邮件和密码，然后它将执行以下操作：

1. 验证用户输入 -> 如果有效，则执行点 #2
2. 从数据库中获取用户详细信息 -> 如果查询成功，请执行 point#3
3. 验证密码 -> 如果已验证，则登录到系统

我知道我可以通过使用纯嵌套来实现上述目标，如果...否则条件，但那样会非常乏味且难以阅读。 所以，我这样试过——

// Empty array to hold processing errors
$processing_errors = array();
// Check for valid input
if(!$is_valid_input) array_push($processing_errors, "Invalid inputs");
// Get user details
if($get_user_details)
{   
// Verify the user
if(!$is_correct_user) array_push($processing_errors, "Incorrect username or password.");
}
else array_push($processing_errors, "Database query error");
// Check if processing errors is empty 
if(empty($processing_errors)) echo "Successfully logged in."; // Success
else foreach($processing_errors as $errors) echo $errors; // Errors

但是上述方法的问题在于，即使它返回 false 并且会花费不必要的处理时间，它仍然会在Check for valid input后执行代码。

编写整洁干净的可读代码的最佳方法是什么？

Answer 1

您可以将这些方法分解为仅公开登录请求方法并将其他所有内容保留在内部(如private方法中(的方法，如下所示。userLogin是请求的用武之地。

片段：

class LoginController extends CI_Controller{
private $errors;
function __construct(){
$this->errors = []; // initialize error bag to an empty array
/*
load needed models
*/
}
private function validateInput($post_data){
/*
validate user details
*/
if(!$is_valid_input){
$this->errors[] = '';
}
return $is_valid_input;
}
private function getUserDetails($post_data){
/*
get the user data using models
*/
return $user_data;
}
private function verifyUser($post_data){
$user_data = $this->getUserDetails($post_data);
$is_correct_user = $user->email == $post_data['email'] && password_verify($post_data['password'],$user->password);
if(!$is_correct_user){
$this->errors[] = 'Incorrect username or password.';
}
return $is_correct_user;
}
private function displayErrors(){
foreach($this->errors as $current_error){
echo $current_error,"<br/>";
}
}
public function userLogin(){
$data = $this->input->post();
try{
if($this->validateInput($data) && $this->verifyUser($data)){
echo "Successfully logged in";
/* Your futher code */
}else{
$this->displayErrors();
}
}catch(Exception $e){
echo $e->getMessage(); // if you throw some Exception in your model for some situations. Better to have exact class name of Exception for faster lookup of the class
}       
}
}