iterator

Ist dir klar das du hier:

for(int i=0;i<=10; ++i)
    cout<<coeff.values()[i]<<endl;

jedes mal die Funktion values() aufrufst. D.h. jedesmal ein Objekt zurückgeben lässt.
Was natürlich nicht gerade effizient ist.

Try this

int main(){
    Coefficients coeff;
    coeff.read("myCoeff.txt");

    vector<double> val = coeff.values();

    for(int i=0;i<=10; ++i)
           cout<<val[i]<<endl;

    for(vector<double>::iterator p_h = val.begin(); p_h != val.end(); ++p_h)
           cout<< *p_h <<endl;

}

THX 1138

Eins versteh ich noch nicht.
Willst Du immer genau 10 Werte ausgeben, oder bis ans Ende des Vectors?
Du solltest beim Iterator auf jedenfall immer prüfen, ob Du nicht schon am Ende des Vectors angekommen bist.

ssm

Deine Get Methode ist fehlerhaft
Versuch mal so:

const vector<double> &values() const
{
   return _values;
}

ssm schrieb:

Deine Get Methode ist fehlerhaft

Wieso ???

Sie kopiert das Objekt und du gibst eine Konstante Referenz darauf zurück ...

carmen

@ssm
und es funktioniert

DANKE!

Kannst du bitte mir erklären, warum kam zur richtigen Ausgabe mit cout<<coeff.values()[i] und fehlerhaft mit cout<<*p_h

DANKEDANKEDANKE

THX 1138

Freak_Coder schrieb:

ssm schrieb:

Deine Get Methode ist fehlerhaft

Wieso ???

Sie kopiert das Objekt und du gibst eine Konstante Referenz darauf zurück ...

Das kopieren ist aber nicht "schön".

ssm

Freak_Coder schrieb:

Wieso ??? Sie kopiert das Objekt und du gibst eine Konstante Referenz darauf zurück ...

Weil carmen wollte, dass das richtig funktioniert:

for(vector<double>::iterator p_h = coeff.values().begin(); 
                p_h != coeff.values().end(); p_h++) 
{ 
   cout  << *p_h << endl; 
}

oder?

carmen

@ ssm Ja es funktioniert und meine algorithmen werden auch in der Klasse Filter mit iteratoren funktionieren, aber ich möchte verstehen warum???? sonst werde ich den Fehler wiederholen

DANKE ssm

ssm

http://www.c-plusplus.net/forum/viewtopic-var-t-is-134700.html

carmen

dafür habe ich keine Zeit

LordJaxom

Weil in der alten Version coeff.values() eine Kopie ist. Davon (von der Kopie!) holst Du Dir den begin() Iterator. Danach wird die Kopie gelöscht, sie wurde ja nur gebraucht um .begin() anzuwenden. Der Iterator zeigt nun auf eine nicht (mehr) vorhandene Collection. Der Rest ergibt sich

carmen

AH!Verstanden! aber warum brauche ich const vector<double>& und nicht nur vector<double> &??

Wenn du kein const davor hast, kann jeder der die Referenz erhält das Objekt, d.h deine PRIVATE Variable ändern !