Objekte dynamisch erzeugen, in (z. B.) Vector schreiben, auf die Objekte zugreifen

Cordula

@brotbernd: Wow! Vielen Dank für die Mühe die du dir gemacht hast Ich werde schon Klassen nehmen, werde also dein Code anpassen

Nur so zur Info - ist das mit dem map sehr schlecht, abgesehen davon dass es viel uneffektiver ist als find_if? Ich will was lernen Danke nochmal.

brotbernd

find_if ist eine simple Schleife die alle Elemente im vector anguckt, bis was passendes gefunden ist. Es kommt drauf an, was du mit effizient meinst. Das Suchen geht in der map viel schneller. Dafür dauert das Einfügen länger und mehr Speicher wird benötigt. Wenn Du eine Datei einliest und danach keine Elemente mehr hinzukommen, wäre ein sortierter Vector wohl am effizientesten.
Wenn du von Programmierfiizienz redest, ist die einfachste Lösung die beste und das ist wohl vector + find_if.

Vicious Falcon

brotbernd schrieb:

menschen.push_back(Mensch("Chantalle", Mensch::Mann, 185));

Das sind doch die, die in Nachtbars als "Bardamen" arbeiten, oder?

Edit: Sorry, offtopic

Cordula

Danke für die ausführliche Erklärung

Den Code hab ich auch schon _fast_ angepasst

Einen Fehler werd ich nicht los ..

invalid use of non-static member function 'bool Read::Fraumit7Fingern(const Mensch&)'

Jemand ne Idee?

Michael E.

Du rufst die Funktion ohne Instanz auf:

Mensch foo;

// du machst:
Read::Fraumit7Fingern(foo);

// richtig wäre, da es sich nicht um eine statische Funktion handelt:
Read reader;
reader.Fraumit7Fingern(foo);

Das ist aber nur ne Vermutung, weil du keinen Quelltext zeigst.

Cordula

Danke, das hat gestimmt.

Aber danach bekomm ich trotzdem andere Fehler zurück. Hier der Code:

vector<Mensch> MenschVec;
vector<Mensch>::iterator itMe;

MenschVec.push_back(Mensch(attr1, attr2, attr3));
MenschVec.push_back(Mensch(attr4, attr5, attr6));
MenschVec.push_back(Mensch(attr7, attr8, attr9));

std::vector<Mensch>::const_iterator itMe = std::find_if(MenschVec.begin(), MenschVec.end(), FrauMit7Fingern);
cout << itMe -> Haare << endl;

Also soll die Frau mit den 7 Fingern gefunden werden und ihre Haarfarbe zurückgegeben werden.
Nach der Korrektur heißt es:

'itMe' has previous declaration as '__gnu_cxx::__normal_iterator<Mensch*,std::vector<Mensch,std::allocator<Mensch>>>'
No matching function for call to 'Read::Fraumit7Fingern'
Conflicting declaration '__gnu_cxx::__normal_iterator<const Mensch*std::vector<MEnsch,std::allocator<Mensch>>>itMe'

Braunstein

Zum 1. und 3. Fehler: du hast itMe 2mal deklariert. Lösch einfach vector<Mensch>::iterator itMe;

zum 2. Fehler: Dein Compiler findet Read::Fraumit7Fingern' nicht.
Dazu kann man wegen fehlenden Code nichts sagen.

Cordula

Ich möchte nicht übertreiben - aber ich bin grad schon übermüdet, also eine letzte Frage:

Was heißt

argument of type 'bool (Read:: )(const Mensch&)' does not match 'bool (Read::*)(const Mensch&)'

Ich hab eine Erklärung über calling a member function using a pointer-to-member function gefunden, aber ehrlich gesagt hab ich sie nicht wirklich verstanden. Und da ich damit keine Erfahrung habe ist es noch schwieriger

Danke für eure Antworten.

Braunstein

Scheinbar rufst du FrauMit7Fingern nicht korrekt auf.
Warum muß diese Funktion denn eine Memberfunktion sein?

Ich bevorzuge an der Stelle eigentlich Funktoren. Also etwa so:

struct FrauMitAnzahlFingern {
 private:
  int m_anzahl;
 public:
  FrauMitAnzahlFingern(int anzahl) : m_anzahl(anzahl) {}
  bool operator()(const Mensch& mensch) {
    return mensch.Geschlecht() == Mensch::Frau && mensch.Finger() == m_anzahl;
  }
};

// Aufruf dann so
std::vector<Mensch>::const_iterator itMe = std::find_if(MenschVec.begin(), MenschVec.end(), FrauMitAnzahlFingern(7));

Da kann man dann auch gleich nach anderen Anzahlen suchen. Außerdem hat es der Compiler etwas leichter mit dem Optimieren.

Cordula

Läuft

Find ich auch übersichtlicher mit dem Funktor

Danke vielmals, und noch nen schönen Abend.
C.