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

Question

エクリプス（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) 

    @content = content_text.strip() 
    @title = header["title"] 
end 

私の理解では、私は安全にこの警告を無視するということです。しかし、私はなぜこの警告が生成されたのか迷っています。 $1.sizeと$1の方法をどのように抽出できるのか分かりません。

Answer 1

Reekは、同じクラスの2つのプロパティを追加しているため、計算は実際にはStringに属している必要があることを伝えています。文字列の長さを追加する場合、これは当然のナンセンスですが、あなたのケースでコードは$&（完全マッチした文字列）を使用して単純化することができます。

input_text[$&.size..-1]