北京华夏安泰技术论坛

 找回密码
 立即注册

扫一扫,访问微社区

QQ登录

只需要一步,快速开始

只需一步,快速开始

搜索
热搜: 活动 交友 discuz
12
返回列表 发新帖
楼主: 王健

展示你的样板代码-提高代码质量活动第一期

[复制链接]

9

主题

187

帖子

763

积分

内部工作人员

积分
763
QQ
发表于 2017-9-27 18:51:42 | 显示全部楼层
本帖最后由 ghostman 于 2017-9-27 18:55 编辑
于凯奇 发表于 2017-9-24 21:12
业务申请查看公共判断页签内容显示js
编写时间:最近
项目路径:/atnk/WebRoot/jsp/expenses/businessappli ...

2个片段代码,除了“空格”格式规范不符合要求,其它还是蛮好滴。nice
有以下两个建议:



本帖子中包含更多资源

您需要 登录 才可以下载或查看,没有帐号?立即注册  

x
回复

使用道具 举报

2

主题

188

帖子

784

积分

内部工作人员

积分
784
发表于 2017-9-28 09:15:45 | 显示全部楼层
点评朱胜志代码:第一段不知道有没有做异常处理;方法参数可以简单说明一下
回复

使用道具 举报

0

主题

4

帖子

158

积分

注册会员

Rank: 2

积分
158
发表于 2017-9-29 16:00:18 | 显示全部楼层
leiql 发表于 2017-9-24 09:30
js 函数  供应商抽取
   1.点击抽取按钮 触发的函数
  

个人觉得,代码注释太少,类似某个变量是干什么呢,不能很好的找。如果要修改代码还要理一下整个的逻辑。
风格的话,可以参照王建的代码,也可以去网上找一些你认为符合你个人风格的代码。还有可能有意义的变量最好命名时,有意义点。
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册  

本版积分规则


小黑屋|手机版|Archiver|北京华夏安泰技术论坛  

GMT+8, 2019-2-17 01:09 , Processed in 0.171875 second(s), 23 queries .

Powered by Discuz! X3.2

© 2001-2013 Comsenz Inc.

快速回复 返回顶部 返回列表