Неэффективное именование методов Ruby: передача пространства имен в качестве аргумента для вызова методов

Question

Должен быть более эффективный способ сделать это в Ruby. У меня есть список методов, которые обрабатывают одни и те же вещи (название, цена) на нескольких сайтах, но немного по-разному, основываясь на коде в каждом магазине. Например:

def store1_get_title
def store1_get_price

def store2_get_title
def store2_get_price

def store3_get_title
def store3_get_price

При вызове всех этих функций я просто хотел бы, чтобы общий вызов с параметром «пространство имен» вызывал любой из этих методов без необходимости набирать все из них, что-то вроде:

for get_all_stores().each do |store|
     store::get_title
     store::get_price
end

... который будет вызывать store1_get_title, store1_get_price, store2_get_title, store2_get_price, как я хочу. Есть что-то подобное или лучший способ сделать это?

Надеюсь, это имеет смысл. Спасибо за любой вклад!

Редактировать: эти задачи находятся в коде задачи rake.

Answer 1

Это идеальное использование для занятий. Если вы найдете два магазина с одним и тем же программным обеспечением (например, магазины Yahoo или EBay), вы можете создавать экземпляры классов с разными параметрами.

class Amazon
  def get_price; end
  def get_title; end
end

class Ebay
  def initialize seller; end
  def get_price; end
  def get_title; end
end

[Amazon.new, Ebay.new("seller1"), Ebay.new("seller2")] each do |store|
   store.get_price
   store.get_title
end

И вы можете сделать это на любом другом объектно-ориентированном языке, определив базовый класс или интерфейс, который все хранилища реализуют / наследуют.

Answer 2

Я не понимаю логику вашего приложения. Возможно, вам следует подумать об определении класса (см. Ответ Кена Блумса).

Тем не менее, вы можете попробовать динамический вызов с send:

def store1_get_title
  p __method__
end
def store1_get_price
  p __method__
end

def store2_get_title
  p __method__
end
def store2_get_price
  p __method__
end

def store3_get_title
  p __method__
end
def store3_get_price
  p __method__
end

all_stores = ['store1', 'store2', 'store3']
all_stores.each do |store|
  send("#{store}_get_title")
  send("#{store}_get_price")
end

Вы не определили, что get_all_stores возвращает. В моем примере я использовал строки. Вы можете добавить синтаксический сахар и расширить строку (я не рекомендую это)

class String
  def get_title()
    send("#{self}_get_title")
  end
  def get_price()
    send("#{self}_get_price")
  end
end

all_stores.each do |store|
  store.get_title
  store.get_price
end

---

Последнее замечание. Вы написали

for get_all_stores().each do |store|

each одного должно быть достаточно. for не похож на рубин и в сочетании с each не выглядит для меня разумным.