Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

No Matching Function Call to Base class

Here I am trying to make a subclass of the base class, Airplane. In my main code I haven't tried to use either constructor yet as I am just trying to make sure I can make the subclass Fighter to properly work.

The exact error its giving me is

no matching function for call to 'Airplane:Airplane()'

And says it pertains to this line of code in the Fighter.cpp

Fighter::Fighter(int engi, int seat, string description)

Fighter.cpp

#include "Fighter.h"
Fighter::Fighter(int engin, int seat, string description)
{
    fNumEngines = engi;
    fNumSeats = seat;
    rangeAndSpeedDesc = description;
}

Fighter.h

#include "Airplane.h"

using namespace std;

#ifndef FIGHTER_H_
#define FIGHTER_H_

class Fighter:public Airplane {
private:
    int fNumSeats;
    int fNumEngines;
    string rangeAndSpeedDesc;
}

Airplane.cpp

#include "Airplane.h"

using namespace std;
Airplane::Airplane(int engines, int seats)
    {
        numSeats = seats;
        numEngines = engines;
    }
like image 234
4Percent Avatar asked Oct 30 '15 17:10

4Percent


2 Answers

Fighter::Fighter(int engines, int seats, string desc)
{
    fNumEngines = engines;
    fNumSeats = seats;
    rangeSpeed = desc;
}

is equivalent to:

Fighter::Fighter(int engines, int seats, string desc) : Airplane()
{
    fNumEngines = engines;
    fNumSeats = seats;
    rangeSpeed = desc;
}

The base class object is initialized using the default constructor unless another constructor is used to initialize the base class in the initialization list in the constructor implementation.

That's why the compiler cannot compile that function.

You need to:

  1. Add a default constructor to Airplane, or
  2. Use the available constructor of Airplane in the the initialization list.

Looking at your posted code, option 2 is going to work.

Fighter::Fighter(int engines, int seats, string desc) :
        Airplane(engines, seats), rangeSpeed(desc)
{
}

Suggestion for cleanup

I don't see why you need the members fNumEngines and fNumSeats in Fighter. The base class already has the members to capture that information. I suggest that you should remove them.

like image 116
R Sahu Avatar answered Oct 28 '22 20:10

R Sahu


When this constructor is called

Fighter::Fighter(int engines, int seats, string desc)
{
    fNumEngines = engines;
    fNumSeats = seats;
    rangeSpeed = desc;
}

then it calls the default base class constructor. However class Airplane does not have the default constructor. It has a constructor with parameters

Airplane(int, int);

So you need explicitly calll the constructor in the mem-initializer list of the constructor Fighter

For example

Fighter::Fighter(int engines, int seats, string desc) : Airplane( engines, seats )
{
    fNumEngines = engines;
    fNumSeats = seats;
    rangeSpeed = desc;
}

Also it is not clear why the data members of the base class and the derived class are duplicated.

like image 1
Vlad from Moscow Avatar answered Oct 28 '22 19:10

Vlad from Moscow