Справка, осуществляющая рефакторинг этот противный Ruby, если/еще оператор
Вы могли использовать CSS для достижения этого. Путем определения списка в цвете и стиле по Вашему выбору, можно тогда также определить текст как различный цвет.
Следуют примеру в http://www.echoecho.com/csslists.htm .
5 ответов
Попробуйте это. Я переписал его, используя регистр и регулярные выражения. Я также использовал : символы вместо "строк" для возвращаемых значений, но вы можете изменить это обратно.
tracking_service = case number
when /^.Z/ then :ups
when /^Q/ then :dhl
when /^96.{20}$/ then :fedex
when /^[HK].{10}$/ then :ups
else
check_response(number) if num_length == 18 || num_length == 20
case num_length
when 17 then :dhlgm
when 13, 20, 22, 30 then :usps
when 12, 15, 19 then :fedex
when 10, 11 then :dhl
else false
end
end
В зависимости от того, является ли код отслеживания рубиновым объектом, вы также можете поместить помощника в определение его класса:
class TrackingCode < String
# not sure if this makes sense for your use case
def ups?
self[1,1] == 'Z'
end
def dhl?
self[0,1] == 'Q'
end
def fedex?
self.length == 22 && self[0, 2] == '96'
end
# etc...
end
Тогда ваше условное выражение станет:
if number.ups?
# ...
elsif number.dhl?
# ...
elseif number.fedex?
end
Одно упрощенное условное выражение, где вы находитесь работает с подразумеваемой функцией кода отслеживания. Точно так же, если бы вы использовали циклический подход, ваш цикл также был бы чище:
%w(ups? dhl? fedex?).each do |is_code|
return if number.send(is_code)
end
или даже:
%w(ups? dhl? fedex?).each do |is_code|
yield if number.send(is_code)
end
Хотя это решение длиннее, чем решение jtbandes, оно может вам понравиться, поскольку оно немного более декларативно:
class Condition
attr_reader :service_name, :predicate
def initialize(service_name, &block)
@service_name = service_name
@predicate = block
end
end
CONDITIONS = [
Condition.new('ups') { |n| n[1] == 'Z' },
Condition.new('dhl') { |n| n[0] == 'Q' },
Condition.new('fedex') { |n| n[0..1] == '96' && n.size == 22 },
Condition.new('ups') { |n| n[0] == 'H' && n.size == 11 },
Condition.new('ups') { |n| n[0] == 'K' && n.size == 11 },
Condition.new('dhlgm') { |n| n.size == 17 },
Condition.new('usps') { |n| [13, 20, 22, 30].include?(n.size) },
Condition.new('fedex') { |n| [12, 15, 19].include?(n.size) },
Condition.new('dhl') { |n| [10, 11].include?(n.size) },
]
def tracking_service(tracking_number)
result = CONDITIONS.find do |condition|
condition.predicate.call(tracking_number)
end
result.service_name if result
end
Я не рассматривал здесь вызов метода check_response, так как считаю, что вам, вероятно, следует заняться этим в другом месте (при условии, что он не возвращает имя службы отслеживания).
Этот метод выглядит так, как будто он был написан для скорости. Вы можете использовать minhash в качестве замены, но я думаю, что код довольно чистый и не требует рефакторинга. Rubyists, как правило, испытывают отвращение к ненужной структуре, но часто это необходимо для моделирования реальных ситуаций и / или повышения производительности. Ключевое слово должно быть ненужным.
I полагаю, что это достаточно сложно, чтобы заслужить собственный метод.
Кстати, если длина равна 20, тогда исходная функция возвращает все, что возвращает check_response (n) , но затем пытается (и всегда будет терпеть неудачу) вернуть 'usps' .
@lenMap = Hash.new false
@lenMap[17] = 'dhlgm'
@lenMap[13] = @lenMap[20] = @lenMap[22] = @lenMap[30] = 'usps'
@lenMap[12] = @lenMap[15] = @lenMap[19] = 'fedex'
@lenMap[10] = @lenMap[11] = 'dhl'
def ts n
len = n.length
return false if len < 8
case n
when /^.Z/
return 'ups'
when /^Q/
return 'dhl'
when /^96....................$/
return 'fedex'
when /^[HK]..........$/
return 'ups'
end
return check_response n if len == 18 or len == 20
return @lenMap[len]
end
# test code...
def check_response n
return 'check 18/20 '
end
%w{ 1Zwhatever Qetcetcetc 9634567890123456789012 H2345678901
K2345678901 hownowhownowhownow hownowhownowhownow90
12345678901234567
1234567890123
12345678901234567890
1234567890123456789012
123456789012345678901234567890
123456789012
123456789012345
1234567890123456789
1234567890
12345678901 }.each do |s|
puts "%32s %s" % [s, (ts s).to_s]
end