MFG
Daniel

Die Fehler:

1. Fehler 1 error LNK2019: Verweis auf nicht aufgelöstes externes Symbol ""public: __thiscall Bus::Bus(int,class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> >,int,int,int)" (??0Bus@@QAE@HV?$basic\_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@HHH@Z)" in Funktion "_main". C:\Users\q\Desktop\Uni\SS 13\C++\Üben\Übung4\Übung4\main.obj

2)Fehler 2 error LNK1120: 1 nicht aufgelöste externe Verweise. C:\Users\q\Desktop\Uni\SS 13\C++\Üben\Übung4\Debug\Übung4.exe

Meine Klasse Bus:

#include <iostream>
#include <string> 

class Bus 
{ 
  public: 
	  Bus();
	  Bus(int, std::string, int, int, int);
	  Bus(const Bus &copyBus);
	  ~Bus();
	  void printmethode();

  private:
	  int Buslinie;
	  std::string Fahrtziel;
	  int Sitzplätze;
	  int Stehplätze;
	  int Passagiere;

};

Meine Klassendeklaration:

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

Bus::Bus()                      //Defaultkonstruktor
{
  Buslinie=0;
  Fahrtziel= "nirgendwo";
  Sitzplätze=40;
  Stehplätze=50;
  Passagiere=0;
}

Bus::~Bus()                      //Destruktor
{
	cout<<"destruktor aktiviert";
}

Bus::Bus(const Bus &copyBus)     //Copykontruktor
{
   Buslinie=copyBus.Buslinie;
   Fahrtziel=copyBus.Fahrtziel;
   Sitzplätze=copyBus.Sitzplätze;
   Stehplätze=copyBus.Stehplätze;
   Passagiere=copyBus.Passagiere;
}

void Bus::printmethode()
{ cout<<Buslinie<<endl<<Fahrtziel<<endl<<Sitzplätze<<endl<<Stehplätze<<endl<<Passagiere<<endl;
}

Und mein Mainprogramm:

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

int main()
{
	Bus Bus1(1, "Baunatal", 10, 10, 5);
	Bus copyBus1(Bus1);

	Bus1.printmethode();

	system("pause");
	return 0;
}

So, dein Fehler liegt in Zeile 8 der Klassendeklaration, sowie in Zeile 7 deiner main Datei und sie liegt eben nicht in deiner Bus.cpp.

So, dein Fehler liegt in Zeile 8 der Klassendeklaration, sowie in Zeile 7 deiner main Datei und sie liegt eben nicht in deiner Bus.cpp.

im Gegenteil, der Fehler liegt sehr wohl in der Bus.cpp. Da fehlt nämlich der in Zeile 8 (Bus.h) deklarierte und in Zeile 7 (main.cpp) verwendete Konstruktor.

PS: ok, man könnte den Konstruktor auch inline schreiben. Dann wäre der Fehler in der Bus.h.

PPS: der copy-Konstruktor ist überflüssig. Der default copy-Konstruktor macht genau das schon, was du da hingeschrieben hast

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

Bus::Bus(int, std::string, int, int, int)                      //Defaultkonstruktor
{
  Buslinie=0;
  Fahrtziel= "nirgendwo";
  Sitzplätze=40;
  Stehplätze=50;
  Passagiere=0;
}

Bus::~Bus()                      //Destruktor
{
	cout<<"destruktor aktiviert";
}

Bus::Bus(const Bus &copyBus)     //Copykontruktor
{
   Buslinie=copyBus.Buslinie;
   Fahrtziel=copyBus.Fahrtziel;
   Sitzplätze=copyBus.Sitzplätze;
   Stehplätze=copyBus.Stehplätze;
   Passagiere=copyBus.Passagiere;
}

void Bus::printmethode()
{ cout<<Buslinie<<endl<<Fahrtziel<<endl<<Sitzplätze<<endl<<Stehplätze<<endl<<Passagiere<<endl;
}

Aufgabe ist bis jetzt gewesen:
Defaultkonstruktor der den Bus mit Standardwerten initialisiert
• Destruktor (mit leerem Rumpf),
• Konstruktor zum Setzen der Member,
• Copykonstruktor,

Danke schonmal für die Hilfe

Skym0sh0 schrieb:

So, dein Fehler liegt in Zeile 8 der Klassendeklaration, sowie in Zeile 7 deiner main Datei und sie liegt eben nicht in deiner Bus.cpp.

im Gegenteil, der Fehler liegt sehr wohl in der Bus.cpp. Da fehlt nämlich der in Zeile 8 (Bus.h) deklarierte und in Zeile 7 (main.cpp) verwendete Konstruktor.

PS: ok, man könnte den Konstruktor auch inline schreiben. Dann wäre der Fehler in der Bus.h.

Nein, du hast mich nicht verstanden. Daniel1990 deklariert neben dem Defaultconstructor noch einen speziellen Konstruktor, diesen definiert er aber nicht.

Meine Antwort zielte darauf, dass er diesen Specialconstructor deklariert, nutzt aber eben nicht implementiert ->

Skym0sh0 schrieb:

und sie liegt eben nicht in deiner Bus.cpp.

im Gegenteil, der Fehler liegt sehr wohl in der Bus.cpp. Da fehlt nämlich der in Zeile 8 (Bus.h) deklarierte und in Zeile 7 (main.cpp) verwendete Konstruktor.

PS: ok, man könnte den Konstruktor auch inline schreiben. Dann wäre der Fehler in der Bus.h.

Nein, du hast mich nicht verstanden. Daniel1990 deklariert neben dem Defaultconstructor noch einen speziellen Konstruktor, diesen definiert er aber nicht.

Meine Antwort zielte darauf, dass er diesen Specialconstructor deklariert, nutzt aber eben nicht implementiert -> Skym0sh0 schrieb:
und sie liegt eben nicht in deiner Bus.cpp.

Also der Fehler liegt in diesen Zeilen:

Bus.h:

Bus(int, std::string, int, int, int);

main.cpp:

Bus Bus1(1, "Baunatal", 10, 10, 5);

wie sollte ich es deiner Meinung nach ändern? ich kann dir da gerade leider nicht Folgen sry

class Bus
{
public:
    Bus(); // <-- Default CTor
};

Jeder andere Konstruktor nimmt Parameter (auch der Copy-CTor oder der Move-CTor).

Aber ein Objekt will ja richtig initialisiert werden, wenn es erzeugt ist und danach voll benutzbar sein. Daher kann es nötig sein, dass ein CTor auch Parameter nehmen kann/will/muss/darf/soll.

class Bus
{
public:
    Bus(); // <-- Default CTor
    Bus(int, std::string, int, int, int); // Anderer CTor
};

Soweit so gut. Lassen wir das Programm jetzt "laufen":

int main()
{
    Bus b1; // Default konstruiert
    Bus b2(1, "Lol", 2, 3, 4); // anders konstruiert
}

Dann kompiliert das zwar ohne Probleme, aber bevor es startet, merkt der Linker "Ups da stimmt was nicht". Denn was machen die beiden Konstruktoren? Man weiss es nicht. Der Computer will das im Allgemeinen jedoch wissen, also sagen wirs ihm:

Bus::Bus()
{ }// hier passiert zwar nix, aber es reicht

Bus::Bus(int, std::string, int, int, int)
{ } // hier passiert zwar nix, aber es reicht

Hier haben wir die beiden deklarierten Methoden (eben die beiden CTors) auch definiert bzw. implementiert )Obwohl da nichts passiert, das ist deine Aufgabe).

Dein Fehler war, dass der zweite Konstruktor (der mit den Parametern) eben nirgends definiert und implementiert war.

Klasse:

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

class Bus
{
public: 
	Bus();
	Bus(int, std::string, int, int, int);
	Bus(const Bus &CopyBus);
	void printmethode();
    ~Bus();
private:
	int Buslinie;
	std::string Ziel;
	int Sitzplätze;
	int Stehplätze;
	int Passagiere;

};

Klasse.cpp

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

Bus::Bus(){}                                                                             
Bus::Bus(int Buslinie, std::string Ziel, int Sitzplätze, int Stehplätze, int Passagiere) // Defaultkonstruktor
{
  Buslinie=0;
  Ziel= "nirgendwo";
  Sitzplätze=40;
  Stehplätze=50;
  Passagiere=0;
}
Bus::Bus(const Bus &CopyBus) //Copykonstruktor
{
  Buslinie=CopyBus.Buslinie;
  Ziel=CopyBus.Ziel;
  Sitzplätze=CopyBus.Sitzplätze;
  Stehplätze=CopyBus.Stehplätze;
  Passagiere=CopyBus.Passagiere;
}

void Bus::printmethode()
{cout<<Buslinie<<endl<<Ziel<<endl<<Sitzplätze<<endl<<Stehplätze<<endl<<Passagiere<<endl;}

Bus::~Bus() //Destruktor
{
cout<<"Destruktor"<<endl;
}

MAin:

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

int main()
{
Bus Fernbus;
Bus Fernbus2(1, "Weitweg", 100, 50, 150);
Fernbus.printmethode();
Fernbus2.printmethode();

system("pause");
return 0;
}

btw, Zeile 7 in der Klasse.cpp lese ich // Defaultkonstruktor .
Das ist er aber nicht.

ist das nicht schon mit namespace initialisiert?

Das grenzt an Trolling oO

Falls nicht, nein ist es nicht. Namespaces sind was ganz anderes und die machen auch was ganz anderes.

Daniel1990 schrieb:

ist das nicht schon mit namespace initialisiert?

Das grenzt an Trolling oO

Nein, nicht ganz.

@TE: Nur globale Variablen, sowie Variablen in einem Namensraum und (afaik) statische Variablen werden seit C++11 default-initialisiert (ohne es explizit dahin zu kritzeln). Deine Integer jedoch sind Member einer Klasse und haben mit einem Namensraum recht wenig am Hut.

Wenn mir jetzt noch jemand ein Beispiel oder so dazu geben könnte?

Nö.
Poste mal den Code des Default Constructors.

class Bus
{
public: 
	Bus();
	Bus(int buslinie, std::string ziel, int sitzplätze, int stehplätze, int passagiere);
	Bus(const Bus &CopyBus);
	void printmethode();
    ~Bus();
private:
	int Buslinie;
	std::string Ziel;
	int Sitzplätze;
	int Stehplätze;
	int Passagiere;
};

und in der Bus.cpp

Bus::Bus() // Defaultkonstruktor 
{
  Buslinie=0;
  Ziel= "nirgendwo";
  Sitzplätze=40;
  Stehplätze=50;
  Passagiere=0;
}
Bus::Bus(int buslinie, std::string ziel, int sitzplätze, int stehplätze, int passagiere)
{
  Buslinie=buslinie;
  Ziel= ziel;
  Sitzplätze=sitzplätze;
  Stehplätze=stehplätze;
  Passagiere=passagiere;
}

Deshalb, wie mein Vorredner schon sagte, lösch den ganz raus.