ich arbeite zur Zeit ein C++ Buch durch und steh noch ganz am Anfang.

Die erste Aufgabe habe ich hingekriegt:
Bei der zweiten läuft das Programm und stoppt glaube ich nicht mehr??!!
Die Zahlen kann ich am Anfang noch eingeben aber mehr auch nicht

Aufgabenstellung:

Schreiben sie ein Programm das ihren Namen und ihre Adresse herausgibt:

Mein Programmcode:

#include <stdio.h>
#include <stdlib.h>
#include <iostream>

using namespace std;

int main()
{
int Adressausgabe;

printf(" Wollen Sie Die Adresse des Computernutzers wissen?\n Falls JA druecke 1\n falls Nein eine andere Zahl ");
scanf("%d", &Adressausgabe);

system ("Pause");

if (Adressausgabe == 1)
{
printf(" max musterman \n schillerstraße 9 \n 999999 alaska \n ");
system("Pause");

}else
{
printf("Vielen Dank \n");
system("Pause");
}}

Aufgabe 2:
Schreiben sie ein Programm dass ganze Zahlen von der Tastatur einliest und anschließend deren Summe, Differenz, Quotient, Produktund den Divisionsrest auf dem Bildschirm ausgibt
was passiert wenn man durch 0 dividiert?

Mein Code:

#include <stdio.h>
#include <stdlib.h>
#include <iostream>

using namespace std;

int main()
{
int zahl1, zahl2;
int summe, differenz,produkt, quotient, divrest;
int k;

printf( "Guten Tag: " );
printf(" Geben Sie zwei ganze Zahlen ein!: ");
scanf("%d %d ", &zahl1, &zahl2);
system("Pause");

if(zahl1 >= zahl2)
{
summe = zahl1 + zahl2;
differenz = zahl1 - zahl2;
quotient= zahl1 / zahl2;
k = quotient * zahl2 ;
divrest = zahl1 - k;

}
else {
summe = zahl2 + zahl1;
differenz = zahl2 - zahl1;
quotient = zahl2 / zahl1;
k= quotient * zahl1 ;
divrest = zahl2 - k ;
}

printf( "Ergebnisse: \n");
printf("Summe: %d\n", summe);
printf("Differenz: %d\n", differenz);
printf("produkt: %d\n", produkt);
printf("quotient: %d\n", quotient);
printf("divrest: %d\n", divrest);
system("Pause");
return 0
}

Bitte um Hilfe!!
Wie hätte ich Aufgabe 1 besser lösen können?

Bitte um Hilfe!!
Wie hätte ich Aufgabe 1 besser lösen können?

Wenn in deinem Buch tatsächlich noch printf verwendet wird, solltest du dir ein aktuelles Buch laufen. printf sollte eigentlich in C++ nicht mehr unbedingt verwendet werden. Daher besteht der Verdacht, dass das Buch veraltet ist.

system("pause") ist meiner Meinung nach unschön, weil man seinen Code unnötigerweise Plattformabhängig macht.

Dein Programm kompiliert wahrscheinlich nicht einmal, da hinter deinem letzten return ein Semikolon fehlt. Lies die Fehlermeldungen deines Compilers.

Dein Programm sollte in etwa so aussehen:

#include <cstdlib>
#include <iostream>

using namespace std;

int main()
{
    int zahl1, zahl2;
    int summe, differenz,produkt, quotient, divrest;

    cout << "Guten Tag: " << endl;
    cout << " Geben Sie zwei ganze Zahlen ein!: " << ends;
    cin >>zahl1 >> zahl2;

    if(zahl1 >= zahl2)
    {
        summe = zahl1 + zahl2;
        differenz = zahl1 - zahl2;
        quotient= zahl1 / zahl2;
        produkt = quotient * zahl2 ;
        divrest = zahl1 % zahl2;
    } else {
        summe = zahl2 + zahl1;
        differenz = zahl2 - zahl1;
        quotient = zahl2 / zahl1;
        produkt = quotient * zahl1 ;
        divrest = zahl1 % zahl1 ;
    }

    cout << "Ergebnisse:" << endl;
    cout << "Summe:" << summe << endl;
    cout << "Differenz: " << differenz << endl;
    cout << "produkt: " << produkt << endl;
    cout << "quotient: " << quotient << endl;
    cout << "divrest: " << divrest << endl;

    return EXIT_SUCCESS;
}

Btw.: Verwende die Code-Tags, wenn du Quelltext postest.

Danke für deine Hilfe

Im Zweifelsfall bitte auch folgende Hinweise beachten:
C/C++ Forum :: FAQ - Sonstiges :: Wohin mit meiner Frage?

Dieses Posting wurde automatisch erzeugt.

Die erste Aufgabe könnte man vielleicht so lösen:

#include <iostream>
#include <cstdlib>

int main()
{
	std::cout<<" Wollen Sie Die Adresse des Computernutzers wissen?\n Falls JA dr\201cken Sie 1\n falls Nein eine andere Zahl\n";

	if (std::cin.get())
		std::cout<<" max musterman \n schillerstra\341e 9 \n 999999 alaska\n"<<std::endl;

	std::cout<<"Vielen Dank !"<<std::endl;
	system("pause");
}

Noch dazu: Du inkludierst <iostream>, obwohl du keine I/O Streams benutzt (errinert mich an einen anderen Post, von nem Typ der in einem 'C++' Programm eine einzige, unnötige Zeile C++ benutzt hat - eben die).

Die erste Aufgabe könnte man vielleicht so lösen:

#include <iostream>
#include <cstdlib>

int main()
{
	std::cout<<" Wollen Sie Die Adresse des Computernutzers wissen?\n Falls JA dr\201cken Sie 1\n falls Nein eine andere Zahl\n";

	if (std::cin.get())
		std::cout<<" max musterman \n schillerstra\341e 9 \n 999999 alaska\n"<<std::endl;

	std::cout<<"Vielen Dank !"<<std::endl;
	system("pause");
}

Wie bitte soll florii als Anfänger das denn verstehen? Er sollte schon "using namespace std;" nutzen, da das Programm dann anschaulicher wird. Deine if-Bedingung kann auch nur ein Fortgeschrittener verstehen. Jedoch bin ich mir nicht so ganz sicher. false ist ja 0 und true 1, aber ich glaube true ist auch alles >1

ich arbeite zur Zeit ein C++ Buch durch und steh noch ganz am Anfang.
...

[...alle variablen am anfang definiert...]
[...printf...]
[...scanf...]

(meine Betonung)

Wenn Du ganz am Anfang stehst, hast Du ja noch Gelegenheit, Dir ein besseres Buch zu besorgen; denn das, was Du da im Moment hast, scheint Dir kein C++ sondern "C mit #include <iostream> und using namespace std;" beizubringen.

#include <iostream>
#include <cstdlib>

using namespace std;

int main()
{
    cout<<" Wollen Sie Die Adresse des Computernutzers wissen?\n Falls JA dr\201cken Sie 1\n falls Nein eine andere Zahl\n";

    int a(0);

    cin>>a;

    if (a == 1)
        cout<<" max musterman \n schillerstra\341e 9 \n 999999 alaska\n";

    cout<<"Vielen Dank !"<<endl;
    system("pause");
}

Das dürfte ein Anfänger verstehen.

@Hacker: Da fehlt noch ein return .

Das ist bei main() nicht nötig – steht es nicht, entspricht es einem return 0; am Ende von main() .

#include <cstdlib>

Buuuuh!

Hacker schrieb:

using namespace std;

Buuuuh!

Hacker schrieb:

int a(0);

Buuuuh!

Hacker schrieb:

system("pause");

Buuuuh!

Noch dazu ist alles hässlich. Tu uns allen einen Gefallen und verschone uns mit deinem Code.

using namespace std;

wegen einem Rat aus der vorherigen Seite.

system("pause")

#include <cstdlib>

hab ich auch nicht gern(ich nutze es ja selbst auch nie), ich hab es nur dagelassen weil flori es auch genutzt hat und cin.get() für Sie noch zu "fortgeschritten" sein scheint.

und was ist an

int a(0);

so schlimm ? Der Bezeichner ?

Sag mal was, anstatt nur die ganze Zeit zu Buuhen.
Der Code ist, wie gesagt (und von dir ignoriert), so konstruiert das jemand wie flori ihn verstehen kann.

Und was genau war daran hässlich ?

Hacker schrieb:

using namespace std;

Buuuuh!

Nix da. Das ist gut und richtig so. Nur weil es nicht DEIN Stil ist, heisst es nicht, dass es schlecht oder falsch ist. Hier siehst du sogar ein Paradebeispiel für richtige Benutzung. Nebenbei Buuhst du ihn dafür aus, nachdem er darauf hingewiesen wurde, dass es GENAU SO wie es jetzt ist lesbarer ist. Du stehst also mit deiner Meinung ziemlich alleine da.

Hacker schrieb:

int a(0);

Buuuuh!

Auch das ist völlig in Ordnung so. Man könnte sogar behaupten, dass dies pre C++0x eine der besten Generalisierungen zur Erzeugung von einzelnen Variablen ist (du würdest auch immer T a(0) schreiben, wenn T ein beliebiges Template ist und nicht T a = 0)

Und hässlich ist der Code auch nicht. Nur ein wenig zu viele Leerzeilen, aber das wird sich schon geben. Ich fürchte, dass es für einen Anfänger so auch einfacher zu erfassen ist, weil der Code nicht wie ein kryptischer Block aussieht.

Lass mal endlich diese Kinderkacke sein. Dein Templateschwachsinn ist auch nicht besser. Verschone uns einfach mit deinen nervigen rants, die nichts, aber auch gar nichts beitragen.

Man könnte sogar behaupten, dass dies pre C++0x eine der besten Generalisierungen zur Erzeugung von einzelnen Variablen ist

Könnte man zwar, aber deswegen diese Schreibweise generell zu bevorzugen, wäre doch ein wenig übertrieben.

Nix da. Das ist gut und richtig so. Nur weil es nicht DEIN Stil ist, heisst es nicht, dass es schlecht oder falsch ist. Hier siehst du sogar ein Paradebeispiel für richtige Benutzung. Nebenbei Buuhst du ihn dafür aus, nachdem er darauf hingewiesen wurde, dass es GENAU SO wie es jetzt ist lesbarer ist. Du stehst also mit deiner Meinung ziemlich alleine da.

Ich habe nichts gegen using namespace; aber für was soll denn das ein Paradebeispiel sein? "Globalisierung" des Namensraums wegen einer Funktion? Generell gilt: man sollte den Zugang zum Namensraum möglichst nur auf die Funktionen beschränken, die es auch brauchen.

Zur Initialisierung mit int a(0); :
dies ist zwar keine weit verbreitete Schreibweise, aber sie ist C++ pur und definitiv akzeptabel. IMO ist jedoch die von C stammende Schreibweise zu bevorzugen, da auf diese Weise für das menschliche Auge schneller ersichtlich wird, dass es sich um eine Initialisierung handelt.

Ich habe nichts gegen using namespace; aber für was soll denn das ein Paradebeispiel sein? "Globalisierung" des Namensraums wegen einer Funktion?

Es ging nur darum, florii nicht zu irritieren. Um nix anderes. Hier ist allen völlig klar, dass man das auch "sauberer", bzw. anders machen könnte.

Nix da. Das ist gut und richtig so. Nur weil es nicht DEIN Stil ist, heisst es nicht, dass es schlecht oder falsch ist. Hier siehst du sogar ein Paradebeispiel für richtige Benutzung.

Natürlich ist das schlecht. using namespace XY ist immer schlecht. Alleine aus Konsistenzgründen. Und brauchst du alles aus dem std-namespace? (Ich tippe jetzt einfach mal auf nein.) Und wenn du zu faul bist, std:: dranzuschreiben, dann nimm using std::XY;.

otze schrieb:

Auch das ist völlig in Ordnung so. Man könnte sogar behaupten, dass dies pre C++0x eine der besten Generalisierungen zur Erzeugung von einzelnen Variablen ist (du würdest auch immer T a(0) schreiben, wenn T ein beliebiges Template ist und nicht T a = 0)

Nein, das ist nicht in Ordnung. Der Anfänger wird sich fragen, wieso da kein = ist. Aber wenn du so auf "Generalisierung" bist, wieso nimmst du dann nicht immer {}? Geht doch nun bei allem. Die Initialisierung ist übrigens sowieso unnötig.

otze schrieb:

Und hässlich ist der Code auch nicht. Nur ein wenig zu viele Leerzeilen, aber das wird sich schon geben.

Die Leerzeilen sind gar nicht mal der Kritikpunkt. Seine Leertaste ist wohl kaputt oder so.

otze schrieb:

Dein Templateschwachsinn ist auch nicht besser.

Welcher Templateschwachsinn?

Edit: Sorry, ganz vergessen. endl ist natürlich auch Mist.

Hier mal ne verbesserte Version...

#include <iostream>

int main()
{
    // So lokal wie möglich halten.
    using std::cin;
    using std::cout;

    // Konstanten sind sexy. Ob die Nummern stimmen, weiß ich gerade nicht. Das zweite erzeugt vermutlich einen Überlauf.
    char const ue = '\201';
    char const sz = '\341';

    // Leerzeichen sind nicht böse, Zeilenumbrüche bei Ausgaben könnte man auch in mehrere Codezeilen schreiben.
    cout << "Wollen Sie Die Adresse des Computernutzers wissen?\n"
         << "Falls Ja dr" << ue << "cken Sie 1\n"
         << "Falls Nein eine andere Zahl\n";

    // Initialisierung unnötig.
    int entered;
    cin >> entered;

    if(entered == 1)
    { // Blöcke kann man auch hinzufügen.
        cout << "Max Musterman\n"
             << "Schillerstra" << sz << "e 9\n"
             << "999999 alaska\n";
    }

    cout << "Vielen Dank!\n";
    cin.get(); // Wie auch immer. Alles ist besser als system(). Sogar getch oder kbhit.
}

otze schrieb:

Auch das ist völlig in Ordnung so. Man könnte sogar behaupten, dass dies pre C++0x eine der besten Generalisierungen zur Erzeugung von einzelnen Variablen ist (du würdest auch immer T a(0) schreiben, wenn T ein beliebiges Template ist und nicht T a = 0)

Nein, das ist nicht in Ordnung. Der Anfänger wird sich fragen, wieso da kein = ist.

[...]

Hier mal ne verbesserte Version...

#include <iostream>

int main()
{
    // So lokal wie möglich halten.
    using std::cin;
    using std::cout;

    // Konstanten sind sexy.
    wchar_t const ue = '\201';
    wchar_t const sz = '\341';

    // Leerzeichen sind nicht böse, Zeilenumbrüche bei Ausgaben könnte man auch in mehrere Codezeilen schreiben.
    cout << "Wollen Sie Die Adresse des Computernutzers wissen?\n"
         << "Falls Ja dr" << ue << "cken Sie 1\n"
         << "Falls Nein eine andere Zahl\n";

    // Initialisierung unnötig.
    int entered;
    cin >> entered;

    if(entered == 1)
    { // Blöcke kann man auch hinzufügen.
        cout << "Max Musterman\n"
             << "Schillerstra" << sz << "e 9\n"
             << "999999 alaska\n";
    }

    cout << "Vielen Dank!\n";
    cin.get(); // Wie auch immer. Alles ist besser als system(). Sogar getch oder kbhit.
}

Was ist das für eine Logik ? Kein Anfänger wird diesen Code ohne weiteres verstehen.

Leerzeilen ? Wo zum teufel willst du in dem Code Leerzeichen ?
Völlig unnötige Konstanten, lateinische Umlaute brauchst du nur jeweils ein mal (!).
Endl hab ich nicht benutzt.

Und auf using wären hier mindestens auch die hälfte (inklusive mir) gekommen, aber wenn du einfach zu faul bist meine verdammten Posts zu lesen, wird dir kein Mensch mehr helfen können.

std::cout << a << b << c << d << e;

sieht schon besser aus als

std::cout<<a<<b<<c<<d<<e;

Aber im Endeffekt ziemlich wayne und sieht am Ende iwie doch nur nach ner Rumpickerei auf einem poster auf. Wieso er den gesamten namespace per using-deklaration einbindet wurde nun schon oft genug gesagt.

Hat denn der Threadersteller noch fragen ?

Was ist das für eine hirngestörte Logik ? Kaum ein Anfänger wird diesen Code verstehen.

Natürlich wird er ihn verstehen. Du musst ihm nur erklären, was using tut und was der std-namespace ist.

Hacker schrieb:

Leerzeilen ? Wo zum teufel willst du in dem Code Leerzeichen ?

Verwechsle nicht Leerzeilen mit Leerzeichen. Leerzeichen gehören bei cout << XY hin. Leerzeilen sind ganz okay.

Hacker schrieb:

Völlig unnötige Konstanten, lateinische Umlaute brauchst du nur jeweils ein mal (!).

Und woher soll der Anfänger wissen, was dieses komische \xyz in deinem Code ist? Die Konstanten geben dem Ding einen Namen, da ist völlig egal, wie oft die verwendet werden.

Hacker schrieb:

Endl hab ich nicht benutzt.

Hast du. Bei deiner letzten Ausgabe.

Hacker schrieb:

Und auf using wären hier mindestens auch die hälfte (inklusive mir) gekommen

Und warum hast du es dann nicht gleich richtig gemacht?

Hacker schrieb:

aber wenn du einfach zu faul bist meine verdammten Posts zu lesen, wird dir kein Mensch mehr helfen können.

Ich habe deine Posts gelesen, sonst würde ich nicht auf ihnen herumreiten.

Hacker schrieb:

Was ist das für eine hirngestörte Logik ? Kaum ein Anfänger wird diesen Code verstehen.

Natürlich wird er ihn verstehen. Du musst ihm nur erklären, was using tut und was der std-namespace ist.

Und warum tust du es dann nicht ?

314159265358979 schrieb:

Hacker schrieb:

Leerzeilen ? Wo zum teufel willst du in dem Code Leerzeichen ?

Verwechsle nicht Leerzeilen mit Leerzeichen. Leerzeichen gehören bei cout << XY hin. Leerzeilen sind ganz okay.

Ja, das war ein Schreibfehler. Nein, bei cout hät ich es nicht bemerkt.

314159265358979 schrieb:

Hacker schrieb:

Völlig unnötige Konstanten, lateinische Umlaute brauchst du nur jeweils ein mal (!).

Und woher soll der Anfänger wissen, was dieses komische \xyz in deinem Code ist? Die Konstanten geben dem Ding einen Namen, da ist völlig egal, wie oft die verwendet werden.

Ja, aber welche Werte wieso eingesetzt ist trotzem nicht erklärt. (komische Logik)

314159265358979 schrieb:

Hacker schrieb:

Endl hab ich nicht benutzt.

Hast du. Bei deiner letzten Ausgabe.

Übersehen.

314159265358979 schrieb:

Hacker schrieb:

Und auf using wären hier mindestens auch die hälfte (inklusive mir) gekommen

Und warum hast du es dann nicht gleich richtig gemacht?

Weil floori es verdammt nochmal nicht versteht !? (-> siehe erstes Kommentar)

314159265358979 schrieb:

Hacker schrieb:

aber wenn du einfach zu faul bist meine verdammten Posts zu lesen, wird dir kein Mensch mehr helfen können.

Ich habe deine Posts gelesen, sonst würde ich nicht auf ihnen herumreiten.

Wahrscheinlich. Mindestens ein/zweimal hab ich darauf hingewiesen, dass es verständlich und nicht effizient oder gar sauber sein muss, trotzdem ist ersichtlich dass du es ignorierst.

Und warum tust du es dann nicht ?

Keine Lust.

Hacker schrieb:

Ja, aber welche Werte wieso eingesetzt ist trotzem nicht erklärt.

Für sowas gibts dann Kommentare. Ich habe mir diese mal erspart, da die Werte sowieso falsch sind. Kannst sie ja mal korrigieren.

Hacker schrieb:

Weil floori es verdammt nochmal nicht versteht !?

Du verwechselst Fahigkeiten mit Wissen mein lieber Padawan. Er versteht es nicht, weil es ihm niemand erklärt hat. Täte das jemand, verstünde er es auch. (Konjunktiv ohne 'würde' ist übrigens viel toller.)

Hacker schrieb:

Wahrscheinlich. Mindestens ein/zweimal hab ich darauf hingewiesen, dass es verständlich und nicht effizient oder gar sauber sein muss, trotzdem ist ersichtlich dass du es ignorierst.

Siehe oben. Wenn er was nicht kennt, erklärs ihm. Deshalb fragt er im Forum, um Hilfe zu bekommen. Dass du dann deinen Code mit Absicht scheiße schreibst, geht gar nicht.

Und wenn du sogar offen zugibst keine Lust zu haben, TEs zu helfen sondern nur rumzumeckern, wieso klinkst du dich dann überhaupt hier ein ?

Ein paar Links für floori, damit die Codes klarer werden, wenn er dennoch eine Erklärung will soll er die Bekommen.

Hier ein Link zur doku für Umlaute.
Hier einer zu Namensräumen und deren Schlüsselwörtern in C++.

Edit: Ich glaube floori ist inzwischen gar nicht mehr dabei.