Frage zur Schreibweise innerhalb einer Klasse

bthight

Ich habe nun folgenden Code:

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

class Box
{
public:
    int width { 0 };
    int length { 0 };
    int height { 0 };
    Box(int width, int length, int height):
    width(w),length(l),height(h)
    {}
    void setWidth (int w){width =w;}
    int getWidth (){return width;}
    void setLength (int l){length =l;}
    int getLength (){return length;}
    void setHeight (int h){height = h;}
    int getHeight(){return height;}
    int Volume() {return width * height * length;}

};

int main()
{   Box box1=Box(1,1,1);
    cout << "box1.Volume: " << box1.Volume() << endl;
    cout << Box.getHeight()<<endl;
}

Die fett markierte Stelle wird nicht compiliert. "Not declared in this scope".
** width(w),length(l),height(h)**

Aber in meinem Codebeispiel aus dem Buch compiliert er das problemlos.
Habe ich hier etwas falsch initialisiert?

Beste Grüße und vielen Dank an Euch alle.

5cript

w, l und h sind nicht deklariert.

Ich tippe auf ein flüchtigkeitsfehler, deine parameter heißen anders.

bthight

@5cript

aahh ok. Ich habe angenommen, dass ich das nicht exra deklarieren muss.

In meinem Code Beispiel aus dem Buch waren die in meinen "Anfänger" Augen nicht deklariert:

#include <iostream>
#include <string>
using namespace std;
class teller
{
float groesse;
string farbe;
string material;
float preis;
public:
teller(float g, string f, string m, float p) :
groesse(g),
farbe(f),
material(m),
preis(p)
{}
void setGroesse (float g)
{
groesse = g;
} float getGroesse ()
{
return groesse;
} void setFarbe (
string f)
{
farbe = f;
} string getFarbe ()
{
return farbe;
} void setMaterial (
string m)
{
material = m;
} string getMaterial ()
{
return material;
} void setPreis (
float p)
{
preis = p;
} float getPreis ()
{
return preis;
}
void rabatt ()
{
preis *= 80/100;
}
};
int main ()
{
teller Teller1 = teller(21, "braun", "Porzellan", 4.99);
cout << Teller1.getGroesse() << endl;
cout << Teller1.getFarbe() << endl;
cout << Teller1.getMaterial() << endl;
cout << Teller1.getPreis() << endl;
}

Mein Code:

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

class Box
{
public:
    int width { 0 };
    int length { 0 };
    int height { 0 };
    int w;
    int l;
    int h;
    Box(int width, int length, int height):
    width(w),length(l),height(h)
    {}
    void setWidth (int w){width =w;}
    int getWidth (){return width;}
    void setLength (int l){length =l;}
    int getLength (){return length;}
    void setHeight (int h){height = h;}
    int getHeight(){return height;}
    int Volume() {return width * height * length;}
};
int main()
{   Box box1= Box(1,1,1);
    cout << "box1.Volume: " << box1.Volume() << endl;
    cout << box1.getHeight()<<endl;
}

Gibt leider nicht mein Gewünschten Output aus. Ich möchte eine 1 erhalten in beiden Fällen.
Es wird 0 angezeigt. (Das sind soweit ich verstehe meine Default-Werte int width { 0 }; int length { 0 };int height { 0 };)

5cript

@bthight sagte in Frage zur Schreibweise innerhalb einer Klasse:

Box(int width, int length, int height):
width(w),length(l),height(h)
{}

Box(int width, int length, int height): // <- ne hier :P
    width(w),length(l),height(h) // <-- w, l, h müssen die parameter sein ofc
    {}

// --> also
Box(int w, int l, int h)
  : width{w}, length{l}, height{h}
{
}

hätte mehr dazu schreiben sollen.

EDIT: die können auch genauso heißen:

// --> also
Box(int width, int length, int height)
  : width{width}, length{length}, height{height} // initialisiere member width, mit parameter width.
{
}

bthight

@5cript sagte in Frage zur Schreibweise innerhalb einer Klasse:

@bthight sagte in Frage zur Schreibweise innerhalb einer Klasse:

Box(int width, int length, int height):
width(w),length(l),height(h)
{}
Box(int width, int length, int height): // <- ne hier :P
    width(w),length(l),height(h) // <-- w, l, h müssen die parameter sein ofc
    {}

// --> also
Box(int w, int l, int h)
  : width{w}, length{l}, height{h}
{
}
hätte mehr dazu schreiben sollen.

EDIT: die können auch genauso heißen:
// --> also
Box(int width, int length, int height)
  : width{width}, length{length}, height{height} // initialisiere member width, mit parameter width.
{
}

Vielen lieben Dank dafür.

Eine Frage habe ich noch.
Warum funktioniert die Initialisierungsliste auch mit {} statt ()

Box(int w, int l, int h):
width{w},length{l},height{h}
{}

oder

Box(int w, int l, int h):
width(w),length(l),height(h)
{}

bthight

@bthight ```cpp
#include <iostream>
#include <string>
using namespace std;

class Box
{
private:
int width { 0 };
int length { 0 };
int height { 0 };

public:

Box(int w, int l, int h):
width{w},length{l},height{h}
{}
void setWidth (int w){width =w;}
int getWidth (){return width;}
void setLength (int l){length =l;}
int getLength (){return length;}
void setHeight (int h){height = h;}
int getHeight(){return height;}
int Volume() {return width * height * length;}

};

int main()
{ Box box1= Box();
cout << "box1.Volume: " << box1.Volume() << endl;
cout << box1.getHeight()<<endl;
}


Box box1= Box();

wieso gibt der Compiler mir nicht als Box 1 die Defaultwerte aus der private aus?

5cript

@bthight sagte in Frage zur Schreibweise innerhalb einer Klasse:

Warum funktioniert die Initialisierungsliste auch mit {} statt ()

Das ist uniform initialization, eine alternative schreibweise für das Initialisieren von Variablen und Konstanten.
Was man grundsätzlich benutzt, es sei denn man muss das andere Nutzen hängt am persönlichen Geschmack. Es gibt einen Unterschied in manchen Fällen, also ist es nicht immer equivalent.
Für built-in types (int, float, ...) ist es allerdings egal.
Ich könnte Beispiele nennen, aber das holt grad zu weit aus, weil dann muss ich die auch erklären.

@bthight sagte in Frage zur Schreibweise innerhalb einer Klasse:

wieso gibt der Compiler mir nicht als Box 1 die Defaultwerte aus der private aus?

Der compiler gibt sowieso nichts aus nach übersetzung
Soweit ich das sehen kann sollte der das Programm nicht übersetzen, es gibt kein Konstruktor für Box, der keine Argumente nimmt.

füge Box() = default; in deine Klasse ein. Dadurch wird ein Standardkonstruktor wieder automatisch generiert, was er in deinem Beispiel nicht macht, weil du einen eigenen definiert hast.

codegod70

Danke für die Tipps. Ich bin gerade am Einstieg und versucher mich hier weiterzubilden!

bthight

füge Box() = default; in deine Klasse ein. Dadurch wird ein Standardkonstruktor wieder automatisch generiert, was er in deinem Beispiel nicht macht, weil du einen eigenen definiert hast.

Das habe ich unter Public eingefügt. Es funktioniert.

int main()
{ Box box1; // Default Konstruktor
Box box2= Box(1,2,3); // Box 2
cout << "box1.Volume: " << box1.Volume() << endl;
cout << "box2.Volume: " << box2.Volume() << endl;
cout << box1.getHeight()<<endl;
}

bthight

@bthight sagte in Frage zur Schreibweise innerhalb einer Klasse:

füge Box() = default; in deine Klasse ein. Dadurch wird ein Standardkonstruktor wieder automatisch generiert, was er in deinem Beispiel nicht macht, weil du einen eigenen definiert hast.

Das habe ich unter Public eingefügt. Es funktioniert.

int main()
{ Box box1; // Default Konstruktor
Box box2= Box(1,2,3); // Box 2
cout << "box1.Volume: " << box1.Volume() << endl;
cout << "box2.Volume: " << box2.Volume() << endl;
cout << box1.getHeight()<<endl;
}

Wenn ich die Box mit folgendem Code schreibe:

Box(int w=2, int l=4, int h=6):
   width{w},length{l},height{h}
   {}

Habe ich die Box auch per Default Konstruktor belegt?
Bzw. ist Box()=default;

das selbe?

*john 0

@bthight sagte in Frage zur Schreibweise innerhalb einer Klasse:

Wenn ich die Box mit folgendem Code schreibe:
Box(int w=2, int l=4, int h=6):
   width{w},length{l},height{h}
   {}
Habe ich die Box auch per Default Konstruktor belegt?

Nein, der Default Konstruktor wird immer mit leerer Argumentsliste definiert und auch aufgerufen. Wenn dieser nicht vorhanden ist, und wie hier ein Konstuktor mit drei Argumenten, die jeweils ein Default haben, definiert wird, dann wird nicht der Default Konstruktor sondern der Konstruktor mit den drei Defaults aufgerufen. Defaults sind nur bedingt sinnvoll, und man muss bei Konstruktoren noch auf einen besonderen Aspekt hinweisen. Konstruktoren mit einem Argument (oder Konstruktoren mit mehreren Default Werten, die nur mit einem expliziten Argument aufgerufen werden können) funktionieren als Konversionkonstruktor.

Beispiel

const short s = -1;
Box bb(1,2,3);
bb = s;

Was passiert in Zeile drei? Die short Variable s wird in ein int konvertiert, und das lässt sich dank Deines Konstuktors mit Defaults in eine Box konvertieren.

Damit man das ausschließen kann, gibt es dann Schlüsselwort explicit, damit wird die ungewollte Konversion verhindert.