重构php遗留代码的核心是识别“坏味道”并逐步优化，必须先建立完整的测试体系以确保功能一致性；2. 使用phpstan或psalm进行静态分析可发现类型错误、未使用变量等潜在问题，需通过composer安装、配置规则文件并运行分析命令；3. 编写有效的单元测试应遵循测试先行、高覆盖率、独立性、可读性和快速执行原则，使用phpunit等框架覆盖各类分支场景；4. 处理遗留代码中的数据库问题需淘汰过时的mysql_*函数，改用pdo或mysqli并采用参数化查询防止sql注入，推荐使用orm框架如doctrine或eloquent提升可维护性，同时利用连接池优化数据库连接管理；5. 针对常见坏味道应实施提取函数/类、引入参数对象、以多态取代条件表达式等重构手法，提升代码内聚性与扩展性，整个重构过程应持续迭代、小步推进，确保系统稳定性不受影响。
重构PHP代码，特别是那些遗留代码，核心在于识别代码中的“坏味道”，然后针对性地进行优化。这不仅关乎代码的可读性，更影响系统的可维护性和扩展性。
解决方案
重构是一个迭代的过程，并非一次性完成。首先，我们需要建立一套完整的测试体系，确保重构后的代码功能与之前保持一致。这包括单元测试、集成测试，甚至端到端测试。没有测试，重构就是一场豪赌。
接下来，识别代码中的“坏味道”。这可以通过静态代码分析工具，如PHPStan、Psalm等，也可以通过人工代码审查。常见的坏味道包括：
立即学习“PHP免费学习笔记（深入）”；
过长函数/类： 函数或类承担了过多的职责，导致难以理解和维护。重复代码： 在多个地方出现相同的代码片段，违反了DRY（Don’t Repeat Yourself）原则。过大的参数列表： 函数的参数过多，难以理解参数的含义和使用方式。发散式变化： 一个类因为多种原因而修改，违反了单一职责原则。霰弹式修改： 一次修改需要在多个类中进行，代码耦合度过高。依恋情结： 一个类过于依赖另一个类，导致代码耦合度过高。数据泥团： 多个类中出现相同的数据项，应该将这些数据项封装成一个类。基本类型偏执： 使用基本类型来表示复杂的概念，应该使用对象来表示。switch语句/if-else嵌套过深： 难以理解和维护，可以使用多态来替代。注释过多： 代码本身应该具有良好的可读性，过多的注释可能意味着代码需要重构。
针对这些坏味道，我们可以采用一些常见的重构手法：
提取函数/类： 将过长的函数或类拆分成更小的、更易于理解的单元。提取方法对象： 将复杂的算法封装成一个方法对象，提高代码的可读性和可维护性。函数组合成类： 将多个相关的函数封装成一个类，提高代码的内聚性。引入参数对象： 将多个相关的参数封装成一个参数对象，减少参数列表的长度。分解条件表达式： 将复杂的条件表达式拆分成更小的、更易于理解的单元。以多态取代条件表达式： 使用多态来替代switch语句/if-else嵌套，提高代码的可扩展性。移除设值函数： 如果一个字段不应该被修改，则应该移除设值函数。以工厂模式取代构造函数： 使用工厂模式来创建对象，可以隐藏对象的创建逻辑。
举个例子，假设我们有一个函数，用于计算订单的总金额，其中包含了大量的条件判断：
function calculateOrderTotal($order, $discountCode, $shippingAddress) { $total = $order['subtotal']; if ($discountCode == 'SUMMER20') { $total *= 0.8; // 20% off } elseif ($discountCode == 'WINTER10') { $total *= 0.9; // 10% off } if ($shippingAddress['country'] == 'US') { $total += 5; // Shipping fee for US } elseif ($shippingAddress['country'] == 'CA') { $total += 10; // Shipping fee for CA } // ... more conditions return $total;}
这个函数违反了单一职责原则，应该将其拆分成更小的函数，并使用策略模式或多态来处理不同的折扣和运费计算。
如何使用PHPStan或Psalm进行静态代码分析？
PHPStan和Psalm都是强大的PHP静态代码分析工具，可以帮助我们发现代码中的潜在问题。它们通过分析代码的语法和语义，可以检测出类型错误、未使用的变量、潜在的bug等。
使用方法通常如下：
安装： 通过Composer安装PHPStan或Psalm。
composer require --dev phpstan/phpstan# 或者composer require --dev vimeo/psalm
配置： 创建一个配置文件，例如
phpstan.neon
或
psalm.xml
，指定要分析的代码目录和规则。
# phpstan.neonparameters: level: 5 paths: - src
运行： 在命令行中运行PHPStan或Psalm，指定要分析的代码目录。
./vendor/bin/phpstan analyse src# 或者./vendor/bin/psalm
分析结果： 分析结果会显示在命令行中，包括错误类型、错误位置等。根据分析结果，我们可以逐步修复代码中的问题。
PHPStan和Psalm都支持自定义规则，可以根据项目的需要进行配置。例如，可以指定代码的编码风格、命名规范等。
如何编写有效的单元测试来支持PHP代码重构？
单元测试是重构的基础。没有单元测试，重构就像是在黑暗中摸索，无法保证重构后的代码功能与之前保持一致。
编写有效的单元测试需要遵循一些原则：
测试先行： 在编写代码之前，先编写单元测试。这可以帮助我们更好地理解需求，并设计出更易于测试的代码。覆盖率： 单元测试应该覆盖代码的所有分支和边界情况。可以使用代码覆盖率工具来衡量测试的覆盖率。独立性： 单元测试应该独立运行，不依赖于外部环境。可以使用Mock对象来模拟外部依赖。可读性： 单元测试应该具有良好的可读性，易于理解和维护。可以使用清晰的命名和注释来提高可读性。快速： 单元测试应该快速运行，以便可以频繁运行。
例如，对于上面的
calculateOrderTotal
函数，我们可以编写如下的单元测试：
use PHPUnitFrameworkTestCase;class OrderTest extends TestCase{ public function testCalculateOrderTotalWithNoDiscountAndNoShippingFee() { $order = ['subtotal' => 100]; $shippingAddress = ['country' => 'Other']; $discountCode = null; $total = calculateOrderTotal($order, $discountCode, $shippingAddress); $this->assertEquals(100, $total); } public function testCalculateOrderTotalWithSummerDiscount() { $order = ['subtotal' => 100]; $shippingAddress = ['country' => 'Other']; $discountCode = 'SUMMER20'; $total = calculateOrderTotal($order, $discountCode, $shippingAddress); $this->assertEquals(80, $total); } // ... more test cases}
这些单元测试可以覆盖
calculateOrderTotal
函数的不同分支，确保重构后的代码功能与之前保持一致。
如何处理PHP遗留代码中的数据库访问问题？
遗留代码中常见的数据库访问问题包括：
*直接使用`mysql_
函数：** 这些函数已经过时，不安全，应该使用
mysqli_*`或PDO来替代。SQL注入漏洞： 没有对用户输入进行充分的验证和过滤，导致SQL注入漏洞。硬编码SQL语句： SQL语句硬编码在代码中，难以维护和修改。数据库连接管理不当： 没有正确地打开和关闭数据库连接，导致资源泄漏。
针对这些问题，我们可以采取以下措施：
升级数据库访问方式： 将
mysql_*
函数替换为
mysqli_*
或PDO。PDO提供了更好的安全性和可扩展性。使用参数化查询： 使用参数化查询可以防止SQL注入漏洞。参数化查询可以将用户输入作为参数传递给SQL语句，而不是直接拼接到SQL语句中。使用ORM框架： ORM框架可以将数据库表映射到对象，简化数据库访问。常见的PHP ORM框架包括Doctrine和Eloquent。使用连接池： 使用连接池可以提高数据库连接的效率。连接池可以预先创建一些数据库连接，并在需要时从连接池中获取连接，而不是每次都重新创建连接。
例如，使用PDO进行数据库访问：
try { $dbh = new PDO('mysql:host=localhost;dbname=test', 'user', 'password'); $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); $stmt = $dbh->prepare("SELECT * FROM users WHERE id = ?"); $stmt->execute([$_GET['id']]); $user = $stmt->fetch(); // ... $dbh = null; // 关闭连接} catch (PDOException $e) { print "Error!: " . $e->getMessage() . "
"; die();}
这段代码使用了PDO来连接数据库，并使用了参数化查询来防止SQL注入漏洞。
总而言之，PHP代码重构是一个需要耐心和技巧的过程。通过识别代码中的“坏味道”，并针对性地进行优化，我们可以提高代码的可读性、可维护性和可扩展性。同时，建立一套完整的测试体系，可以确保重构后的代码功能与之前保持一致。
以上就是PHP代码重构实战：坏味道识别 如何优化和重构遗留PHP代码的方法的详细内容，更多请关注php中文网其它相关文章！
版权声明：本文内容由互联网用户自发贡献，该文观点仅代表作者本人。本站仅提供信息存储空间服务，不拥有所有权，不承担相关法律责任。
如发现本站有涉嫌抄袭侵权/违法违规的内容， 请发送邮件至 chuangxiangniao@163.com 举报，一经查实，本站将立刻删除。
发布者：程序猿，转转请注明出处：https://www.chuangxiangniao.com/p/1267172.html
微信扫一扫 
支付宝扫一扫 
