常见的代码质量问题-- 汇总版 

访问量: 702

写代码的原则: 简单,易于沟通。 建议看的书: <<代码之美>>, 以及 <>
# 不要把有意义的变量名 命名成 foo, bar,  例如下面的,改成叫 guide 或者叫 guide_property 比较合适。
foo = Ti.App.Properties.getString("guide")
# 对于coffeescript , 最后一行默认是return , 所以 下面代码的最后一行的 return 是多余的,应该去掉。
back_me = ->
  Ti.App.addEventListener "close_choice", ->
    $.choice_by_self.close()
    return
#这里要把地址写到配置文件中去。 不要写死在代码中
home_competition_update = ->
    get_json("http://106.185.52.66:1111/interface/games")

#刷新三大成指的数据
there_stock_update = ->
    get_market_json('http://hq.sinajs.cn/list=s_sh000001,s_sz399001,s_sz399006')

# 不要使用 “魔法数"  ,把下面的 1, 2 换成有意义的名字。 例如: 
#点击股票行情                  
SHOULD_SHOW_LEFT = 1
go_to_market_view = (e) ->     
    market_view_select(2)   # 改成: market_view_select( SHOULD_SHOW_LEFT)
    market_click_style(2)      
    there_stock_update()  
#视图切换判定方法           
market_view_select = (num) ->  
    switch num
      when 1   # 改成:  when SHOULD_SHOW_LEFT 
        $.id_market_left_view_style.show()
        $.id_market_right_view_style.hide()
      when 2
        $.id_market_left_view_style.hide()
        $.id_market_right_view_style.show()
# 两个不同的方法,要以一个空行来分割。 
go_to_collection = (e) ->
    alert '收藏'
show_message = (e) ->            # 这一行前面要有个空格
    alert '暂无新消息'  
# 文件命名 不清晰,例如:,item_3.coffee 
看不懂这个是什么。。。
# 无内容的文件,把它删掉。 例如: 
titanium 项目中:app/styles 里面的:  buy_detail.tss,  chart_detail.tss ...
rails 项目中 app/helper 里面的也是。

订阅/RSS Feed

Subscribe

分类/category