C++ - Schleifen und boolesche Ausdrücke

Question

Ich habe Probleme, diese Aufgabe für meine Klasse für ein paar Tage zu tun, und möchte etwas Hilfe.

Die Aufgabe besteht darin, ein Programm zu schreiben, das den Benutzer auf der Grundlage seiner Hoch- und Gewichtsqualifikation abhängig von seinem Geschlecht über seinen Akzeptanzstatus informiert.

Am Ende des Programms möchte es die Anzahl der Kandidaten, die akzeptiert wurden, und den Durchschnitt der akzeptierten Kandidaten für die Gesamtzahl der Kandidaten ausgeben.

Zuordnung - https://www.saddleback.edu/faculty/slinker/CS1A/CS1A_Fall2013/Assignment8.pdf

Wir nicht Schalter, bedingte Operatoren verwenden können und die Auswahl (nur für die richtige Botschaft an die Ergebnisse ausgibt). Wir können nur Loops verwenden und komplexe Boolesche Ausdrücke

Die Probleme, die ich habe, ist:

1. Wenn alle 3 meiner Eingänge gültig sind, warum sind sie die Ausgabe nicht, wenn sie akzeptiert werden und wenn einer von die Eingabe (Höhe oder Gewicht) oder beide wurden abgelehnt, warum gibt sie es nicht aus? Ist meine boolesche Variable falsch? Wenn ja, wie repariere ich es?

2. Warum kann ich die Schleife/das Programm nicht verlassen, wenn ich X eingebe? Ist meine While-Schleife korrekt oder nicht?

3. Gibt es irgendwelche Auswahlanweisungen, die ich in "Nicht-Auswahlanweisungen" ändern kann?

Hier ist mein Code

#include <iostream> 
#include <iomanip> 
using namespace std; 

int main() 
{ 

    char gender; 
    int height; 
    int weight; 
    bool heightOK; 
    bool weightOK; 
    int candidateCount; 
    int validCandidateCount; 
    bool invalidGender; 
    bool invalidHeight; 
    bool invalidWeight; 
    float percentOutput; 

    candidateCount = 0; 
    validCandidateCount = 0; 

    cout << "Please enter the candidate's information (enter 'X' to exit)." 
    << endl; 

    do 
    { 
     cout << "Gender: "; 
     cin.get(gender); 

     cin.ignore (1000,'\n'); 
     invalidGender = (!(gender == 'm' || 
          gender == 'M' || 
          gender == 'f' || 
          gender == 'F')); 

     candidateCount = candidateCount + 1; 

     if(invalidGender) 
     { 
      cout << "***** Invalid gender; please enter M or F*****" <<  endl; 
     } 

    }while(invalidGender); 

    while (gender != 'X' || gender != 'x') 
    { 
     candidateCount = candidateCount + 1; 

     do 
     { 
      cout << "Height: "; 
      cin >> height; 

      invalidHeight = height < 24 || height > 110; 

      heightOK = ((gender == 'm' || gender == 'M') && 
         (height > 65 && height < 80)); 

      heightOK = heightOK || ((gender == 'f' || gender == 'F') && 
            (height > 62 && height < 75)); 

      if(invalidHeight) 
      { 
       cout << "***** Invalid height; please enter a height in inches between 24 and 110. *****" 
        << endl; 
      } 

     }while(invalidHeight); 


     do 
     { 
     cout << "Weight: "; 
     cin >> weight; 

     invalidWeight = weight < 50 || weight > 1400; 

     weightOK = ((gender == 'm' || gender == 'M') && 
         (weight > 130 && weight < 250)); 

     weightOK = weightOK || ((gender == 'f' || gender == 'F') && 
        (weight > 110 && weight < 185)); 

     if(invalidWeight) 
      { 

       cout << "***** Invalid weight; please enter a weight in lbs between 50 and 1400." 
        << endl; 
      } 

     }while(invalidWeight); 



     if(heightOK && weightOK) 
     { 
      cout << "This candidate has been ACCEPTED!" << endl; 

      validCandidateCount = validCandidateCount + 1; 
     } 
     else if (!heightOK) 
     { 
      cout << "This candidate has been rejected based on the HEIGHT requirement." 
       << endl; 
     } 
     else if (!weightOK) 
     { 
      cout << "This candidate has been rejected based on the WEIGHT requirement." 
       << endl; 
     } 
     else if (!(heightOK && weightOK)) 
     { 
      cout << "This candidate has been rejected based on the HEIGHT and WEIGHT requirements" 
       << endl; 
     } 
     do 
     { 
      cout << "Gender: "; 
      cin.get(gender); 
      cin.ignore (1000,'\n'); 

      candidateCount = candidateCount + 1; 

      if(invalidGender) 
      { 
      cout << "***** Invalid gender; please enter M or F*****" <<  endl; 
      } 
     }while(invalidGender); 


    } 

    cout << validCandidateCount << " candidate(s) accepted!" << endl; 

    percentOutput = validCandidateCount/candidateCount; 

    cout << "That's " << percentOutput <<"%!" << endl; 



return 0; 
} 

Answer 1

Der Haupt while-Schleife sollte und Zustand.

while(gender !='X' && gender!='x) 

Und Ihr Auswahlcode hat falsche bedingte Anweisungen.

if(heightOK && weightOK) 
    { 
     cout << "This candidate has been ACCEPTED!" << endl; 

     validCandidateCount = validCandidateCount + 1; 
    } 
    else if (!heightOK) // you have written else if(heightOK) 
    { 
     cout << "This candidate has been rejected based on the HEIGHT requirement." 
      << endl; 
    } 
    else if (!weightOK) // you have written else if(weightOK) 
    { 
     cout << "This candidate has been rejected based on the WEIGHT requirement." 
      << endl; 
    } 
    else if (!(heightOK && weightOK)) 
    { 
     cout << "This candidate has been rejected based on the HEIGHT and WEIGHT requirements" 
      << endl; 
    } 

Sie sollten diese invalidgender Zustand in der letzten do while-Schleife entfernen, sonst wird es zu einer Endlosschleife führen, auch wenn Sie durch Drücken der Taste X. Anstelle der ungültigen Geschlecht Bedingung kann an den Start gebracht werden verlassen wollen von Haupt While-Schleife.

Und invalidGender-Variable sollte wieder ihren Wert zugewiesen werden andernfalls wird es den zuvor gespeicherten Wert übernehmen.

invalidGender = (!(gender == 'm' || 
         gender == 'M' || 
         gender == 'f' || 
         gender == 'F')); 

der gesamte Code

#include <iostream> 
#include <iomanip> 
using namespace std; 

int main() 
{ 

char gender; 
int height; 
int weight; 
bool heightOK; 
bool weightOK; 
int candidateCount; 
int validCandidateCount; 
bool invalidGender; 
bool invalidHeight; 
bool invalidWeight; 
double percentOutput; 

candidateCount = 0; 
validCandidateCount = 0; 

cout << "Please enter the candidate's information (enter 'X' to exit)." 
<< endl; 

    cout << "Gender: "; 
    cin.get(gender); 

while (gender != 'X' && gender != 'x') 
{ 
    candidateCount = candidateCount + 1; 

    do 
    { 
     invalidGender = (!(gender == 'm' || 
         gender == 'M' || 
         gender == 'f' || 
         gender == 'F')); 

     if(invalidGender) 
     { 
      cout << "***** Invalid gender; please enter M or F*****" <<  endl; 
      cout << "Gender: "; 
      cin>>gender; 
      cin.ignore (1000,'\n'); 
     } 
    }while(invalidGender); 

    do 
    { 
     cout << "Height: "; 
     cin >> height; 

     invalidHeight = height < 24 || height > 110; 

     heightOK = ((gender == 'm' || gender == 'M') && 
        (height > 65 && height < 80)); 

     heightOK = heightOK || ((gender == 'f' || gender == 'F') && 
           (height > 62 && height < 75)); 

     if(invalidHeight) 
     { 
      cout << "***** Invalid height; please enter a height in inches between 24 and 110. *****" 
       << endl; 
     } 

    }while(invalidHeight); 


    do 
    { 
    cout << "Weight: "; 
    cin >> weight; 

    invalidWeight = weight < 50 || weight > 1400; 

    weightOK = ((gender == 'm' || gender == 'M') && 
        (weight > 130 && weight < 250)); 

    weightOK = weightOK || ((gender == 'f' || gender == 'F') && 
       (weight > 110 && weight < 185)); 

    if(invalidWeight) 
     { 

      cout << "***** Invalid weight; please enter a weight in lbs between 50 and 1400." 
       << endl; 
     } 

    }while(invalidWeight); 



    if(heightOK && weightOK) 
    { 
     cout << "This candidate has been ACCEPTED!" << endl; 

     validCandidateCount = validCandidateCount + 1; 
    } 
    else if (!heightOK) 
    { 
     cout << "This candidate has been rejected based on the HEIGHT requirement." 
      << endl; 
    } 
    else if (!weightOK) 
    { 
     cout << "This candidate has been rejected based on the WEIGHT requirement." 
      << endl; 
    } 
    else if (!(heightOK && weightOK)) 
    { 
     cout << "This candidate has been rejected based on the HEIGHT and WEIGHT requirements" 
      << endl; 
    } 


     cout << "Gender: "; 
     cin>>gender; 
     cin.ignore (1000,'\n'); 

} 

cout << validCandidateCount << " candidate(s) accepted!" << endl; 

percentOutput = (double)validCandidateCount/(double)candidateCount; 

cout << "That's " << percentOutput*100 <<"%!" << endl; 



return 0; 
}