Wisst ihr was dieser Code bewirkt:

Tier Hase[5];

Hase[0]=Tier(1,"Max");

Bei Tier Hase[5] werden ja 5 Objekte erzeugt, bei denen der Standardkonstruktor aufegerufen wird?
Was passiert dann aber bei der Zeile

Hase[0]=Tier(1,"Max");

Danke!

Viele Grüße

Usw.

Simon

Ich habe jetzt folgendes in einer Klasse Pferd gemacht:

Pferd() : name(" "), identnr(999){}

 Pferd(string n, int id): name(n), identnr(id) {}

In der main:

Pferd pferde[5];

 pferde[0] = Pferd("Sepp",0);

Also bei Pferd pferde[5] wird der Standardkonstruktor aufgerufen.
Bei pferde[0] = Pferd("Sepp",0) wird Konstruktor mit Initialisierungsliste aufgerufen?

Das klappt auch alles.

Nur wenn ich z.B. den Namen ausgeben lassen möchte, dann erscheint nichts

Wenn ich anstatt

pferde[0] = Pferd("Sepp",0);

einfach

pferde[0].setAttribute("Sepp",0)

aufrufen würde und dann den Namen ausgeben möchte, klappt das schon.

Wisst ihr, woran das liegen kann?

Danke!

hier der Code:

#include <cstdlib>
#include <iostream>

using namespace std;

class Elefant 
{
  //Deklaration der Attribute
  private:
  string name; //für den Namen
  int identnr; //für die Identifizierung
  int reservierung[10];

  public:
  //der Konstruktor mit Initialisierungsliste
  //hier werden Defaultwerte gesetzt
  Elefant() : name(" "), identnr(999) {

                 for(int i = 0; i < 10; i++) {

                  reservierung[i] = 999;

                  } 

          cout <<"NAME: " <<name <<endl;
          cout <<"IDENTNR: " <<identnr <<endl;

          }    

 Elefant(string n, int id): name(n), identnr(id) {}

 string getName() {

 return name;       
}

int getident() {

    return identnr;
}

int getReservierung() {

    return reservierung[9];
}

};

int main(int argc, char *argv[])
{

    Elefant fanti[5];

    fanti[0] = Elefant ("Max",11);
    cout <<endl << endl;
    cout << "NAME NEU: " << fanti[0].getName() <<endl;
    cout << "IDENT NEU: " << fanti[0].getident() <<endl;

    //HIER STIMMT ES NICHT
    cout << "HIER SOLLTE 999 stehen: " << fanti[0].getReservierung();

    system("PAUSE");
    return EXIT_SUCCESS;
}

Ich glaube der Fehler liegt daran, dass ich die Reservierung nicht neu gesetzt habe?
Warum muss ich eigentlich die Reservierung neu setzen, sie wurde ja bereits im Standardkonstruktor neu gesetzt?

Ist das mit der Reservierung und der for SChleife im Konstruktor korrekt oder eher schlechter Stil?

Danke"!

Viele Grüße

Warum muss ich eigentlich die Reservierung neu setzen, sie wurde ja bereits im Standardkonstruktor neu gesetzt?

Aber du konstruierst ein neues Objekt Elefant ("Max",11) und benutzt dabei einen Konstruktor, der die Reservierung nicht setzt. Und dann kopierst du dieses neue Objekt mit der uninitialisierten Reservierung über das alte.

NAME:
IDENTNR: 999
NAME:
IDENTNR: 999
NAME:
IDENTNR: 999
NAME:
IDENTNR: 999
NAME:
IDENTNR: 999

NAME NEU: Max     // < funkst bei mir also
IDENT NEU: 11
HIER SOLLTE 999 stehen: 1990298837

Zu deiner Reservierung. Es ist durchaus korrekt, deine Variablen im Konstruktor mit Werten vorzufüllen. Du solltest aber bedenken, das du dies bei deinem 2. Konstuktor NICHT tust. Somit ist der Wert in reservierung natürlich uninitialisiert.

Also läuft das so ab:

1. Array fanti[5] erzeugen => Standardkonstruktor wird aufgerufen
2. neues Objekt erstelen => fanti[0] = Elefant ("Max",11); ==> der andere
Konstruktor wird aufgerufen
3. das neue Objekt wird dann in fanti[0] kopiert?

Ist das dann vergleichbar wie in Java:

Elefant fanti[5]

fanti [0] = new Elefant("Max",11)??

Danke!

Viele Grüße

Ist das dann vergleichbar wie in Java:

Nein, bei Java hast du zunächst ein Array von Nullzeigern. Anschliessend lässt du den einen Zeiger auf ein neues Objekt verweisen. Bei C++ hingegen hantierst du hier direkt mit Objekten.

Wichtig ist der Unterschied Wertsemantik <-> Referenzsemantik.

Es ist durchaus korrekt, deine Variablen im Konstruktor mit Werten vorzufüllen.

Die Werte sollten aber nicht Magic Numbers sein (999 ist für mich eine gültige Reservierung), sondern ganz klar falsch sein oder irgendwo mit const bennennt werden. Ich hätte hier -1 genommen:

Elefant(string n, int id): name(n), identnr(id) { std::fill(reservierung, reservierung+10, -1); }

C++-Code mit Java zu vergleichen ist im Allgemeinen eine schlechte Idee. Wenn du gleiches Verhalten willst, solltest du boost::ptr_vector verwenden. Ansonsten empfehle ich dir, einen std::vector<int> anstatt ein Array zu verwenden.
Also sowas:

#include <iostream>
#include <vector>
#include <string>

class elefant {
public:
  elefant() : name(), identnr(999), reservierung(10, -1) { }
  elefant(const std::string& n, int id)
    : name(n), identnr(id), reservierung(10, -1); { }

  const std::string& name()         const { return name;            }
  int                ident()        const { return identnr;         }
  int                reservierung() const { return reservierung[9]; }

private:
  std::string name;
  int identnr;
  std::vector<int> reservierung;
};

int main()
{
  std::vector<elefant> fanti(5);
  fanti[0] = elefant ("Max", 11);
  std::cout << "NAME NEU: "  << fanti[0].name()  << std::endl;
  std::cout << "IDENT NEU: " << fanti[0].ident() << std::endl;
  std::cout << "HIER SOLLTE 999 stehen: " << fanti[0].reservierung() << std::endl;
}

Habs nur im Antwortfenster geschrieben und nicht getestet, vielleicht sind ein paar syntaxfehler drin.

Ich habe noch eine Frage:

Das gleiche Thema mit einer anderen Klasse

Kundschaft[i] = Kunde(gehege1.kundeVornameAnlegen(), gehege1.kundeNachnameAnlegen(), gehege1.kundeTelefonnummerAnlegen(), i);

in den Methoden "...Anlegen" werden hier die Werte erfasst.

Mein Frage:

Ist es normal, das zuerst die Methode kundeTelefonnumerAnlegen(), dann KundeNachnameAnlegen() und zum Schluss kundeVornameAnlegen() aufgerufen wird?

Danke!

Viele Grüße

Kundschaft[i] = Kunde(gehege1.kundeVornameAnlegen(), gehege1.kundeNachnameAnlegen(), gehege1.kundeTelefonnummerAnlegen(), i);

in den Methoden "...Anlegen" werden hier die Werte erfasst.

Mein Frage:

Ist es normal, das zuerst die Methode kundeTelefonnumerAnlegen(), dann KundeNachnameAnlegen() und zum Schluss kundeVornameAnlegen() aufgerufen wird?

Die Auswertungsreihenfolge ist nicht definiert.

#include <cstdlib>
#include <iostream>

using namespace std;

class Elefant 
{
  //Deklaration der Attribute
  private:
  string name; //für den Namen
  int identnr; //für die Identifizierung
  int reservierung[10];

  public:
  //der Konstruktor mit Initialisierungsliste
  //hier werden Defaultwerte gesetzt
  Elefant() : name(" "), identnr(999) {

                 for(int i = 0; i < 10; i++) {

                  reservierung[i] = 999;

                  } 

          cout <<"NAME: " <<name <<endl;
          cout <<"IDENTNR: " <<identnr <<endl;

          }    

 Elefant(string n, int id): name(n), identnr(id) {}

 string getName() {

 return name;       
}

int getident() {

    return identnr;
}

int getReservierung() {

    return reservierung[9];
}

};

int main(int argc, char *argv[])
{

    Elefant fanti[3];

  //  fanti[0] = Elefant ("Max",11);

fanti[0] = Elefant ("BLA",0);

fanti[1] = Elefant ("BLu",1);

fanti[2] = Elefant ("BLi",2);

//wenn man diese Zeile auskommentiert klappt es wieder??
fanti[3] = Elefant ("BLo",3);

fanti[4] = Elefant ("BLu",4);

    cout <<endl << endl;
    cout << "NAME NEU: " << fanti[0].getName() <<endl;
    cout << "IDENT NEU: " << fanti[0].getident() <<endl;

    //HIER STIMMT ES NICHT
    cout << "HIER SOLLTE 999 stehen: " << fanti[0].getReservierung();

    system("PAUSE");
    return EXIT_SUCCESS;
}

Das Programm bricht natürlich ab.

Allerdings läuft es wenn ich fanti[3] ausklammer??
Warum funktioniert das denn?

Viele Grüße

ich habe nur versehentlich 3 eingegen, eigentlich sollte es 5 sein.

Dann habe ich ein bisschen rumgespielt und gemerkt dass es wenn man fanti[3] auskommentiert kein Fehler mehr erscheint?

Ich habe immer gelernt dass dann das Programm abbrechen sollte?

Viele Grüße

Und nein, gar nichts muss abbrechen. Auf ungültige Array-Indizes zuzugreifen resultiert in undefiniertem Verhalten, also kann alles passieren. Anders wäre das, wenn du intelligente Arrays wie std::tr1::array oder boost::array oder in C++0x std::array verwenden würdest, die könnten dir nämlich mit einer Assertion sofort Fehler aufzeigen.