爲什麼程序通過了所有的測試，如果我在方法中使用常規的if語句，但是在使用三元運算符時表示「堆棧層次太深」？

Question

我正在研究叫做機器人名稱的編碼挑戰。我也有測試。該計劃通過了所有的測試。代碼如下..

class Robot 
    attr_accessor :name 
    @@robots = [] 
    def initialize 
    @name = self.random_name 
    @@robots << self.name 
    end 

    def random_name 
    name = '' 
    2.times do 
     name << ('a'..'z').to_a.sample 
    end 
    3.times do 
     name << (1..9).to_a.sample.to_s 
    end 
    no_duplicate(name.upcase) 
    end 

    def reset 
    @name = self.random_name 
    end 

    def no_duplicate(name) 
    if @@robots.include? name 
     reset 
    else 
     name 
    end 
    end 
end 

如果你需要看測試文件，你可以看看它在這裏robot_name_tests。

然後我開始重構，其中的第一件事是重構no_duplicate方法。因此，重構代碼看起來像這樣

class Robot 

    ... 
    # the rest of code stayed the same 

    def no_duplicate(name) 
    @@robots.include? name ? reset : name 
    end 
end 

使用此版本所有測試顯示SystemStackError: stack level too deep。爲什麼它考慮到提供的代碼在這兩種情況下都會出現這種錯誤以及幕後發生了什麼？謝謝！

Answer 1

我喜歡你的詩歌模式代碼但它導致你在這裏陷入困境。

一種方法還挺保持在詩歌模式，但是解決您的運營商優先考慮的問題是要做到這一點：

def no_duplicate(name) 
    (@@robots.include? name) ? reset : name 
    end 

更新：如果你在大公司工作的編碼規範，你需要使它多一點無聊。我認爲這是顯而易見的，但畫廊正確地注意到通常的解決方案：

@@robots.include?(name) ? reset : name