Kevin33
unregistriert
 |
|
hmm ok . Wieso soll man goalsreceived.total usw weg lassen ?
Ich dachte ,dass das auch zum Endergebnis sozusagen gehört
| 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:
|
void CTeam::addResult(unsigned int goalsScored,unsigned int goalsReceived){
if(goalsScored>goalsReceived ){
m_winsTotal = m_winsTotal + 1 ;
}
if(goalsScored<goalsReceived ){
m_lostTotal = m_lostTotal + 1;
}
if( goalsScored== goalsReceived){
m_equalTotal= m_equalTotal +1;
}
}
|
|
|
|
22.10.2017 15:49 |
|
|
as_string 
Haudegen
Dabei seit: 06.11.2013
Beiträge: 638
Herkunft: Heidelberg
|
|
Ich hab nicht gesagt, dass man das weglassen soll, wo liest Du das denn raus??? Ich habe nur gesagt, dass die beiden Werte nichts damit zu tun haben, ob das aktuelle Spiel gewonnen ist oder nicht!
Und was soll denn überhaupt goalsreceived.total sein???
Das hier
m_winsTotal = m_winsTotal + 1 ;
entspricht
m_winsTotal += 1;
was wiederum dem hier entspricht:
m_winsTotal++;
Alle diese Anweisungen zählen die Variable um eins hoch und das ist hier auch richtig.
Wenn Du aber die Gesamttore zählen willst, musst Du diese ja nicht um 1hoch zählen, sondern eben um die Anzahl der jeweils gefallenen Tore. Also nicht eine 1 addieren sondern den jeweiligen Zahlenwert. Verstehst Du den Unterschied oder ist selbst sowas triviales für Dich schon zu viel?
|
|
|
22.10.2017 16:08 |
|
|
Kevin33
unregistriert
|
|
Dann hatte ich dich wohl falsch verstanden .
Weil dauernd gesagt wird , ich hätte keinen Plan , dachte dass wäre falsch . Aber ok .
m_goalsReceivedTotal++; müsste ich nicht this. ...machen ?
Wieso soll ich überhaupt die Gesamtrore aufsummieren ? Wo steht das?
Ich dachte nur die Variablen die ich gemacht habe
|
|
|
22.10.2017 16:18 |
|
|
Kevin33
unregistriert
|
|
| 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:
|
void CTeam::addResult(unsigned int goalsScored,unsigned int goalsReceived){
if(goalsScored>goalsReceived && m_goalsScoredTotal>m_goalsReceivedTotal){
m_winsTotal += 1;
m_winsTotal = m_winsTotal+m_goalsScoredTotal;
}
if(goalsScored<goalsReceived && m_goalsScoredTotal<m_goalsReceivedTotal ){
m_lostTotal += 1;
m_lostTotal= m_lostTotal+m_goalsReceivedTotal;
}
if(m_goalsScoredTotal==m_goalsReceivedTotal && goalsScored== goalsReceived){
m_equalTotal += 1;
}
}
|
|
|
|
|
22.10.2017 17:13 |
|
|
|
as_string
Haudegen
Dabei seit: 06.11.2013
Beiträge: 638
Herkunft: Heidelberg
|
|
| Zitat: |
Original von Kevin33
| 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:
|
void CTeam::addResult(unsigned int goalsScored,unsigned int goalsReceived){
if(goalsScored>goalsReceived && m_goalsScoredTotal>m_goalsReceivedTotal){
m_winsTotal += 1;
m_winsTotal = m_winsTotal+m_goalsScoredTotal;
}
if(goalsScored<goalsReceived && m_goalsScoredTotal<m_goalsReceivedTotal ){
m_lostTotal += 1;
m_lostTotal= m_lostTotal+m_goalsReceivedTotal;
}
if(m_goalsScoredTotal==m_goalsReceivedTotal && goalsScored== goalsReceived){
m_equalTotal += 1;
}
}
|
|
|
Was soll denn das hier überhaupt???:
| code: |
1:
2:
|
m_winsTotal += 1;
m_winsTotal = m_winsTotal+m_goalsScoredTotal; |
|
Du addierst zuerst 1 und dann nochmal m_goalsScoredTotal zu m_winsTotal??? Also letztlich addierst Du m_goalsScoredTotal+1 zu dem Wert, der in m_winsTotal ist?
Jetzt mal im Ernst: Hast Du auch nur irgendetwas dabei Dir gedacht oder hast Du das einfach mal so hingerotzt?
Mal abgesehen davon, dass das im if ja auch wieder überhaupt keinen Sinn ergibt, wie ich Dir jetzt schon mehrfach versucht hatte zu erklären: Du sollst im if() eine Bedingung haben, die wahr wird, wenn ein Spiel gewonnen ist, im nächsten, wenn es verloren ist, im letzten, wenn es unentschieden ist. Verstehst Du das wenigstens? Warum kommst Du dann auf die Idee hier immer noch die Total-Variablen mit zu überprüfen? Das muss man doch mit normalem gesunden Menschverstand schon verstehen dass das einfach nur Blödsinn ist!
Ich habe das ja noch nie gesagt, aber Du bist wirklich ein hoffnungsloser Fall. Da Du ja auch keinen einzigen Ratschlag befolgen willst, wird sich daran auch nie etwas ändern.
Gruß
Marco
|
|
|
22.10.2017 19:37 |
|
|
Kevin33
unregistriert
|
|
m_goalsScoredTotal += goalsScored;
m_goalsReceivedTotal += goalsReceived;
Oder ist der letzten Aufgabenteil so gemeint ?
|
|
|
22.10.2017 17:17 |
|
|
|
as_string
Haudegen
Dabei seit: 06.11.2013
Beiträge: 638
Herkunft: Heidelberg
|
|
| Zitat: |
Original von Kevin33
m_goalsScoredTotal += goalsScored;
m_goalsReceivedTotal += goalsReceived;
Oder ist der letzten Aufgabenteil so gemeint ? |
Ja, natürlich! Was denn sonst???
|
|
|
22.10.2017 19:27 |
|
|
Kevin33
unregistriert
|
|
Aber dann müssen doch diese if Bedingungen richtig sein oder ?
| code: |
1:
2:
3:
4:
5:
6:
7:
8:
|
if(goalsScored>goalsReceived Spiel gewonnen
goalsScored<goalsReceived Mehr Tore kassiert ,also verloren
goalsScored== goalsReceived Gleichstand
|
|
Sind diese if Bedingungen jetzt richtig?
Aus meiner Sicht schon 
|
|
|
22.10.2017 20:46 |
|
|
as_string
Haudegen
Dabei seit: 06.11.2013
Beiträge: 638
Herkunft: Heidelberg
|
|
So sind die Bedingungen, richtig.
Ohne Dich noch mehr verwirren zu wollen: Du solltest hier mit if() {} else if() {} else{} arbeiten. Dann wird das zweite if nur überprüft, wenn das erste nicht zugetroffen hat und die letzte Bedingung ausgeführt, wenn die ersten beiden ifs nicht wahr geworden sind. Wenn das eine weder kleiner noch größer als das andere ist, muss es ja gleich sein, so dass Du Dir die letzte Überprüfung dann sparen könntest.
Aber das ist nicht weiter wichtig, wenn Du das nicht gleich verstanden hast, dann lass das einfach erstmal so, wie Du es jetzt hast. Das ist schon auch richtig.
So, jetzt musst Du also überlegen, was passieren soll in den einzelnen Fällen. Du sollst ja im Falle eines Sieges die eine Member-Variable für die Summe aller gewonnen Spiele hoch zählen, im zweiten Fall die für die verlorenen und im dritten dann die für unentschieden.
Das Hochzählen mach am besten mit ++, also Das ist die kürzeste Form, eine Ganzzahl-Variable um eins hoch zu zählen. Keiner schreibt oder sogar wenn das auch alles richtig ist.
Unabhängig von dem if musst Du die beiden total-Variablen für die Tor/Gegentor-Summen jeweils um die Anzahl der für dieses Spiel erzielten Tore bzw. kassierten Gegentore hochzählen.
Dann endlich wären wir eventuell einen Schritt weiter... 
Gruß
Marco
|
|
|
22.10.2017 22:09 |
|
|
Kevin33
unregistriert
|
|
Zuerst mal noch eine Frage das mit dem m_lostTotal+=1 , sollte ich doch machen oder ?
Das was du willst ist ja : m_goalsReceivedTotal++;
Und in dem anderen Fall m_goalsScoredTotal++, machen .
Muss ich diese beide Sachen zusätzlich machen ?
Weil das stand ja nicht so in der Aufgabe
|
|
|
22.10.2017 22:17 |
|
|
|
as_string
Haudegen
Dabei seit: 06.11.2013
Beiträge: 638
Herkunft: Heidelberg
|
|
| Zitat: |
Original von Kevin33
Zuerst mal noch eine Frage das mit dem m_lostTotal+=1 , sollte ich doch machen oder ?
Das was du willst ist ja : m_goalsReceivedTotal++;
Und in dem anderen Fall m_goalsScoredTotal++, machen .
Muss ich diese beide Sachen zusätzlich machen ?
Weil das stand ja nicht so in der Aufgabe
|
Nein, Du verstehst ja schon wieder gar nichts! Was hab ich oben geschrieben, was macht += 1, was macht ++? Was willst Du mit den einzelnen Membervariablen machen?
Nochmal von vorne: mache mir zuerst eine Tabelle schreibe in die Spalten die fünf Membervariablen. Dann mache vier Zeilen mit 3:2 gewonnen, 2:2 unentschieden, 1:2 verloren und 0:0 unentschieden. Schreibe für die 4 Ergebnisse jeweils die Werte der Variablen auf, so wie wenn diese Spiele nacheinander stattgefunden hätten.
Mach die Tabelle, dann sehen wir weiter!
|
|
|
22.10.2017 23:20 |
|
|
Kevin33
unregistriert
|
|
| 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:
|
#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_goalsScoredTotal++;
}
if(goalsScored<goalsReceived ){
m_lostTotal += 1;
m_goalsReceivedTotal += goalsReceived;
m_goalsReceivedTotal++;
}
if( goalsScored== goalsReceived){
m_equalTotal += 1;
m_equalTotal++;
}
}
|
|
So würde das dann als Code aussehen .
Die fragen sind ja im vorletzten Beitrag 
|
|
|
22.10.2017 22:28 |
|
|
|
as_string
Haudegen
Dabei seit: 06.11.2013
Beiträge: 638
Herkunft: Heidelberg
|
|
| Zitat: |
Original von Kevin33
Ich habe den weiteren Aufgabenteil auch schon ein wenig bearbeitet :
Mal sehen ob ich die Funktion richtig implementiert hab
| code: |
1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
|
unsigned int const CTeam::getTotalPoints(){
int tmp = 0;
if(m_goalsScoredTotal>m_goalsReceivedTotal ){
tmp += 3;
return tmp;
}
if(m_goalsScoredTotal<m_goalsReceivedTotal ){
return tmp;
}
if(m_goalsScoredTotal==m_goalsReceivedTotal ){
tmp += 1;
return tmp;
}
|
|
|
Auch wieder komplett falsch. Das ist schon ziemlich heftig. In der Aufgabe ist recht genau beschrieben, wie man das rechnen muss und jeder, der schon mal Bundesliga geschaut hat, kennt die Berechnungsmethode.
Um die Punkte in der Tabelle auszurechnen, interessieren da die gesamten Tore und/oder Gegentore? Warum vergleichst Du die denn? Was ist in der Aufgabe geschrieben, wie das berechnet werden sollte? Da steht: "für jedes gewonnene Spiel erhält das Team 3 Punkte". Wieviele Spiele hat das Team denn gewonnen, in welcher Membervariable steht das denn drin? Hast Du überhaupt irgendetwas von dem verstanden, was wir hier machen???
Also, das was Du ablieferst hat nichts mit "ich kann halt nicht programmieren" zu tun. Noch nicht mal mit Faulheit, glaube ich. Da fehlts einfach irgendwo komplett. Ich mache mir da keine Hoffnungen mehr...
|
|
|
22.10.2017 23:33 |
|
|
|
Kevin33
unregistriert
|
|
| Zitat: |
Original von as_string
| Zitat: |
Original von Kevin33
Ich habe den weiteren Aufgabenteil auch schon ein wenig bearbeitet :
Mal sehen ob ich die Funktion richtig implementiert hab
| code: |
1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
|
unsigned int const CTeam::getTotalPoints(){
int tmp = 0;
if(m_goalsScoredTotal>m_goalsReceivedTotal ){
tmp += 3;
return tmp;
}
if(m_goalsScoredTotal<m_goalsReceivedTotal ){
return tmp;
}
if(m_goalsScoredTotal==m_goalsReceivedTotal ){
tmp += 1;
return tmp;
}
|
|
|
Auch wieder komplett falsch. Das ist schon ziemlich heftig. In der Aufgabe ist recht genau beschrieben, wie man das rechnen muss und jeder, der schon mal Bundesliga geschaut hat, kennt die Berechnungsmethode.
Um die Punkte in der Tabelle auszurechnen, interessieren da die gesamten Tore und/oder Gegentore? Warum vergleichst Du die denn? Was ist in der Aufgabe geschrieben, wie das berechnet werden sollte? Da steht: "für jedes gewonnene Spiel erhält das Team 3 Punkte". Wieviele Spiele hat das Team denn gewonnen, in welcher Membervariable steht das denn drin? Hast Du überhaupt irgendetwas von dem verstanden, was wir hier machen???
Also, das was Du ablieferst hat nichts mit "ich kann halt nicht programmieren" zu tun. Noch nicht mal mit Faulheit, glaube ich. Da fehlts einfach irgendwo komplett. Ich mache mir da keine Hoffnungen mehr... |
Das Problem ist das ich in der Funktion ,also in der if V`Bedingung das goalsReceived und goalsscored nicht benutzen konnte ?
Da er es in der Funktion nicht erkannte .
Sonst könnte man eigentlich die gleichen if Bedingungen fast wie bei addrESULT NUTZENß
|
|
|
22.10.2017 23:39 |
|
|
|
|
  |
|
