问题标签 [reek]

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 投票
4 回答
371 浏览

ruby-on-rails - 如果我们将参数缓存到操作中的本地变量中,它会有所帮助还是一样?

因此,作为项目的一部分,我们偶尔会运行一个名为reek的代码质量工具。该工具主要查找代码异味并报告它们。在这里,我们观察到每次我们尝试params多次访问一个键时都会产生“重复”的气味(就像我们使用相同的参数进行两次方法调用或者我们正在复制一个 if 条件等)。然而,params只是一个Hash,对吧?当多次访问其密钥时,其他哈希不会产生重复气味。

为什么会这样?究竟是params什么?params缓存在局部变量中然后使用它们是否有意义?它会有所帮助还是一样?还是工具有问题?帮助!

0 投票
1 回答
903 浏览

ruby - 为什么 Eclipse 在我的代码中抱怨“功能嫉妒”的气味?

Eclipse (RedRails)在以下代码中抱怨“功能嫉妒”:

我的理解是我可以安全地忽略这个警告。但我在徘徊为什么会产生这个警告。我无法理解如何为$1.sizeand提取方法$1

0 投票
4 回答
436 浏览

ruby - Ruby:使用 Reek 作为训练工具

Reek 在培训红宝石菜鸟的良好实践方面是否有用,还是需要经验丰富的红宝石眼来使用和解释?

我有多年或编程经验,但主要是 C 变体。在过去的几年里,我很少使用 Ruby 作为实用脚本语言,但我的 ruby​​ 代码显然大部分只是音译 C。现在我希望将它用作一个严肃的工具,并且我想学习“ruby 方式”。

我计划从一开始就使用 TDD/BDD 来提供必要的培训反馈,看起来 Reek 可能有助于提供有关非标准形式和使用的反馈。但是,根据长期的经验,我知道这些工具可能是两刃剑,需要有经验才能使用,并且在新手手中会导致更多的问题而不是解决的问题。

有没有人以这种方式使用 reek(或类似工具)的经验?

如果这很重要,我最初将专注于为 MacOS X 编写独立的应用程序,即没有导轨、服务器等。

0 投票
3 回答
6044 浏览

ruby-on-rails - 方法中的布尔参数。为什么不?

reek在我的代码中用作最佳实践的分析工具。但是最近我发现如果在方法中,我有布尔参数,例如。

reek给我一个警告。为什么reek告诉我这是一个警告?

0 投票
1 回答
395 浏览

ruby-on-rails - 运行 rake reek 任务时没有要加载的文件

https://github.com/kevinrutherford/reek/wiki/Rake-Task

我按照上面页面中的所有说明安装并运行了 reek gem,但是在运行 rake reek 任务时出现以下错误。

gem list --local(显示已安装的正确 gem)

0 投票
1 回答
457 浏览

ruby-on-rails - 如何纠正来自 reek 的 DuplicateMethodCall

如何DuplicateMethodCall从以下电话中纠正臭味

我应该如何处理 params[:some] 我应该单独声明它。

0 投票
3 回答
1774 浏览

ruby - 臭味代码重复方法调用修复

我收到以下错误reek

下面是方法 move_forward

我想删除所有错误,尤其是重复的方法调用。在这种情况下,解决所有警告的最佳方法是什么?

0 投票
2 回答
371 浏览

ruby - 臭味错误重复条件

我有以下错误:

从以下代码生成:

我们将如何摆脱这种重复?

0 投票
1 回答
229 浏览

ruby - Ruby 的 Reek 质量检查

我的机器人课程出现以下错误:

这是导致它的有问题的代码:

我们将如何重新组织它以避免此警告?

0 投票
1 回答
106 浏览

ruby-on-rails - DuplicateMethodCall format.json 的恶臭投诉

我有这个代码恶臭抱怨 DuplicateMethodCall (调用 format.json 两次)

在rails中最好的方法是什么?或者这是误报?