Visual studio C++

Question

Visual studio C++でサイコロを50回振り、1〜6の数字が何回出たかを調べるということをやろうとしています。

#include "stdafx.h"
#include <time.h>
#include <iostream>
#include <string>
using namespace std;

int main()
{
int N[50], kazu[7];
int count[6 + 1] = { 0 };//さいころの目　１～６ごとのカウント
for (int J = 1; J <= 50; J++) { N[J] = 0; };
kazu[1] = 0;
kazu[2] = 0;
kazu[3] = 0;
kazu[4] = 0;
kazu[5] = 0;
kazu[6] = 0;
kazu[7] = 0;

//乱数生成の準備
srand((unsigned)time(NULL));
//50個の乱数を表示する
for (int i = 1; i <= 50; ++i) {
N[i] = rand() % 6 + 1;
if (N[i] == 1) { kazu[i] = kazu[1] + 1; }
if (N[i] == 2) { kazu[i] = kazu[2] + 1; }
if (N[i] == 3) { kazu[i] = kazu[3] + 1; }
if (N[i] == 4) { kazu[i] = kazu[4] + 1; }
if (N[i] == 5) { kazu[i] = kazu[5] + 1; }
if (N[i] == 6) { kazu[i] = kazu[6] + 1; }
if (N[i] == 7) { kazu[i] = kazu[7] + 1; }
cout << N[i] << " " << kazu[i] << "¥n";
};
return 0;
}

このコードでやっていて、1が1回出現するたびに増やす2が1回出るたびに〜〜という感じにやろうと思ってましたが、全く上手くいきません…

どうすればいいでしょうか？

Comments

[Bull]

最終的に欲しいのは 1～6 の出た回数ですよね。
とすると、N の役割は何でしょうか？
Nを配列にする必要はありますか？

Answer 1

C/C++では配列を

int N[50];

と宣言したら、その配列の添字は0～49までになります。

なので、

for (int i = 0; i < 50; i++)

でfor文を回さないとだめです。
どうしても[1]～[50]でアクセスしたければ

int N[51];

と宣言すればできます。
-858993460 とかの数値が表示されるのは、配列をオーバーアクセスしているためです。（運良く暴走してないだけ）

Answer 2

多分これでなおるはず・・・

if (N[i] == 1) { kazu[i] = kazu[1] + 1; }
if (N[i] == 2) { kazu[i] = kazu[2] + 1; }
if (N[i] == 3) { kazu[i] = kazu[3] + 1; }
if (N[i] == 4) { kazu[i] = kazu[4] + 1; }
if (N[i] == 5) { kazu[i] = kazu[5] + 1; }
if (N[i] == 6) { kazu[i] = kazu[6] + 1; }
if (N[i] == 7) { kazu[i] = kazu[7] + 1; }

↓

if (N[i] == 1) { kazu[1] = kazu[1] + 1; }
if (N[i] == 2) { kazu[2] = kazu[2] + 1; }
if (N[i] == 3) { kazu[3] = kazu[3] + 1; }
if (N[i] == 4) { kazu[4] = kazu[4] + 1; }
if (N[i] == 5) { kazu[5] = kazu[5] + 1; }
if (N[i] == 6) { kazu[6] = kazu[6] + 1; }
if (N[i] == 7) { kazu[7] = kazu[7] + 1; }

色々まずそう。
とりあえず動きそうな感じに修正するとこんな感じ。

#include <iostream>
#include <time.h>
#include <stdio.h>
using namespace std;

int main()
{
int N[50], kazu[7];
int count[6 + 1] = { 0 };//さいころの目　１～６ごとのカウント
for (int J = 1; J <= 50; J++) { N[J] = 0; };
kazu[1] = 0;
kazu[2] = 0;
kazu[3] = 0;
kazu[4] = 0;
kazu[5] = 0;
kazu[6] = 0;
//kazu[7] = 0; // int N[50], kazu[7];と定義されてるのでkazu[7]にアクセスしちゃだめ

//乱数生成の準備
srand((unsigned)time(NULL));
//50個の乱数を表示する
for (int i = 1; i <= 50; ++i) {
N[i] = rand() % 6 + 1;
if (N[i] == 1) { kazu[1] = kazu[1] + 1; }
if (N[i] == 2) { kazu[2] = kazu[2] + 1; }
if (N[i] == 3) { kazu[3] = kazu[3] + 1; }
if (N[i] == 4) { kazu[4] = kazu[4] + 1; }
if (N[i] == 5) { kazu[5] = kazu[5] + 1; }
if (N[i] == 6) { kazu[6] = kazu[6] + 1; }
//if (N[i] == 7) { kazu[7] = kazu[7] + 1; } // ここいらない
};
for(int i=1;i<=6;i++){
cout << kazu[i] << endl;
}
return 0;
}

Comments

[kk3taro]

やってみたんですけど、実行結果（回数）が-858993460 とか出て来てしまいました…

[yokuda]

なるほど・・・
よく見てみると、いろいろ怪しいところがありました。

たとえば、iが50のとき以下のコードが実行された場合、Nとkazuのサイズをオーバーしてアクセスしてしまいますね。
cout << N[i] << " " << kazu[i] << "¥n";

[yokuda]

サイコロの出目をカウントするfor文と集計を表示するfor文を分けたほうがいいでしょう。

[yokuda]

とりあえず動きそうな感じに修正するとこんな感じ。

#include <iostream>
#include <time.h>
#include <stdio.h>
using namespace std;

int main()
{
int N[50], kazu[7];
int count[6 + 1] = { 0 };//さいころの目　１～６ごとのカウント
for (int J = 1; J <= 50; J++) { N[J] = 0; };
kazu[1] = 0;
kazu[2] = 0;
kazu[3] = 0;
kazu[4] = 0;
kazu[5] = 0;
kazu[6] = 0;
//kazu[7] = 0; // int N[50], kazu[7];と定義されてるのでkazu[7]にアクセスしちゃだめ

//乱数生成の準備
srand((unsigned)time(NULL));
//50個の乱数を表示する
for (int i = 1; i <= 50; ++i) {
N[i] = rand() % 6 + 1;
if (N[i] == 1) { kazu[1] = kazu[1] + 1; }
if (N[i] == 2) { kazu[2] = kazu[2] + 1; }
if (N[i] == 3) { kazu[3] = kazu[3] + 1; }
if (N[i] == 4) { kazu[4] = kazu[4] + 1; }
if (N[i] == 5) { kazu[5] = kazu[5] + 1; }
if (N[i] == 6) { kazu[6] = kazu[6] + 1; }
//if (N[i] == 7) { kazu[7] = kazu[7] + 1; } // ここいらない
};
for(int i=1;i<=6;i++){
cout << kazu[i] << endl;
}
return 0;
}

[yokuda]

アドバイスするとすれば、
* countが使われていないので不要
* 使わなくて済む変数はできるだけ作らない

このコードをもう少しスリムに書くとこんな感じになります。
#include <iostream>
#include <time.h>
#include <stdio.h>
using namespace std;

int main(void){
int count[7]={0};
srand((unsigned)time(NULL));
for(int i=0;i<50;i++){
count[rand()%6+1]++;
}
for(int i=1;i<=6;i++){
cout << count[i] << endl;
}
return 0;
}

[yokuda]

ちなみに、Visual Studioを使っているのであれば、デバッグでステップ実行が簡単にできるので、
一度、そこで挙動を見てみるのもおすすめですよ。