• if a || b 的重构 at 2013年09月10日
  • if a || b 的重构 at 2013年09月10日

    #18 楼 @Victor 坏习惯就如同我儿子吃自己的指甲,梅西也咬指甲,但那是坏习惯。 异常应该放到 get_foo() 里 raise,这样可以测试 get_foo 是否在条件下抛出了特定的 exception。 按照上面的写法,就不是最小化测试,或者单元测试了。

  • if a || b 的重构 at 2013年09月09日

    #15 楼 @hpyhacking 这在重构和测试上是不允许的,在阅读感上也是最差的。当然,它是最直白的表达方式。

  • if a || b 的重构 at 2013年09月08日

    #8 楼 @kenshin54 and 查询和 or 查询的语境不同。

    #9 楼 @zj0713001 是别人写的代码。我在研究重构方案,自己也提升一下重构和测试的能力。

    #10 楼 @hooopo 被 8 楼带跑偏了,8 楼请喝茶。。我们讨论的是 if 中的 or 查询。

    个人习惯上,只有在赋值的时候才用 || ,在判断的时候一概用 or 这个关键词。 这样,我的代码一定是 user = current_user||User.new 的样子。

    a 的存在和 a 的关联关系存在,属性存在,是两种情形。我一般会在外部先确保 a 存在,在 if 里再做条件判断。“是否存在”不是 if 应该做的。

  • if a || b 的重构 at 2013年09月08日

    #5 楼 @hpyhacking 不好。我感觉未到非常非常必要,不要用 try。因为我可以在这个方法外围的适当的位置 begin...rescue...一个 exception 或者自定义的 exception,告诉我发现了什么错误,而不是回避错误。

  • if a || b 的重构 at 2013年09月08日

    #1 楼 @kenshin54 从重构的角度看,这种代码可以,因为它的含义是,user 要么是 vip,要么是五个以上订单,是符合测试时候的语境的。have_more_than_five_orders 也是一个重构的形式。 我现在面对的是未重构的代码,我在探讨重构方式。

  • if a || b 的重构 at 2013年09月08日

    #1 楼 @kenshin54 我指的是 or 中的判断。if 是要取 true 和 false 的。

  • 涨姿势。。。

  • 瓶颈期,求指点. at 2013年09月07日

    我是希望 lz 真心的把 ruby 和 rails 当成职业,这样你会享有它的回馈。技术范围内的问题不是问题,态度才决定一切。不要生气。

  • 别人的代码,我也不知道什么剧情。

  • 瓶颈期,求指点. at 2013年09月07日

    #104 楼 @hellomac 可能你觉得自己是那个 special one,其实你只是 one of ...不要浪费时间去回复啦,搞一个 gem 或者某个开源软件的插件吧,对你有益。

  • 瓶颈期,求指点. at 2013年09月06日

    #87 楼 @Ddl1st 你的回复我没读懂,我的话不爱听,然后用自己的话转述一遍,就爱听了,是么?,。。。。另外,思考是人人都会的,问题是我们怎么解决?我给 lz 提出了 3 个建议。

  • 瓶颈期,求指点. at 2013年09月06日

    个人对这种野生的程序员很畏惧,因为,一,会把代码搞的“又臭又长”,二,完全不理解测试的含义,三,如果不擅长沟通,会成为无法理喻的那个人。 建议:一、多用休息时间搞一些自己的小项目,小插件,gem 等,提高对项目的理解,不要成为会吃饭的代码机器。 二、一定要有一个自己的技术博客。 三、不要理会别人如何看你,听从自己内心的声音,如果你觉得 Rails 开发已经到达极限,无法在短期内突破,那么可以换换职业发展的思路。

  • 暂时用 google draw 画图,用 excel 描述 scenario 和测试用例以及分配任务。 说到实践也是第一次搞这么复杂的测试和重构,周末会写一个简单的程序管理测试用例,希望未来能集成到 redmine 里面去。

  • 一个收钱的东西木有测试,那才叫 垂死病中惊坐起,笑问客从何处来。。。。。。。

  • 去 freelancer 接项目吧。

  • wow。。。收下了。。观望中。。。