ich hab mir zwei klassen gemacht welche eine Musiksammlung realisieren sollen.
Beim compilieren bekomme ich in der zeile

list<CLied> m_musiksammlung;  // erzeugt eine leere Liste des Typs CLied

in der klasse CMusiksammlunn folgende Fehldermeldung: ISO C++ forbids declaration of 'list' with no type.

Weiß jemand was ich da falsch mache?

LG

using namespace std;

class CLied {
  bool found;
  string m_type;
  string m_titel;
  string m_interpret;
  string m_info;

  public:
    CLied () : m_type(""), m_titel(""), m_interpret(""), m_info("") {
      }

    CLied (string type, string titel, string interpret, string info)
      : m_type(type), m_titel(titel), m_interpret(interpret), m_info(info) {
    }

    /* Kopier-Konstruktor */
    CLied (CLied& lied)
      : m_type(lied.m_type), m_titel(lied.m_titel), m_interpret(lied.m_interpret), m_info(lied.m_info) {
    }

    string getLied () {
      return(m_type+", "+m_titel+", "+m_interpret+", "+m_info);
    }

    bool getFound () {
      return (found);
    }

    void resetFound() {
      found = TRUE;
    }

    bool isFoundInType (string type) {
      if ( m_type.find(type,0) == string::npos ) {
        found = FALSE;
        return (found);
      }
      else {
        return (found);
      }
    }

    bool isFoundInTitel (string titel) {
      if ( m_titel.find(titel,0) == string::npos ) {
        found = FALSE;
        return (found);
      }
      else {
        return (found);
      }
    }

    bool isFoundInInterpret (string interpret) {
      if ( m_interpret.find(interpret,0) == string::npos ) {
        found = FALSE;
        return (found);
      }
      else {
        return (found);
      }
    }

};

class CMuskisammlung {
  list<CLied> m_musiksammlung;  // erzeugt eine leere Liste des Typs CLied
};

Ich hab einfach nur die zeile

#include <list>

vergessen.

LG

CLied (string type, string titel, string interpret, string info) 
      : m_type(type), m_titel(titel), m_interpret(interpret), m_info(info) { 
    } 

    /* Kopier-Konstruktor */ 
    CLied (CLied& lied) 
      : m_type(lied.m_type), m_titel(lied.m_titel), m_interpret(lied.m_interpret), m_info(lied.m_info) { 
    }

besser:

CLied (const string& typ, const string& titel, const string& interpret, const string& info)
...

// Sowie
CLied (const CLied& lied)
...

Und mische nicht dauernd englisch und deutsch sondern einige dich auf eine Sprache.

MfG SideWinder

Allerdings stehe ich leider vor meinem nächsten Problem:

class CMusiksammlung {
  list<CLied> m_musiksammlung;  // erzeugt eine leere Liste des Typs CLied
  list<CLied>::iterator m_itor_musiksammlung;

  public:
    CMusiksammlung () {
      m_itor_musiksammlung = m_musiksammlung.begin(); // Iterator an den Beginn der Liste
    }

    ~CMusiksammlung () {
      m_musiksammlung.clear(); // Inhalt der Liste löschen, Speicherplatz freigeben
      }

    void addLiedToMusiksammlung (string type, string titel, string interpret, string info) {
      CLied temp_lied (type, titel, interpret, info);
      cout << temp_lied.getLied() << endl << endl;
      m_musiksammlung.push_back( temp_lied );
    }
};

Wenn ich die Funktion "addLiedToMusiksammlung" nun aus dem main mit

CMusiksammlung meine_musiksammlung;
  meine_musiksammlung.addLiedToMusiksammlung("Pop", "Rocket Man", "Elton John", "asdf");

aufrufe, dann kommt eine ganz liste von compiler fehlern. bei der Zeile

m_musiksammlung.push_back( temp_lied );

Bin leider neu auf dem Gebiet der Objektorientieren Programmierung. Bin also für jede Hilfe dankbar.

LG

Es wäre natürlich sehr hilfreich die Fehlermeldungen zu posten
Als einzigstes Problem sehe ich den Kopierkonstruktor von CLied, den die Referenz muß als const übergeben werden, sonst kommt die list-interne Verwaltung nicht damit zurecht wenn Kopieen erzeugt werden müßen.

/* Kopier-Konstruktor */
    CLied (const CLied& lied)

Siehe auch Const Correctness

bis bald
akari

hab mich leider bist jetzt immer gegen const gewehrt weil ich es für überflüssig gehalten habe. danke du hast mich vom gegenteil überzeugt.

LG

... mach das mal so ^^ ist ein wenig logischer ... vor allem das mit m_found als Membervariable von CLied war nicht so sinnvoll ...

vielen dank jedenfalls,

LG