测试用于TAOCP Knuth入门练习的计数器

Question

Donald在“计算机编程艺术”第一卷第一章中的练习与使用欧几里德的gcd算法取剩余步骤的平均值有关。我的代码完美地返回了GCD，并且已经过了测试。我无法让它返回剩余步骤，而在代码中加上注释的第二个标记为失败的测试无法获得正确的剩余步骤，并且只返回gcdRemainerSteps在gcdTestObject上的1。

require 'minitest/autorun'


class GCDTest < Minitest::Test 
  def test_euclid_gcd
    gcdTestObject=GCD.new(20,5)
    assert gcdTestObject.euclidGcd==5
        assert gcdTestObject.gcdRemainderSteps==1

    end

    def test_euclid_two
        gcdTestObject=GCD.new(13,8)
        assert gcdTestObject.euclidGcd==1
    #Failing TEST Passes on 1 not on 5 
    assert gcdTestObject.gcdRemainderSteps==5
    end
end



class GCD
    attr_accessor :m,:n 
    def initialize(m,n)
        @m=m
        @n=n
    end

    def euclidGcd
        r= @m % @n
        until r==0 
            @m=@n
            @n=r
            r= @m % @n
        end
        return @n   
    end

    def gcdRemainderSteps
        r=@m % @n
        counter=1
        until r==0 
            counter+=1
            @m=@n
            @n=r
            r=@m % @n

        end
        return counter
    end
end

我编写的第二段代码是用来测试计数的，它非常出色地工作，并且完美地通过了所有的测试。

如预期的那样，这将返回100的计数器，并且测试处于绿色状态。

#until_loop_test.rb
require 'minitest/autorun'


class Until_test < Minitest::Test

    def test_till_100_steps
        myUntilTestObject=UntilTester.new

        assert myUntilTestObject.untilLoopCount==100

    end
end


class UntilTester

    def untilLoopCount
        x=0
        counter=0
        until x==100
            x+=1
            counter+=1
        end
        return counter
    end
end

Answer 1

也许GCD的方法不应该修改其成员。只有在gcdRemainderSteps()修改了成员变量之后才调用euclidGcd()。

我做了这个改变，测试通过了：

def euclidGcd
    m = @m
    n = @n
    r = m % n
    until r==0
        m = n
        n = r
        r  = m % n
    end
    return n
end

测试结果：

# Running:

..

Finished in 0.001809s, 1105.5202 runs/s, 2211.0405 assertions/s.

2 runs, 4 assertions, 0 failures, 0 errors, 0 skips