1

ユニットを含むさまざまなハッシュがあり、ユニット タイプにグループ化されています。私のコードは、さらに処理するためにどの単位タイプを返す必要があるかを判断することを目的としています。ただし、各リストを調べると、多くの重複が発生しています。最初の if は、最初の elsif とまったく同じことをしています。可能な限り最善の方法でコードを DRY するにはどうすればよいですか?

from_unit = "gr"
to_unit = "kg"

WEIGHT = {
"gr" => 1000.0,
"kg" => 1.0,
}

MEASURE = {
"mm" => 1000.0,
"cm" => 100.0,
"m" => 1.0
}

if WEIGHT.has_key?(from_unit) or WEIGHT.has_key?(to_unit)
  if WEIGHT.has_key?(from_unit) && WEIGHT.has_key?(to_unit)
    return WEIGHT
  elsif WEIGHT.has_key?(from_unit)
    raise RuntimeError, "#{to_unit} is not a known unit"
  else
    raise RuntimeError, "#{from_unit} is not a known unit"
  end
elsif MEASURE.has_key?(from_unit) or MEASURE.has_key?(to_unit)
  if MEASURE.has_key?(from_unit) && MEASURE.has_key?(to_unit)
    return WEIGHT
  elsif MEASURE.has_key?(from_unit)
    raise RuntimeError, "#{to_unit} is not a known unit"
  else
    raise RuntimeError, "#{from_unit} is not a known unit"
  end
else
  raise RuntimeError, "You can't convert #{from_unit} into #{to_unit}"
end
4

2 に答える 2

6

簡単にするために、このスニペットはあなたのものよりも少ないチェックを行います (本当に必要ですか?) が、仕事は完了します:

def get_table(from_unit, to_unit)
  [WEIGHT, MEASURE].detect do |table|
    table[from_unit] && table[to_unit]
  end or fail("You can't convert #{from_unit} into #{to_unit}")
end
于 2012-12-09T14:27:35.050 に答える
1

次のようなことができます。

if !MEASURE.has_key?(from_unit) and !WEIGHT.has_key?(from_unit)
    raise RuntimeError, "#{from_unit} is not a known unit"

if !MEASURE.has_key?(to_unit) and !WEIGHT.has_key?(to_unit)
    raise RuntimeError, "#{to_unit} is not a known unit"

if WEIGHT.has_key?(from_unit) and WEIGHT.has_key?(to_unit)
    return WEIGHT

if MEASURE.has_key?(from_unit) and MEASURE.has_key?(to_unit)
    return MEASURE # Was this a typo?

raise RuntimeError, "You can't convert #{from_unit} into #{to_unit}"
于 2012-12-09T13:57:49.313 に答える