
1 hafeyang 2012 年 11 月 3 日 如果说是否实现了功能,确实是实现了功能,但是要说代码的质量,太多的可以提高的地方。 |
4 hpyhacking 2012 年 11 月 3 日 淡定一些 |
5 chone 2012 年 11 月 3 日 这样的"历史遗留"问题其实很常见,快速迭代的话似乎能多留出一些机会消灭它。 |
6 CoX OP @Livid 说的有一定道理。不过老话说的好,“磨刀不误砍柴工”,用这个模板再添加几组case,那又要改很多地方。因为涉及到很多人的很多case,貌似我也不太好直接给人反映这个问题。 @hpyhacking 我也就是发个牢骚,现在淡定很多。 @chone 也可能 team leader没想着去消灭它吧,都这么长时间过去了,还在这么用着呢。 |
7 conis 2012 年 11 月 3 日 很多人喜欢用find/replace/ctrl+v/ctrl+c |
8 dreampuf 2012 年 11 月 4 日 只要是paste的代码就有问题。 |
9 noahua 2012 年 11 月 4 日 写成这样后面怎么维护?调试起来会极其痛苦 |
10 liuyao729 2012 年 11 月 4 日 第一次要写好 |
11 CoX OP @conis @dreampuf @noahua @liuyao729 这些人以前主要是写c 或者c++的,所以写Javascript写成这样也算是情有可原吧。可能leader只关注功能实现了。 我重新写了下,稍做优化,其实也没花太多时间的。 https://gist.github.com/4014930 |
12 ledzep2 2012 年 11 月 5 日 copy & paste正是很多对编程无爱的程序员的工作方式。 |
13 paloalto 2012 年 11 月 5 日 虽不明但觉厉 |
15 leecade 2012 年 11 月 5 日 测试 case 不就该这样么 |
16 CoX OP |
17 m4ker 2012 年 11 月 6 日 是因为没有规范,没有审核,也没有review,这样是很正常,你没规定人家不可以这样写啊. |
18 zzcflying 2012 年 11 月 6 日 变量名好蛋疼。 |
19 bengle 2012 年 11 月 6 日 思路倒是蛮奇葩的。 |