Hilfee!!!!!

  • Hilfee!!!!!

    Hallo zusammen.

    Bin neu hier und Programmieranfänger , vielleicht kann mir jemand nen Tipp geben was in meinem Quellcode falsch ist.

    Es dreht sich hier um einen Rechner , bei jeder Ausgabe wird nach der letzten Zeile die zweite Zahl auf dem Bildschirm mit ausgegeben ;(

    komme einfach nicht mehr weiter,im voraus vielen Dank .mfg. muffti_*

    Quellcode

    1. #include <cstdlib>
    2. #include <iostream>
    3. #include<cmath>
    5. using namespace std;
    7. double add(double x, double y){
    8. return x + y;
    9. }
    10. double sub(double x, double y){
    11. return x - y;
    12. }
    13. double divi(double x, double y){
    14. return x / y;
    15. }
    16. double multi(double x, double y){
    17. return x * y;
    18. }
    20. double alle(double x, double y){
    21. cout << "Ergebnis alle: \n";
    22. cout << "\n\n Das Ergebnis der Addition ist: " << add(x,y) << "\n\n";
    23. cout << "\n\nDas Ergebnis der Subtraktion ist: " << sub(x,y) << " \n\n";
    24. cout << "\n\nDas Ergebnis der Division ist: " << divi(x,y) << " \n\n";
    25. cout << "\n\nDas Ergebnis der Multiplikation ist: " << multi(x,y) << " \n\n";
    26. return (x, y);
    27. }
    29. int main(int argc, char *argv[])
    30. {
    31. double z1, z2;
    32. char operation;
    34. cout << " Geben Sie die erste Zahl ein: ";
    35. cin >> z1;
    37. cout << "\nGeben Sie die zweite Zahl ein: ";
    38. cin >> z2;
    40. cout << "\nWelche Rechenoperation moechten Sie ausfuehren ? < +, -, /, * > :";
    41. cin >> operation;
    43. switch(operation){
    45. case'+':
    46. cout << "\n\n Das Ergebnis der Addition ist: " << add(z1,z2) << "\n\n";
    47. break;
    49. case'-':
    50. cout << "\n\nDas Ergebnis der Subtraktion ist: " << sub(z1,z2) << " \n\n";
    51. break;
    53. case'/':
    54. if(z2 == 0){
    55. cout << "\nDivision durch Null ist nicht erlaubt! Das Programm wird beendet:\n\n";
    56. break;
    57. }
    58. cout << "\n\nDas Ergebnis der Division ist: " << divi(z1,z2) << " \n\n";
    59. break;
    61. case'*':
    62. cout << "\n\nDas Ergebnis der Multiplikation ist: " << multi(z1,z2) << " \n\n";
    63. break;
    64. }
    66. cout << alle(z1, z2) << "\n";
    68. system("PAUSE");
    70. return EXIT_SUCCESS;
    71. }
    Alles anzeigen

  • ist es Absicht, dass die Funktion "alle" immer ausgeführt wird? Sollte es vieleicht so sein, dass das nur ausgeführt wird, wenn keines der 4 Rechenzeichen erkannt wurde? Dann muss es in den default-case rein.

    Dass die zweite Zahl nochmal ausgegeben wird liegt an der Zeile: cout << alle(z1, z2) << "\n";
    Die Funktion alle gibt ja selbst schon alles nötig aus, daher ist es nichtmehr nötig den return wert von alle an cout zu leiten.. Daher würde

    Quellcode

    1. alle(z1,z2);
    reichen.
    Und wo wir schon beim return wert sind: return (x, y); in der Funktion alle Funktioniert nicht so. Eine Funktion muss immer eine einzigen Rückgabewert haben. Du könntest die Werte in ein Array speichern und dann zurückgeben oder irgendwas anderes damit machen, aber auf diese Weise funktioniert es nicht. Abgesehen davon ist die Rückgabe aber auch unnötig, denn x und y sind ja die Übergabeparameter (und wurden auch nicht geändert), daher macht es wenig Sinn diese wieder zurückzuliefern.

  • Ausgabe mit cout

    Hallo . Danke für die schnelle Hilfe.

    Das Problem lag tatsächlich in der Ausgabeanweisung mit cout. Hab es so geschrieben wie von Rondrer geschildert .

    Im übrigen , die Funktion alle() muss laut Aufgabenstellung jedesmal mit ausgegeben werden.

    Einmal ist`s mit dem Gnu Compiler geschrieben , also dieser Quelltext und einmal mit Visual C++ 2008 in .NET :)