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