师傅领错门，害了你个人 - ruby/rails新手避免入错门

光说不练呢。。。 没啥意思。

从偶尔泄露的代码看，这边ruby水平感觉不错的，快客王，gigix，其他的人印象不深。

兄弟，来点实际点的代码吧

估计不能说话了.....
----------------------------
鸟的...算错数了......
本板要50点....

哦，抱歉，看错了。编辑掉。。。

严重怀疑那个titans是个马甲书托。反正不用多理他，浪费时间。何必这么多人跟他一个人过不去。大家眼睛都很亮。

请问，优点是什么，如果你清楚，请诉说一，二

技术是严肃的，并非大家和他过不去

你不看偶后面的文字么？

在改成这样的情况下代码应该如何重构还用偶多说么？虽然偶是Ruby/Rails菜鸟，但是这个代码也是呼之逾出阿：

def pro_activate
  user = User.find_by_name_and_active_code_and_active(params[:name], params[:active_code], false)
  if user
    flash[:notice] = "您的账户已经处于激活状态,请勿重复激活!"
  else
    user.update_attribute(:activated , true)
    flash[:notice] = "恭喜您,您已经成功激活了您的账户!"
  end
end

至于你说的明码密码，你随便找一本Rails的书，比如Rails作者DHH的Agile Web Development with Rails，里面都是 t.column :hashed_password, :string

偶承认偶是个rails菜鸟，菜鸟都能从这个书中找出那么多最佳实践的反面教材，这本书不如叫Rails敏捷开发最佳菜鸟鉴定术第一卷...

差点漏了这个，您一定觉得，if (1 == 2) == false 这个代码可读性不错了...

我来来个稍微简洁点的版本

def pro_activate
  if user = User.find_by_name_and_active_code_and_active(params[:name], params[:active_code], false)
    user.update_attribute(:activated , true)
    flash[:notice] = "恭喜您,您已经成功激活了您的账户!"
  else
    flash[:notice] = "激活失败，您的账户可能不存在或者已经激活，请重新检查你的帐户。"
  end
end

这个函数是用来激活用户的，那么出现特殊情况的，要么就是用户没有注册，尝试进行攻击，或者已经激活过了。所以该写成这样就可以满足使用了。

汗，这个代码，如果我看到组员要这样写，是要游街的。