编写函数sumVector:不返回值

writing a function sumVector : not returning a value

本文关键字：返回值 sumVector 函数      更新时间：2023-10-16

我正在练习向量类上的函数。我已经编写了一个函数"sumVector"来将向量"ages"的所有元素相加，但它并没有返回总数我正在复制我的代码块；我知道STL提供了一个求和函数，但这是我自己练习的。如果有人能指出我做错了什么，我将不胜感激

    #include <vector>
    #include <fstream>
    #include <sstream>
    #include <iostream>
    #include <iterator>
    #include <algorithm>
    using namespace std;
    template<typename T>
    int sumVector(const vector<T> &integers2)
    {
         typename vector<T>::const_iterator constIterator;
         //display vector elements using constIterator
         for 
         (constIterator=integers2.begin();                                             
         constIterator!=integers2.end();++constIterator);
         int sum=0;
         sum+=*constIterator;
         return sum;
         cout<<sum;
    }


    int main ()
    {
    vector<int> ages;
    vector<float>var;
    ifstream inFile;
    inFile.open("data.txt");
    if(!inFile)
    {
        cout<<"didn't work";
        cin.get();
        exit (1);
    }
    string line;
    getline(inFile, line);
    istringstream in(line);
    copy(std::istream_iterator<int>(in),
      std::istream_iterator<int>(),
      std::back_inserter(ages));
    cout<<ages.at(2)<<endl;
    getline(inFile, line);
    istringstream gin(line);
    copy(istream_iterator<float>(gin),
                                     istream_iterator<float>(),
                                     back_inserter(var));
    cout<<var.at(0)<<endl;
    cout<<"whats the sum";
    sumVector (ages);
    cin.get();
    inFile.close();
    cin.get();
    }

for循环不正确，cout << sum出现在sumVector()的return之后。返回类型（和sum的类型）也应该是T，否则sumVector()被限制为int的元素的vectors（或可转换为ints的类型）。

我认为以下是你想要的东西：

template<typename T>
T sumVector(const vector<T> &integers2)
{
    typename vector<T>::const_iterator constIterator;
    T sum=0;
    //display vector elements using constIterator
    for (constIterator=integers2.begin();
         constIterator!=integers2.end();
         ++constIterator)
    {
        sum+=*constIterator;
    }
    cout<<sum;
    return sum;
}

考虑不在sumVector()内打印sum的值作为（IMO），这更清楚：

cout << "whats the sum: " << sumVector(ages);

还可能需要将sumVector()的结果存储在变量中，并且在计算总和时不向标准输出写入任何内容。

for后面的"；"是错误的——按照您的编写方式，分号将是for循环的主体。

我没有触及你的命名（例如整数2有点刺激）。

template<typename T>
int sumVector(const vector<T> &integers2)
{
     int sum=0;
     typename vector<T>::const_iterator constIterator;
     //display vector elements using constIterator
     for (constIterator=integers2.begin();constIterator!=integers2.end();++constIterator) {
         sum+=*constIterator;
     }
     cout<<sum;
     return sum;
}

我还重新订购了

  • 和的打印和返回，因为返回端是函数的执行
  • i=0的初始化也必须在循环之前完成
相关文章：