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.