0〜49の範囲の1000個のランダムな整数を生成するプログラムを作成します。 （C++）

Question

私はここで質問された他の質問を見回しましたが、私が必要とするものを見つけることができません。

この質問があります。

0-49の範囲で1000個のランダムな整数を生成するプログラムを作成します。それは、どのくらいの数が0-24の範囲にあり、いくつが25-49の範囲に含まれているかを数えて報告する必要があります。

これはこれまでのところです。

#include<iostream> 
#include<ctime> //For time() 
#include<cstdlib> //For rand() and srand() 


using namespace std; 
int main() 
{ 
    int x; 
    int counter = 0; 
    int y = 0; 
    int z = 0; 

    srand(time(NULL)); 

    for (counter = 0; counter < 1000; counter++) { 
    { 
     x = rand() % 49 + 1; 
     if (x >= 0 && x <= 24) 
     { 
      y++; 
     } 
     else if (x > 24 && x <= 49) 
     { 
      z++; 
     } 
    } 

    cout << "Number of numbers between 0-24: " << y << endl; 
    cout << "Number of numbers between 25-49: " << z << endl; 

    system("pause"); 
    return 0; 
    } 
} 

かつてはループが、私は1000の数字が生成され、分類されるまで何度もループにそれを取得する方法を理解することはできません。

私はC++には新しく、誰かがこれをループする方法を説明することができましたか？私は何かを見逃したことがありますか？

Answer 1

あなたのforループに多すぎる中括弧が1つあります。あなたのreturnとcoutのループは、forループ内で起こっています。それは1つの数字を取得し、プリントアウトしてから戻ってくる前に返されます。これを試してください：あなたのコードについて

#include<iostream> 
#include<ctime> //For time() 
#include<cstdlib> //For rand() and srand() 

using namespace std; 
int main() 
{  
    srand(time(NULL)); 

    for (counter = 0; counter < 1000; counter++) 
    { 
     int x;   //See Note #3 Below 
     int counter = 0; 
     int y = 0; 
     int z = 0; 
     x = rand() % 50; //See Note #1 Below 
     if (x < 25)  //See Note #2 Below 
     { 
      y++; 
     } 
     else 
     { 
      z++; 
     } 
    } 

    cout << "Number of numbers between 0-24: " << y << endl; 
    cout << "Number of numbers between 25-49: " << z << endl; 

    system("pause"); 
    return 0; 
} 

3ノート：

1. あなたの乱数生成器のみを取得している数字1-49、0-49ありません。
2. if文をそのように構造化する必要はありません。あなたがしなければならないことは、「25未満の場合は1つのバケットに入れ、それ以外の場合はそれを別のバケットに入れてください」と言うだけです。
3. 変数は、使用されるスコープで宣言することができます。この場合、ループはforです。