Helfen Sie beim Refactoring dieser bösen Ruby if / else-Anweisung

Ich habe also diese große, haarige if / else-Anweisung. Ich übergebe ihm eine Tracking-Nummer und ermittle dann, um welche Art von Tracking-Nummer es sich handelt.

Wie kann ich das vereinfachen? Speziell um die Anzahl der Codezeilen zu reduzieren.

if num_length < 8
  tracking_service = false
else
  if number[1, 1] == 'Z'
    tracking_service = 'ups'
  elsif number[0, 1] == 'Q'
    tracking_service = 'dhl'
  elsif number[0, 2] == '96' && num_length == 22
    tracking_service = 'fedex'
  elsif number[0, 1] == 'H' && num_length == 11
    tracking_service = 'ups'
  elsif number[0, 1] == 'K' && num_length == 11
    tracking_service = 'ups'
  elsif num_length == 18 || num_length == 20
    check_response(number)
  else
    case num_length
    when 17
      tracking_service = 'dhlgm'
    when 13,20,22,30
      tracking_service = 'usps'
    when 12,15,19
      tracking_service = 'fedex'
    when 10,11
      tracking_service = 'dhl'
    else
      tracking_service = false  
    end  
  end
end

Ja, ich weiß. Es ist fies.

Antworten auf die Frage(5)

Ihre Antwort auf die Frage