我有以下红宝石葫芦步骤:
When(/^I enter (phone number|email)$/) do |method|
login_type = case method
when 'phone number'
true
when 'email'
false
else
raise("#{method} is not supported")
end
verify_login_page(type: login_type)
aggregator = case method
when 'phone number'
Aggregator::PHONE
when 'email'
Aggregator::EMAIL
else
Aggregator::ALL
end
get_and_enter_code(aggregator)
end
虽然看起来很清楚,但我很确定我可以从设计角度做得更好。特别是我不喜欢的是具有true / false赋值的login_type case方法。如何做到这一点更好?谢谢
您可以使用三元运算符赋值:
When(/^I enter (phone number|email)$/) do |method|
unless ['phone number', 'email'].include?(method)
raise "#{method} is not supported"
else
login_type = method == 'phone number' ? true : false
end
verify_login_page(type: login_type)
aggregator = login_type ? Aggregator::PHONE : Aggregator::EMAIL
get_and_enter_code(aggregator)
end
由于你提出异常Aggregator::ALL
案件应该永远不会发生。