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

[CodingNET]

讲师介绍

杨周
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 文档中，支持团队成员直接访问、加密分享。

点击观看课程回放