|
锁定老帖子 主题：一个应该忏悔的代码片段
精华帖 (0) :: 良好帖 (13) :: 新手帖 (0) :: 隐藏帖 (0)
|
|
|---|---|
| 作者 | 正文 |
|
发表时间：2009-04-25
hsbljyy 写道 这就应该忏悔啦？我碰到过从头到尾一大堆注释掉的代码，可是解释都不解释为什么，而且代码功能也都不写注释，最为关键的是，我要去修改、维护这些代码，而我又不是那种喜欢说三道四的人，给我任务，我会蒙头把代码一行行看完，弄清楚是为什么再改，这样速度就慢了，慢了就有人催，催着就急着做，急着做就容易出问题，结果老是被骂，有的时候真的想说一点三字经，骂远些写代码的人。PS:其实Java代码还好，比较规范，容易懂，你想想看，已要维护一个JSP页面4K多行的JS代码时，你想想看那痛苦程度了！而且都是一坨一坨的。
说道这个，我向各位兄弟说一下。在上海找工作千万不要去一家叫“协同”的公司，他们就是JSP里面7，8K行代码的。 我一个师弟做了两个月就“逃”出来了。这公司还有分公司呢，看起来蛮大的。 |
| 返回顶楼 | |
|
发表时间：2009-04-25
项目开发的时候 先做测试是必要的
用Junit进行单体测试 或者用DBunit测试都可以 我是习惯现在测试表间关系 以及配置关系 然后再具体的开发功能 可能前期很话时间 真的可以避免很多问题 最近的一个项目 一同事觉得测试费时间 就直接上手现实功能部分 要数据就自己在数据表里面添加 然后就出错误了 而且还很难找到原因 最后发现是oracle下建的序列没有开始滚动 而我之先 对自己负责的表关系都进行了增加删除 级联的操作 就没有出现这样的问题 同样 后面要数据 也是在数据表里面加的 测试费事 但却是保证后续开发的高效完成 |
| 返回顶楼 | |
|
发表时间：2009-04-25
这个- 我以前也是都加comments的，
后来见别人从来都不加的， 我也就没加了 - -。 |
| 返回顶楼 | |
|
发表时间：2009-04-25
傻了吧
我们这样的代码是会打回去的 要么就打开，要么就删除 如果注释掉 必须注释上理由 |
| 返回顶楼 | |
|
发表时间：2009-04-25
yangtao309 写道 项目开发的时候 先做测试是必要的
用Junit进行单体测试 或者用DBunit测试都可以 我是习惯现在测试表间关系 以及配置关系 然后再具体的开发功能 可能前期很话时间 真的可以避免很多问题 最近的一个项目 一同事觉得测试费时间 就直接上手现实功能部分 要数据就自己在数据表里面添加 然后就出错误了 而且还很难找到原因 最后发现是oracle下建的序列没有开始滚动 而我之先 对自己负责的表关系都进行了增加删除 级联的操作 就没有出现这样的问题 同样 后面要数据 也是在数据表里面加的 测试费事 但却是保证后续开发的高效完成 你是用DBUnit做的？ 我最近也碰到一个问题，DBA把五年以来所有sql，sp都给我，让我在本机oracle测试是否跑的通，结果测试过程那是非常痛苦。不知道如果用DBunit会怎么样。因为很多公司都不愿意给个人这么多时间准备测试，唉。 |
| 返回顶楼 | |
|
发表时间：2009-04-25
按楼主的说法，时间很久了，如果已经进入维护阶段，任何代码的更改必然是需求变更或者bugfix引起的。去翻一翻bug跟踪系统的历史。
如果是开发阶段留下的，那就不要管它了，当时的需求就是这个逻辑。有需求文档吧？别说找不到了。 |
| 返回顶楼 | |
|
发表时间：2009-04-25
基于这种情况 我和我项目组的人做法是:一定要写清楚注释!而且注释一定要包含有用的信息 哪怕是"xx年xx月xx日与某某确认 修改此处为xxxxxx" 注释一定要能帮助你或他人以后能知道这些代码修改的来龙去脉 只写了功能变更的注释在有些情况下是没意义的
|
| 返回顶楼 | |
|
发表时间：2009-04-26
为什么不加comments？
我现在平时写点小代码玩，都是自己装subversion，提交时写注释，习惯了。 注释这东西，万一有需要又没有写的时候，很吐血的 |
| 返回顶楼 | |
|
发表时间：2009-04-26
lin5061 写道 基于这种情况 我和我项目组的人做法是:一定要写清楚注释!而且注释一定要包含有用的信息 哪怕是"xx年xx月xx日与某某确认 修改此处为xxxxxx" 注释一定要能帮助你或他人以后能知道这些代码修改的来龙去脉 只写了功能变更的注释在有些情况下是没意义的
这是好习惯，不过我认为只有在提交文件时候写清楚，但是在代码里最好不这样。因为代码里写注释这种事情很多人都坚持不下来的。 |
| 返回顶楼 | |
|
发表时间：2009-04-26
最后修改：2009-04-26
i am new hand，不过有一次我也注释过这个东西，我当时是这样的
我加 像if (!this.existInTodayFile(msisdnArray[i]))条件判断的时候是想这个条件成立以后再执行下面的程序，开始觉得这样就行，后来想到程序不能碰到这个下面就不能执行了吧，最好采用异常处理，就决定改成抛出异常了，如果这个条件不成立就转到异常处理，不过又觉得写异常也不是太好，就先留着，注释掉了，看看以后是不是真的需要异常处理 |
| 返回顶楼 | |
