推荐关注
› Meteor
› JSLint - a JavaScript code quality tool
› jsFiddle
› D3.js
› WebStorm
推荐书目
› JavaScript 权威指南第 5 版
› Closure: The Definitive Guide
CoX

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

  •  
  •   CoX · Nov 1, 2012 · 7854 views
    This topic created in 5117 days ago, the information mentioned may be changed or developed.
    一直觉得自己的javascript水平属于半瓶子醋的样子,然后今天看到另外一个team的测试case代码,瞬间给跪了。整个team的很多case都在用这个模板,已经使用了快1年多了。

    19 replies  •  1970-01-01 08:00:00 +08:00
    hafeyang
    hafeyang
    Nov 3, 2012❤️1
    1
    如果说是否实现了功能,确实是实现了功能,但是要说代码的质量,太多的可以提高的地方。
    CoX
    CoX
    OP
    Nov 3, 2012
    2
    @hafeyang 我只是觉得,这样的代码,整个team少说也有五六个人,都在用,就没人想到要优化一下;真的很无语。
    Livid
    Livid
    MOD
    PRO
    Nov 3, 2012❤️2
    3
    @CoX 每次想要优化什么的时候,发现似乎总有更重要的事情排在前面。于是,能 working 的东西就不去碰了。
    hpyhacking
    hpyhacking
    Nov 3, 2012❤️1
    4
    淡定一些
    chone
    chone
    Nov 3, 2012❤️1
    5
    这样的"历史遗留"问题其实很常见,快速迭代的话似乎能多留出一些机会消灭它。
    CoX
    CoX
    OP
    Nov 3, 2012
    6
    @Livid 说的有一定道理。不过老话说的好,“磨刀不误砍柴工”,用这个模板再添加几组case,那又要改很多地方。因为涉及到很多人的很多case,貌似我也不太好直接给人反映这个问题。

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

    @chone 也可能 team leader没想着去消灭它吧,都这么长时间过去了,还在这么用着呢。
    conis
    conis
    Nov 3, 2012❤️1
    7
    很多人喜欢用find/replace/ctrl+v/ctrl+c
    dreampuf
    dreampuf
    Nov 4, 2012❤️1
    8
    只要是paste的代码就有问题。
    noahua
    noahua
    Nov 4, 2012❤️1
    9
    写成这样后面怎么维护?调试起来会极其痛苦
    liuyao729
    liuyao729
    Nov 4, 2012❤️1
    10
    第一次要写好
    CoX
    CoX
    OP
    Nov 5, 2012
    11
    @conis
    @dreampuf
    @noahua
    @liuyao729

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

    我重新写了下,稍做优化,其实也没花太多时间的。
    https://gist.github.com/4014930
    ledzep2
    ledzep2
    Nov 5, 2012❤️2
    12
    copy & paste正是很多对编程无爱的程序员的工作方式。
    paloalto
    paloalto
    Nov 5, 2012❤️1
    13
    虽不明但觉厉
    haohaolee
    haohaolee
    Nov 5, 2012❤️1
    14
    @CoX C++程序员躺枪,就算是C/C++也不会写成这样啊。不管哪门语言,DRY都是重要准则
    leecade
    leecade
    Nov 5, 2012❤️1
    15
    测试 case 不就该这样么
    CoX
    CoX
    OP
    Nov 5, 2012
    16
    @paloalto 只是不淡定的时候胡言乱语一番而已

    @haohaolee 本意是想说他们可能对javascript不熟,所以才出现这种情况

    @leecade 你可能没看完代码,后面部分是吐槽点
    m4ker
    m4ker
    Nov 6, 2012
    17
    是因为没有规范,没有审核,也没有review,这样是很正常,你没规定人家不可以这样写啊.
    zzcflying
    zzcflying
    Nov 6, 2012
    18
    变量名好蛋疼。
    bengle
    bengle
    Nov 6, 2012
    19
    思路倒是蛮奇葩的。
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Privacy   ·   Solana   ·   3488 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 71ms · UTC 04:18 · PVG 12:18 · LAX 20:18 · JFK 23:18
    ♥ Do have faith in what you're doing.