Code Review之代码规范篇

news/2025/6/16 23:30:20
由于先前公司做了一个外国的项目,那个项目好像基于掌上电脑的一个小网站(具体的不敢问头太多),结果我们的HTML代码写的不好,
被国外的人说我们的代码结构不好,CSS里属性值老多重复等.在我看来可能当时这样一种网站由于时间进度\项目规模等各种原因,没有足够的人手去把那些东西写好.

结果后来这个单子就丢了.
头就急了,后来就让我负责全部开发团队的代码规范的Code Review事宜.他在向其他人宣布我的这一职责也明确说了.Code Reveiw中我主要检查代码规范,至于具体其他性能上,设计上的就不是我的范畴了

但问题是:代码规范都是静态的东西.怎么执行?执行到什么程度好呢?你比如说:几乎所有的语言的代码缩进都要求4个空格,如果让团队每个人都做到这一点?把它们的各种开发工具编辑器的都设置一遍吗?让他们自己设置?如果他们重新安装的电脑了怎么办?
看似代码缩进4个空格的这个简单问题,执行起来确有各种可能,这里绝对不能有教条思维,

还比如javascript,变量没有申明前,就不要直接使用.但是你如果真的用了,也没有关系,程序也能跑.这一点怎么执行呢?我还要辛辛苦苦用jslint.com的工具来检查.
程序员出现这种问题,只是道德问题,而不是法律问题;而头非要反过来说是警察和小偷的问题,是法律问题.

争论归争论,做还得做;

我初步的步骤如下;
 1)调度svn命令,每天增量下载所有要Reivew项目代码的Source Code.
2)Javascript:  jslint.com
HTML: http://validator.w3.org/
CSS: http://jigsaw.w3.org/css-validator/
JSP: PMD
Java: PMD/Checkstyle
C#: unknown
aspx: unknown

用这些工具检查下载下来的各种类型文件,然后形成一个Jira Issues,提交到SVN版本库里的相应的作者.

这个过程是蛮麻烦和蛮琐碎的.

  1)整合这些工具,形成jira issue比较麻烦.
  2) jira issues还要跟踪,重新下载被Review的svn代码,检查,然后要向发生问题的解释,然后让他执行,这个比较费时,工作量无法估量

最麻烦的是头还要我形成一个报告,列出哪些是团队经常犯的代码规范问题 ?哪些人经常犯?我说这些问题你直接去看jira,他说他不看那些detail的报告,他要那些总结性的.
我的哥,这些报告如果没有每一个平台去自动形成,那么就意味着我要自己手工去归纳,我归纳出来的这样的报告有意义吗?这样的报告只是静态的.这样的工作持续会持续多长时间?
  我倒是想写一个软件平台,把上面的代码检查工具整合一下,自动形成jira issue,然后自动得出报告,但是这要费大量时间,而我又有其他工作.

各位,你们是如果做代码规范的Code Review的?
文章来源:https://blog.csdn.net/taikeqi/article/details/83711894
本文来自互联网用户投稿,该文观点仅代表作者本人,不代表本站立场。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如若转载,请注明出处:https://dhexx.cn/news/show-4055916.html

相关文章

OpenCMS7.5 + Apache2.2+Tomcat6集成 去掉2个opencms路径

1.修改去掉第一个/opencms 1.1 将opencms安装成tomcat根路径&#xff0c;一般修改servlet.xml中加入&#xff1a; <Context path"" docBase"D:/tomcat-whirlpool/opencms" /> 就行了。同时servlet.xml应该有具体某一个site的port配置: <Connec…

WORD,EXCEL文档标题栏(任务栏)显示路径

1.在“工具”菜单中&#xff0c;指向“宏”&#xff0c;并单击“宏”&#xff0c;打开其对话框。   2.在“宏名”框中&#xff0c;键入“AutoOpen”&#xff0c;单击“创建”按钮 &#xff08;如图2&#xff09;&#xff0c;打开“Microsoft Visual Basic”编辑窗口。如果在…

MAVEN编译时报错Could not reserve enough space for object heap

公司某个项目在初次使用&#xff0c;尝试运行代码 maven在package时报错 [ERROR] Error occurred during initialization of VM [INFO] Could not reserve enough space for object heap [ERROR] Error: Could not create the Java Virtual Machine. [ERROR] Error: A fatal …

Notes 显示较大字体

C:/Program Files/IBM/Lotus/Notes/notes.ini 加上&#xff1a; Display_font_adjustment1

关闭公司电脑的强制屏幕保护

公司电脑安静10分钟之后自动会黑屏 开始以为是进入睡眠状态了&#xff0c;将电源选项全部改成从不睡眠但是一点效果都没有 最近终于发现是在哪里的限制在运行>rsop.msc其中%systemroot%\system32\scrnsave.scr就是当前使用的屏幕保护程序应该是公司通过域管理强制设置了屏幕…

听力训练的经典方法

在很长一段时间内,我所选的听力资料将会是VOA的Special级别的慢速英语.这主要是出于两个目的:一是打好基本功;二是从慢速英语入手是科学的方法,是实现标准英语听力突破的有效途径. 先说第一点. 关于听力基本功的认识问题我建议大家去买一本钟道隆编的《逆向法巧学英语》一书仔细…

ansible初步了解

playbook&#xff1a;字面意思 剧本 就是里面写了什么就照着剧本从上往下演呗~~嗯跟脚本的含义差不多简单了解yaml语法playbook各个模块&#xff1a; template file shell command学习参考 Ansible中文权威指南 http://www.ansible.com.cn/index.html http://blog.csdn.net/m…

SQL Server 2008 Express Manage Studio 的安装方法 Windows 2003 Server

安装步骤 删除文件C:\Program Files\Microsoft SQL Server\100\Setup Bootstrap\Log\20111021_174732\ConfigurationFile.ini1) install dotnetfx35_sp1.exe2) install Power Shell 1.1WindowsServer2003-KB926140-v5-x86-CHS.exe3) Install Windows Installer 4.5WindowsServ…

HBASE表数据迁移到另一台服务器上

1.首先命令行删除原来hbase表 ./hbase shell disable TABLE1 drop TABLE1删除之后hdfs上的表数据文件会被移除2.hdfs数据迁移 进入hadoop home ./hdfs dfs -cp hdfs://fromip/hbase/data/default/TABLE1 hdfs://dstip/hbase/data/default/TABLE1 数据量大拷贝可能很慢3.hbase修…

mysql 存储引擎和中文存储的关系

1)当用INNO DB Client: latin1 Server:UTF8 控制台下&#xff0c;能正确存储中文。 mysqldump 命令导出中文正确。 -------------------- 当用MyISAM Client:UTF8 Server:UTF8 如此方能正确存储中文。 控制台下&#xff0c;不能正确显示中文。 mysqldump 命令导出中…