原文出自【听云技术博客】:http://blog.tingyun.com/web/article/detail/1094
最近团队开始抓代码质量了,总结一下自己的经验,先看看坏代码有哪些特点:
“都一样,不幸的家庭却各有不同”,这句话放到代码里也同样适用。接下来,我们聊一聊如何解决坏代码问题。
如果我问你,“你们是如何保证团队代码质量的”,你的回答可能是:“我们每次写完代码,都会花一些时间review一下。”
恩,做的确实不错,但是,做的还不够,除非你是门门考试都100的学霸,否则,借助一些工具还是比较稳妥的办法。
在这里简单介绍一个代码分析工具RubyCritic,这是一个专门针对Ruby的一个静态代码分析工具。其它语言的,也有相似功能的工具链,我就不做介绍了。
这是一个命令行工具,第一步就是添加到你的gem库中,当然,还可以使用guard自动化分析。(Ruby的世界,你懂得~) 第二step,在console运行【RubyCritic】命令,就像这样:
在命令的最后,会生成一个静态页面。长这个样子:
x轴代表改动频率,Y轴代表代码复杂程度
这是分析结果的overview,超过200的复杂度的,基本都是坏代码。
再看看code里的内容:
对不同文件按照改动频率、复杂度、重复度和坏味道4个维度进行综合评定代码质量等级(和美国考试的成绩打分规则一样)。
再看看Smell里的内容 :
RubyCritic对代码分析的原理,其实就是分析一些,被它认为是坏代码的点。注意,我这里使用的措辞是“被它认为,所以,有时候,它不是绝对的正确。” 还可以查看具体的类文件中的代码质量问题。
更多的介绍,详见 https://github.com/whitesmith/rubycritic
下面,我们针对RubyCritic给我们的一些坏代码的点,有针对性的做些代码调整。
这里使用git diff 比较新旧版本的差异。operator原来是实例方法,代码行7,并且里面还有一个if结构体。started_time_and_node 原来是实例方法,代码行4,并且里面还不止一个if结构体。
笔者review的方式:
1.实例方法修改为类方法(减少混入方法,解耦合,减低负责度)
2.多使用Ruby原生链式操作(减少中间变量,更少的代码,对于脚本语言,就是更快的执行效率,而且很多原生方法是C语言实现。)
3.去掉结构体 (现代编程语言的结构体,让代码具有丰富的逻辑性和可读性,但是缺点就是cpu的额外开销。)
以上部分,属于语法层面的奇技淫巧。
第二部分,我们从设计角度分析一下。
它的代码行只有141行,方法也只有7个。但是评级却是C。再看看代码分析细节,这里就展现一小部分,简直就是惨不忍睹,不好意思全展现给大家看了。
没有人会一开始就这样写代码,这种坏代码,永远都是渐渐变馊的。不过笔记仔细想来,当年遇到过比着还馊1000倍的代码(1000倍都不算过分)。
这是笔者做的第一版重构结果。
这里使用了策略模式。Stats_hash不再是充当一个集合的作用,现在变成了一个环境类,将原来依赖if结构数据分装到不同的行为类中。
第二版的改动计划是,引入work-job的模式,并发执行4个job。
第三版改动计划就是利用回调方式,去掉与该类不相干的代码,将逻辑分装到行为类里。
好了,写到这里,基本的代码层面的优化思路就这些了,其它就是开支散叶的过程,这里就不冗余了。下一节,咱俩聊一聊性能优化的一些思路。
相关推荐
标题中的“ChatGPT+低代码,将干掉40%的程序员”直指当前IT行业的一个热门话题:人工智能技术,特别是OpenAI的ChatGPT,与低代码平台结合,可能对未来程序员的需求产生深远影响。这个观点暗示了编程工作的一部分可能...
标题与描述中的“用9行代码干掉Windows XP,2000”这一知识点,实际上是在探讨一种在特定条件下使Windows XP或Windows 2000操作系统陷入死锁的编程技巧。这种技巧并非用于破坏系统,而是程序员在探索系统边界、进行...
服务可以通过`startService()`启动,此时服务会执行`onStartCommand()`方法中的代码,直到`stopService()`或`stopSelf()`被调用。服务还可以通过`bindService()`绑定,这时它会在客户端和服务端之间创建一个连接,...
“能轻松干掉你电脑里的假文件”可能是指它能够识别并结束那些占用资源却无实际作用或者恶意的进程。 【知识点详解】: 1. **进程管理**:操作系统中,进程是程序在内存中的运行实例,每个进程都有自己的独立内存...
在VB(Visual Basic)编程环境中,编写代码来“干掉”或结束运行中的360保险箱进程,涉及到了进程管理和Windows API调用的相关知识。360保险箱是一款安全软件,其目的是保护用户计算机免受恶意软件的侵害。然而,在...
"一键干掉win10更新"这个话题,就是针对如何禁用或阻止Windows 10系统的自动更新进行探讨。 **Windows 10自动更新的重要性** 1. **安全补丁**:自动更新能够及时安装最新的安全补丁,防御病毒和恶意软件的攻击。 2....
浅谈Spring Boot中如何干掉if else的方法 在软件开发中,如果else语句泛滥会对代码的可读性和维护性产生严重影响。特别是在订单系统中,对订单的不同类型作出不同的处理,传统实现方法将导致一堆的if else语句,使...
策略模式在 Java 中的应用 - 剔除 if-else 代码,提高代码可读性 在软件开发中,if-else 语句是非常常见的,但是大量的 if-else 语句会使代码变得难以阅读和维护。今天,我们将讨论如何使用策略模式来取代 if-else ...
描述中的“使用驱动程序干掉 360”意味着这个教程可能会涉及利用驱动程序自身的机制来移除360的对应驱动。值得注意的是,任何对系统核心组件的修改都应谨慎进行,以免引起系统不稳定或安全风险。作者虽然给出了免责...
在 Spring Boot 中,如何干掉 if else
### 命令行干掉进程 在计算机操作系统中,有时我们需要通过命令行来结束某个进程,这在解决系统故障、资源占用问题等方面非常有用。本文将详细介绍如何使用不同的命令来实现这一目标。 #### 一、`tasklist` `...
该项目采用注解策略模式重构Java代码,旨在消除if-else和case-switch结构,实现代码的优雅和易于维护。项目包含30个文件,其中包括15个Java源文件、7个XML文件、3个Markdown文件、2个Git忽略文件、1个Maven构建文件...
标题《当《头号玩家》照见现实:干掉你的车钥匙》所涉及的知识点主要围绕着物联网(IoT)技术、信息安全、数字身份认证、数字车钥匙技术及自动驾驶车辆。以下是对这些知识点的详细解读: 1. 物联网(IoT):在描述中...
标题中的“干掉红蜘蛛的批处理 双击即可”暗示了这是一个使用批处理(Batch Script)技术来解决与“红蜘蛛多媒体教室”软件相关问题的脚本。批处理是Windows操作系统中的一种简单编程方式,它允许用户通过一系列...
标题中的“给页面加速,干掉Dom Level 0 Event”是指优化网页性能的一个关键策略,主要涉及JavaScript事件处理。在早期的DOM(文档对象模型)规范中,即DOM Level 0,事件处理主要通过两种方式:`element.onclick = ...
标题“vb写的干掉输入指示器”涉及到的是一种编程技术,使用Visual Basic(VB)语言编写程序来消除或隐藏电脑屏幕上的输入指示器或者特定窗口。在Windows操作系统中,输入指示器通常指的是那些提示用户进行操作的...
了解哪些行为或特点会导致个人更容易被裁掉,能够帮助职场人士自我反省和提升,避免成为裁员浪潮中的牺牲品。 首先,职场中的“高薪低能”现象指的是那些薪水高于平均水平,但实际贡献平平的员工。在经济不景气或...
这个可以干掉360 这个可以干掉360 这个可以干掉360 这个可以干掉360 这个可以干掉360
【标题】"干掉kv2008vb源码"所涉及的知识点主要集中在VB(Visual Basic)编程语言以及安全软件领域,特别是针对KV2008这款杀毒软件的反制技术。VB是一种由微软开发的面向对象的编程语言,它在Windows平台上广泛应用...
因此,网络上出现了一款名为“完美干掉易游 易乐游广告”的工具,这可能是一个软件程序,旨在帮助用户有效去除或拦截这些平台上令人烦恼的广告。 提及“易游”和“易乐游”,这可能是指为用户提供游戏和娱乐内容的...