不要过分暴露变量

相信有不少人也和我一样,这样写 destroy action.

# app/controllers/goods_controller.rb
def destroy
  @goods = Goods.find(params[:id])
  @goods.destroy
  redirect_to admin_goods_index_url
end

这个对象删除后直接调转到新页面了。view 根本用不到 @goods ,用一个局部变量足矣。

这是第二次犯这种错误了,以后要常常提醒自己

  1. 不要过分暴露变量的作用域,
  2. view 中用不到的变量统统使用局部变量。
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值