看到这样的Javascript代码，你会做何感想

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

推荐关注

› Meteor

› JSLint - a JavaScript code quality tool

› jsFiddle

› D3.js

› WebStorm

推荐书目

› JavaScript 权威指南第 5 版

› Closure: The Definitive Guide

这是一个创建于 4411 天前的主题，其中的信息可能已经有所发展或是发生改变。

一直觉得自己的javascript水平属于半瓶子醋的样子，然后今天看到另外一个team的测试case代码，瞬间给跪了。整个team的很多case都在用这个模板，已经使用了快1年多了。

Team

Case

代码

19 条回复 • 1970-01-01 08:00:00 +08:00

hafeyang

2012-11-03 00:48:39 +08:00

如果说是否实现了功能，确实是实现了功能，但是要说代码的质量，太多的可以提高的地方。

CoX

2012-11-03 01:01:31 +08:00

@hafeyang 我只是觉得，这样的代码，整个team少说也有五六个人，都在用，就没人想到要优化一下；真的很无语。

Livid

MOD

2012-11-03 01:11:01 +08:00

@CoX 每次想要优化什么的时候，发现似乎总有更重要的事情排在前面。于是，能 working 的东西就不去碰了。

hpyhacking

2012-11-03 01:18:24 +08:00

淡定一些

chone

2012-11-03 01:24:16 +08:00

这样的"历史遗留"问题其实很常见，快速迭代的话似乎能多留出一些机会消灭它。

CoX

2012-11-03 01:51:24 +08:00

@Livid 说的有一定道理。不过老话说的好，“磨刀不误砍柴工”，用这个模板再添加几组case，那又要改很多地方。因为涉及到很多人的很多case，貌似我也不太好直接给人反映这个问题。

@hpyhacking 我也就是发个牢骚，现在淡定很多。

@chone 也可能 team leader没想着去消灭它吧，都这么长时间过去了，还在这么用着呢。

conis

2012-11-03 09:49:21 +08:00

很多人喜欢用find/replace/ctrl+v/ctrl+c

dreampuf

2012-11-04 17:52:31 +08:00

只要是paste的代码就有问题。

noahua

2012-11-04 20:02:46 +08:00

写成这样后面怎么维护？调试起来会极其痛苦

liuyao729

2012-11-04 21:25:40 +08:00

第一次要写好

CoX

2012-11-05 10:19:41 +08:00

@conis
@dreampuf
@noahua
@liuyao729

这些人以前主要是写c 或者c++的，所以写javascript写成这样也算是情有可原吧。可能leader只关注功能实现了。

我重新写了下，稍做优化，其实也没花太多时间的。
https://gist.github.com/4014930