首页
学习
活动
专区
圈层
工具
发布
首页
学习
活动
专区
圈层
工具
MCP广场
社区首页 >问答首页 >活动记录查询-如果未提供任何值,则查找的默认值

活动记录查询-如果未提供任何值,则查找的默认值
EN

Stack Overflow用户
提问于 2013-03-13 21:28:57
回答 2查看 1.8K关注 0票数 1

有时,params:user_id将为空,在这种情况下,搜索应该会得到current_user。

代码语言:javascript
运行
复制
@user = User.find(params[:user_id]||=current_user)

代码语言:javascript
运行
复制
@user = User.where('id = ?', params[:user_id]).first || current_user

问题是:

1)哪种搜索方法更合适,以及2)如果params:user_id为空,是否有方法绕过所有搜索?

我确信我可以想出一个if/else语句来完成第二行,但这对我来说有点不对劲。但是,也许不是呢?

谢谢!

EN

回答 2

Stack Overflow用户

回答已采纳

发布于 2013-03-13 21:36:57

这是一个三元运算符的完美位置。

代码语言:javascript
运行
复制
@user = params[:user_id] ? User.find(params[:user_id]) : current_user

我想再补充一点。

代码语言:javascript
运行
复制
I'm sure I could come up with a if/else statement to accomplish line 2 but that
feels hackish to me. But, maybe it's not?

许多开发人员喜欢使用巧妙的单行程序,而不是更具可读性的if/else控制流。我也这么做了!

当你写代码的时候,试着记住可读性远比巧妙的一行代码重要得多。话虽如此,您的特定示例是使用三元运算符的好地方,但替代的if/else也是可以接受的。Ruby甚至可以让它比其他一些语言更酷。

代码语言:javascript
运行
复制
@user = if params[:user_id]
  User.find(params[:user_id])
else
  current_user
end
票数 4
EN

Stack Overflow用户

发布于 2013-03-13 21:35:29

我不知道你要说什么,但对我来说,第一个更像是Rails。

您不需要=

代码语言:javascript
运行
复制
@user = User.find(params[:user_id]||current_user)
票数 1
EN
页面原文内容由Stack Overflow提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接:

https://stackoverflow.com/questions/15386734

复制
相关文章

相似问题

领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档