Problema em chamar construtor de classe base na herança

AlvimIgor
(usa Ubuntu)

Enviado em 10/09/2017 - 02:34h

Pessoal, estou pagando estruturas de dados na faculdade em C++ e essa linguagem é muito louca >.<
Estou tentando chamar o construtor da classe mãe no construtor das classes herdeiras, mas aparece o seguinte erro:


Aí vai o código:
veiculo.h

#ifndef VEICULO_H_
#define VEICULO_H_
#include <iostream>
using namespace std;

class Veiculo {

protected:
	string nome;

public:

	Veiculo(const char *nome) {
		this->nome = string(nome);
		cout << "Criação de Veículo" << nome << endl;
	}

	~Veiculo(){
		cout << "Destruição de Veículo" << nome << endl;
	}

};

class Terrestre : public Veiculo {
public:

	Terrestre() : Veiculo(nome){
		this->nome = Veiculo::nome;
		cout << "Criação de Terrestre" << nome << endl;
	};

	~Terrestre() : Veiculo() {
		cout << "Destruição de Terrestre" << nome << endl;
	}
};

class Aquatico : public Veiculo {
public:

	Aquatico() : Veiculo(nome) {
		this->nome = Veiculo::nome;
		cout << "Criação de Aquatico" << nome << endl;
	};

	~Aquatico() {
		cout << "Destruição de Aquatico" << nome << endl;
	}

};

class Aereo : public Veiculo {
public:

	Aereo() : Veiculo(nome) {
		this->nome = Veiculo::nome;
		cout << "Criação de Aereo" << nome << endl;

	};

	~Aereo() {
		cout << "Destruição de Aereo" << nome << endl;
	}

};


#endif /* VEICULO_H_ */
 

principal.cpp:

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

using namespace std;

int main() {
	cout << "Segunda pratica de AED em C++" << endl;

	Veiculo v1("v1");
	Terrestre t1("t1");
	Aquatico aq1("aq1");
	Aereo ar1("ar1");
}
 

phoemur
(usa Debian)

Enviado em 10/09/2017 - 20:17h

Seu veículo aceita const char* (string literal), porém você está chamando com std::string...
Essa conversão implícita não funciona sempre em duas vias...

dito issso:

Para funcionar, sugiro trocar tudo por std::string que é mais difícil dar tiro no pé.
Consertei alguns erros de digitação tb.

#ifndef VEICULO_H_
#define VEICULO_H_

#include <iostream>
#include <string>
using namespace std;

class Veiculo {
protected:
        string nome;

public:
        Veiculo(const string& arg) 
            : nome{arg}
        {
                cout << "Criação de Veículo " << nome << endl;
        }

        ~Veiculo()
        {
                cout << "Destruição de Veículo " << nome << endl;
        }
};

class Terrestre : public Veiculo {
public:
        Terrestre(const string& arg)
            : Veiculo(arg)
        {
                cout << "Criação de Terrestre " << nome << endl;
        }

        ~Terrestre()
        {
                cout << "Destruição de Terrestre " << nome << endl;
        }
};

class Aquatico : public Veiculo {
public:
        Aquatico(const string& arg)
            : Veiculo(arg)
        {
                cout << "Criação de Aquatico " << nome << endl;
        }

        ~Aquatico()
        {
                cout << "Destruição de Aquatico " << nome << endl;
        }
};

class Aereo : public Veiculo {
public:
        Aereo(const string& arg)
            : Veiculo(arg)
        {
                cout << "Criação de Aereo " << nome << endl;
        }

        ~Aereo()
        {
                cout << "Destruição de Aereo " << nome << endl;
        }
};

#endif /* VEICULO_H_ */
 

Se o construtor da classe base não aceitasse nenhum argumento (default constructor), o c++ chamaria ele sozinho pra você, porém como você precisa chamar com o nome (uma std::string), você precisa chamar explicitamente assim como no meu exemplo.

Outra coisa a se lembrar é que não é boa prática de programação colocar

 using namespace std;  

em um header file, pois pode "contaminar" o namespace global... Deixa o cliente decidir como quer fazer.

Outra coisa é que como você definiu um destructor, o compilador não vai mais gerar automaticamente os construtores que geralmente faz (copy constructor, copy assignment, move constructor, move assignment, destructor).
Se você definir algum desses recomenda-se que defina todos os outros, nem que seja um =default;
Mas isso é mais pra frente, a depender do comportamento que você quer pra sua classe.

Outra coisa é que o this->nome é desnecessário pois a classe já herdou o nome e é só usar direto.