Ich soll in einem Schulprojekt ein kleines Programm zur berechnung von Leiterquerschnitten und zur Auswahl von Leitungsschutzorganen schreiben.
Das Programm funktioniert soweit auch ganz gut. Mit einer kleinen Ausnahme:

Ich kann das Programm nicht wie gewünscht mit Drücken von "B" beenden.

#include <cstdlib>
#include <iostream>
#include <conio.h>

using namespace std;

char auswahlroutine (char programmwahl);

int main(int n)

    {

     char programmwahl;   
     bool schleife = n;

      while (schleife)
      {
      system("cls");
          //weite wahlmöglichkeiten
	  cout << "\t(B) Programm Beenden: " << endl << endl;
	  cin >> programmwahl;
      auswahlroutine (programmwahl);
      }

      return 0;
      system("PAUSE");

}

//********************************************Hauptprogramm auswahlstruktur*************************************************

char auswahlroutine (char programmwahl)
     {
     int n;
     system("cls");
     switch (programmwahl)
     {

             case ('b'):
             {
             n = 0;    
             main (n);
             }   

             case ('B'):
             {      
             n = 0;
             main (n);
             }

             //Weitere verzweigungen
             //...

                default:
                    {             
                    cout << endl << endl;
                    cout << "\tAnscheinend haben Sie sich vertippt." << endl << endl;
                    cout << "\tBitte entscheiden Sie sich fuer eine der Auswahlmoeglichkeiten" << endl;
                    cout << "\tund bestaetigen Sie diese mit eingabe von ENTER:" << endl;
                    cout << endl;
                    wasnun();
                    }
      }
}

Ich habs mal auf das wesentliche Reduziert.
Wenn ich nun B Drücke, nachdem ich aus einer Anderen Funktion ins Hauptmenü zurückgekehrt bin, tut sich überhaupt nichts (oder das Hauptmenü wird erneut aufgerufen, genau kann ich das nicht sagen). zu den anderen Funktionen springt das Programm Problemlos.

Kann mir jemand weiterhelfen? ich bin echt vorm verzweifeln und habe schon so viel ausprobiert und gegooglet.

ich hänge mal den vollständigen quelltext mit an falls dieser benötigt wird.

Ich bin für jeden Hinweis dankbar.
(Bitte leicht verständlich )

http://www.schmerzkeks.net/PQ/ProjektSchule.cpp

Ich habe nun folgendes geändert:

case ('b'):
             {
             n = true;    
             return n;

             }   

             case ('B'):
             {      
             n = true;
             return n;
             }

ausserdem habe ich char auswahlroutine (char programmwahl); zu bool auswahlroutine(char programmwahl); gemacht

aber am Ergebnis ändert sich leider nichts.
was mach ich nur falsch?

Danke für weitere Hilfe!

switch (Konstante)
{
    case 1:
    case 2:
       TuEtwas();           // für die Fälle 1 und 2
       break;               // damit nächste Anweisungen nicht ausgeführt werden
    case 3:
       TuEtwasAnderes();    // für Fall 3
       break;
    default:                // für alle anderen Fälle
       TuNichts();
}

Da bei den Fällen 'b' und 'B' das Gleiche passiert, kannst du sie zusammenfassen (siehe oben bei Fall 1 und 2). Aber bei sowenig Anweisungen kannst du eigentlich auch eine If-Else-Verzweigung durchführen.
Hast du jetzt eine Schleife eingebaut, anstatt main() aufzurufen?

Und statt

n = true;
return n;

kannst du auch schreiben:

return true;

Mit dem cahr kann es nicht funktionieren.

Mit dem cahr kann es nicht funktionieren.

Switch-Variablen müssen integrale Konstanten sein, was bei einem char ebenso erfüllt ist. Ein char wird nur von den Ausgabeströmen als Zeichen ausgegeben, ist jedoch eine Ganzzahl.

int main (){
...

In den Klammern von main gehören auch keinen Parameter. Du musst nichts übergeben!

Wie kann er dann das Problem mit dem char lösen? Dann darf er also keine Buchstaben einlesen oder muss den ASCII-Code abfragen?

case ('B'):
             case ('b'):   
             return true;
             break;

Allerdings wieder ohne erfolg. Das programm beendet sich imemrnoch nicht.

Nexus Hast du jetzt eine Schleife eingebaut, anstatt main() aufzurufen?

Was meinst du mit einer schleife ich verstehe das nun so das ich mit return true aus der funktion nach main zurückkehre.

bool laufend = true;
while (laufend)
{
    // ...
    laufend = auswahlroutine();  // wenn Funktion false zurückgibt, wird Schleife verlassen
}

Falls nein, wie sieht dein Programm jetzt aus, was willst du genau tun und wo liegt das Problem?

Gibt die Funktion true zurück, wenn das Programm weiterlaufen soll und false , wenn man es beenden will? Falls ja, kannst du das folgendermassen machen:

bool laufend = true;
while (laufend)
{
    // ...
    laufend = auswahlroutine();  // wenn Funktion false zurückgibt, wird Schleife verlassen
}

Falls nein, wie sieht dein Programm jetzt aus, was willst du genau tun und wo liegt das Problem?

Es funktioniert Ich könnt dich knutschen. Ich muss zwar 2 mal b + Enter eingeben aber wenigstens lässt es sich beenden total super!
Vielen Dank.

Ich muss zwar 2 mal b + Enter eingeben aber wenigstens lässt es sich beenden total super!

Willst du das nicht auch noch ausbessern?
Das sollte eigentlich nicht passieren... Die Eingabe von 'b' wird ja in der Funktion auswahlroutine() gehandhabt. Und falls diese false zurückgibt, wird die Schleife verlassen und das Programm beendet. Wahrscheinlich ist ein Fehler innerhalb der Funktion vorhanden...

Ach ja, mit der Zeit würde ich mir unbedingt einen Umstieg auf eine modernere Entwicklungsumgebung als Dev-C++ überlegen (z.B. auf die kostenlose Express-Version von Microsoft Visual C++ 2008). Besonders bei grösseren Projekten helfen einem eingebaute Tools enorm bei der Übersicht. Am Anfang ist es jedoch ein bisschen nachteilig, weil für jede kleine Anwendung ein neues Projekt erstellt werden muss.

int main(int n) 
    { 
     char programmwahl;   
     bool schleife = n;

macht wahrscheinlich (bin gerad nicht ganz sicher) nicht das, was es soll ^^
main wird aufgerufen:

main (1, "Pfad\\Dateiname_der_exe.exe")
    du bekommst nur
main (1)

und wenn man iwas übergibt statt der 1 halt ne 2 oder so
also is schleife immer true

also schreib

int main ()
{
//...
}

für deine schleife kannste das au so machen:

#include <iostream>

int main ()
{
do
{

} while (auswahlroutine ());
}

bool /*again*/ auswahlroutine () 
     { 
char eingabe (0);
std::cout << "..." << std::endl;
std::cin >> eingabe;
     system("cls"); //würd ich weglassen - wenn es stört, machste halt paar zeilenumbrüche hin?!
     switch (programmwahl) 
     {
             case ('b'): 
             case ('B'):
               return false;
             case ('_'): //weiter oder so...
               return true;
             default: 
               {             
                 cout << endl << endl; 
                 cout << "\tAnscheinend haben Sie sich vertippt.\r\n" << endl
                   << "\tBitte entscheiden Sie sich fuer eine der Auswahlmoeglichkeiten" << endl
                   << "\t\t {b, B} = beenden, ....."
                   << "\tund bestaetigen Sie diese mit eingabe von ENTER:\r\n"
                 << endl;
                 return auswahlroutine;
               } 
      } 
}

bb : )

Freut mich für dich

Ach ja, mit der Zeit würde ich mir unbedingt einen Umstieg auf eine modernere Entwicklungsumgebung als Dev-C++ überlegen (z.B. auf die kostenlose Express-Version von Microsoft Visual C++ 2008). Besonders bei grösseren Projekten helfen einem eingebaute Tools enorm bei der Übersicht. Am Anfang ist es jedoch ein bisschen nachteilig, weil für jede kleine Anwendung ein neues Projekt erstellt werden muss.

Also Microsoft Visual könnte ich sogar als vollversion von der schule bekommen die sind in soeinem Microsoft Partnerprogram. Wir lernen allerdings auf Dev-cpp daher habe ich das bisher benutzt, werde aber mal Visual ausprobieren. Danke für den Tip.

unskilled schrieb:

std::cout << "..." << std::endl;
std::cin >> eingabe;

Dir auch erstmal danke für deine Hinweise
Den gequoteten abschnitt haben wir in der schule leider noch nicht behandelt daher lass ich das erstmal lieber weg.

Die warnung lautet:

(277) : warning C4244: '=': Konvertierung von 'double' in 'float', möglicher Datenverlust

Ich kann aber niergends das "double" entdecken

float querschnittsberechnung (float temperatur, float laenge, float leistung, float Inenn)
{
float spezifischerleitungswiederstand, querschnittergebnis, quer;

              Inenn = leitungsschutzorganberechnung (leistung);  

  /*Zeile 277 in meinem Quelltext --->*/            spezifischerleitungswiederstand = 0.0178 * (1 + (0.0038) * (temperatur - 20));
                                                    querschnittergebnis = (spezifischerleitungswiederstand * 2 * laenge * Inenn * 100) / (3*230);

                       if (querschnittergebnis < 1.5)
                          {
                          quer = 1.5;
                          return quer;
                          }

                                else

                       if (querschnittergebnis < 2.5)
                          {
                          quer = 2.5;
                          return quer;
                          }
                                else

                          //...etc.

Bin wiedermal sehr dankbar für Hilfe!

Vielen Dank!