In C++ Dantenströme übergeben

[HasProblemsWithCpp (Gast)]

Tschou Zämä

Ich versuche, ein Struct für eine imaginäre Zahl ein- und auszulesen. 
Dafür möchte ich iostream verwenden und diesen in die Methode übergeben. 
Irgendwie geht's nicht und die Fehlermeldung hilft mir auch nicht 
weiter...
Wer findet das Problem?
Vielen Dank im Voraus!

#include <iostream>
#include <cmath>
#include <cstring>
using namespace std;
typedef string::iterator itStr;
double epsilon = 0.00001;

struct cNumb{
    double a; // a+
    double b; //b*i
};

cNumb add(const cNumb c1, const cNumb c2);
cNumb operator + (const cNumb c1, const cNumb c2);
cNumb sub(const cNumb c1, const cNumb c2);
cNumb operator - (const cNumb c1, const cNumb c2);
cNumb multiply(const cNumb c1, const cNumb c2);
cNumb operator * (const cNumb c1, const cNumb c2);
cNumb complement(const cNumb c1);
double radOrAbs(const cNumb c1); // radius == Betrag
// Divisor, Dividend
cNumb divise(const cNumb c1, const cNumb c2);
cNumb operator / (const cNumb c1, const cNumb c2);
bool operator ==(const cNumb c1, const cNumb c2);
bool operator != (const cNumb c1, const cNumb c2);

cNumb read(istream& in, const cNumb c1);
//cNumb operator >> (istream& in, const cNumb c1);

void write( ostream& out,const cNumb c1);
//void operator << ( ostream& out, const cNumb c1);

int main(){
    bool run = true;
    while(run){
        cNumb c; 
        read(&cin, c); 
        write(&out, c); 
        cout << "\nWieder?";
        cin >> run;
    }
    return 0;
}

void write(ostream& out, const cNumb c1){
    (&out) << " " << c1.a << " + "<< c1.b << "i";
}
/*
void operator << (ostream& out, const cNumb c1){
    write(out, c1);
}
*/
cNumb read(istream& in, const cNumb c1){
    string input;
    &in >> input;
    c1.a = 0;
    c1.b = 0;
    bool readingA = true;
    bool readingB = false;
    bool aNegative;
    bool bNegative;

    for(itStr i = input.begin(); i<input.end(); i++){
        if(readingA && *i== '-'){
            aNegative = true;
        }
        if(readingA && *i>47 && *i<58){
            c1.a = c1.a*10 +*i;
        }else{
            readingA = false;
            readingB = true;
        }
        if(readingB && *i== '-'){
            bNegative = true;
        }
        if(readingB&& *i>47 && *i<58){
            c1.b = c1.b*10 +*i;
        }else if(*i == 105){
            break;
        }
        // Alternative: Ganz normal output : cout << "a = "; cout << "b = "
    }
}

[Mark B. (markbrandis)]

g++ -Wall -Wextra main.cpp -o main.exe

main.cpp: In function `int main()':
main.cpp:37: error: invalid initialization of non-const reference of type 'std::istream&' from a temporary of type 'std:
:istream*'
main.cpp:27: error: in passing argument 1 of `cNumb read(std::istream&, cNumb)'
main.cpp:38: error: `out' was not declared in this scope
main.cpp:38: warning: unused variable 'out'
main.cpp: In function `void write(std::ostream&, cNumb)':
main.cpp:46: error: invalid operands of types `std::ostream*' and `const char[2]' to binary `operator<<'
main.cpp: In function `cNumb read(std::istream&, cNumb)':
main.cpp:55: error: no match for 'operator>>' in '+in >> input'
main.cpp:56: error: assignment of data-member `cNumb::a' in read-only structure
main.cpp:57: error: assignment of data-member `cNumb::b' in read-only structure
main.cpp:68: error: assignment of data-member `cNumb::a' in read-only structure
main.cpp:77: error: assignment of data-member `cNumb::b' in read-only structure

Na nu komm aber. Zumindest ein paar von diesen Fehlern kannst Du selbst 
beheben.

[Klaus W. (mfgkw)]

Zudem sollte man ihm vielleicht noch verraten, daß man das eh mit 
Überladen des Operators << wesentlicher eleganter und C++-konformer 
machen kann?

[Rolf Magnus (Gast)]

Mal ganz davon abgesehen, daß es in C++ schon ein Template für komplexe 
Zahlen gibt. Da sind diese ganzen Operationen und noch einige mehr schon 
definiert.
Siehe http://www.cplusplus.com/reference/complex/ und
http://www.cplusplus.com/reference/complex/complex/operators/

[HasProblemsWithCpp (Gast)]

Es geht um die Theorie, darum, das Struct zu verstehen etc.
Den Operator << möchte ich später auch noch einführen, für Addition und 
weiteres habe ich das schon gemacht.
Was sind denn diese 'offensichtlichen' Fehler?

Von Datenströmen verstehe ich noch recht wenig.

[Matthias H. (hallamen)]

HasProblemsWithCpp schrieb:
> Von Datenströmen verstehe ich noch recht wenig.

Du meintest wohl "Von C++ verstehe ich noch recht wenig." :)

Zu den fehlern:
In den Zeilen 37,38,46,55:
Das '&' vor den Objekten weglassen bzw. das Kapitel 'Referenzen' in 
einem C++ Buch deiner Wahl durchlesen.

Zeilen 56,57,68,77:
'const' sagt dem Compiler, dass ein Parameter einer Funktion nicht 
verändert wird, du versuchst es aber trotzdem...

[Karl H. (kbuchegg)]

Matthias H. schrieb:

> 'const' sagt dem Compiler, dass ein Parameter einer Funktion nicht
> verändert wird, du versuchst es aber trotzdem...


Wobei die Sache hier noch weiter geht. Das ist die bewusste Funktion

cNumb read(istream& in, const cNumb c1){
....

Wenn die Funktion read ein cNumb Objekt als Returnwert liefert, wozu 
braucht es dann noch eines als Funktionsargument? Was macht es damit? 
Wozu ist dieses nötig?

Bei write ist das so ok, denn write muss ja wissen, welches cNumb Objekt 
ausgegeben werden soll. D.h. das muss man dem übergeben. Aber read?

Mir scheint, da gibt es noch mehr Probleme ganz banalerer Natur als nur 
Strukturen.

[Karl H. (kbuchegg)]

cNumb read(istream& in, const cNumb c1){
    string input;
    &in >> input;

Die Sache mit den Referenzen hast du nicht wirklich verstanden. Oder?
Das Referenz-& in der Argumentliste bedeutet nicht, dass du einfach dann 
überall ein & schreiben musst. Ganz im Gegenteil.

[PittyJ (Gast)]

Warum nimmt man ein struct für complexe Zahlen und nicht gleich eine 
Klasse?

[Dr. Sommer (Gast)]

PittyJ schrieb:
> Warum nimmt man ein struct für complexe Zahlen und nicht gleich eine
> Klasse?
Um das "public:" zu sparen :D

[Fred (Gast)]

Ein struct ist doch auch nur eine Klasse.

[Mark B. (markbrandis)]

/me empfiehlt die Lektüre eines C++ Buches

[Mark B. (markbrandis)]

A propos, was sind eigentlich Dantenströme? Hat das was hiermit zu tun?

http://de.wikipedia.org/wiki/G%C3%B6ttliche_Kom%C3%B6die

;-)

[HasProblemsWithCpp (Gast)]

Es geht nicht um eine alte Komödie, aber auch mir scheint Kerberos den 
Eintritt in die C++-Welt zu verwehren.

Vieles konnte ich beheben! Ein Fehler bleibt:

Es geht um den iostream, genauer um den istream, um cin. Ich kann den 
cin nicht anderorts verwenden, nachdem ich ihn read() übergeben habe.

Das Programm sollte fragen, ob es nochmals durchlaufen soll.
Für andere Projekte hat es wunderbar funktioniert -> das Problem liegt 
beim cin.

--------------------Aktuelle Version------------------

#include <iostream>
#include <cmath>
#include <cstring>
using namespace std;
typedef string::iterator itStr;
double epsilon = 0.00001;

struct cNumb{
    double a; // a+
    double b; //b*i
};

cNumb read(istream& in);
cNumb operator >> (istream& in, cNumb c);

void write(ostream& out, const cNumb c);
void operator << ( ostream& out, const cNumb c1);

int main(){
    bool run = true;
    while(run){
        cNumb c;
        c = read(cin);
        write(cout, c);
        cout << "\nWieder?";
        cin >> run;
    }
    return 0;
}

void write(ostream& out, const cNumb c1){
    out << " " << c1.a<< " + "<< c1.b << "i";
}

void operator << (ostream& out, const cNumb c1){
    write(out, c1);
}

cNumb read(istream& in){
    cNumb res;
    res.a = 0;
    res.b = 0;
    string input;
    while(in.peek()!= 'i'){
            input += in.get();
    }
    input += 'i';
    bool readingA = true;
    bool aNegative = false;
    bool bNegative = false;
    for(itStr i = input.begin(); i<input.end(); i++){
        if(readingA){
            if(*i == '-'){
                aNegative = true;
                i++;
            }
            while(*i>='0' && *i<='9'){
                res.a = res.a*10 +((*i)-48);
                i++;
            }
            readingA = false;
        }else{
            if(*i== '-'){
                bNegative = true;
                i++;
            }
            while(*i>='0' && *i<='9'){
                res.b = res.b*10 + ((*i)-48);
                i++;
            }
        }
    }
    if(aNegative){
        res.a = -res.a;
    }
    if(bNegative){
        res.b = -res.b;
    }
    return res;
}

cNumb operator >> (istream& in, cNumb c1){
    return read(in);
}

[Rolf Magnus (Gast)]

HasProblemsWithCpp schrieb:
> Es geht um den iostream, genauer um den istream, um cin. Ich kann den
> cin nicht anderorts verwenden, nachdem ich ihn read() übergeben habe.

Bei der Eingabe drückst du doch bestimmt am Ende Enter, oder? Ich sehe 
nirgends, daß das abgeholt wird, also steht es noch im Stream-Puffer 
wenn du "Wieder?" fragst, und dann wird das gelesen.

[HadProblemsWithCpp (Gast)]

Problem gelöst: Es fehlten sentry, clear() und ignore().

Vielen Dank für die Hilfe.

[Karl H. (kbuchegg)]

So ganz versteh ich noch nicht, warum read nicht einfach so aussehen 
kann

cNumb read(istream& in)
{
  cNumb res;
  char c;
  char imag;

  in >> res.a >> c >> res.b >> imag;

  return res;
}

aber das wirst du schon wissen, warum das so (in meinen Augen) 
kompliziert sein muss.

[Karl H. (kbuchegg)]

PS:
Hast du schon von Referenzen gelernt?

Denn hier

cNumb add(const cNumb c1, const cNumb c2){
...

kannst du dir das const auch sparen.
Beim Funktionsaufruf wird sowieso eine Kopie des Arguments des Aufrufers 
gemacht. Ob du die in der Funktion veränderst oder nicht spielt keine 
Rolle, da diese Kopie nach Beendigung der Funktion sowieso zerstört 
wird.

So hingegen

cNumb add(const cNumb& c1, const cNumb& c2){
...

ist das const wichtig, weil jetzt keine Kopie mehr gemacht wird, sondern 
die Funktion mit den Originalen des Aufrufers arbeitet. Und in dem Fall 
ist es wichtig, dem Aufrufer die Zusicherung zu geben: Die Funktion wird 
die beim Aufruf angegebenen Argumente nicht verändern.