Это может имеет смысл извлекать префиксы в массив первых, когда список становится больше:
URL_PREFIXES = %w[ scp http ]
я могу думать о Serveral способов использование это URL_PREFIXES
постоянный. Например, я часто предпочитаю if...else
блоки по одной гильзе причин читаемости:
if URL_PREFIXES.any? { |p| @path.start_with?(p) }
@source = 'url'
else
@source = 'local'
end
Или это:
@source = 'local'
@source = 'url' if URL_PREFIXES.any? { |p| @path.start_with?(p) }
Или определить префикс первого:
prefix = @path.split(':').first
@source = URL_PREFIXES.include?(prefix) ? 'url' : 'local'
Если вы работаете с Rails вы можете написать это вместо этого:
prefix = @path.split(':').first
@source = prefix.in?(URL_PREFIXES) ? 'url' : 'local'
Что вы хотите получить от этого нового префикса? – engineersmnky
Для обзоров кода есть еще один сайт: http://codereview.stackexchange.com – Meier
@Meier спасибо, я никогда не видел этот сайт раньше! Потрясающие :) – valadzko