Informatiker Board | Druckvorschau: Implementierungsproblem

Informatiker Board (http://www.informatikerboard.de/board/index.php)
- Themengebiete (http://www.informatikerboard.de/board/board.php?boardid=1)
-- Praktische Informatik (http://www.informatikerboard.de/board/board.php?boardid=6)
--- Implementierungsproblem (http://www.informatikerboard.de/board/thread.php?threadid=3737)

Geschrieben von Kevin33 am 23.10.2017 um 00:37:

großes Grinsen

ok gehe pennen .
Bin mit den Nerven am Ende großes Grinsen

Morgen geht es weiter mit Operator Ausgabe großes Grinsen

Geschrieben von as_string am 23.10.2017 um 00:44:

Zitat:

Original von Kevin33
großes Grinsen

ok gehe pennen .
Bin mit den Nerven am Ende großes Grinsen

Morgen geht es weiter mit Operator Ausgabe großes Grinsen

Mach doch endlich mal meine Tabelle! Das mit dem Aufsummieren stimmt doch immer noch nicht!

Geschrieben von Kevin33 am 23.10.2017 um 11:20:

Bin gegen Abend wieder da. Mache dann deine Tabelle großes Grinsen

Poste sie dann.
Wie soll ich die genau haben ?

Aber ich glaube ,dass ich vielleicht verstehe was du meinst .
Mann muss ja auch z.B bei einem Sieg auch die kassierten Tore mitzählen oder ?

Nur als Bsp:

Meinst du die Logik so?

code:

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:


    if(goalsScored>goalsReceived ){

        m_winsTotal += 1;
        m_goalsScoredTotal += goalsScored;
        m_goalsReceivedTotal += goalsReceived; das auch noch hochzählen?
        





    }

Geschrieben von as_string am 23.10.2017 um 13:36:

Du musst immer die jeweiligen Tore mitzählen, völlig unabhängig davon ob gewonnen oder verloren. Mach das einfach raus aus dem ganzen if-Zeugs!

Geschrieben von Kevin33 am 23.10.2017 um 16:51:

Was meinst du jetzt genau String ?

Geschrieben von as_string am 23.10.2017 um 17:05:

Zitat:

Original von Kevin33
Was meinst du jetzt genau String ?

Was meine ich wie genau? Noch genauer wäre es nur, wenn ich gleich den Quelltext hinschreiben würde. Willst Du darauf hinaus, dann sag’s gleich!
Ich glaube Du willst uns alle nur verarschen!

Geschrieben von Kevin33 am 23.10.2017 um 17:40:

Nein großes Grinsen

. Ich poste später meinen Ansatz

Geschrieben von Kevin33 am 23.10.2017 um 23:35:

code:

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33:
34:
35:
36:
37:
38:
39:
40:
41:
42:
43:
44:
45:
46:
47:
48:
49:
50:
51:
52:
53:
54:
55:
56:
57:
58:
59:
60:


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

CTeam::CTeam(string name ){

    m_teamName = name;
}

std::string CTeam::getName(){
    return m_teamName;

    }

void CTeam::addResult(unsigned int goalsScored,unsigned int goalsReceived){


    if(goalsScored>goalsReceived ){

        m_winsTotal += 1;
        m_goalsScoredTotal += goalsScored;
        m_goalsReceivedTotal += goalsReceived;






    }
    if(goalsScored<goalsReceived   ){

            m_lostTotal += 1;
            m_goalsReceivedTotal += goalsReceived;
            m_goalsScoredTotal += goalsScored;




        }
    if( goalsScored== goalsReceived){

                m_equalTotal += 1;
                m_goalsScoredTotal += goalsScored;
                m_goalsReceivedTotal += goalsReceived;
            }

}

unsigned int const CTeam::getTotalPoints(){

	return 3*m_winsTotal+1*m_equalTotal;


}

Mach mich jetzt an die Operator Ausgabe .
Wenigstens paar Tipps dafür ?

Bevor ich wieder dumme Ansätze bringe ? großes Grinsen

Geschrieben von Kevin33 am 24.10.2017 um 01:10:

habe die Operator Ausgabe implementiert ?
Die kommt im Header oder ?

code:

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33:
34:
35:
36:
37:
38:
39:
40:

* CTeam.h
 *
 *  Created on: 21.10.2017
 *      Author: Medion
 */

#ifndef CTEAM_H_
#define CTEAM_H_
#include<iostream>
using namespace std;
#include<string>

class CTeam{
private:
	string m_teamName;
	unsigned int m_winsTotal=0;
	unsigned int m_lostTotal =0;
	unsigned int m_equalTotal= 0;
	unsigned int m_goalsScoredTotal = 0;
	unsigned int m_goalsReceivedTotal=0;

public:
	CTeam(string name = "NoName");
	string getName();
	void addResult(unsigned int goalsScored,unsigned int goalsReceived);
	unsigned int const getTotalPoints();
	friend ostream& operator <<(ostream& out,CTeam const& rop){

		out << "Bayern Muenchen" << " " << ":" << rop.addResult << rop.1*m_equalTotal << " " << "Tore"<<rop.goalsScored<<":"<<rop.goalsReceived << endl;
		return out;
	}
	bool operator< (const CTeam& rop);

};




#endif /* CTEAM_H_ */

Geschrieben von as_string am 24.10.2017 um 01:16:

Um das vielleicht doch noch irgendwie hin zu bekommen: Schau mal, die zwei Zeilen

code:
1: 2:	m_goalsScoredTotal += goalsScored; m_goalsReceivedTotal += goalsReceived;

hast Du doch jetzt in jedem if-Block drin. Das bedeutet doch, dass diese beiden immer ausgeführt werden sollen, richtig? Warum schreibst Du sie dann aber überhaupt in einen bedingten Block rein und dafür dreimal dasselbe und nicht einfach vor oder nach den ganzen if Blöcken???
Abgesehen davon sollte man wirklich if(){} else if() {} else {} verwenden. Die drei Bedingungen schließen sich ja gegenseitig aus und decken umgekehrt gemeinsam aber alle Möglichkeiten ab.

Gute Tipps haben Dich bisher doch auch nicht von dummen Ansätzen abgehalten. Was sollte das also bringen?

Geschrieben von Kevin33 am 24.10.2017 um 01:18:

Zitat:

Original von Kevin33
habe die Operator Ausgabe implementiert ?
Die kommt im Header oder ?

code:

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33:
34:
35:
36:
37:
38:
39:
40:

* CTeam.h
 *
 *  Created on: 21.10.2017
 *      Author: Medion
 */

#ifndef CTEAM_H_
#define CTEAM_H_
#include<iostream>
using namespace std;
#include<string>

class CTeam{
private:
	string m_teamName;
	unsigned int m_winsTotal=0;
	unsigned int m_lostTotal =0;
	unsigned int m_equalTotal= 0;
	unsigned int m_goalsScoredTotal = 0;
	unsigned int m_goalsReceivedTotal=0;

public:
	CTeam(string name = "NoName");
	string getName();
	void addResult(unsigned int goalsScored,unsigned int goalsReceived);
	unsigned int const getTotalPoints();
	friend ostream& operator <<(ostream& out,CTeam const& rop){

		out << "Bayern Muenchen" << " " << ":" << rop.addResult << rop.1*m_equalTotal << " " << "Tore"<<rop.goalsScored<<":"<<rop.goalsReceived << endl;
		return out;
	}
	bool operator< (const CTeam& rop);

};




#endif /* CTEAM_H_ */

Stimmt auch wieder großes Grinsen

Du hast meinen Ansatz übersehen? großes Grinsen

Geschrieben von as_string am 24.10.2017 um 08:49:

Kurz zwei Punkte:
Seit wann implementieren wir direkt in der Header-Datei?
Was ist, wenn das Team nicht Bayern München sondern Eintracht Frankfurt ist? Meinst Du, die würden sich über die Ausgabe freuen? Ich finde halt, so was muss einem schon auffallen, wenn man so was hin schreibt. Deshalb glaube ich Dir auch nicht mehr was Du erzählst. Du willst uns nur verarschen...

Generell: ließ doch irgendwie erst ein Buch oder auch online über Operatorüberladung

Geschrieben von Kevin33 am 24.10.2017 um 09:44:

code:

1:
2:
3:
4:
5:
6:
7:
8:


friend ostream& operator <<(ostream& out,CTeam const& rop){

		out << m_teamName << " " << ":" << rop.addResult << rop.1*m_equalTotal << " " << "Tore"<<rop.goalsScored<<":"<<rop.goalsReceived << endl;
		return out;
	}

Soll ich nicht besser m_goalsScoredTotal nehmen ?

Bei goalsScored zeigt er ja Fehler an?

Geschrieben von Kevin33 am 24.10.2017 um 09:48:

Ich denke ,dass ich die member Variablen nutzen muss oder ?

Die anderen variablen erkennt er ja nicht ?

code:

1:
2:
3:
4:
5:
6:
7:
8:


ostream& operator <<(ostream& out,CTeam const& rop){

		out << rop.m_teamName << " " << ":" << rop.addResult << rop.1*m_equalTotal << " " << "Tore"<<rop.m_goalsScoredTotal<<":"<<rop.m_goalsReceivedTotal ;
		return out;
	}