出目が0~3のサイコロ(その２)

Question

出目が0~3のサイコロを振って、それぞれの目が揃うまで降り続けその回数をカウントするコードを作りました。
それを1000回繰り返してその平均値を割り出すコードを付け加えました。
少数点以下第１位まで求めたいのですが、実行すると小数点以下第３位まで表示されてしまいます。どうしたらよいでしょうか？

sum = 0
1000.times do
	n = 4
	num = Array.new(n)
	
	count = 0
	
	
	until num.all?
		deme = Random.rand(n)
		num[deme] = true
		count += 1
		
	end
	sum += count
end
p sum/1000.round(1)   #ここにround(1)を付け加えましたがうまく計算してくれません

---

実際実行した結果

ruby 2.3.3p222 (2016-11-21 revision 56859) [x64-mingw32]

C:\Users\owner>cd c:\ruby23-x64\ruby

c:\Ruby23-x64\ruby>ruby test1.rb
8.379     #小数点以下第３位まで出てしまいました

c:\Ruby23-x64\ruby>

よろしくお願いいたします。

Answer 1

y.rb

N = 4 # サイコロの目の種類
PLAY_COUNT = 1000

# すべての目がでるまでサイコロを振った回数を返す
def play
  nums = []
  nums << Random.rand(N) while nums.uniq.size != N
  nums.size
end

sum = 0
PLAY_COUNT.times { sum += play }
puts(format('%.1f', sum.to_f / PLAY_COUNT))

前の質問への回答
https://teratail.com/questions/92645
に1000回繰り返してその平均値を割り出すコードをつけてみました。

表示する小数点以下の桁数は format を使って制御しています。
sum / PALY_COUNT では、整数/整数 なので計算結果が整数になってしまいます。

sum.to_f とすることで、 FLOAT/整数 となるようにしています。こうすると計算結果も FLOAT になります。

実行例

Comments

[kintarock777]

回答ありがとうございます。
katoyさんのコードだと、より簡潔になりますね。
def を使ったコードが、よりプログラムっぽくなっていてカッコイイです。
untilを使った前出のコードでも、defを使ったコーディングを試してみたのですがうまくいきません。おそらく変数が２つ以上あるためplayに値が代入できないと思うのですが、変数を１つに絞ったコードが作れません。
ただいま、試行錯誤中です。

[kintarock777]

N = 4
count = 0
nums = Array.new(N)

until nums.all?
nums[Random.rand(N)] = true

count += 1
end

p count

ここまでシェイプアップできましたが（笑）
untilした回数自体をカウントするメソッドとか、あるのでしょうか？

[kintarock777]

.

[katoy]

play メソッドを until で書いてみました。(play2, play3 の２通リで)

def play2
count = 0
nums = Array.new(N)

until nums.all?
nums[Random.rand(N)] = true
count += 1
end
count
end

def play3
nums = Array.new(N, 0)
until nums.all? { |x| x > 0 }
nums[Random.rand(N)] += 1
end
nums.sum
end

play3 は、nums[x] には xの目が出た回数を格納するようにしています。
すべての目の出た回数を合計した値を返すようにすることで、 count 変数を使わずに済ませています。

[kintarock777]

回答ありがとうございます。
早速試してみました。play2は試したところ、うまく機能しました。
変数が２個あっても大丈夫だったということは、最後の処理の値(count)がplay2の値として代入されるのでしょうか？とにかく正常に実行できました。
play3は、試してみたところ、”nums.sum”のところがうまく計算できませんでした

def play3
nums = Array.new(N, 0)
until nums.all? { |x| x > 0 }
nums[Random.rand(N)] += 1
end
nums.sum　　#<=ここの部分が機能しない
end

結局　for文を追加して、

def play3
nums = Array.new(N, 0)
until nums.all?{ |x|x > 0 }
nums[Random.rand(N)] += 1
end
sum = 0
for i in 0..3
sum += nums[i]
end
end

こんな感じになってしまい、また変数が増えてしまいました。

結論として、katoyさんに書いていただいた最初のコード
"while nums.uniq.size ～ "のヤツが一番シンプルできれいなコードの
ように感じました。

とにかく新しい書き方が出てくると、ワクワクします。
意味を考えるのに必死ですが（笑）

[katoy]

配列の合計を求めることについては次を参照していろいろ試してみてください。
(ruby の version がふるいと sum のメソッドは動作しません。)

[kintarock777]

いつも回答ありがとうございます。大変心強いです。
またこれからもいろいろ教えてください。

Answer 2

Rubyでは、メソッド実行のほうが演算子より優先されるので、sum/(1000.round(1))と解釈されてしまいます。

p (sum/1000.0).round(1)とするか、printfで書式指定をかけてprintf '%.1f', sum/1000.0の夜にしましょう（1000.0としているのは、整数÷整数では結果が整数になってしまうので、浮動小数点数演算に持ち込むためです）。