0

我试图hours通过遍历我的向量来获取所有变量的总和。现在我收到这个矢量订阅错误,我不知道如何解决它。这是我的代码:

int hulpInt = hours[0];

for (int i = hours[1]; i != hours[hours.size()-1]; i++)
{
    hulpInt = hulpInt + i;
}

int HOUR = hulpInt + hours[hours.size()];

如果您需要更多代码,请发表评论。

4

3 回答 3

4

解决这个问题的两种方法。

#include <functional>
#include <numeric>
#include <vector>

方法一:使用vector.size()

int hulpInt =0;
for(int i=0;i<hours.size();i++){
    hulpInt+= hours[i];
}

方法 2:使用 std::accumulate

int hulpInt = std::accumulate(hours.begin(),hours.end(),0);
于 2012-06-14T19:55:00.810 回答
3

请参阅本文档

你应该这样做:

int sum = 0;

for (int i = 0; i < hours.size(); ++i) {
    sum += hours[i];
}
于 2012-06-14T19:52:18.303 回答
2

当您遇到错误时,最好也将其包括在内。

具体来说,我猜您的索引超出了范围,hours[hours.size()]因为向量是基于零的索引。

STL 容器有漂亮的迭代器,有助于保持界限:

int h[] = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12};
std::vector<int> hours(h, h+12);
int total(0);
for (std::vector<int>::const_iterator i(hours.begin()); i != hours.end(); ++i)
    total += *i;

STL 也有使用这些迭代器为你做常见事情的算法:

#include <numeric>

int h[] = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12};
std::vector<int> hours(h, h+12);
int total = std::accumulate(hours.begin(), hours.end(), 0);

当然你也可以滚动你自己的循环:

int h[] = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12};
std::vector<int> hours(h, h+12);
int total(0);
for (int i(0); i < int(hours.size()); ++i)
    total += hours[i];

以上所有这些都得出了相同的结果,它们只是以稍微不同的方式进行。正如他们所说,给猫剥皮的方法总是不止一种。

于 2012-06-14T19:55:01.810 回答