CODING DevOps 代码质量实战系列第二课： PHP 版

讲师介绍

杨周

CODING DevOps 架构师

CODING 布道师

连续创业者、DIY/Linux 玩家、知乎小 V，曾在创新工场、百度担任后端开发。十余年一线研发和带队经验，经历了 ToB、ToC、O2O、国内、出海各种项目，见证了云计算时代的诞生，擅长研发最佳实践：Code Review、DevOps、Git Workflow、敏捷开发、架构、极客办公硬件。

PHP 老项目常见质量问题

问题一：未提交 composer.lock

Composer 官方文档（getcomposer.org）提醒：一定要提交 composer.lock 文件到代码库，才能确保所有人、所有机器安装的依赖包都是同一版本。

问题二：提交了 vendor

把第三方包的代码提交到了自己的代码库里，导致代码库过大，拉取速度慢，并且无法跟进第三方包的升级。所以，应该使用.gitignore 忽略vendor 目录。

解决方案：删除 vendor，执行 composer install，再把 composer.lock 提交到代码库。

结果报错：

1.使用了废弃的包；

2.使用了已废弃的语法；

3.框架命令执行失败。

查看 PHP 官方文档可以发现：PHP 7.4 不赞成使用花括号语法来访问数组元素和字符串偏移量。

修改之后即可解决，从这个技术细节可以看出此开源项目无法跟上 PHP 官方的版本升级，在我们做技术选型时要避免使用这种落后的项目、框架，而应使用全球知名框架，能够紧跟语言官方版本升级，目前 PHP 7.1 已经废弃，7.2 即将结束维护升级，建议大家使用 7.4 。

修改 composer 包

如果需要修改 vendor 中的第三方包，有两个办法：

1.Fork：修改代码→制作补丁，自己用 → 发起合并请求，回馈开源；

2.私有 composer 仓库：把修改后的包发布到 coding.net 制品库。

Lint：增量检查代码规范

清理了第三方代码之后，可以通过 Lint（代码静态分析工具）程序扫描自己的代码，PHP Lint 常用的是 CodeSniffer，内置知名的「PSR12」规范，包括很多规则，比如：

每行代码最大长度 120 个字符；
运算符左右各 1 个空格；
文件结尾必须有一个空行。

老项目有成千上万的报错，一次难以清理干净，建议使用增量检查，具体请访问 CODING 帮助文档，搜索「增量检查」。

PHP 自动化测试

自动化测试要点

可测性：不要在系统中使用系统时间、随机数、实例化对象。

Fake：用于伪造测试数据，而不要在测试代码中写死 123、12:00:00，因为可能 999、00:00:00 时会报错。

Mock：拦截所有的第三方服务，比如测微信登录、支付，不可能每次测试都真的付款。

PHPUnit 在自动化测试时，可同时生成测试覆盖率的 HTML 报告，可以看到哪一行代码没有覆盖。这种报告包含代码，所以需要私有访问，可使用 CODING 通用报告功能，在持续集成中自动上传即可。

PHP 代码自动生成 API 文档

在项目开发中，最好让代码先行，在代码里写注释，这样可以保证注释与代码一起维护，然后通过工具自动生成文档。

在持续集成中自动生成文档，可上传到 CODING API 文档中，支持团队成员直接访问、加密分享。

点击观看课程回放

 关于 CODING，了解更多