重构检查env变量

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

这一切都与编程风格的品味有关。在我看来，除非您在整个应用程序的不同位置使用相同的代码块，否则您的代码看起来不错。在这种情况下，您应该将其包装在类或方法中，以确保设置或不设置必要的应用程序部分。