重構檢查環境變量

Question

我有這段代碼來檢查環境變量是否爲空，並引發異常。

sample_id  = ENV['SAMPLE_ID'] 
sample_password = ENV['SAMPLE_PASSWORD'] 
raise "ENV variables are empty!" if ENV['SAMPLE_ID'].nil? || ENV['SAMPLE_PASSWORD'].nil? 

有什麼辦法可以用更有說服力的方式編寫這些代碼嗎？我使用變量env兩次。

Answer 1

是否這樣？

sample_id  = ENV['SAMPLE_ID'] 
sample_password = ENV['SAMPLE_PASSWORD'] 

raise 'ENV variables are empty!' unless sample_id && sample_password 

Answer 2

這一切都涉及編程風格的味道問題。在我看來，你的代碼看起來很好，除非你在應用程序的不同地方使用相同的代碼塊。在這種情況下，您應該將其包裝在類或方法中以確保設置或不設置必要的應用程序部分。