代码质量

2023-12-11  本文已影响0人  TomyZhang

一、代码静态检查

使用 SVACE 静态检查系统,每天都会有检查报告。

静态检查问题举例:

1.FALL_THROUGH

使用 switch-case 时,出现 one case falls through to the next case 情况。

2.FB.BC_UNCONFIRMED_CAST

使用类型转换时,出现 Unchecked/unconfirmed cast from ... to ... 情况。

3.FB.RV_RETURN_VALUE_IGNORED_BAD_PRACTICE

调用有返回值的方法时,忽略掉了返回值。

4.FB.ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD

在实例方法中修改静态变量的值。

5.NO_LOCK.STAT.EX

使用变量时没有加锁,然而该变量在其它地方使用时都有加锁。

二、代码安全审查

使用安全审查系统,一年提交一次审查。

安全审查问题举例:

1.open_ad_sdk.aar(文件安全)

问题描述:This code will change path "/data/user/0/xxx/cache/Download" permission to "777"

对应文件:class com.bytedance.sdk.openadsdk.downloadnew.a
class LibHolder

修改要求:Change permissions to "664" or lower.

2.open_ad_sdk.aar(网络安全)

问题描述:HostnameVerifier always returns true without verifying any host name

对应文件:class HTTPSTrustManager

修改要求:The configuration makes the HTTPS connection easy to suffer MITM() attack.

Suggestion: to check the host name in HostnameVerifier, verify the server certification or use SSLSocketFactory.STRICT_HOSTNAME_VERIFIER mode in TLSSocketFactory.

3.open_ad_sdk.aar(密码安全)

问题描述:Some symmetric secret keys are written in code.
Although these secret keys are in base64 form, they are decoded easily.

对应文件:class com.bytedance.sdk.openadsdk.core.b
class com.bytedance.sdk.openadsdk.core.a

修改要求:1: the first two letters are not used for secret key.
2: some keys are repeated twice or more times, and then get substring of the concatenated keys.

三、软件架构成熟度

使用 CQA 系统,每天都会有检查报告。

SAM(S/W Architecture Maturity,软件架构成熟度)指标:

1.CC(Cyclomatic Complexity)圈复杂度

① 定义

圈复杂度也称为条件复杂度,是模块结构复杂度的度量,数量上表现为独立路径的条数,即合理的预防错误所需测试的最少路径条数。

② 计算方法

圈复杂度 =(1 + ifs + loop + case),其中

③ 阈值

Cyclomatic Complexity Risk Evaluation
1-10 一个没有太大风险的简单模块
11-20 具有中等风险的更复杂模块
21-50 高风险的复杂模块
51 and greater 风险极高的不稳定项目

④ 降低方法

分两个方向降低圈复杂度,一是拆分函数,二是尽量减少 if、else、while、case 等这些流程控制语句。

2.DC(Duplicate Code)重复代码

3.MCD (Module Circular Dependency) 模块循环依赖

① 定义

模块循环依赖是指两个或多个模块之间相互依赖,形成了一个循环的依赖关系。

② 解决方法

4.CBO(Coupling Between Objects)对象间耦合

① 定义

类耦合也称为对象间耦合,类耦合是衡量单个类使用多少类的指标。

② 解决方法

遵循单一职责原则,分离职责,减少类耦合。

5.LOC(Lines Of Code of class)代码行数

① 定义

代码行数是一种用于衡量软件规模的指标,他表示源代码中的代码行数。

② 减少方法

精简代码,进行模块化设计,减少不必要的代码行数。

6.DEP(Dependency Complexity)模块依赖复杂度

7.GM(God Module)上帝模块

上一篇 下一篇

猜你喜欢

热点阅读