Question

我写了一个小程序，生成唯一的随机数。我首先使用我所知道的数组编写它来加载和打印数字。我试图用一个向量替换数组，所以如果我想复制列表，我可以更容易。我收到了一个错误。

  error: cannot convert 'std::vector<int>' to "std::vector<int>*' for argument '1' to bool numInList(std::vector<int>*, int)'

当我调用numInList函数时会发生此错误。

我是使用矢量的新手，但我认为你可以使用像数组这样的矢量，它具有内置函数的优点，没有固定的大小，并且能够将一个矢量复制到另一个矢量中。

这是我的代码：

#include <iostream>
#include <string>
#include <vector>
#include <fstream>
#include <cstdlib>
#include <ctime>


using namespace std;
bool numInList(vector <int> randNumbers[], int num);

const int length = 100;

int main()
{
int countCheck = 0;
vector <int> randNumbers(length);
vector <int> newlist();

srand(time(0));

 while(countCheck < length){
    int num = rand() % 90000 +10000;

    if (!numInList(randNumbers, num)){
        randNumbers[countCheck] = num;
        cout << "The Random Number  " << randNumbers[countCheck] << endl;
        countCheck++;
    }
 }

cout << "\n\n\n";
newlist[] = randNumbers[];

return 0;
}
bool numInList(vector<int> randNumbers[], int num){
for (int index = 0; index < length; index++){
    if (randNumbers[index] == num){
        return true;
    }
}
return false;
}

我尝试取消引用，希望能解决问题

  if (!numInList(&randNumbers, num))

然后我在函数numInList

中的IF语句上出错

 error: ISO C++ forbids comparisons between pointers and integer [f-permissive]

非常感谢任何帮助。

我已经改变了一些东西，现在我没有得到任何编译错误，但程序在执行时崩溃了...任何建议???

    #include <iostream>
    #include <string>
    #include <vector>
    #include <fstream>
    #include <cstdlib>
    #include <ctime>


    using namespace std;
    bool numInList(vector <int> randNumbers, int num);

    const int length = 100;

    int main()
    {
    int countCheck = 0;
    vector <int> randNumbers;
    vector <int> newlist;

    srand(time(0));

    while(countCheck < length){
        int num = rand() % 90000 +10000;

        if (!numInList(randNumbers, num)){
            randNumbers.push_back(num);
            cout << "The Random Number  " << randNumbers[countCheck] << endl;
            countCheck++;
        }
     }

     cout << "\n\n\n";
     newlist = randNumbers;

     return 0;
     }
    bool numInList(vector<int> randNumbers, int num){
    for (int index = 0; index < length; index++){
        if (randNumbers[index] == num){
            return true;
        }
    }
    return false;
   }

Answer 1

您应该通过引用传递向量（除非您要复制它）。如果您不打算修改矢量，请将其设为 const 参考：

bool numInList(const vector<int>& randNumbers, int num)

更多信息：How to pass objects to functions in C++?（这些是C ++的基础知识。正确使用它们。）

此外，您可以使用vector::size来获取向量中的元素数量。这种方式index永远不会超过向量的大小，无论向量的大小如何，您都不会有越界访问：

for (int index = 0; index < randNumbers.size(); index++)

在这里，我重写了整篇文章，展示了如何使用矢量。仔细阅读并确保您了解所有内容（包括为什么您的原始方法不是最佳设计选择）：

int main()
{
  const int length = 100;
  vector<int> randNumbers;
  vector<int> newList;

  srand(time(0));

  while(randNumbers.size() < length){
    int num = rand() % 90000 +10000;

    if (!numInList(randNumbers, num)){
      randNumbers.push_back(num);
      cout << "The Random Number  " << randNumbers.back() << endl;
    }
  }

  cout << "\n\n\n";
  newList = randNumbers;

  return 0;
}

bool numInList(const vector<int>& randNumbers, int num){
  for (int index = 0; index < randNumbers.size(); index++){
    if (randNumbers[index] == num){
      return true;
    }
  }
  return false;
}

Answer 2

numInList vector的参数randNumbers []相当于vector * randNumbers将参数更改为

bool numInList(vector<int> &randNumbers, int num)

使用向量而不是数组

2 个答案: