模板 C++ 错误：无法推断模板参数

Question

我正在尝试添加一个函数模板，如果它包含精度值或阀门和值，它将打印。除此功能外，该程序的其余部分均有效。我不确定我做错了什么，但我收到的错误是：

错误 C2784：“void printInstrumentDetail(const I *const)”：无法从“std::vector<_ty>”推导出“const I *const”的模板参数

#include <iostream>
#include <vector>
#include <iomanip>
#include <string>
#include "Instruments.h"
#include "Brass.h"
#include "Strings.h"

using namespace std;

//template<typename I>                    <---Problem
//void printInstrumentDetail(const I * const a)
//{
//      for (size_t i = 0; i < 6; i ++)
//  {
//      cout << "The details for " << a[i]->getName()
//          << ": " << a[i]->print();
//  }
//}
int main()
{
    double total = 0;

    Strings violin("Violin", 553.90, 3);
    Strings cello("Cello", 876.45, 3);
    Strings viola("Viola", 200.50, 23);
    Brass tuba("Tuba", 1400.10, 1.23);
    Brass trumpet("Trumpet", 500.00, 4.32);
    Brass sax("Sax", 674.78, .99);

    vector <Instruments *> band(6);

    band[0] = &violin;
    band[1] = &tuba;
    band[2] = &cello;
    band[3] = &trumpet;
    band[4] = &viola;
    band[5] = &sax;

    cout << fixed << setprecision(2);

    cout << "The instruments in the band are:\n";
    //Get name and cost of each
    for (size_t i = 0; i < 6; i ++)
    {
        cout << band[i]->getName() << "     $" 
            << band[i]->getCost() << endl;
    }

    cout << "\nThen band is warming up..." << endl;
    //Get descrition of how sound is made of each
    for (size_t i = 0; i < 6; i ++)
    {
        cout << "This " << band[i]->getName()
            << " makes sounds by " ;
        band[i]->playSound();
    }
    cout << "\nTotal cost of the band is: $" ;
    //Get total cost of all instruments
    for (size_t i = 0; i < 6; i ++)
    {

        total = band[i]->getCost() + total;
    }
    cout << total << endl;

    //printInstrumentDetail(band);                     <--Problem

    return 0;
}

这是基类：

#ifndef INSTRUMENTS_H
#define INSTRUMENTS_H

#include <string>

using namespace std;

class Instruments
{
public:
    Instruments(string, double);

    void setName(string);
    virtual string getName();

    void setCost(double);
    virtual double getCost();

    virtual void print();

    virtual void playSound();

private:
    string name;
    double cost;
};
#endif

#include <iostream>
#include "Instruments.h"

using namespace std;

Instruments::Instruments(string n, double c)
{
    name = n;
    cost = c;
}
void Instruments::setName(string n)
{
    name = n;
}
string Instruments::getName()
{
    return name;
}
void Instruments::setCost(double c)
{
    cost = c;
}
double Instruments::getCost()
{
    return cost;
}
void Instruments::print() 
{

}
void Instruments::playSound()
{
    //empty
}

派生类低音：

#ifndef BRASS_H
#define BRASS_H

#include <string>
#include "Instruments.h"

using namespace std;

class Brass : public Instruments
{
public:
    Brass(string, double, double);

    void setPrecisionValue(double);
    double getPrecisionValue();
    void print() ;
     void playSound();

private:
    double precision;
    string sound;
};
#endif

#include <iostream>
#include "Brass.h"

using namespace std;

Brass::Brass(string n, double c, double p)
:Instruments(n, c)
{
    precision = p;
}
void Brass::setPrecisionValue(double p)
{
    precision = p;
}
double Brass::getPrecisionValue()
{
    return precision;
}
void Brass::print() 
{
    cout << getPrecisionValue() << endl;
}
void Brass::playSound()
{
    cout << "blowing in a mouthpiece." << endl;
    Instruments::playSound();
}

派生类字符串：

#ifndef STRINGS_H
#define STRINGS_H

#include <string>
#include "Instruments.h"

using namespace std;

class Strings : public Instruments
{
public:
    Strings(string, double, int);

    void setValves(int);
    int getValves();
    void print();
    void playSound();

private:
    int valves;
};
#endif

#include <iostream>
#include "Strings.h"

using namespace std;

Strings::Strings(string n, double c, int v)
    :Instruments(n, c)
{
    valves = v; 
}
void Strings::setValves(int v)
{
    valves = v;
}
int Strings::getValves()
{
    return valves;
}
void Strings::print() 
{
    cout<< getValves() << endl;
}
void Strings::playSound()
{
    cout << "striking with a bow." << endl;
    Instruments::playSound();
}

Answer 1

好吧，首先我不相信你可以将向量作为 const * I const at 传递

printInstrumentDetail(band);

Answer 2

将指针传递给向量

printInstrumentDetail(&band);   

在printInstrumentDetail内部

(*a)[i]->getName();

Answer 3

嗯，问题是你的模板需要一个指针：

template<typename I>   
void printInstrumentDetail(const I * const a);

但你给它的是一个向量，而不是一个指针：

vector <Instruments *> band(6);
...
printInstrumentDetail(band);

您可以通过将指针传递给printInstrumentDetail 函数来解决这个问题，如下所示：

printInstrumentDetail(&band[0]);

但实际上，您最好修改 printInstrumentDetail 以获取一个容器或一对迭代器：

template <typename ContainerT>   
void printInstrumentDetail(const ContainerT& a)

或

template <typename IteratorT>
void printInstrumentDetail(IteratorT first, IteratorT last)

对函数的定义进行适当的修改。

Answer 4

Vector 不能只转换为指针。一种可行的解决方案是：

template <typename T>
void printInstrumentDetail( const std::vector<T*>& band )
{
    for ( size_t i = 0; i < band.size(); ++i )
        cout << "The details for " << band[i]->getName()
             << ": " << band[i]->print();
}

还有很多其他的，包括迭代器、函子、STL 算法等。

Answer 5

您正在尝试将对象传递给需要指针的接口。

void printInstrumentDetail(const I * const a)

将此转换为参考。

void printInstrumentDetail(I const I& a)

但要符合 C++ 中常见的模式。您应该将序列的开头和结尾作为参数传递。即更改您的函数以采用迭代器而不是指针。

Answer 6

而不是传递指针：

printInstrumentDetail(const I * const a)  

你可以通过参考：

printInstrumentDetail(const I& a)

其他一切都保持不变。

Answer 7

首先，PrintInstrumentDetail 似乎根本没有理由成为模板——它适用于指向基类的指针，除非您可能有其他类型的 getName() 和 print()可能应用它的成员，它可以/可以/应该只使用指向基类的指针。

其次，我会认真考虑改变你的工作方式。而不是每个 Instrument 中的成员函数，以及 PrintInstrumentDetail 循环遍历所有工具，我会认真考虑为 Instrument 定义 operator<<，并使用标准算法打印出详细信息。

看着它，我认为其他一些事情也应该改变。首先，除非你正在处理非常不寻常的乐器，否则铜管乐器上的阀门数量是永远固定的——所以它不应该有SetValve() 成员。相反，阀门的数量应该在施工时设定，而不是开放后改变。

弦乐器根本没有阀门（至少大多数普通乐器没有），所以它们不应该有 SetValves()、GetValves() 或任何与阀门相关的东西。

同样，除非您在做一些非常不寻常的事情，否则仪器的成本永远不会改变 - 您支付了您所支付的费用，因此应该在施工期间确定成本，而不是以后更改。

编辑：另一件事：不要到处硬编码6，而是使用band.size() 循环遍历乐队中的所有乐器。