Array beschreiben und auslesen

artyfical

Hallo liebe Forengemeinde,

vllt finde ich bei euch die benötigte Idee mein Problem umzusetzen, bisher erhalte ich nur mir unverständliche Fehler bei VS2008. Folgendes Problem ich möchte in ein Array einen neuen Spieler eintragen lassen und mir dieses Array dann wiederum auf der Konsolehttp://forum.chip.de/# ausgeben lassen, nur steh ich bei der Methodenprogrammierung etwas auf dem Schlauch. Vielen Dank für eure Mühe im Vorraus.

Problemmethoden in CKader.cpp kommentiert:

C

// Kader.cpp


#include "CKader.h"
#include "CSpieler.h"
#include <iostream>


using namespace std;


    // Konstruktor
    CKader::CKader(char* vorname, char* nachname, char* geboren, unsigned int groesse, unsigned int gewicht, unsigned int nummer, char* verein, unsigned int seit, E_POSITION position)
    {

    }
    // Destruktor
    ~CKader::CKader()
    {
        delete spielerArray[i];
    }


    // Methoden
     bool CKader::AddSpieler() 
    {
        for(i = 0; i < cMaxSpieler; i++)
      {
          //Array Spieler hinzufügen
          spielerArray[i]= new CSpieler();
      }
    return true;
    }
    bool CKader::PrintKader()
    {
        for (int i = 0; i < cMaxSpieler; i++)
    {
        // Array auf Bildschirm ausgeben
        spielerArray[i]->CSpieler::Print();
    }
    return true;
    }

Alles anzeigen

Code

// CKader.h


const int cMaxSpieler = 50;
int i;


class CKader {


public: 


    enum E_POSITION {Torwart, Abwehr, Mittelfeld, Sturm, Ersatzbank};
    int *spielerArray[];


    //Konstruktor 
    CKader(char* vorname, char* nachname, char* geboren, unsigned int groesse, unsigned int gewicht, unsigned int nummer, char* verein, unsigned int seit, E_POSITION position); 
    ~CKader();


    //Methoden
    bool AddSpieler();;
    bool PrintKader();
};

Alles anzeigen

Code

// CSpieler.h


const int cMaxStrLength = 50;    


class CSpieler {        


public:
        char vorname[cMaxStrLength];
        char nachname[cMaxStrLength];
        char geboren[cMaxStrLength];


        unsigned int groesse_cm;
        unsigned int gewicht_kg;


        unsigned int aktNummer;
        char aktVerein[cMaxStrLength];
        unsigned int aktVereinSeit_jahr;


        enum E_POSITION 
        {
            Torwart, Abwehr, Mittelfeld, Sturm, Ersatzbank
        };
        E_POSITION aktPosition;


        //Konstruktor 
        CSpieler(char* vorname, char* nachname, char* geboren, unsigned int groesse, unsigned int gewicht);
        ~CSpieler();


        //Methoden
        bool SetNummer(unsigned int nummer);
        bool SetVerein(char* verein, unsigned int seit);
        bool SetPosition(E_POSITION position);
        bool Print();
};

Alles anzeigen

C

// CSpieler.cpp


#include "CSpieler.h"
#include <iostream>


using namespace std;


//Konstruktor
    CSpieler::CSpieler(char* vorname, char* nachname, char* geboren, unsigned int groesse, unsigned int gewicht)
    {
        strcpy(this->vorname, vorname);        
        strcpy(this->nachname, nachname);    
        strcpy(this->geboren, geboren);        
        this->groesse_cm = groesse;            
        this->gewicht_kg = gewicht;
        this->aktNummer = 00;
        strcpy(this->aktVerein, "Kein Verein");
        this->aktVereinSeit_jahr = 0001;
        this->aktPosition = Ersatzbank;
    }


    //Methoden
    bool CSpieler::SetNummer(unsigned int nummer) {  this->aktNummer = nummer; return true; }
    bool CSpieler::SetVerein(char *verein, unsigned int seit){ strcpy(this->aktVerein, verein);    this->aktVereinSeit_jahr = seit; return true; }
    bool CSpieler::SetPosition(E_POSITION position) {  this->aktPosition = position; return true; } 
    bool CSpieler::Print()
    {
        cout << "Spieler Vorname: " << vorname << "\n" << endl;    
        cout << "Spieler Nachname: " << nachname << "\n" << endl;
        cout << "Spieler wurde geboren am: " << geboren << "\n" << endl;
        cout << "Spieler Groesse in cm: " << groesse_cm << "\n" << endl;
        cout << "Spieler Gewicht in kg: " << gewicht_kg << "\n" << endl;
        cout << "Spielernummer: " << aktNummer << "\n" << endl;
        if (CSpieler::aktPosition == Torwart) { cout << "Position: " << "Torwart\n" << endl; }
        else if (CSpieler::aktPosition == Abwehr) { cout << "Position: " << "Abwehr\n" << endl; }
        else if (CSpieler::aktPosition == Mittelfeld) { cout << "Position: " << "Mittelfeld\n" << endl; }
        else if (CSpieler::aktPosition == Sturm) { cout << "Position: " << "Sturm\n" << endl; }
        else if (CSpieler::aktPosition == Ersatzbank) { cout << "Position: " << "Ersatzbank\n" << endl; }
        cout << "Spielt seit Jahr: " << aktVereinSeit_jahr << " im Verein: " << aktVerein << "\n" << endl; 
        return true; 
    }

Alles anzeigen

The User

Hmm, warum hat der Kader ein Gewicht, ein Geburtsdatum usw.??
Also ich würde als erstes darauf verzichten, arrays aus Pointern zu verwenden, vorallem müssten es Pointer auf CSpieler und nicht auf int sein.
Benutz doch einen vector:
In der Klasse:

Code

vector<CSpieler> spielerArray;

Du kannst einen Vektor beliebig lang machen mit push_back:

Code

void CKader::AddSpieler() // void ist sinnvoll ;)
    {
        for(i = 0; i < cMaxSpieler; i++)
      {
          spielerArray.push_back(CSpieler());
      }
    }

Einfacher wäre es im Konstruktor:

Code

CKader::CKader() : spielerArray(cMaxSpieler)
{


}

Habe mir die Freiheit genommen, die Position und das Gewicht etc. zu entfernen, kannst du natürlich auch wieder einbauen.
Anstelle von char*, bzw char[] solltest du strings benutzen:

Code

string vorname, nachname, geboren, aktVerein;

Du solltest das keins Falls vergessen:

C

#include <vector>
#include <string>

Bool Rückgaben sind ziemlich blödsinnig, wenn es sowieso immer true ist, benutze lieber void als Rückgabewert und spar dir das return true;.
Das mit dem Print ist eigentlich okay, es gibt allerdings noch eine praktischere Variante:

Code

ostream& operator<<(ostream& out, CSpieler s)
{
        out << "Spieler Vorname: " << s.vorname << endl; // Das endl sorgt schon für einen Zeilenumbruch!    
        out << "Spieler Nachname: " << s.nachname << endl;
        out << "Spieler wurde geboren am: " << s.geboren << endl;
        out << "Spieler Groesse in cm: " << s.groesse_cm << endl;
        out << "Spieler Gewicht in kg: " << s.gewicht_kg << endl;
        out << "Spielernummer: " << s.aktNummer << endl;
        if (CSpieler::aktPosition == Torwart) { out << "Position: " << "Torwart" << endl; }
        else if (CSpieler::aktPosition == Abwehr) { out << "Position: " << "Abwehr" << endl;
        else if (CSpieler::aktPosition == Mittelfeld) { out << "Position: " << "Mittelfeld" << endl; }
        else if (CSpieler::aktPosition == Sturm) { out << "Position: " << "Sturm" << endl; }
        else if (CSpieler::aktPosition == Ersatzbank) { out << "Position: " << "Ersatzbank" << endl; }
        out << "Spielt seit Jahr: " << aktVereinSeit_jahr << " im Verein: " << aktVerein << endl; 
        return out;
}

Alles anzeigen

Die hat einen praktischen Vorteil: Du kannst nun direkt cout << spieler aufrufen.
Durch den Aufruf von cout << spieler wird diese Funktion dann mit cout als Parameter out aufgerufen, in das cout werden dann die Ausgaben geschrieben.
Eine Zeile musst du dann allerdings in die Klasse CSpieler einfügen:

Code

friend ostream& operator<<(ostream&, CSpieler);

Das bewirkt, dass die Ausgabe-Funktion auf die privaten Elemente zugreifen darf.
Noch ein Denkanstoß:
Soll aktVerein wirklich nur eine Zeichenkette sein, oder lieber ein Zeiger auf einen Verein (ist CKader ein Verein, muss da eben stehen CKader* aktVerein, bei CVerein eben entsprechendes...).

Es ist sehr schön, dass du die tolle Objektorientierung aus C++ verwendest, allerdings nimm es als allgeminen Ratschlag, in nicht Hardware-nahen Sachen, auf char*, arrays und übermäßig viele Pointer zu verzichten. Das bringt nur Probleme mit sich, Alternativen sind eben strings und vectoren.

Viele liebe Grüße
The User

PS:
Schonmal code::Blocks probiert?

Unregistriert

Erstmal danke schön und es hat geholfen. Leider verlangt es meine diese unnützen Bool-Rückgaben zu verwenden, da ich sie auch nicht für sehr sinnig halte. Zum Thema Objektorientierung, ich komme aus der Javaecke ;D

artyfical

Beitrag oben von mir und code::Blocks werd ich mal antesten. Danke

The User

Wer ist denn "meine"??
Also false würde ich vielleicht zurückgeben, wenn ein Fehlerfall halbwegs wahrscheinlich ist, z. B. wenn man eine Datei einlesen möchte und die Datei nicht existiert oder so. Aber wenn das wirklich eine Ausnahme ist, würde ich auch zur Ausnahmenbehandlung greifen. Die kommen ja von C++, übrigens kannst du in C++ alle Datentypen werfern, sie müssen nicht throwable sein, das throws kann man sich auch sparen, möchte man es dennoch haben, geht es so:

Code

void fehlerAnfaelligeFunktion() : throw(exception, runtime_error, bool, my_exception);

Ich komme von C++ und benutze ab und an Java, du wirst bestimmt auch noch tolle Sachen in C++ kennenlernen, die in Java nicht gehen.

artyfical

Ahh Text vergessen. Mit meine, meinte ich meinen Prof.

Tags