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