Buchstabe in Dezimalzahlwert Fehler

jkhsjdhjs

Hier ist jetzt der fertige Code:

Wutz

Furchtbar.
- viel zu viel Code für diese triviale Aufgabenstellung
- goto ist Anfängermüll
- Note 6 setzen

jkhsjdhjs

Dann schreib doch nen kürzeren Code, der auch gegen jede möglichen Fehleingaben geschützt ist und als Ausgabe auch nur Dezimalzahlen und keine Fehler ausgibt.

Außerdem bin ich ja auch Anfänger.

Wutz

Da hast du hier was falsch verstanden.
Ich schreibe dir hier nicht deine Hausaufgaben.
Du ignorierst die hier gegebenen Hinweise, aber genau dafür ist dieses Forum ja auch gedacht: Hinweise auf konkrete Fragen zu geben.
Das ignorierst du und bist deshalb ein Noob.
Naiv auch dein anfängerhafter Glaube:
Es funktioniert jetzt einwandfrei!!
mit dem du implizierst, dass der Code korrekt ist. Ist er nicht.
Und standardkonform schon gar nicht.
Und float-Gleichheitsabfragen sind auch Schrott.
Und ...

Wutz macht hier eigentlich gar nichts außer blöd rumzuflamen. Ignorier ihn. Selbst wenn der code nicht unbedingt "optimal" ist.

jkhsjdhjs

Du sollst mir auch nicht deine Hausaufgaben schreiben. Die hab ich ja schon.

Außerdem brauchst du auch nicht auf Threads zu antworten nur um irgendwelche Leute zu beleidigen.

Du kannst gerne Kritik äußern, aber dann lass bitte nächstes mal Bemerkungen wie "Note 6 setzen" oder "bist deshalb ein Noob" weg.

Wutz

Schon wieder falsch verstanden.
Ich lasse mir von dir nichts vorschreiben.
Wenn dein Lehrer halbwegs Ahnung hat, wird er dir evtl. eine 5 geben.

Nathan

@TE, Zagatho:
Mein Beispiel konnte gar nicht funktionieren, das ! muss natürlich weg...
Mein Fehler. Aber im Ansatz wäre das die Alternative, die Zagatho meinte.
Sie ist aber nicht zur Nachahmung geeignet, weil das schon ein bisschen unleserlicher ist, als die ursprüngliche Version.

Eisflamme

Zutz:
Konstruktiv? Ich denke nicht.

Ein paar Dinge zu Herzen nehmen solltest Du Dir wohl schon. Wenn ihr Schleifen noch nicht hattet, kann der ~~Fehler~~Lehrer Dein goto eigentlich nicht kritisieren. Gut wäre natürlich trotzdem, wenn Du es selbst anlernst.

jkhsjdhjs

Ok, Danke. Schleifen werde ich mir mal ansehn.

Davon abgesehen:

if (b2 == 0) {
                cout<<"y = "<<m2<<" * x"<<endl;
                goto punkt6;
        }

        if (b2 < 0) {
                cout<<"y = "<<m2<<" * x - "<<b4<<endl;
                goto punkt6;
        }

        cout<<"y = "<<m2<<" * x + "<<b2<<endl;
punkt6:

Besser:

if (b2 == 0) {
                cout<<"y = "<<m2<<" * x"<<endl;
        }
        else if (b2 < 0) { //wenn das erste if falsch war, prüfe b2 <0
                cout<<"y = "<<m2<<" * x - "<<b4<<endl;
        } 
        else cout<<"y = "<<m2<<" * x + "<<b2<<endl; //wenn auch das zweite if falsch war (und nur dann) führe das hier aus

Macht soweit ich sehen kann das selbe, aber goto ist so unübersichtlich, sicher sein kann man sich da fast nie...

jkhsjdhjs

Achso. Danke, ich werds mal ändern

Aber müsste hinter dem else nicht noch ne geschweifte Klammer geöffnet werden?

volkard

kleiner Troll schrieb:

Besser:

if (b2 == 0) {
                cout<<"y = "<<m2<<" * x"<<endl;
        }
        else if (b2 < 0) { //wenn das erste if falsch war, prüfe b2 <0
                cout<<"y = "<<m2<<" * x - "<<b4<<endl;
        } 
        else cout<<"y = "<<m2<<" * x + "<<b2<<endl; //wenn auch das zweite if falsch war (und nur dann) führe das hier aus

Macht soweit ich sehen kann das selbe, aber goto ist so unübersichtlich, sicher sein kann man sich da fast nie...

Oder sogar

cout<<"y = "<<m2<<" * x";
        if (b2 > 0) {
                cout<<" + "<<b2;
        } 
        else if(b2 < 0 ) {
                cout<<" - "<<b4;
        }
        cout<<'\n';

Nicht unbedingt. Wenn hinter einem If- abfrage oder Schleife nur eine Anweisung steht, kann die Klammer entfallen. Zur besseren Lesbarkeit sollte man sie aber doch schreiben.

jkhsjdhjs

Das ist jetzt natürlich ultra kurz

Danke, ich werds ändern

jkhsjdhjs

@Zagatho, oke, Danke für die Info. Ich hab Sie lieber mal hingeschrieben.