Question

我一直在尝试让我的c ++程序检查数组中的重复项，因为我需要这个用于我正在编写的程序。但是，它不起作用，我想我知道问题是怎么回事，但我不知道如何修复它。

#include <iostream>
#include <cstdlib>
#include <ctime>
#include <algorithm>

using namespace std;

int asknum(int ask);
int check(int checking);

bool error = false;

int test [10] = {};

int i = 0;

int gen = 0;

int main() {
    //int gen = 0;

    for (i=0; i<10; i++) {
        gen = asknum(gen);
    }
    for (int b=0; b<10; b++) {
        cout << test [b] << endl;
    }

    return 0;
}

int asknum(int ask) {
    ask = 0;
    gen = 0;
    int nouse;
    cout << "Please enter a value: ";
    cin >> ask;
    error = 0;
    nouse = check(ask);
}

int check(int checking) {
    int n = 0;
    for (n=0; n<10; n++) {
        int temp = test [n];
        if (temp == checking) {
            cout << "Value is the same. Enter another value.\n";
            error = 1;
            gen = asknum(gen);
        }

    }

    if (n == 10) {

        if (error == 0) {
            gen = 0;
            test [i] = checking;
        }
    }   
}

我的代码出了什么问题？我第一次编写代码时效果很好。但是当我注意到一个缺陷时，如果你经常输入一个当前的数字，它会停止要求一个新的数字，我决定将代码移动到函数中，但现在它无法正常工作！这是输出：

Please enter a number: 1
Please enter a number: 2
Please enter a number: 3
Please enter a number: 3
That number already is used. Please type in another value.
Please enter a number: 4
Please enter a number: 5
Please enter a number: 6
Please enter a number: 7
Please enter a number: 8
Please enter a number: 9
Please enter a number: 10
1
2
3
3
5
6
7
8
9
10

我认为沟通有问题。请帮忙！

谢谢！

Answer 1

在添加副本3之后，程序从gen = asknum(gen);继续，并在函数asknum(int ask)中将错误分配为等于0。这就是它传递if条件以在数组中分配ith元素的原因。从error=0移除asknum(int ask)，然后将其放在check

中的条件之后

编辑：检查上面的解决方案仍然提供相同的输出，但由于在asknum内调用asknum，错误值相同。使其工作更改为主要的cin循环：

for (i=0; i<10; i++) {

    gen = asknum(gen);
      if(error==1)
      {
          i--;
          error=0;
      }
    }

并查看：

int check(int checking) {
    int n = 0;
    for (n=0; n<10; n++) {
        int temp = test [n];
        if (temp == checking) {
            cout << "Value is the same. Enter another value.\n";
            error = 1;
            //fflush(stdout);
            //cin.clear();
            //gen = asknum(gen);
        }

    }
    //cout<<error<<"\n";

        if (error == 0) {
            gen = 0;
            test [i] = checking;
            cout<<checking<<" "<<i<<"\n";
        }
    //error=0;
}

请检查这是否有帮助。

Answer 2

Okey，所以你要做的是询问用户一个整数，如果容器中不存在该整数，你想将它添加到容器中。

在c ++中，我建议避免使用c风格的数组，而是使用其中一个c ++容器，例如std::set。

那么参考页面告诉我们有关std::set的内容是什么？

集合是按特定订单存储唯一元素的容器。

听起来就像我们想要的！

最大的优点是std::set具有内置功能，例如，根据元素的值查找现有元素。

这看起来如何：

void getNumberFromUser(set<int>& numbers);

int main()
{
    set<int> numbers;

    for (int i = 0; i < 10; ++i) {
        getNumberFromUser(numbers);
    }

    for (auto& value : numbers) {
        cout << value << endl;
    }

    return 0;
}


void getNumberFromUser(set<int>& numbers)
{
    int value;

    cout << "Please enter a value: ";
    cin >> value;

    auto result = numbers.insert(value);

    while (!result.second()) {
        cout << "That number is already used. Please type in another value.\n";
        cout << "Please enter a value: ";
        cin >> value;

        result = numbers.insert(value);
    }
}

自set::insert

以来

返回一对，其成员对:: first设置为指向新插入元素或集合中已有的等效元素的迭代器。如果插入了新元素，则对中的pair :: second元素设置为true;如果已存在等效元素，则设置为false。

我们要做的就是检查返回对的第二个值是true还是false，以确定该值是否已被使用。

我希望这有点帮助，不要害怕看c++ reference pages，它们并不像你想象的那么可怕！

如何确保数组中没有重复项？

2 个答案: