問題タブ [reek]

Eclipse (RedRails)は、次のコードで「 Feature envy 」について不満を述べています。

私の理解では、この警告は無視しても問題ありません。しかし、この警告が生成される理由をさまよっています。$1.sizeとのメソッドを抽出する方法がわかりません$1。

Reek は、Ruby 初心者を適切な方法でトレーニングするのに役立ちますか? それとも、使用および解釈するのに経験豊富な Ruby の目が必要ですか?

私はつぶやく年またはプログラミングの経験がありますが、ほとんどが C バリアントです。ここ数年、Ruby をユーティリティ スクリプト言語として軽く使用してきましたが、Ruby コードは明らかに C の音訳にすぎません。今は本格的なツールとして Ruby を使用したいと考えており、「Ruby の方法」を学びたいと考えています。

必要なトレーニング フィードバックを提供するために、最初から TDD/BDD を使用する予定でした。非標準のフォームや使用法に関するフィードバックを提供するには、おそらく Reek が役立つようです。しかし、長年の経験から、そのようなツールは、使用するのに事前の経験が必要な両刃の剣であり、初心者の手に渡った場合、解決するよりも多くの問題を引き起こす可能性があることを私は知っています.

この方法で reek (または同様のツール) を使用した経験のある人はいますか?

問題があれば、最初は MacOS X 用のスタンドアローン アプリケーション、つまりレールやサーバーなどを使わないものを書くことに集中します。

reekコードのベスト プラクティスの分析ツールとして使用しています。しかし最近、メソッド内に次のようなブール値のパラメーターがあることがわかりました。

reek私に警告を与えます。reekが警告だと言うのはなぜですか?

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

上記のページのすべての手順に従って reek gem をインストールして実行しましたが、rake reek タスクの実行中に次のエラーが発生します。

gem list --local (インストールされている適切な gem を表示します)

DuplicateMethodCall次の呼び出しで悪臭を放つ方法

params[:some] を個別に宣言する必要があります。

から以下のエラーが発生していますreek:

以下はメソッド move_forward です

すべてのエラー、特に重複したメソッド呼び出しを削除したい。この場合、すべての警告を修正する最善の方法は何でしょうか?

Robot クラスで以下のエラーが発生します。

これは、それを引き起こしている問題のあるコードです:

この警告を回避するには、これをどのように再編成しますか?