вызова метода в зависимости от вариантов (лучший способ реорганизовать)

Question

У меня есть функция, где я передать строку и возможность использовать его в качестве регулярного выражения

def regexp_this?(string, arg1, arg2, regx = false) 
    if regx 
    method1 %r{#{string}:someconstantstring} 
    method2 %r{#{string}:someconstantstring:#{arg1}} 
    method3 %r{#{string}:someconstantstring:#{arg1}:anotherconstant:#{#arg2}} 
    else 
    method1 "#{string}:someconstantstring" 
    method2 "#{string}:someconstantstring:#{arg1}" 
    method3 "#{string}:someconstantstring:#{arg1}:anotherconstant:#{#arg2}" 
    end 
end 

method1, method2 и method3 призывы expire_fragment. Я создаю кеш, основанный на текущем состоянии страницы. Если arg1 или arg2 объявлен или нет, у меня есть разные ключи кеша, которые мне нужно закончить.

Есть ли способ реорганизовать это?

Answer 1

Для условно, по крайней мере, вы должны быть в состоянии сделать что-то вроде этого:

def regexp_this?(string, arg1, arg2, regx = false) 
    # Select whether you want a String or Regexp parameter. 
    argument_klass = (regx ? Regexp : String) 

    method1 argument_klass.new("#{string}: ...") 
    method2 argument_klass.new("#{string}: ...") 
    method3 argument_klass.new("#{string}: ...") 
end 

Answer 2

У вас есть несколько обязанностей по одному методу, отделите их и назовите их соответствующим образом. У вас есть проверка регулярных выражений, несколько аргументов и несколько методов в методе. Рекомендую прочитать the SOLID principles.