ruby - SECADO de tijeras de papel roca

methods dry

Question

Soy un programador de rubí novato y, aunque este código funciona, me pregunto cómo puedo mejorarlo. Tengo un conocimiento muy limitado sobre lambdas y procs y similares, pero cualquier consejo sería genial. ¿Hay alguna manera de simplificar las declaraciones if else en cada caso? Además, ¿hay alguna manera alternativa de omitir el enunciado de case lugar de hacer que casi la totalidad del código sea una declaración grande si no?

def rps(roll)
    roll_ops = ["rock", "paper", "scissors"]
    pick = roll_ops.sample
    result = nil
    if roll == pick
        result = "tie"
    else
    case roll
    when "scissors" then
        if pick == "paper"
            result = "win"
        else
            result = "lose"
        end
    when "rock" then
        if pick == "scissors"
            result = "win"
        else
            result = "lose"
        end
    when "paper" then
        if pick == "rock"
            result = "win"
        else
            result = "lose"
        end
    else
        puts "Please input rock paper or scissors"
    end
    end

    puts "#{pick}, #{result}"
end

rps("scissors")

Answer 1

Aquí hay un par de formas:

# 1 Usar ciclo de matriz

OPS = %w[rock paper scissors]

def rps(roll)
  pick = OPS.sample
  enum = OPS.cycle
  (prev = enum.next) until enum.peek == roll
  return [pick, "lose"] if prev == pick
  enum.next
  return [pick, "win"] if enum.peek == pick
  [pick, "tie"]
end

rps "scissors" #=> ["scissors", "tie"] 
rps "scissors" #=> ["scissors", "tie"] 
rps "scissors" #=> ["rock", "win"] 
rps "scissors" #=> ["paper", "lose"] 

# 2 Lleva la respuesta de @MiFoFoX un paso más allá

def rps(roll)
  roll_ops = %w|rock paper scissors|
  h = (roll_ops + [roll_ops.first]).each_cons(2).
    with_object(Hash.new("tie")) { |(a,b),h| h[[a,b]]="lose"; h[[b,a]]="win" }
  pick = roll_ops.sample
  [pick, h[[roll,pick]]]
end

rps "scissors" #=> ["rock", "lose"] 
rps "scissors" #=> ["scissors", "tie"] 
rps "scissors" #=> ["paper", "win"] 

Aquí:

h #=> {["rock", "paper"]=>"lose", ["paper", "rock"]=>"win",
  #    ["paper", "scissors"]=>"lose", ["scissors", "paper"]=>"win",
  #    ["scissors", "rock"]=>"lose", ["rock", "scissors"]=>"win"} 

y debido al valor predeterminado "empate":

h[["rock", "rock"]]         #=> "tie"
h[["paper", "paper"]]       #=> "tie"
h[["scissors", "scissors"]] #=> "tie"

Answer 2

Puede construir un hash que contenga la selección y qué opción perder contra esa selección:

hash = {''scissors'' => ''paper'', ''rock'' => ''scissors'', ''paper'' => ''rock''}

Luego, comprueba si la selección de la máquina es la misma que la que hizo:

roll_ops = ["rock", "paper", "scissors"]
pick = roll_ops.sample
if roll == pick

Y la condición de ganar / perder se convierte en algo como esto:

if hash[roll] == pick
  "win"
else
  "lose"
end

Agradable y limpio con solo 2 condiciones.

Answer 3

Creo que un proceso u otra configuración similar es probablemente excesivo. Solo use ifs en línea:

def rps(roll)
  raise "Choose rock, paper, or scissors" if roll.nil?
  roll_ops = ["rock", "paper", "scissors"]
  pick = roll_ops.sample
  result = if roll == pick
    "tie"
  else
    case roll
    when "scissors"
      pick == "paper" ? ''win'' : ''lose''
    when "rock"
      pick == "scissors" ? ''win'' : ''lose''
    when "paper" then
      pick == "rock" ? ''win'' : ''lose''
    end
  end
  puts "#{pick}, #{result}"
end

rps("scissors")

Eliminé tu extra adicional que se suponía que manejaba sin entrada. Es mejor usar errores en tal caso.

Hay un par de trucos aquí:

1- Los ifs en línea. Esos deberían ser bastante claros.

2- La variable de resultado se establece igual al valor de retorno de la expresión if. Este es un truco útil que puedes usar porque en Ruby, ¡todo es una expresión!

Si está interesado en usar una lambda para esto, también debería funcionar bastante bien:

def rps(roll)
  raise "Choose rock, paper, or scissors" if roll.nil?
  roll_ops = ["rock", "paper", "scissors"]
  pick = roll_ops.sample
  did_win = lambda do |choice|
    return choice == pick ? ''win'' : ''lose''
  end
  result = if roll == pick
    "tie"
  else
    case roll
    when "scissors"
      did_win.call(''paper'')
    when "rock"
      did_win.call(''scissors'')
    when "paper" then
      did_win.call(''rock'')
    end
  end
  puts "#{pick}, #{result}"
end

Answer 4

ROLL_OPS = %w[rock paper scissors]
RESULTS = %w[tie lose win]
def rps(roll)
  unless i = ROLL_OPS.index(roll)
    return puts "Please input rock paper or scissors".freeze
  end
  pick = ROLL_OPS.sample
  puts "#{pick}, #{RESULTS[(i - ROLL_OPS.index(pick)) % 3]}"
end