Un modo più elegante per farlo in Ruby

Question

Ho iniziato con Ruby e sto trovando nuovi, più brevi ed eleganti modi di scrivere codice ogni giorno.

nel risolvere i problemi Project Euler, ho scritto un sacco di codice come

if best_score < current_score 
    best_score = current_score 
end 

C'è un modo più elegante di scrivere questo?

Answer 1

best_score = [best_score, current_score].max 

vedere: Enumerable. max

---

disclaimer: anche se questo è un po 'più leggibile (imho), è meno performante:

require 'benchmark' 

best_score, current_score, n = 1000, 2000, 100_000 

Benchmark.bm do |x| 
    x.report { n.times do best_score = [best_score, current_score].max end } 
    x.report { n.times do 
    best_score = current_score if best_score < current_score 
    end } 
end 

si tradurrà in (con Ruby 1.8.6 (2008-08-11 patchlevel 287)):

user  system  total  real 
0.160000 0.000000 0.160000 ( 0.160333) 
0.030000 0.000000 0.030000 ( 0.030578) 

Answer 2

Questo può essere fatto su una sola riga:

best_score = current_score if best_score < current_score 

Answer 3

Forse una battuta?

best_score = current_score if best_score < current_score 

Answer 4

Non

sicuro che sarebbe qualificarsi come "più elegante", ma se non si vuole riscrivere la se ogni volta ...

def max(b,c) 
if (b < c) 
    c 
else 
    b 
end 
end 

best = 10 
current = 20 
best = max(best,current) 

Answer 5

Questo è abbastanza elegante. È leggibile e facile da mantenere.

Se volete più breve, si può andare:

best_score = current_score if best_score < current_score 

o

best_score = current_score unless best_score >= current_score 

... ma non è necessariamente un miglioramento in tutti i casi (tenere a mente la leggibilità).

Answer 6

O questo modo

(current_score > best_score) ? best_score = current_score : best_score 

Answer 7

Sembra proprio il modo in cui ce l'hai già. Cambierei solo il confronto in modo che legge:

Se il punteggio attuale è maggiore miglior punteggio

È anche possibile creare un metodo e chiami. Questo è più OO per me.

def get_best_score() 
     current_score > best_score ? 
      current_score : 
      best_score 
end 

Questo è ciò di cui si occupa OOP no? Tieni lo stato dell'oggetto.

best_score = get_best_score() 

Answer 8

Dal momento che non riesco a vedere al di sopra, mi appoggio verso questo uso del ternary operator:

best_score = current_score > best_score ? current_score : best_score 

e c'è anche questa versione un po 'meno frequentemente incontrati:

best_score = (best_score > current_score && best_score) || current_score 

... che è più difficile da leggere, ma mostra un (a me) effetto collaterale leggermente inaspettato di corto circuito. (Vedi this blog post.)