Help with switch statements inside while a loop

[JackHawk]

Trying to write a program that can calculate the volume, surface area, or "girth" of a box. I want to be able to enter a letter for each subject and after it calculate that subject enter a letter for the next subject. Also want the program to end when a Q is typed. I believe my math is right but this sucker wont work for me. It compiles fine but the Volume wont calculate at all though the other two will. I also get three lines of answers, and when I go to enter a second letter for a second value the program runs like crazy. Any and all help is much appreciated. Thanks for your time.

#include <iostream>
#include <iomanip>

using namespace std;

const char SENTINEL = 'Q';//found this suggestion online to quit a program

int main ()
{
   
    char V, A, G, inputChar;
    float length, width, depth, totalV, totalA, totalG;
    
    cout<<"Enter character V for Volume, character A for Surface Area,\n";
    cout<<"character G or Girth plus Depth or Q to quit\n"<<endl;
    cin>>inputChar;// pick letter
    while (inputChar!=SENTINEL)// if not Q then check the if-ele statments
    {
          switch (inputChar)
          case 'V':
          {
                    cout<<"Enter Length, Width, and Depth for Volume\n";
                    cin>>length, width, depth;
                    totalV=length*width*depth;//math for volume
                    cout<<"The Volume = "<<totalV<<endl;
                    break;
          }
          case 'A':
          {
                   cout<<"Enter Length, Width, and Depth for Surface Area\n";
                   cin>>length, width, depth;
                   totalA=2*length*width+2*width*depth+2*length*depth;//math for area
                   cout<<"The Surface Area = "<<totalA<<endl;
                   break;
          }
          case 'G':
          {
                   cout<<"Enter Length, Width, and Depth for Girth\n";
                   cin>>length, width, depth;
                   totalG=2*(length+width)+depth;//math for girth
                   cout<<"The Girth = "<<totalG<<endl;
                   break;
          }          
    }
          system ("pause");
          return 0;
}

[YokoTsuno]

The reason why the program goes "haywire" is that cin<< isn't called anymore within the while loop.

[SECMoG1T]

Hi
cin>>length, width, depth; is an error replace with cin>> length>> width>> depth;

You might consider a default case too.

cin>> inputChar;
 while (inputChar!=SENTINEL)// if not Q then check the if-ele statments 
      { 
          switch (inputChar) 
             {
                case 'V': 
                          { 
                               cout<<"Enter Length, Width, and Depth for Volume\n"; 
                               cin>>length>>width>>depth; 
                               totalV=length*width*depth;//math for volume 
                               cout<<"The Volume = "<<totalV<<endl; 
                               break;
                          } 
              case 'A':
                          { 
                               cout<<"Enter Length, Width, and Depth for Surface Area\n"; 
                               cin>>length>> width>>epth; 
                               totalA=2*length*width+2*width*depth+2*length*depth;//math for area 
                               cout<<"The Surface Area = "<<totalA<<endl;
                               break;
                          } 
               case 'G':
                          { 
                                cout<<"Enter Length, Width, and Depth for Girth\n"; 
                                cin>>length>> width>>depth; 
                                totalG=2*(length+width)+depth;//math for girth 
                                cout<<"The Girth = "<<totalG<<endl;
                                break; 
                          } 
              default:  
                          {
                                cout <<"error wrong entry\n";  break; 
                          }
            }
    
            cin>> inputChar; 
     }

Note you need to put your switch into a block lile i have done.

[ne555]

http://www.cplusplus.com/doc/tutorial/operators/ (comma operator)

//cin>>length, width, depth;
cin>>length>>width>>depth;

[JackHawk]

Thank you so much for the help. :)