函数调用中的参数太多。 C++答案

【问题标题】：Too many arguments in function call. c++函数调用中的参数太多。 C++
【发布时间】：2020-01-03 15:19:28
【问题描述】：

所以我的目标是将这个向量 vector 'Beer' allBeers 传递给函数 UnitedStatesBeer::getBeerTop()，但是当我尝试这样做时，我得到了函数调用中参数过多或我的对象未初始化的错误。

我该如何解决这个问题？感谢您的帮助！

#include <iostream>
#include <fstream>
#include <string>
#include <iomanip>
#include <vector>

using namespace std;

class RatingCalculator
{
public:
     virtual void showCountryTop() = 0;
     virtual void getCountryTop() {};
};

class Beer
{
public:
    string name;
    string rating;
    string country;
    string alc;
    string type;
};

class UnitedStatesBeer : public RatingCalculator
{
private:
    string name;
    string rating;
    string country;
    string alc;
    string type;
public:
    void showCountryTop() {};
    void getCountryTop(vector<Beer> allBeers){};

int main()
{
        ifstream file("beer.txt");
        Beer currentBeer;
        vector<Beer> allBeers;

        for (int i = 0; !file.eof(); i++)
        {
            getline(file, currentBeer.name, '\t');
            getline(file, currentBeer.rating, '\t');
            getline(file, currentBeer.country, '\t');
            getline(file, currentBeer.alc, '\t');
            getline(file, currentBeer.type, '\n');

            allBeers.push_back(currentBeer);    //copy all the information to allBeers vector 
        }
        file.close();

        /*if I do it this way*/
        UnitedStatesBeer UsReassign;          
        UsReassign->getCountryTop(allBeers);   //<- expression (UsReassign) must have pointer type/ Using uninitialized memory

    //  RatingCalculator* UsReassign = new UnitedStatesBeer();
    //  UsReassign-> getCountryTop(allBeers);  //<- too many arguments in function call


    }

【问题讨论】：

请创建一个minimal reproducible example 向我们展示，我们可以复制粘贴并自行尝试，无需以任何方式编辑或修改它。另外，请将完整且完整的错误输出复制粘贴到问题中（作为文本），因为通常会有有用的信息说明可以为问题提供提示。
一个提示：要覆盖虚函数，函数签名（例如参数）必须与基类中的完全相同。
Why is iostream::eof inside a loop condition considered wrong?
为什么UnitedStatesBeer 有country 属性？我希望看到class UnitedStatesBeer : public Beer，因为来自美国的啤酒是一种啤酒（嗯，至少有一部分是），而不是一种计算器。
请注意，您没有正确覆盖您的虚拟。 RatingCalculator::getCountryTop() 与 UnitedStatesBeer::getCountryTop(vector<Beer> allBeers).

标签： c++ function class object vector

【解决方案1】：

首先，由于您要覆盖虚函数，因此您需要将基类中的函数定义（即RatingCalculator）与UnitedStatesBeer 中的函数定义相匹配。这就是它在RatingCalculator 中的样子：

  virtual void getCountryTop(vector<Beer> allBeers) {};

在UnitedStatesBeer

 void getCountryTop(vector<Beer> allBeers){};

其次，既然你想获得顶级啤酒，你可能想通过引用传递，在变量beers前面添加&。

第三， UsReassign->getCountryTop(allBeers); //<- expression (UsReassign) must have pointer type/ Using uninitialized memory.. 这是因为 UsReassign 是一个对象而不是一个指针。对象使用. 而不是-> 引用。

最后， // UsReassign-> getCountryTop(allBeers); //<- too many arguments in function call 发生这种情况是因为正在调用没有参数的函数getCountryTop()。 “首先”中提到的步骤应该可以解决它。

【讨论】：

在编写 RatingCalculator::getCountryTop(vector allBeers) 后，出现错误：C2065: 'Beer': undeclared identifier
和 C2923: 'std::vector': 'Beer' 不是参数 '_Ty' 的有效模板类型参数
@tendor 尝试将Beer 类放在RatingCalculator 之前