如何识别和消除 PHP 中的代码异味

代码异味是指代码中表明存在潜在问题或效率低下的某些特征或模式。识别和解决代码异味对于维护干净、可维护和可扩展的代码库至关重要。

在本文中，我们将探讨 PHP 中的一些常见代码异味并提供示例和解决方案。

长方法：

代码异味:长方法是指代码中的方法过长，通常超过 20 行。长方法通常表明该方法试图同时执行太多操作，这可能会导致以下问题：

代码难以理解和维护。
代码难以测试。
代码难以扩展。

// Example of a long method
class OrderProcessor {
    public function processOrder($order) {
        // ... many lines of code ...
    }
}

解决方案：

将其分解为更小、更集中的函数。这些函数应该具有明确的职责，并使用有意义的名称。

class OrderProcessor {
    public function processOrder($order) {
        $this->validateOrder($order);
        $this->calculateTotal($order);
        $this->applyDiscount($order);
        // ... other specific tasks ...
    }

    private function validateOrder($order) {
        // validation logic
    }

    private function calculateTotal($order) {
        // calculation logic
    }

    private function applyDiscount($order) {
        // discount logic
    }
}

大类：

代码异味:大类是指类中包含太多职责和方法的类。大类会导致以下问题：

难以理解和维护
难以测试
容易出错

// Example of a large class
class Order {
    public function process() {
        // ... 处理订单的逻辑 ...
    }

    public function calculateTotal() {
        // ... 计算订单总额的逻辑 ...
    }

    // ... 其他方法 ...
}

解决方案：

将其划分为更小、更集中的类，每个类负责特定方面的功能。

class Order {
    private $processor;
    private $calculator;

    public function __construct(OrderProcessor $processor, OrderCalculator $calculator) {
        $this->processor = $processor;
        $this->calculator = $calculator;
    }

    public function process() {
        $this->processor->processOrder($this);
    }

    public function calculateTotal() {
        $this->calculator->calculateTotal($this);
    }
}

class OrderProcessor {
    public function processOrder(Order $order) {
        // ... 处理订单的逻辑 ...
    }
}

class OrderCalculator {
    public function calculateTotal(Order $order) {
        // ... 计算订单总额的逻辑 ...
    }
}

重复代码：

代码异味:重复代码是指代码库中存在多个相同或相似的代码块。重复代码会导致以下问题：

维护困难：修改一个代码块需要修改所有相同或相似的代码块。
错误风险增加：如果一个代码块存在错误，则所有相同或相似的代码块都可能存在错误。

class UserController {
    public function createUser($data) {
        // ... 验证逻辑 ...
        $user = new User();
        $user->name = $data['name'];
        $user->email = $data['email'];
        $user->save();
    }

    public function updateProfile($data) {
        // ... 验证逻辑 ...
        $user = Auth::user();
        $user->name = $data['name'];
        $user->email = $data['email'];
        $user->save();
    }
}

解决方案：

将通用功能提取到单独的方法或类中。

class UserController {
    public function createUser($data) {
        $this->validateAndSaveUser($data);
    }

    public function updateProfile($data) {
        $this->validateAndSaveUser($data, Auth::user());
    }

    private function validateAndSaveUser($data, $user = null) {
        // ... validation logic ...
        if (!$user) {
            $user = new User();
        }
        $user->name = $data['name'];
        $user->email = $data['email'];
        $user->save();
    }
}

过多的注释：

代码异味：过多的注释会导致以下问题：

代码变得冗余和难以阅读。
注释可能与代码不一致或过时。
注释可能被忽略或删除。

// 过度使用注释的示例
class  Calculator  { 
    // 将两个数字相加的函数
    public  function  add ( $a , $b ) { 
        return  $a + $b ; 
    } 

    // 两个数字相减的函数
    public  function  minus ( $a , $b ) { 
        return  $a - $b ; 
    } 
}

解决方案：

编写干净且不言自明的代码，无需过多注释。

class Calculator {
    public function add($a, $b) {
        return $a + $b;
    }

    public function subtract($a, $b) {
        return $a - $b;
    }
}

复杂条件语句：

代码味道：复杂条件语句是指条件过于复杂，包含多个嵌套条件。复杂条件语句会导致以下问题：

理解和维护代码变得具有挑战性。
增加错误的风险。

class PaymentProcessor {

    public function processPayment($order) {
        if ($order->isConfirmed()) {
            if ($order->hasItems()) {
                if ($order->isPaymentDue()) {
                    // 处理付款
                } else {
                    // 付款已完成
                }
            } else {
                // 订单中没有商品
            }
        } else {
            // 订单未确认
        }
    }
}

解决方案：

将其分解为更小、更易于管理的部分。

class PaymentProcessor {

    public function processPayment($order) {
        if (!$this->isValidOrder($order)) {
            // 处理无效订单
            return;
        }

        // 处理付款
    }

    private function isValidOrder($order) {
        return $order->isConfirmed() && $order->hasItems() && $order->isPaymentDue();
    }
}

无限循环：

代码异味：无限循环是指循环没有终止条件，导致代码永远执行下去。无限循环可能是一个严重的问题，导致应用程序挂起或崩溃。

class TaskScheduler {

    public function runTasks() {
        while ($this->hasPendingTasks()) {
            // 执行任务
            // ...
        }
    }

    private function hasPendingTasks() {
        // 检查是否有待处理的任务
        // 该方法可能没有正确更新状态
    }
}

解决方案：

确保循环具有适当的终止条件以防止无限循环。

class TaskScheduler {

    public function runTasks() {
        $maxIterations = 1000; // 设置最大迭代次数以防止潜在的无限循环

        for ($i = 0; $i < $maxIterations && $this->hasPendingTasks(); $i++) {
            // 执行任务
            // ...
        }
    }

    private function hasPendingTasks() {
        // 检查是否有待处理的任务
        // ...
    }
}