Compiler Fehler in C++

Question

Hier ist der Fehler, den ich bekomme.

CruiseShip.h:10: error: expected ‘;’ before ‘::’ token

CruiseShip.cpp:8: error: expected ‘)’ before ‘name’ make: ***

[CruiseShip.o] Error 1

CruiseShip.h

CruiseShip(std::string name,std::string year, int maxPassengers)::Ship(std::string name,std::string year); 

CruiseShip.cpp

CruiseShip(string name, string year, int maxPassengers)::Ship(string name, string year){ 
    maxPassengers=0; 
} 

Dies sind die Linie ist, wo der Fehler auftritt. Hier

ist der Rest des Codes: CruiseShip.cpp

#include <iostream> 
#include "Ship.h" 
#include "CruiseShip.h" 
using namespace std; 



CruiseShip(string name, string year, int maxPassengers)::Ship(string name, string year){ 
    maxPassengers=0; 
} 

void CruiseShip::setPass(int maxPassengers){ 
    this->maxPassengers=maxPassengers; 
} 
int CruiseShip::getPass(){ 
    return maxPassengers; 
} 

void CruiseShip::print(){ 
    cout<<"The name of the ship is "<<getName()<<endl; 
    cout<<"The capacity of the ship is "<<maxPassengers<<endl; 

} 

CruiseShip.h

#ifndef CRUISESHIP_H_ 
#define CRUISESHIP_H_ 
#include <string> 
class CruiseShip: public Ship{ 
protected: 
    int maxPassengers; 


public: 
    CruiseShip(std::string name,std::string year, int maxPassengers)::Ship(std::string name,std::string year); 
    void setPass(int); 
    int getPass(); 
    virtual void print(); 

}; 
#endif 

Answer 1

Diese Linie scheint keinen Sinn zu machen.
Was denkst du soll es tun?

CruiseShip(std::string name,std::string year, int maxPassengers)::Ship(std::string name,std::string year); 

Es sieht aus wie der Anfang eines Konstruktor für class CruiseShip, aber hat dann eine Scoping (::) vor für class Ship wie der Konstruktor zu suchen beginnen. Hier

ist, was ich denke du meinst:

In Header (.h) Datei:

#pragma once 
#include <string> 
using std::string; 
class CruiseShip : 
    public Ship // Class inherits from base-class Ship 
{ 
    // Constructor takes 3 parameters: 
    CruiseShip(const string& name, const string& year, int maxPassengers); 
}; 

In-Implementierung (CPP) File:

// Implementation of the Constructor, which begins by passing 
// name and year to the Base-Class constructor. 
// Then completes the constructor by handling the maxPassengers parameter. 
CruiseShip::CruiseShip(const string& name, const string& year, int maxPassengers): 
    Ship(name, year) // Call the base-class constructor 
{ 
    this->maxPassengers = maxPassengers; // Also assign member variable. 
} 

---

Ein paar andere Anmerkungen:

• Sie sollten Variablen in der Regel durch const-Referenz übergeben, wenn Sie keinen guten Grund haben, nach Wert zu übergeben. Dies vermeidet überflüssige Kopierkonstruktoren.

• Vermeiden Sie den gesamten #ifdef - #endif Schutz mit #pragma once, die jetzt von den meisten großen Compiler unterstützt wird.

• Tun Sie nicht using namespace std;. Es bringt den gesamten Namensraum, der wirklich groß ist. Importieren Sie einfach, was Sie brauchen: using std::string; (siehe This Topic)

Answer 2

Offenbar CruiseShip erbt von Ship.

sollte die Erklärung nur sagen, was der Prototyp Konstruktor ist

CruiseShip(std::string name, std::string year, int maxPassengers); 

und die Definition funktioniert die Initialisierung:

CruiseShip::CruiseShip(string name, string year, int maxPassengers) 
    : Ship(name, year), 
    maxPassengers(maxPassengers) 
{ 

} 

Beachten Sie, dass nur es gibt einen einzigen Doppelpunkt und dass die Initialisierung der Basisklasse doesn Erwähnen Sie die Typen nicht wie einen Funktionsaufruf.
Auch die Konstruktordefinition benötigt die Bereichsspezifikation CruiseShip::.

Answer 3

Ihre Schiffsklasse hat, so etwas haben:

Ship(std::string,std::string); 

in öffentlichen Erklärungen. Denn das ist, was Sie fordern, wenn Sie Parameter geben in CruiseShip

Die Art und Weise eine korrekte Konstruktor mit inherence machen, ist auf diese Weise:

CruiseShip::CruiseShip(string name, string year, int maxPassengers):Ship(name,year){ 
    maxPassengers=0; 
} 

Sie rufen den Konstruktor mit Parametern Ship(std::string,std::string), die Parameter gegeben nimmt durch CruiseShip. Und Sie sagen einfach dem Programm, welche Variablen Sie geben

Ihre CruiseShip-Klasse, weil es falsch ist. Sie sagen dem Programm nicht, zuerst anzurufen.

#ifndef CRUISESHIP_H_ 
#define CRUISESHIP_H_ 
#include <string> 
class CruiseShip: public Ship{ 
protected: 
    int maxPassengers; 


public: 
    CruiseShip(std::string name,std::string year, int maxPassengers); 
    void setPass(int); 
    int getPass(); 
    virtual void print(); 

}; 
#endif