6

如果我phploc针对我的一个 PHP 项目（开源）运行，我会看到这个输出。

phploc 1.6.4 by Sebastian Bergmann.

Directories:                                          3
Files:                                               33

Lines of Code (LOC):                               2358
  Cyclomatic Complexity / Lines of Code:           0.08
Comment Lines of Code (CLOC):                       903
Non-Comment Lines of Code (NCLOC):                 1455

Namespaces:                                           0
Interfaces:                                           3
Classes:                                             28
  Abstract:                                           1 (3.57%)
  Concrete:                                          27 (96.43%)
  Average Class Length (NCLOC):                      49
Methods:                                            149
  Scope:
    Non-Static:                                     128 (85.91%)
    Static:                                          21 (14.09%)
  Visibility:
    Public:                                         103 (69.13%)
    Non-Public:                                      46 (30.87%)
  Average Method Length (NCLOC):                      9
  Cyclomatic Complexity / Number of Methods:       1.69

Anonymous Functions:                                  0
Functions:                                            3

Constants:                                            9
  Global constants:                                   0
  Class constants:                                    9

通过这个静态分析，如何判断这个代码库是好是坏？或者写得好不好？

  1. 如果圈复杂度太低或太高，它好吗？
  2. 有3.57%抽象类是好是坏？
  3. 14.09%静态方法。它应该在 OOP 代码库上更低吗？
  4. 没有使用命名空间，是好是坏？

最后一个问题你如何分析带有 phploc 输出的代码库？

4

1 回答 1

10

圈复杂度低好，高不好；静态很难进行单元测试，但是（虽然有些人认为它们与 eval 一样糟糕）它们确实有目的；phploc 的其他措施有待解释。

但是你真的应该将 phploc 与pdepend和phpmd等其他工具一起使用。pdepend 站点特别解释了许多使用的分析，并且 phpmd 输出非常不言自明

编辑

作为比较，我目前正在处理的代码（https://github.com/MarkBaker/PHPGeodetic）：我对抽象/具体类的级别相当满意，尽管它可能会更高一些；我有一种具有更高圈复杂度的方法，足以稍微扭曲这些数字，但这并不容易被分解；和一些更长的方法（但不足以触发 phpmd 警告）。

Lines of Code (LOC):                               4003
  Cyclomatic Complexity / Lines of Code:           0.07
Comment Lines of Code (CLOC):                      1580
Non-Comment Lines of Code (NCLOC):                 2423

Namespaces:                                           0
Interfaces:                                           1
Traits:                                               0
Classes:                                             25
  Abstract:                                           4 (16.00%)
  Concrete:                                          21 (84.00%)
  Average Class Length (NCLOC):                     103
Methods:                                            160
  Scope:
    Non-Static:                                     129 (80.62%)
    Static:                                          31 (19.38%)
  Visibility:
    Public:                                         131 (81.88%)
    Non-Public:                                      29 (18.12%)
  Average Method Length (NCLOC):                     16
  Cyclomatic Complexity / Number of Methods:       2.12

Anonymous Functions:                                  2
Functions:                                            0

Constants:                                           66
  Global constants:                                   0
  Class constants:                                   66

总的来说，我很想为公共/私人、非静态/静态和混凝土/抽象使用 80:20 规则；但很大程度上取决于您实际编码的内容

可能更重要的是圈复杂度/方法数量……我不喜欢这个数字太高；但如果它高于 2.5 的平均值，我会更仔细地查看 phpmd 统计信息

于 2012-12-18T16:51:50.867 回答