2

原谅这个问题的简单性。我正在学习 TDD 并有以下陈述。

def test_equilateral_triangles_have_equal_sides
   assert_equal :equilateral, triangle(2, 2, 2)
   assert_equal :equilateral, triangle(10, 10, 10)
end

def test_isosceles_triangles_have_exactly_two_sides_equal
  assert_equal :isosceles, triangle(3, 4, 4)
  assert_equal :isosceles, triangle(4, 3, 4)
  assert_equal :isosceles, triangle(4, 4, 3)
  assert_equal :isosceles, triangle(10, 10, 2)
end

def test_scalene_triangles_have_no_equal_sides
  assert_equal :scalene, triangle(3, 4, 5)
  assert_equal :scalene, triangle(10, 11, 12)
  assert_equal :scalene, triangle(5, 4, 2)
end

我为这个问题做了一个非常基本的解决方案，并希望从更有经验的程序员那里获得关于替代解决方案的反馈。

我的代码：

def triangle(a, b, c)
  if (a == b) && (a == c) && (b == c)
    :equilateral
  elsif (a == b) && ((a || b) != c)
    :isosceles
  elsif (a == c) && ((a || c) != b)
    :isosceles
  elsif (b == c) && ((b || c) != a)
    :isosceles
  else
    :scalene
end
4

4 回答 4

7

您可以通过对边进行排序来简化条件检查

sides = [a, b, c].sort
return :equilateral if sides[0] == sides[2]
return :isosceles if sides[0] == sides[1] || sides[1] == sides[2]
return :scalene

或者更简单的方法是使用.uniq

sides = [a, b, c].uniq
type = case sides.length
        when 1 then :equilateral 
        when 2 then :isosceles 
        when 3 then :scalene
       end
于 2012-12-06T21:05:06.397 回答
2

你的很多条件都是多余的。如果 a==b 和 a==c，则 b==c 必然。此外，如果第一个条件为假且 a==b，则 c 不能等于其中任何一个，否则第一个条件为真。所以你可以简化为：

def triangle(a, b, c)
  if (a == b) && (a == c)
    :equilateral
  elsif (a == b) || (a == c) || (b == c)
    :isosceles
  else
    :scalene
  end
end
于 2012-12-06T20:40:42.337 回答
2

两条评论。首先，通常您希望通过测试来确认阳性病例，以及检查失败情况 ( assert_not_equal)。

triangle我想你可以稍微收紧你的方法。

考虑：如果a == b并且b == c那么 是真的a == c

考虑：如果或a == b然后三角形是等腰三角形b == ca == c

考虑：满足第一个条件将阻止其他人执行，并排除其他情况。

于 2012-12-06T20:41:16.410 回答
0

这应该涵盖您：

def triangle(a, b, c)

  raise TriangleError, 'Sum of two sides must be greater than the third.' if a + b < c || b + c < a || a + c < b # not a triangle!
  raise TriangleError, 'Sum of two sides must not be equal to the third.' if a + b == c || b + c == a || a + c == b # degenerate triangle!

  return :equilateral  if a == b && a == c
  return :isosceles    if a == b || b == c || a == c
  return :scalene      if a != b && b != c && a != c

end

感谢您通过 Ruby Koans 取得的进步！:-)

于 2013-05-10T21:18:21.270 回答