Ruby和if語句

Question

讓我先說這個，說我是Ruby的新手。

我試圖做這樣的事情：

raise NoSuchStrategyError unless ((player1[1].downcase && player2[1].downcase) == ("p" || "r" || "s")) 

然而，由於預期它不工作。它只承認第一個參數是否是「p」。如果它是「r」或「s」，它會拋出錯誤。我不得不寫出來的很長的路要走這樣，它的工作：

raise NoSuchStrategyError unless player1[1].downcase == "p" or player1[1].downcase == "s" or player1[1].downcase == "r" 
raise NoSuchStrategyError unless player2[1].downcase == "p" or player2[1].downcase == "s" or player2[1].downcase == "r" 

是否有更好的方法來做到這一點速記？

Answer 1

這是因爲||返回到它的第一個參數是truthy 。在這種情況下，由於"p"是truthy，因此("p" || "r" || "s")總是返回"p"。認識到這一點，你的第一個語句可以等價地改寫爲：

raise NoSuchStrategyError unless ((player1[1].downcase && player2[1].downcase) == "p" 

由於攪拌機暗示在他對Python的評論，你可以這樣做：

raise NoSuchStrategyError unless ['p', 'r', 'y'].include?(player1[1].downcase) && ['p', 'r', 'y'].include?(player2[1].downcase) 

或更簡潔：

raise NoSuchStrategyError unless [player1[1].downcase, player2[1].downcase].all? { |c| %w[p r s].include? c } 

此外，在Ruby中使用and & or時要小心，它們不同於&& & ||。你可以（也應該）read more about the difference。

Answer 2

你可以把它簡化這樣的：

raise NoSuchStrategyError unless (%w(a b c).include?(player1[1].downcase) && %w(a b c).include?(player2[1].downcase)) 

編輯

一個更簡單的解決方案：

raise NoSuchStrategyError if ("pry"[player1[1].downcase] || "pry"[player2[1].downcase]) 

Answer 3

在你的方法('p' || 'r' || 's')總是返回「P」，因爲在Ruby中，除了nil和false一切true包括0。因此，除了「P」，你的方法失敗。

---

嘗試使用Array#include?方法。

plays = ['p', 'r', 's'] 

raise NoSuchStrategyError unless (plays.include?(player1[1].downcase) && 
            plays.include?(player2[1].downcase) 
           ) 

Answer 4

你可能只是做：

"pry".include?(player1[1].downcase) 

與您的代碼真正的問題是你如何構建它。當你發現自己宣稱像PLAYER1和player2變量，然後寫一堆重複的代碼用這些變量工作，其通常是你需要聲明一個「玩家」類線索：

class Player 

    def initialize(name) 
    @name = name 
    @strategy = "goofy" 
    end 

    def valid_strategy? 
    return "pry".include?(@strategy) 
    end 

end 

然後你行看起來像這樣：

raise NoSuchStrategyError unless @player.valid_strategy?