代码质量

一、代码静态检查

使用 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),其中

  • ifs:number of if, else if,else statements in the current function

  • loop:number of for, while, and do-while statements in the current function

  • case:the number of switch branches in the function (without default)

③ 阈值

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)上帝模块

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 194,457评论 5 459
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 81,837评论 2 371
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 141,696评论 0 319
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 52,183评论 1 263
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 61,057评论 4 355
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 46,105评论 1 272
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 36,520评论 3 381
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 35,211评论 0 253
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 39,482评论 1 290
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 34,574评论 2 309
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 36,353评论 1 326
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 32,213评论 3 312
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 37,576评论 3 298
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 28,897评论 0 17
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 30,174评论 1 250
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 41,489评论 2 341
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 40,683评论 2 335

推荐阅读更多精彩内容