问题标签 [ruby-style-guide]

For questions regarding programming in ECMAScript (JavaScript/JS) and its various dialects/implementations (excluding ActionScript). Note JavaScript is NOT the same as Java! Please include all relevant tags on your question; e.g., [node.js], [jquery], [json], [reactjs], [angular], [ember.js], [vue.js], [typescript], [svelte], etc.

0 投票
1 回答
538 浏览

ruby-on-rails - rake 任务中的 rubocop eval

Rubocop 在以下设置中窒息:

你可以,我现在跳过了 eval 部分,所以我可以使用这个 rake 任务,但我想知道是否有更好的方法来实现这一点。当我们扩展应用程序并添加更多模型时,我想通过将它们附加到列表来节省时间。

0 投票
3 回答
75 浏览

ruby-on-rails - rubocop 应用程序控制器功能验证参数整数使用 nil?谓词

我尝试用多种方法重写这个函数来解决这个错误,但是,在禁用它周围的警察之前,我想听从其他专家的意见。

这是这样使用的:

此问题已通过以下方式解决:检查变量是否为整数

在此处输入图像描述

更新尝试:

参考如何确定字符串是否为数字?

新错误: 在此处输入图像描述

0 投票
1 回答
39 浏览

ruby-on-rails - 模型显示重构多个区域设置消息 rubocop

在此处输入图像描述

当我写这个方法时,我知道它可以写得更好,但它是一个 MVP。现在我正在尝试了解专家对这种方法的看法以及如何解决警察问题。不断学习!

这是有问题的控制器方法:

反过来,这些调用一系列路由来根据他们获得的错误类型指导用户去哪里:

对于专家,您将如何处理这个问题?我知道这必须更干净。

0 投票
2 回答
29 浏览

ruby-on-rails - rails 4 范围 NOTNULL 重构

我必须创建一个范围来创建活跃的工作,但这感觉有点奇怪,老实说它与 PosgresSQL 紧密耦合:

你会写这个不同吗?谢谢

0 投票
1 回答
797 浏览

ruby-on-rails - rails 4 rubocop 使用 proc 而不是 Proc.new

在此处输入图像描述

我一直在修补这个问题,但它不断失败。如果我切换到推荐的过程,它会窒息:

在此处输入图像描述

处理此问题并通过警察的最佳方法是什么?

万一其他人落在这里,最终的语法是:

0 投票
1 回答
393 浏览

ruby-on-rails - 更新期间rails 4简单表单嵌套属性多个模型错误

我正在与嵌套属性的错误作斗争,并试图同时修复 cop 错误。所以这里是遍历。优惠券代码可以使用可能影响工作价格的嵌套属性与表单一起提交。仅当优惠券代码有效时才会发生这种情况。在这种情况下,优惠券代码已经分配,​​因此第一个if coupon_code && coupon.nil?被触发。当表单返回时,flash 消息正常工作,但简单表单不显示该值。我可以调整简单的形式来获得具有实例变量的值,但我开始在我的逻辑中闻到一些东西。此外,它的气味Assignment Branch Condition开始让我担心。我可以继续前进,但用户希望看到代码。我也会的。

警察错误

控制器

查看

更新

  1. 重构此代码以使用下面的帖子。工厂修复factorygirl 创建模型关联 NoMethodError: undefined method
0 投票
1 回答
250 浏览

ruby-on-rails - rubocop 字符串插值和大小条件

在此处输入图像描述

在我除了这两种方法之前,我想看看社区中是否有人有更好的想法来构建这些方法并让警察通过。第一个to_s似乎也有点疯狂。我正在考虑重构另一种方法,但那将是一两行。

想法?

代码示例一:

代码示例二:

0 投票
1 回答
41 浏览

ruby-on-rails - 如何改进这个 Oauth Rubocop 代码?

我有以下方法设置来帮助刷新 Oauth 令牌:

我试着把它分解成单独的方法,但它一直在爆炸,所以我要在这里听从读者的意见。我正在寻找通过警察和学习的建议。

0 投票
2 回答
346 浏览

ruby - 使用 `Object#send` 方法作为最佳实践

很难识别如下代码的用法:

还有其他原因表明这是一种反模式吗?

0 投票
1 回答
63 浏览

ruby-on-rails - Rails 4 api默认限制参数不通过警察

这似乎有点疯狂。有没有另一种方法来编写这个函数来让警察通过它,或者我应该让它豁免?

代码

警察错误