woocommerce-code-review

WooCommerce 代码审查

根据 WooCommerce 编码标准和惯例审查代码更改。

需要标记的严重违规

后端 PHP 代码

请参阅 woocommerce-backend-dev 技能，了解详细标准。以这些标准为指导，标记以下违规及类似问题：

架构与结构：

• 独立函数——必须使用类方法（file-entities.md）
• 对由 DI 管理的类使用 new——src/ 中的类必须使用 $container->get()（dependency-injection.md）
• 位于 src/Internal/ 之外的类——除非明确为公共类，否则应使用默认位置（file-entities.md）

命名与惯例：

• camelCase 命名——方法、变量和钩子必须使用 snake_case（code-entities.md）
• Yoda 条件违规——必须遵循 WordPress 编码标准（coding-conventions.md）

文档：

• 缺少 @since 注释——公共/受保护方法和钩子必须添加（code-entities.md）
• 缺少文档块——所有钩子和方法都必须添加（code-entities.md）
• 文档块冗长——保持简洁，最好写成一行（code-entities.md）

数据完整性：

• 缺少验证——删除或修改前必须验证状态（data-integrity.md）

测试：

• 在测试中使用 $instance——变量名必须使用 $sut（unit-tests.md）
• 缺少 @testdox——测试方法的文档块中必须包含（unit-tests.md）
• 测试文件命名——必须遵循 includes/ 与 src/ 各自的命名惯例（unit-tests.md）

UI 文案

请参阅 woocommerce-copy-guidelines 技能。标记以下问题：

• UI 中使用标题式大小写——必须使用句子式大小写（sentence-case.md）
  • 错误："Save Changes"、"Order Details"、"Payment Options"
  • 正确："Save changes"、"Order details"、"Payment options"
  • 例外：专有名词（WooPayments）、缩写词（API）、品牌名称

审查方法

1. 检查上述严重违规
2. 标明指出问题时所依据的具体技能文件
3. 提供技能文档中的正确示例
4. 归类相关问题，让内容更清晰
5. 保持建设性——适当说明制定该标准的原因

输出格式

对于发现的每项违规：

[Issue Type]: [Specific problem]
Location: [File path and line number]
Standard: [Link to relevant skill file]
Fix: [Brief explanation or example]

备注

• 所有详细标准均位于 woocommerce-backend-dev、woocommerce-dev-cycle 和 woocommerce-copy-guidelines 技能中
• 请参阅这些技能，了解完整背景和示例
• 如有疑问，请参考上方链接的具体技能文档