プロジェクトオイラータスク17は、数字

Question

問題の範囲からのすべての文字の数を探します3 + 3 + 5 + 4 + 4 =合計19文字。 1から1000までのすべての数字が単語で書き出された場合、いくつの文字が使用されますか？

私の解決策：他の質問に@Arifにより、privided

require 'humanize' 
arr, total = [], 0 
(1..1000).to_a.map(&:humanize).each {|x| arr << x.delete(" ").delete("-")}.map {|y| total += y.length } 
p total 

他の解決策は、（プロジェクトオイラーによる）正しい答えを与える：

p (1..1000).to_a.map(&:humanize).join.tr(" -", "").size 

私の解決策が間違っているのはなぜ？私の知る限り、これら2つの実装は同じ出力を与える必要があります...

編集： もう1つ奇妙なことが見つかりました。 私は20と実装までの任意の番号に1000を変更した場合は、次のようになります。

(1..20).to_a.map(&:humanize).each {|x| arr << x.delete(" ").delete("-")}.map {|y| total += y.length } 

p (1..20).to_a.map(&:humanize).join.tr(" -", "").size 

は、私は同じ答えを受け取る.....？何？

Answer 1

mapをeachと混乱させているという問題があります。

(1..1000).to_a.map(&:humanize) 
       .each {|x| arr << x.delete(" ").delete("-") } # ??? each 
       .map {|y| total += y.length }     # ??? map 

Enumerable#each戻り、元Enumerable、ないarrまたは何でもあなたが期待きたの呼び出し。はい、あなたはarrを実行しましたが、後者のmapは元の配列で呼び出され、これらのスペースとダッシュがすべて含まれています。

、あなたのようにそれを実装するための正しい方法：

(1..1000).to_a.map(&:humanize) 
       .map { |x| x.delete(" ").delete("-") } # sic!, no need for arr 
       .each { |y| total += y.length } # no need to map here 

あるいは、より多くの慣用的なルビー：

(1..1000).to_a.map(&:humanize) 
       .map { |x| x.delete(" ").delete("-") } 
       .reduce(:+) # reduce by summing everything up 

あるいは、より良い（to_aはここに冗長です）：

(1..1000).map(&:humanize) 
     .reduce(0) { |memo, x| memo + x.delete(" ").delete("-").length }