ruby - Rubyで動的メソッドを使用してインスタンスメソッドをリファクタリングする方法は?

Question

これをリファクタリングしたい：

class Collection
  def artists
    @songs.map { |song| song.artist }.uniq
  end
  def albums
    @songs.map { |song| song.album }.uniq
  end
  def names
    @songs.map { |song| song.name }.uniq
  end
end

このようなものに：

class Collection
  song_attributes = [:artist, :album, :name]
  song_attributes.each do |song_attribute|
    define_method song_attribute do
      @songs.map { |song| song.public_send(song_attribute) }.uniq
    end
  end
end

問題は、複数形にする必要がある行define_method song_attribute doにあります。song_attributeどうすればいいですか？

私はこれしか思いつきませんでしたが、もっと良い方法があると思います：

song_attributes = [[:artists, :artist], [:albums, :album], [:names, :name]]
song_attributes.each do |song_attribute|
  define_method song_attribute[0] do
    @songs.map { |song| song.public_send(song_attribute[1]) }.uniq
  end
end

Answer 1

文字列をdefine_methodに渡す

.....
song_attributes = [:artist, :album, :name]

song_attributes.each do |song_attribute|
  define_method("#{song_attribute}s") do
    @songs.map { |song| song.public_send(song_attribute) }.uniq
  end
end
.....