Skip to content

Commit 5f1b8bd

Browse files
committed
style: safe_openのURL検証を後置unlessで簡潔に記述
1 parent ac923e8 commit 5f1b8bd

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

lib/tasks/fetch_news.rake

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,7 @@ require 'active_support/broadcast_logger'
77

88
def safe_open(url)
99
uri = URI.parse(url)
10-
unless uri.is_a?(URI::HTTP) || uri.is_a?(URI::HTTPS)
11-
raise "不正なURLです: #{url}"
12-
end
10+
raise "不正なURLです: #{url}" unless uri.is_a?(URI::HTTP) || uri.is_a?(URI::HTTPS)
1311

1412
Net::HTTP.start(uri.host, uri.port, use_ssl: uri.scheme == 'https') do |http|
1513
request = Net::HTTP::Get.new(uri)

0 commit comments

Comments
 (0)