Rubin injizieren mit bedingten in Block?

Question

die erste Projekt Euler Frage zu tun: die Vielfachen von 3 und 5 zwischen 1 und 1000 Summieren, kam ich mit diesem nach oben (sehr einfach)

sum = 0 
1.upto(999) { |i| sum += i if 0 == i%3 || 0 == i%5 } 
sum 

aber ich dachte, dies funktionieren würde, aber es funktioniert nicht, kann jemand zeigt mir, was ich falsch mache, oder warum es nicht funktioniert?

1.upto(999).inject(0) { |sum, i| sum + i if 0 == i%3 || 0 == i%5 } 

danke!

Answer 1

inject übergibt das Ergebnis des Blocks an die nächste Iteration als erstes Argument. Ihr Block wird nil zurückgeben, wenn Ihre if-Anweisung falsch ist, die dann als sum zurückgegeben wird.

Um die richtige Antwort zu erhalten, sollte der Block die aktuelle Summe zurück, wenn es falsch ist:

1.upto(999).inject(0) { |sum, i| (0 == i%3 || 0 == i%5) ? sum + i : sum } 

Answer 2

1.upto(999).inject(0) { |sum, i| sum += i if 0 == i%3 || 0 == i%5; sum } 

auch (die += beachten) funktionieren würde.

Answer 3

Komplementäre Antwort: Wenn Sie Euler-Probleme lösen wollen, sollten Sie damit beginnen, eigene Erweiterungen von wiederverwendbarem Code zu erstellen. In diesem Fall wäre die erste Erweiterung Enumerable#sum sein:

module Enumerable 
    def sum 
    inject(0, :+) 
    end 
end 

Und jetzt können Sie eine Lösung schreiben, die den Zustand des summatory trennt (man kann es lesen laut und es macht Sinn, das ist typisch für funktionale/deklarative Stil):

1.upto(999).select { |x| x.divisible_by?(3) || x.divisible_by?(5) }.sum 

Mehr:

1.upto(999).select { |x| x % 3 == 0 || x % 5 == 0 }.sum 

Sie es sogar noch einen Schritt weiter drücken könnte und erstellen Fixnum#divisible_by? so können Sie schreiben hier es ist kein Problem, aber später strenge Implementierungen (diejenigen, die Arrays verwenden) werden zu viel Speicher benötigen. Versuchen Sie dann mit laziness:

require 'lazy' 
1.upto(999).lazy.select { |x| x % 3 == 0 || x % 5 == 0 }.sum 

Answer 4

Oder verwenden & proc, die sich selbst richtet.

(1..999).select{|x| x%3==0||x%5==0}.inject &:+ 

Answer 5

(1..999).to_a.keep_if{|d| d%3 == 0 || d%5 == 0}.reduce(:+) der Vollständigkeit halber.