|
锁定老帖子 主题：一个应该忏悔的代码片段
精华帖 (0) :: 良好帖 (13) :: 新手帖 (0) :: 隐藏帖 (0)
|
|
|---|---|
| 作者 | 正文 |
|
发表时间：2009-04-26
没用的地方直接删了就好 设置不加注释不允许commit
|
| 返回顶楼 | |
|
发表时间：2009-04-27
fantasybei 写道 
原来在js里写了很多注释，现在要求全部删除，说是为了减小文件大小。 分成开发版，测试版和发布版。 发布版没有注释。 开发版尽量注释。 测试版适当注释。 |
| 返回顶楼 | |
|
发表时间：2009-04-27
heavener 写道 大家在真正部署项目的时候都会把注释去掉吗？js都会压缩吗？
完全不会，基本都不用js。。。ajax更是完全不用。。。客户喜欢看实际的，不喜欢花里胡哨的东西 |
| 返回顶楼 | |
|
发表时间：2009-04-28
这样的代码要是在我们的项目中出现，既没编码格式，有没良好的注释，肯定是要被骂死的。 这种情况是极为不允许的。
|
| 返回顶楼 | |
|
发表时间：2009-04-28
我猜应该是无论文件是否存在，均加入缓存写文件吧？存在文件就覆盖原文件，不存在就直接创建。
|
| 返回顶楼 | |
|
发表时间：2009-04-29
iaimstar 写道 heavener 写道 大家在真正部署项目的时候都会把注释去掉吗？js都会压缩吗？
完全不会，基本都不用js。。。ajax更是完全不用。。。客户喜欢看实际的，不喜欢花里胡哨的东西 ajax->花里胡哨？。。。不一定吧，没有必然的联系。。。 |
| 返回顶楼 | |
|
发表时间：2009-05-01
这段程序有两个问题：
1. 在循环中去读取文件并检查字段是非常低效率的，极容易产生性能问题（就如同循环去检查某些值是否存在于数据库中一样）。 2. 从程序中可以看出思路作者是想把msisdn写入某个文件中，却又需要检查文件中是否存在该字符串。这时会存在有读操作和写操作的冲突（首先取决于existInTodayFile函数方法的实现）。即使没有冲突，这样的检查也丝毫没有意义，假设在msisdnArray数组中存在重复项，因为并没有flush并关闭流，在另外的读取流中根本就检查不到，所以这种检查就失去了意义。 看看前面楼层的一些朋友扯到了如何进行配置管理，呵呵，有些偏题了。即使再严格的代码管理机制（比如说check in之前必须有注释之类），都无法解决人的问题（任何工具都无法验证注释是否正确） |
| 返回顶楼 | |
|
发表时间：2009-05-04
jansel 写道 hsbljyy 写道 这就应该忏悔啦？我碰到过从头到尾一大堆注释掉的代码，可是解释都不解释为什么，而且代码功能也都不写注释，最为关键的是，我要去修改、维护这些代码，而我又不是那种喜欢说三道四的人，给我任务，我会蒙头把代码一行行看完，弄清楚是为什么再改，这样速度就慢了，慢了就有人催，催着就急着做，急着做就容易出问题，结果老是被骂，有的时候真的想说一点三字经，骂远些写代码的人。PS:其实Java代码还好，比较规范，容易懂，你想想看，已要维护一个JSP页面4K多行的JS代码时，你想想看那痛苦程度了！而且都是一坨一坨的。
兄台没有碰到因为JSP过长，JVM不能编译的情况吧。我就碰到过，大家可以猜猜里面是啥东东了。 jsp过长，jvm不能编译。这个好像是以前的版本不行，新的版本应该解决了这个问题。 |
| 返回顶楼 | |
|
发表时间：2009-05-05
crazy.j 写道 iaimstar 写道 heavener 写道 大家在真正部署项目的时候都会把注释去掉吗？js都会压缩吗？
完全不会，基本都不用js。。。ajax更是完全不用。。。客户喜欢看实际的，不喜欢花里胡哨的东西 ajax->花里胡哨？。。。不一定吧，没有必然的联系。。。 事实上，有 你是用ajax可能是技术上的考虑，但是从别的角度考虑ajax就是花里胡哨的东西，不管从使用上，还是界面，抑或是技术 |
| 返回顶楼 | |
|
发表时间：2009-05-05
奥，这种事情就不用再追究了，主要是在管理上不规范所致
以后在注释或者是改动同事代码的时候 最好要跟别人交流一下 怕望文生义 |
| 返回顶楼 | |
