reek

-1熱

1答えて

def readable uptime = (Time.now - self).to_i case uptime when 0 then 'just now' when 1 then 'uptime second ago' when 2..59 then uptime.to_s + ' seconds ago' when 60..119

0熱

2答えて

複数のアレイ/ハッシュルックアップを変数

に格納されなければならない、私は私のコードや臭いの1をリファクタリングする最近Reekを使用してきた、DuplicateMethodCall、このようarray[1]やhash[:key]として、配列やハッシュのルックアップに呼び出されています複数回と呼ばれる。私は複数の配列やハッシュ検索が非常に高価なので、変数を直接呼び出すのではなく変数に格納する必要があるのだろうかと思っていました。変数内

1熱

1答えて

機能モジュールが混在している時の羨望

私は現在、非常に簡単なテストツールを書いています。私の "Testable"モジュールを含むクラスを持つことができます。たとえば： class Veilus include Testable end site = Veilus.new テスト可能モジュールは以下があります。 module Testable module_function def inclu

2熱

4答えて

paramsをアクション内のローカル変数にキャッシュすると、それは役に立ちますか？

プロジェクトの一環として、reekというコード品質ツールをしばらく実行します。このツールは基本的にコードの匂いを探して報告します。ここでは、paramsのキーに複数回アクセスしようとするたびに、「Duplication」という匂いが出ることがわかりました（同じパラメータでメソッド呼び出しを2回行っているか、if条件などを複製しているかのように）。ただし、paramsはHashです。それらのキーに複

0熱

1答えて

Eclipseが私のコードで "Feature envy"の匂いを訴えるのはなぜですか？

エクリプス（RedRails）次のコードで「Feature envy」文句を言う： if input_text =~ /^(---\s*\n.*?\n?)(---.*?)/m content_text = input_text[($1.size + $2.size)..-1] # warning in $1 header = YAML.load($1) @cont

5熱

3答えて

メソッドのブール値。何故なの？

私は自分のコードのベストプラクティスの分析ツールとしてreekを使用しています。しかし、最近私は方法では、私はブール値のようなパラメータがあることがわかった。 def method (flag = true) reek私に警告を出します。 reekはなぜそれが警告であると言いますか？

-1熱

1答えて

すくい臭いタスク

https://github.com/kevinrutherford/reek/wiki/Rake-Task を実行中にロードするためにそのようなファイル私は臭いの宝石をインストールし、それを実行するために、上記ページ内のすべての指示に従ったが、すくい臭いのタスクを実行している間、私は次のエラーを取得します。 --local rake reek rake aborted! no such fi