當輸入新變量時，C++ For-Loop會卡住

Question

我是C++的新手，並試圖爲大學項目創建一個彩票遊戲。

我有一個for循環來檢查輸入數組中沒有重複的數字。當你拿出代碼段來產生隨機數時，這種方法非常好。

只要我將隨機數部分添加回來，for循環就會卡住。它會不斷告訴我，當它試圖存儲第一個號碼時，我已經輸入了該號碼。

我附上了我所有的代碼，如果你不需要這些，請致歉。

#include <iostream> 
#include <stdio.h> 
#include <time.h> 
#include <stdlib.h> 

using namespace std; 
//int loto[6]; 
int main() 
{ 
int numbers[6]; 
//void rand_num(int loto[6], int count); 
int loto[6]; //used to store the loto numbers 
//int james = 0; 
//int l,j; //used in checking any duplicated 

    srand(time(0)); 

     for(int count=0; count<6; count++) 
      { 
      loto[count] = (rand()%49)+1; 

      cout << loto[count] << endl; 
      } 



//declares the variable i to increase each time a number is entered. 
//this will only go as high as 6 
for(int i=0;i<6;i++) 
{ 
    cout<<" " << i<<" : Please enter your lottery numbers: "<<endl; 
    cin>>numbers[i]; 
     if ((numbers[i] >= 50) | (numbers[i] == 0)) 
     do 
     { 
      { 
      //checks to see if the first number entered is above 50 or = to 0 and rejects it 
      cout << "The Number must be between 1-49, please select again. " << endl; 
      cin >> numbers[i]; 
      } 
     } 
     while ((numbers[i] >= 50) | (numbers[i] == 0)); 

//---------------------------------------------------------------------------- 

//this section of code is a loop within a loop to check the number entered against all numbers already stored. 
//makes l the same as i effectively 
for(int l=0;l<6;l++) 
{ 
    //makes j one more than l 
    for(int j=l+1;j<7;j++) 
    { 
    if(numbers[l] == numbers[j]) 
     do 
     { 
      { 
      cout << "Number has already been chosen, please re-enter number " << endl; 
      cout << endl; 
      cin >>numbers[i]; 
       //checks the number that is re-entered is not <50 or = 0 
       //if so it rejects it and asks for another as above. 
       if ((numbers[i] >= 50) | (numbers[i] == 0)) 
        do 
        { 
         { 
         cout << "The Number must be between 1-49, please select again. " << endl; 
         cin >> numbers[i]; 
         } 
        } 
        while ((numbers[i] >= 50) | (numbers[i] == 0)); 
      } 
     } 
     while (numbers[l] == numbers[j]); 
    } 
} 
} 

//---------------------------------------------------------------------------- 

//this displays the numbers that have been chosen. 
cout << "Your Numbers are: " << endl; 
for (int i = 0; i<6; i++) 
{ 
    cout << " " << numbers[i]; 
} 

return 0; 
} 

Answer 1

我不確定這是真正的問題，但它是一個錯誤。嘗試糾正它，看看它是否有幫助。

for(int l=0;l<6;l++) 
{ 
    //makes j one more than l 
    for(int j=l+1;j<7;j++) 
    { 
     if(numbers[l] == numbers[j]) 

內循環將達到j == 6，因此您將訪問數組之外​​。外環應該有5個作爲極限，而內環應該有6個作爲極限。

編輯： 在查看了一下你的代碼後，我可以看到你正在使用數字[]而沒有初始化它。兩個嵌套for循環將比較數字中的所有元素。但是，如果用戶只輸入了2個數字，其餘的就會被單元化並可能導致意想不到的結果。

此外 - 你不需要每次都檢查所有元素。只需檢查新輸入的號碼（由i指數）和以前的所有號碼。

最後你可能會需要這樣的東西：

if (!(cin >> numbers[i])) { 
    cout << "Please enter numbers only." << endl; 
    cin.clear(); 
    cin.ignore(10000,'\n'); 
} 

處理的輸入不是整數，例如「text」

對於次要的事情：

您還應該檢查負數。

您正在用途|而不是||。它會正常工作，但||似乎更正確，因爲它是邏輯OR（while |是一個二進制OR）。