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

这本书是gigix做的序 ， 5个作者起码有4个在JE，大家伙基本都认识，看目录的话，感觉编排的内容很丰富

 

 

2. 用户表中直接用明码保存用户的密码
请各位作实际开发新手注意：任何一个应用都不应该犯这样愚蠢的错误

人家是写书，不是教你开发一个权限系统，这样说有些过分。
(下略....)

人家是写书，不是教你开发一个权限系统，这样说有些过分。

数据库命名无可厚非。难道不懂Ruby的人不可以为Ruby程序员设计数据库了吗？这样不太厚道。

尊重别人也是尊重自己。

2. 用户表中直接用明码保存用户的密码
请各位作实际开发新手注意：任何一个应用都不应该犯这样愚蠢的错误

人家是写书，不是教你开发一个权限系统，这样说有些过分。

3.标示用户是否激活的字段名叫is_actived
这不符合rails的最佳实践写法，对于boolean类型的数据，应该省略前面的is_，ActiveRecord会自动加个?号，映射成actived?

数据库命名无可厚非。难道不懂Ruby的人不可以为Ruby程序员设计数据库了吗？这样不太厚道。

5. 设置用户的默认激活为false：@user.is_activated = false
Rails的理念是COC和DRY，这种初始默认值的设定应该在创建数据库定义的时候指定，而且默认的boolean都是false，这里的赋值完全是多此一举。

呵呵，数据库设计这忘记了为Ruby程序员定义初值怎么办？

7. if user != nil && user.is_activated == false
user.is_activated == false ??? 这叫啥代码阿？ 偶的天哪，李刚老师可能是为了保持风格统一，后面果然还有 user.is_activated == true ，这是偶看到搞笑的代码了...

这样的代码可读性不错，当然给初学者看比较合适。这样笑话人家不太地道。

8. Ruby代码 复制代码

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

这样的代码也不见得就可以作为规范，IF嵌套层次多了，可读性不好。

尊重别人也是尊重自己。