Eclipse Fehlermeldungen

[Programmer33]

Ja aber warum ist der Defaulwert im Konstruktor dann auf 12? Siehe header ?

[Finnegan]

@Programmer33 sagte in Eclipse Fehlermeldungen:

Ja aber warum ist der Defaulwert im Konstruktor dann auf 12? Siehe header ?

Was interessiert dich eigentlich der Default-Wert im Header, wenn ihr den so bekommen habt? Deine Aufgabe ist es nicht Kafeesatz zu lesen und irgendetwas in diese 12 hineinzuinterpretieren, sondern den Konstruktor exakt nach Anweisung umzusetzen (Klassendiagramm und Aufgabenstellung). Wenn jemand diese Klasse benutzt, dann kommen da noch ganz andere Werte als 12 über maxTips reingeprasselt. Die 12 ist da nur ein möglicher Wert für diesen Parameter und für dich, der den Konstruktor implementieren soll erstmal ohne weitere Bedeutung.

[Programmer33]

ah ok danke

[Programmer33]

CLottoTicket::CLottoTicket(std::string name,unsigned short maxTips ){

	m_name = name;
	m_countTips = 0;
	
	m_name = name;

		if (m_maxTips > 6)
		{

			m_maxTips = maxTips;

		}
		else
		{
			m_maxTips = 6;
		}

		m_tips = new CTip[m_maxTips];

	
}

Ok so?

[Finnegan]

@Programmer33 sagte in Eclipse Fehlermeldungen:

Ok so?

• name wird zweimal zugewiesen. Nicht falsch, sieht aber schlampig aus.
• Du prüfst mit m_maxTips > 6 eine nicht-initialisierte Variable -> Bug.
• Dein if/else-Code folgt nicht exakt dem Wortlaut der Aufgabenstellung. Nur wenn die Zusicherung verletzt wird, soll der Minimalwert genommen werden. Für 6 wird die Zusicherung nicht verletzt, du setzt aber trotzdem den Minimalwert. Am Ende kommt zwar dasselbe heraus, da ich aber denke, dass die Aufgabe vornehmlich darauf abzielt, Anweisungen exakt umzusetzen, würde ich das vielleicht umformulieren.
• Die Einrückung ist nicht konsistent.

[Programmer33]

Jetzt müsste es passen denke ich.
Ich glaube die memeber Variable kann man in der if Bedingung auch schon nutzen ?

CLottoTicket::CLottoTicket(std::string name,unsigned short maxTips ){

	m_name = name;
	m_countTips = 0;
	

		if (m_maxTips >= 6)
		{

			m_maxTips = maxTips;

		}
		else
		{
			m_maxTips = 6;
		}

		m_tips = new CTip[m_maxTips];

	
}```

Jetzt müsste es passen ,denke ich

[Finnegan]

@Programmer33 sagte in Eclipse Fehlermeldungen:

Jetzt müsste es passen denke ich.

m_maxTips ist in Zeile 8 noch immer nicht initialisiert. Du prüfst also auf einen unspezifizierten Wert, der in keiner Beziehung zu dem übergebenen Parameter maxTips steht.

[Programmer33]

aha ok .Soll ich dann besser maxTips dort noch lassen ?
Weil davor initialisieren soll man glaub ich nach der Aufgabenstellung nicht

Was sagt ihr zu meiner operator += Methode

CLottoTicket& CLottoTicket::operator+= (CTip tip){


	if( m_countTips < m_maxTips){

		CTip* temp = new CTip [m_maxTips];

		for(int i = 0; i < m_maxTips; i++)
		{
			temp[i] = m_tips[i];
		}

			delete m_tips;

		m_tips = new CTip [m_maxTips+12];

		for(int i = 0; i < m_maxTips; i++) {
			m_tips[i] = temp[i];
		  }

		delete[] temp;
		m_maxTips += 12;



	}

   m_tips[m_countTips++]= tip;
   return *this;

}

[Finnegan]

@Programmer33 sagte in Eclipse Fehlermeldungen:

aha ok .Soll ich dann besser maxTips dort noch lassen ?

Prüfe einfach maxTips statt m_maxTips und alles wird gut.

Was sagt ihr zu meiner operator += Methode

Unterirdisch. Erstaunlicherweise sollte sie jedoch fast funktionieren, wenn du das Array nicht nur dann vergrößern würdest, wenn noch genug Platz im Array ist. Schau dir die Bedingung in Zeile 4 nochmal genau an.

Anmerkungen:

• Bedingung in Zeile 4 (s.o.). Auch wenn man euch C++ aus dem letzten Jahrtausend lehrt, solltest du aus der Veranstaltung zumindest mitnehmen wie man Bedingungen richtig formuliert und dein logisches Denken schärfen.
• Warum kopierst du die Array-Elemente zweimal? Einmal reicht. Denke dran, dass du einen Pointer auch einfach neu zuweisen kannst (m_tips = temp), damit kannst du dir eine Kopierschleife sparen.
• In Zeile 13 löschst du kein Array, sondern nur ein einzelnes Element.

Ansonsten sollte die Funktion weitgehend das Richtige tun,wenn ich nichts übersehen habe. Auch wenn es kein schönes und modernes C++ ist.

[Programmer33]

Ich habe es noch ein wenig verbessert .

CLottoTicket& CLottoTicket::operator+= (CTip tip){


	if( m_countTips < m_maxTips && m_countTips >0){

		CTip* temp = new CTip [m_maxTips];

		for(int i = 0; i < m_maxTips; i++)
		{
			temp[i] = m_tips[i];
		}

			delete[] m_tips;

		m_tips = new CTip [m_maxTips+12];

		for(int i = 0; i < m_maxTips; i++) {
			m_tips[i] = temp[i];
		  }

		delete[] temp;
		m_maxTips += 12;



	}

   m_tips[m_countTips++]= tip;
   return *this;

}

So würde es passen oder dann ?
Ich bin schon froh ,dass ich zum ersten mal so ne schwere Methode teilweise selber implementieren konnte

[Finnegan]

@Programmer33 sagte in Eclipse Fehlermeldungen:

So würde es passen oder dann ?

Schau dir die bedingung in Zeile 5 nochmal genau an. m_maxTips ist die Kapazität des Arrays und m_countTips die Anzahl der Tips, die gerade im Array gespeichert sind.

Welche Bedingung ist erfüllt, wenn du keinen Platz mehr im Array hast? Oder anders: Wann ist die Kapazität erschöpft? Welche Bedingung gilt dann? Die, die dort bei dir in Zeile 5 steht, ist jedenfalls komplett falsch.

Der Rest ist nicht schön, sollte aber funktionieren (ohne Garantie! Selbst testen!).

[Programmer33]

@Finnegan sagte in Eclipse Fehlermeldungen:

Welche Bedingung ist erfüllt, wenn du keinen Platz mehr im Array hast? Oder anders: Wann ist die Kapazität erschöpft?

Wenn (m_countTips > m_maxTips && m_countTips >0 )ist ? Dann ist der Speicher voll .

Hatte in der Aufgabe verlesen
Dachte man muss es machen wenn der nicht voll ist

[Swordfish]

@Programmer33 sagte in Eclipse Fehlermeldungen:

new

"nicht schön" fängt schon bei Zeile drei mit new an.

[Programmer33]

Wie hätte man die Aufgabe schöner lösen können ?
Gerne erklären

[Swordfish]

@Programmer33 sagte in Eclipse Fehlermeldungen:

Wie hätte man die Aufgabe schöner lösen können ?

Was ist die Aufgabenstellung?

[Finnegan]

@Programmer33 sagte in Eclipse Fehlermeldungen:

@Finnegan sagte in Eclipse Fehlermeldungen:
Wenn (m_countTips > m_maxTips && m_countTips >0 )ist ? Dann ist der Speicher voll .

Besser, aber immer noch falsch
Wenn m_countTips > m_maxTips, dann bedeutet, das, dass dein operator+ bei einem vorherigen Aufruf irgendwann mal in Zeile 29 ein Element bei m_tips[m_maxTips] eingefügt haben muss: Schliesslich wird m_countTips nur in Zeile 29 durch die Anweisung m_countTips++ größer. Wenn m_countTips nun aber größer als m_maxTips ist, dann war es beim vorangegangen Aufruf von operator+ gleich m_maxTips, da m_countTips in Zeile 29 immer nur um 1 größer wird. Das bedeutet, dass du beim vorherigen Auruf einen Tip an Position m_tips[m_maxTips] eingefügt haben musst.

Ist das korrekt? Bedenke, dass dein Array m_maxTips Elemente hat. Auf welchen Index darfst du also maximal zugreifen, damit du nicht über die Grenzen das Array hinaus schreibst?

Die Bedingung m_countTips > 0 ist übrigens auch nicht notwendig. Ob die Kapazität erschöpft ist, hängt nur von der Relation zwischen m_countTips und m_maxTips ab. Kleiner als 0 können diese Werte auch nicht sein, da es sich um unsigned-Typen handelt.

[Finnegan]

@Swordfish sagte in Eclipse Fehlermeldungen:

Was ist die Aufgabenstellung?

Die wurde in einem vorherigen Beitrag in einem Link zu einem Bild gepostet. Es gilt die Klasse nach einem vorgegebenen UML-Klassendiagramm umzusetzen. Dort ist der Typ des "Array" als CTip* vorgegeben. Die Frage, wie man es schöner machen kann stellt sich also leider nicht, wenn man nicht die Vorgaben im Klassendiagramm verletzen will. Ich denke es ist Teil der Aufgabe, diesen Vorgaben exakt zu folgen. Den Vorwurf mit dem Uralt-C++ muss man hier also wie so oft mal wieder dem Dozenten machen

[Swordfish]

@Finnegan sagte in Eclipse Fehlermeldungen:

Die wurde in einem vorherigen Beitrag in einem Link zu einem Bild gepostet.

Ach so. Das erklärt einiges. Ich lese keinen Text in Bildern.

[Programmer33]

@Finnegan sagte in Eclipse Fehlermeldungen:

Ist das korrekt? Bedenke, dass dein Array m_maxTips Elemente hat. Auf welchen Index darfst du also maximal zugreifen, damit du nicht über die Grenzen das Array hinaus schreibst?

Index 5 ist der letzte Index .

Aber ich bin jetzt verwirrt das mit dem einfügen vor der if Funktion

[Finnegan]

@Programmer33 sagte in Eclipse Fehlermeldungen:

Index 5 ist der letzte Index .

Aber ich bin jetzt verwirrt das mit dem einfügen vor der if Funktion

Drücke diesen letzten Index mal nicht als konkrete Zahl, sondern allgemein unter Verwendung der Variablen m_maxTips aus. Schliesslich kann der Konstruktor auch mit maxTips größer 6 aufgerufen, oder das Array schonmal vergrößert worden sein.