Warum klagen Eclipse über "Feature Neid" Geruch in meinem Code?

Question

Eklipse (RedRails) über „Feature envy“ im folgenden Code beschweren:

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 

Mein Verständnis ist, dass ich sicher diese Warnung zu ignorieren. Aber ich bin verrückt, warum diese Warnung erzeugt wird. Ich kann nicht verstehen, wie ich die Methode für $1.size und $1 extrahieren kann.

Answer 1

Reek sagt Ihnen, dass, weil Sie zwei Eigenschaften der gleichen Klasse hinzufügen, sollte die Berechnung tatsächlich in String gehören. Wenn Stringlängen Zugabe ist dies natürlich Unsinn, aber in Ihrem Fall kann der Code unter Verwendung $& (das komplette angepasst string) vereinfacht werden:

input_text[$&.size..-1]