Yinelenen Koşul Dalları (ARCH6190)

Anahtar ruby.quality.duplicate-branches · Önem warning · Kategori quality

Gövdeleri birebir aynı olan dallar dallanmayı gereksiz kılar; ya koşullar birleştirilmeli ya da yinelenen mantık tek bir yola indirgenmelidir.

Hatalı

def fee(plan)
  if plan == :free
    0
  elsif plan == :trial
    0
  else
    20
  end
end

Doğru

def fee(plan)
  return 0 if [:free, :trial].include?(plan)
  20
end

Düzeltme

Aynı gövdeye sahip dalları tek bir koşulda birleştirin veya ortak mantığı çıkarın.