C++：For循环和向量

Question

我一直致力于 Codility 的 MaxCounters 任务。我被卡住了。不管我怎么尝试，下面的代码都不会循环

// you can also use includes, for example:
// #include <algorithm>

vector<int> solution(int N, vector<int> &A) 
{
    vector<int> counter (N,0);

    for (int i = 0;i < A.size();i++)
    {
        int value = A.at(i);

        if (value = A.size() + 1)
        {

        }
        else
            counter[value - 1] += 1;
    }

    return (counter);
}

我已使用此网站找到解决问题的方法，但无济于事。我不断收到此错误消息。

在 user.cpp:20 中包含的文件中：func.cpp：在函数'std::vector > solution(int, std::vector >&)'中：func.cpp:7：警告：比较 有符号和无符号整数表达式 func.cpp:11：警告：建议 赋值的括号用作真值

任何想法为什么它总是抛出这个错误？

编辑：从其他测试中查看后，很明显无论我做了什么练习，我都会得到这个错误。因此，我得出结论，我还没有理解循环向量。遍历向量的正确方法是什么？

Answer 1

正如编译器所建议的那样：

if (value = A.size() + 1)

正在赋值 A.size() + 1 的值，而不是评估它们的相等性

if (value == A.size() + 1)

可能是你想要的。

此外，size() 返回一个无符号整数，而不是 int，因此编译器对此提出了投诉。

Answer 2

您的程序应按以下方式更新

1. 这应该避免比较有符号和无符号整数之间的比较。

   for (int i = 0;i

到

   for (vector<int>::size_type i = 0;i < A.size();i++)

1. 当您的循环处理 size() 时，您应该使用 [] 运算符而不是 at 作为 在内部，它确实检查了越界访问。这是可以避免的。

   int 值 = A.at(i);

到

 int value = A[i];

1. 这实际上是一个主要问题，并且在您的代码中使用 = 而不是 ==。

   如果（值 = A.size() + 1）

到

   if (value == A.size() + 1)

Answer 3

您收到两个不同的警告。第一个与这个问题有关：

if( value = A.size() + 1)

...这只是一个错字，应该写成：

if( value == A.size() + 1 )

一旦你解决了，你可以继续下一个警告：

int 已签名。 A.size() 返回一个无符号值。您可以将for( int i = 0; i < A.size(); ++i) 更改为for( size_t i = 0....。

由于您存储的所有整数都可能是正数，因此请使用适当的容器类型（例如，无符号）。下面是它的外观：

vector<unsigned> solution(int N, vector<unsigned> &A) 
{
    vector<unsigned> counter (N,0);

    for (size_t i = 0;i < A.size();i++)
    {
        unsigned value = A.at(i);

        if ( value == A.size() + 1 )
        {

        }
        else
            counter[value - 1] += 1;
    }

    return (counter);
}

比较或分配一个无符号到一个有符号值得产生警告的原因是有符号容器和无符号容器的范围是不同的。