C++嘗試創建隨機排列的密碼陣列

Question

我對這個站點（和一般編程）相對較新，所以如果這篇文章像最後一篇那樣糟糕，請告訴我是否應該改變它，我會盡力做到這一點。

因此，我給出了一個陣列密碼中的50個密碼數組，我面臨的直接挑戰是必須隨機化他們的安排，隨機數字發生器的幫助。出於某種原因，使用我所擁有的代碼，我在某處犯了一些錯誤，導致代碼被卡在循環中或其他東西中。我知道我接近正確，但我不知道我犯了什麼錯誤。

我試着創建一個隨機數，檢查它是否已經在數組PasswordsTemp中聲明，然後:(如果它已經在數組中，什麼都不做），（如果它不在循環中，將它添加到下一個可用索引處的循環中）。爲了然後選擇另一個隨機數，該代碼必須循環。我將它設置爲循環600次（實際上）確保它在某個時刻猜測每個數字0到50。在此之後，我只是編寫了一個簡單的cout來查看數組的外觀。

任何幫助/意見，將不勝感激。請注意，我在這裏比較新，所以也許試着對我稍微容易一些，或者只是幫助其他人代替對我進行指責。謝謝。

#include <iostream> 
#include <string> 
#include <fstream> 
#include <time.h> 

int main() 
{ 
std::string firstname = ""; 
std::string lastname = ""; 
std::string lastnametemp = ""; 
std::string fullname = ""; 
std::string Users[50] = {""}; 
std::string Passwords[50] = {""}; 
std::string NewPass = {""}; 
std::string temp = {""}; 
int PasswordsTemp[50] = {60}; 
int randomNumber = 0; 
bool repeat = false; 
int A = 0; 
int i = 0; 

std::ifstream inputHandler; 
inputHandler.open("names.txt"); 

while (!inputHandler.eof())//ignore this part. I know it works. 
{ 
    inputHandler >> firstname >> lastname; //reads first name and last name of one line 
    lastnametemp = lastname.substr(0, 7); // sets temp last name as being at most, 7 letters long 
    fullname = firstname[0] + lastnametemp; // combines the two into a full name 
    Users[i] = fullname; //creates Users array 
    i = i + 1;//indexes data 
} 


std::ifstream indata; 
indata.open("passwords.txt"); 
i = 0; 
    while (!indata.eof()) 
{ 
    indata >> Passwords[i];//reads in passwords into array called Passwords 
    i = i + 1; 
} 
    inputHandler.close(); 


    //******************************************************* 
    //****************Confusion hereafter******************** 
    //******************************************************* 


    for (i = 0; i < 600; i++)//Loops a long time to (practically) ensure that all numbers 0 to 50 are guessed atleast once 
    { 
     srand(time(NULL)); 
     randomNumber = rand() % 51;//Produces random number between 0 and 50 

     for (i = 0; i < 51; i++) 
     { 
      if (randomNumber == PasswordsTemp[i]) 
      { 
       repeat = true;//determines if the random number appears anywhere in the PasswordsTemp array; repeat = true if it appears anywhere 
      } 
     } 

     if (repeat == false)//if the random number doesn't appear anywhere, assign the random number an index in Passwordstemp 
     { 
      PasswordsTemp[A] = randomNumber; 
      A = A + 1; 
     } 
    } 
    //***************************************************************************** 
    //**************************End of Confusion*********************************** 
    //***************************************************************************** 
    for (A = 0; A < 51; A++)//Checks to see if the passwords are randomized 
    { 
     temp = Passwords[PasswordsTemp[A]]; 
     std::cout << temp << std::endl; 
    } 

return 0; 
} 

Answer 1

我可以推薦一種不同的方法嗎？正如你所說，由於你運行了600次，所以不能保證每個號碼至少會被擊中一次。相反，使用一個while循環，上面寫着：雖然PasswordsTemp包含randomNumber，再次隨機化randomNumber：

while(PasswordsTemp.contains(randomNumber) 
{ 
    randomNumber = rand() % 51; 
} 

然後繼續在你的代碼已經行：

PasswordsTemp[A] = randomNumber; 
A = A + 1; 

，並把所有的，在從0到PasswordsTemp長度結束的for循環。此外，PasswordsTemp的長度是50，而不是51，所以嘗試訪問PasswordsTemp [51]將返回錯誤。

Answer 2

即不是容易的，但你可以存儲50首的RNG口角並將其分類到您的列表映射到它...

要明白，這裏的我的意思僞代碼：

for(i=1..50) 
    a[i] <— rng 
end for 
for(i=1..50) 
    b[i] <— a[i] 
end for 

//not being easy 
b <— sort(b) 

for(i=1..50) 
    for(j=1..50) 
     if(b[i]==a[j]) shuffled_list[i] <— original_list[j] 
    end for 
end for 

Answer 3

一個簡單的方法來製作你的隨機密碼的排列是：

1. 創建包含0陣列，以50
2. 申請std::random_shuffle or std::shuffle到數組。這將隨機重新排列元素。

全部完成。

特別獎金編輯：

while (!inputHandler.eof())//ignore this part. I know it works. 
{ 
    inputHandler >> firstname >> lastname; //reads first name and last name of one line 
    lastnametemp = lastname.substr(0, 7); // sets temp last name as being at most, 7 letters long 
    fullname = firstname[0] + lastnametemp; // combines the two into a full name 
    Users[i] = fullname; //creates Users array 
    i = i + 1;//indexes data 
} 

不能工作。它在從文件讀取數據之前測試EOF。直到嘗試讀取數據並失敗後，才能確定是否已到達文件末尾。讀取之前進行測試可確保您將讀取文件末尾並使用未讀垃圾。始終

1. 讀取數據
2. 您閱讀數據
3. 使用數據或退出，這取決於2

測試，以便

while (inputHandler >> firstname >> lastname)// read and test 
{ 
    // now use 
    lastnametemp = lastname.substr(0, 7); // sets temp last name as being at most, 7 letters long 
    fullname = firstname[0] + lastnametemp; // combines the two into a full name 
    Users[i] = fullname; //creates Users array 
    i = i + 1;//indexes data 
} 

還建議，所以你不要限制器對Users[i]數組和using std::getline讀取的數據太多，因此您可以使用「Victor Von Doom」 。