warum bekomme ich nur die Speicheradressen?!?

thx4help

#include <vector>
#include <string>
#include <iostream>
struct DiagRun
{
       DiagRun(Koord startpos, int sc, int lae){ //Constructor
           int score= sc;
           int laenge = lae;
           Koord start = startpos;                     
       }

       int score;
       int laenge;
       Koord start;              
};

struct Koord {  
      Koord(){
             int m,n;
             } 
    Koord(int x, int y){
        m= x;
        n= y;
    }

    int m,n;
};

int main(){

    Koord temp (3,4); 
    Koord start1 (5,7); 

    vector<DiagRun> testvector;
    DiagRun temp2(start1, 15, 12); 
    testvector.push_back(temp2);

    cout << testvector.at(0).start.m; 
        //hier sollte 5 ausgegeben werden (start1.m)
        //es wird eine 7-stellige Zahl ausgegeben (vermutlich eine Adresse?!?)
}

Wieso hast du in den Konstruktoren die Variablen nochmal neu definiert:

DiagRun(Koord startpos, int sc, int lae){ //Constructor
           int score = sc;
           int laenge = lae;
           Koord start = startpos;                    
       }
...
Koord(){
             int m,n;
             }

Sie sind ja schon Elemente der Klasse und wenn du sie nochmal in den Konstruktoren neu definierst und ihn dann was zuweist, dann werden nur den lokalen Variablen was zugewiesen die anderen Elementvariablen werden nicht beachtet, da sie von den lokalen überdeckt werden.

Behebe diesen und paar andere kleine Fehler und es sollte Funktionieren

ich habe copy&Paste gemacht und danch nicht mehr gecheckt, ob es funktioniert! jetzt wird es zwar kompeliert, doch wird immer noch nicht das richtige ergebnis ausgegeben. Anstatt der korrekten Ausgabe 5 kommt jetzt ne null!!

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

using namespace std;

struct Koord { 
        int m;
        int n;
      Koord(){
             int m,n;
             } 

    Koord(int x, int y){
        m= x;
        n= y;
    }

    bool Koord::lt(const Koord& k) const;

     void init (){
          m=0;
          n=0;
     }

};

struct DiagRun
{
       DiagRun(Koord startpos, int sc, int lae){ //Constructor
           int score= sc;
           int laenge = lae;
           Koord start = startpos;                     
       }

       int score;
       int laenge;
       Koord start;              
};

int main(){    

    vector<DiagRun> testvector;
    Koord temp (3,4);
    Koord start1 (5,7);
    DiagRun temp2(start1, 15, 12);
    testvector.push_back(temp2);

    cout << testvector[0].start.m;
        //hier sollte 5 ausgegeben werden (start1.m)
        //es wird nun eine 0 ausgegeben (vorher wars ne Speicheradresse...)
}

PS: ich wusste nicht, dass man beim Constructor die Variablen nicht neu deklarieren muss!!

THX

Koord(){
             int m,n;
             }

Was bringt das?

DiagRun(Koord startpos, int sc, int lae){ //Constructor
           int score= sc;
           int laenge = lae;
           Koord start = startpos;                    
       }

???

mach das erste mal ganz weg und ändere das Zweite in

DiagRun(Koord startpos, int sc, int lae)
{
           score= sc;
           laenge = lae;
           start = startpos;                    
}

struct Koord { 
        int m;
        int n;
      Koord(){
             int m,n;
             } 
           
    Koord(int x, int y){
        m= x;
        n= y;
    }

und bei Konstruktoren immer schön Initialisierungslisten verwenden. Dann passieren solche Fehler erst gar nicht.

struct Koord { 
    int m;
    int n;

    Koord()
        : m( 0 )
        , n( 0 ) 
    {}

    Koord(int x, int y)
        : m( x )
        , n( y ) 
    {}

K.A. schrieb:

bool Koord::lt(const Koord& k) const;

ich unterstelle mal, dass 'lt' "less than" heißen soll. Dann mach doch gleich

bool Koord::operator<(const Koord& k) const;

wobei man 'Koord::' innerhalb der Klassen-Definition i.a. wegläßt.

K.A. schrieb:

void init (){
          m=0;
          n=0;
     }

solche init-Methoden sind immer ein Hinweis auf ein Fehl-Design. Vorschlag: weglassen

Gruß
Werner

DiagRun(Koord startpos, int sc, int lae)

nicht den = Operator implementieren?

Muss man wegen

DiagRun(Koord startpos, int sc, int lae)

nicht den = Operator implementieren?

Man muss nie aufgrund der Konstruktor-Signatur einen Zuweisungsoperator implementieren.