创建一个求和函数，仅对向量的一部分求和

Question

显然我需要一个求和函数，而accumulate不会削减它

我需要创建程序-一个向量-用户可以指定n个元素-并且求和函数只能对正元素求和，即使用户可以输入负元素还有......

在computeSum函数中，我还需要向整个组添加一个“成功”

computeSum (dataVec, howMany, total, sucess);

，并为输入的人创建一个参数 - 所有负数，但想要对它们求和，但无法，因为没有正数

if (success) {
   cout << "The sum is " << total << endl;
}
else {
cerr << "Oops, you cannot add these elements.";
}

所以这是我得到的

#include <iostream>
#include <vector>        // need this in order to use vectors in the program
using namespace std;

int main()
{
    vector<double> dataVec;

    double i, n, howMany, total;
    cout << "How many numbers would you like to put into the vector?";
    cin >> n; 

    dataVec.resize(n);

    for(vector<double>::size_type i=0;i < n;i++)
    {
        cout << "Enter the numbers: \n";
        cin >> dataVec[i];
    }

    cout << "How many POSITIVE numbers would you like to sum?";
    cin >> howMany;
    cout << computeSum (dataVec, howMany, total);
}

 double computeSum (vector<double> &Vec, howMany, total)
 {
      double total =0;
      for(int i=0;i < howMany;i++)
    total+=Vec[i];
      return total;
 }

我似乎也遇到了麻烦编译这个 -computeSum() 在 int main() 中不被理解； computerSum() 中未理解 howMany；在全球范围内，total() 和 howMany() 未声明（我想这意味着我需要全局声明？？？）

Answer 1

您的 computeSum() 函数必须出现在源文件中的 main() 函数上方，才能使其处于作用域内。另外，在您的computeSum()函数签名中，您没有为howMany和total变量指定类型。我猜它们应该是双倍howMany和双倍总计？

Answer 2

事实上，accumulate会“削减它”，使用只考虑正值的适当函子：

int sum_positive(int first, int second) {
    return first + (second > 0 ? second : 0);
}

…

std::accumulate(data.begin(), data.begin() + how_many, 0, sum_positive);

Answer 3

骑上我的爱好马：Boost Range 适配器。与我达到最佳点

#include <boost/range/adaptors.hpp>
#include <boost/range/numeric.hpp>

bool isnatural(int i) { return i>=0; }
using namespace boost::adaptors;

int main(int argc, char** args)
{
    static const int data[] = { -130, -1543, 4018, 5542, -4389, 15266, };

    std::cout << "sum: " << boost::accumulate(data | filtered(isnatural), 0) << std::endl;

    return 0;
}

输出：

总和：24826

---

使用 C++11 awesomeness¹ spice：

std::cout << "sum: " << boost::accumulate(data 
        | filtered([] (int i) { return i>=0; }), 0) << std::endl;

¹：说实话，我真的很讨厌lambda语法的笨拙：

• 必须始终指定参数类型
• 必须将 return 语句拼出
  对于这种情况，似乎
  filtered([] (i) { i>=0 })
  可以被编译器算出来。好吧，也许在 c++22 中:)