DaniWeb Forum Index > Software Development > C++

C++ RSS

Structures

Please support our C++ advertiser: Intel Parallel Studio Home

•

Join Date: Apr 2008

Posts: 59

Reputation: Dannyo329 is an unknown quantity at this point

Solved Threads: 6

Dannyo329

Offline

Junior Poster in Training

Structures

Jan 3rd, 2009

I'm working with structures, but somehow it skips one of the user inputs for the pie's price:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
#include <iostream>
#include <windows.h>
#include <string>
#include <conio.h>
 
using namespace std;
 
const int ESC = 0x1b; 
 
struct pie{
       string brand;
       float price;
       string flavour;
}pies[5];
 
int main()
{
    cout << "Structures.";
    cout << "\nPress c to contine.";
    int a;
    while ( (a = getch()) != ESC)
    if (a == 'c')
    {
          int n;
          int g;
          for (n = 0; n<5; n++)
          {
              cout << "\nEnter brand of pie:";
              getline(cin,pies[n].brand);
              cout << "\nEnter price of pie:";
              cin >> g;
              pies[n].price = g;
              cout << "\nEnter Flavour of pie:";
              getline(cin,pies[n].flavour);
          }
          getch();
          return 0;
    }
}
#include <iostream>
#include <windows.h>
#include <string>
#include <conio.h>

using namespace std;

const int ESC = 0x1b; 

struct pie{
       string brand;
       float price;
       string flavour;
}pies[5];

int main()
{
    cout << "Structures.";
    cout << "\nPress c to contine.";
    int a;
    while ( (a = getch()) != ESC)
    if (a == 'c')
    {
          int n;
          int g;
          for (n = 0; n<5; n++)
          {
              cout << "\nEnter brand of pie:";
              getline(cin,pies[n].brand);
              cout << "\nEnter price of pie:";
              cin >> g;
              pies[n].price = g;
              cout << "\nEnter Flavour of pie:";
              getline(cin,pies[n].flavour);
          }
          getch();
          return 0;
    }
}

When I get up to Enter brand of pie, then I press enter, it skips straight onto Enter Flavour of pie leaving the price of pie blank. I've tried cin.ignore() but that doesn't seem to do it. Any help would be appreciated.

•

Join Date: Feb 2006

Posts: 490

Reputation: Bench has a spectacular aura about

Solved Threads: 49

Bench

Offline

Posting Pro in Training

Re: Structures

Jan 3rd, 2009

There's plenty of ways around the many problems associated with cin >>; IMHO the best way to avoid them is to never use the >> operator in conjunction with cin in the first place (unless its really necessary).

My personal preference is to keep a separate function which grabs a line using getline(), and a stringstream to convert your input to whatever numeric type you need.

 Help with Code Tags 
 cpp Syntax (Toggle Plain Text) 
bool read_float( std::istream& in , float& num )
{
    std::string str;
    std::getline( in, str );
    std::istringstream iss( str );
    iss >> num;
    return iss.good();
}
bool read_float( std::istream& in , float& num )
{
    std::string str;
    std::getline( in, str );
    std::istringstream iss( str );
    iss >> num;
    return iss.good();
}

This has the added bonus of removing the risk that cin will fail due to invalid chars being typed. You can check for this after asking the user to enter the price of the pie.

 Help with Code Tags 
 cpp Syntax (Toggle Plain Text) 
cout << "\nEnter price of pie:";
while ( ! read_float( std::cin , g ) ) 
    std::cout << "Invalid input - Enter price of pie: ";
cout << "\nEnter price of pie:";
while ( ! read_float( std::cin , g ) ) 
    std::cout << "Invalid input - Enter price of pie: ";

One slight nitpick - the default floating-point type in C++ is double. For that reason its usually recommended to always use double whenever you need floating point numbers, unless you have some compelling reason otherwise to use float (which generally suffers from low precision. usually only 5 or 6 s.f. on most modern machines)

edit if you want to be extra-clever you can template the 'read' function to work with other numeric types (which would be handy if you wanted to do the same with int or double)

 Help with Code Tags 
 cpp Syntax (Toggle Plain Text) 
template <typename Ty>
bool read_in( std::istream& in , Ty& num )
{
    std::string str;
    std::getline( in, str );
    std::istringstream iss( str );
    iss >> num;
    return iss.good();
}
template <typename Ty>
bool read_in( std::istream& in , Ty& num )
{
    std::string str;
    std::getline( in, str );
    std::istringstream iss( str );
    iss >> num;
    return iss.good();
}