5
idea 2018.3 换行/空行也计算在代码行数中,导致超过80行
idea 2018.3 换行/空行也计算在代码行数中,导致超过80行
规则说明有写着不包括空行
@imalvisc 按照文本理解的话,我认为他的意思是需要计入空行。
这个规定垃圾吗?这个有啥纠结的!! 1)统计过大量代码得出来的结论,90%以下的都是80行以内的。 2)人的记忆只有三屏,80行预计在两屏至两屏半之间。 3)强制大家在写长方法时,需要拆分主干逻辑和次要逻辑。 4)使大家有抽取公共方法的思维习惯。
@gujin520 对于 1)那么剩下10%的 如果是刚需 怎么办?
特别是一些领域相关的,比如涉及语法解析,或者复杂自动机,这种天然就要几百行,拆出来就有可能会导致性能降低,你打算如何
对于 3)4) 并不是所有场景都能通过抽离公共方法,或者拆分逻辑的方式,无损失地拆出子函数。
举个例子,你现在用的组件中,有一小半,都含有超过80行的代码,你打算标记这些组件全部不合法吗?
一个强制性的硬性规则必须做到兼容全部可能场景,否则就应该降级为非强制性的软性规则
@gujin520
比如 你前司的著名作品fastjson,就包含这种我描述的领域相关的自动机
这个函数是一个400多行的json自动机
@wenshao 你觉得你这个代码写的不符合规范,无法维护吗?应该在流水线中被判定为不合格,打回重做吗?