C / C++ / MFC

char buff[1024];
  unsigned int value;
  int charsRead;

  bool validEntry = false;
  while (!validEntry)
  {
      cout << "Please enter the number of subunits: ";
      cin.getline(buff, 1023, '\n');


         if (sscanf(buff, "%u%n",&value,&charsRead)==1)
         validEntry = true;

     if (validEntry == true)
             if (value >= 1 && value <= 12)
          break;
         else
          validEntry =false;
  }

Harold_Wishes wrote:

I ended up doing this instead:

"The largest fire starts but with the smallest spark." - David Crow

"Judge not by the eye but by the heart." - Native American Proverb

do{    
	cout << "Please ... ";    
	cin >> data2;


} while (data2 < 1 || data2 > 12);

Harold_Wishes wrote:

I might mention that the code below does not test for non-numeric values...

"The largest fire starts but with the smallest spark." - David Crow

"Judge not by the eye but by the heart." - Native American Proverb

// assume min and max are included in range
int getInput(int range_min, int range_max)
{
        int ret = range_min - 1;        // initialized to a invalid entry
        do
        {
                char buffer[100] = {0};
                cout << "Enter a value:  ";
                cin.getline(buffer, 99, '\n');
                ret = atoi(buffer);
                if (range_min > ret || ret > range_max)
                {
                        cout << "Invalid entry!" << endl;
                }
        } while (range_min > ret || ret > range_max);

        return ret;
}

int main()
{
	int count = getInput(1, 11);
	cout << "You entered:  " << count << endl;
}

<br />
char buff[1024];<br />
unsigned int value;<br />
int charsRead;<br />
<br />
bool validEntry = false;<br />
while (!validEntry){ <br />
   cout << "Please enter the number of subunits: ";<br />
   cin.getline(buff, 1023, '\n');<br />
   <br />
   if (sscanf(buff, "%u%n",&value,&charsRead)==1 && value >=1 && value <= 12)<br />
      validEntry = true;<br />
}<br />

<br />
	int test;<br />
	std::string dump;<br />
<br />
	bool valid = false;<br />
	while(!valid) {<br />
		cout << "Please enter a number: ";<br />
		cin >> test;<br />
<br />
		if (!cin.fail())<br />
			valid = true;<br />
		else {<br />
			cin.clear();<br />
			cin >> dump;  //clear input buffer<br />
		}<br />
<br />
		if (valid && (test < 1  ||  test > 12)) {<br />
			valid = false;<br />
			cout << "Out of range" << endl;<br />
		}<br />
			<br />
	}<br />
	cout << "You entered " << test << endl;<br />

#include <cstring>
#include <string>
#include <iostream>


using namespace std;



int main()
{
    int data2;
    char buff[1024];
    unsigned int value;
    int charsRead; 

    char selection;
    cout << "A) Complete  B) Partial  C) UNKNOWN" << endl;
    cin >> selection;



    bool validEntry = false;
	while (!validEntry)
	{ 
		cout << "Please enter the number of subunits: ";
		cin.getline(buff, 1023, '\n');

		if (sscanf(buff, "%u%n",&value,&charsRead)==1 && value >=1 && value <= 12)
		   validEntry = true;
	}

	data2 = value;

  return 0;

}

toxcct wrote:

pthread is a header you wrote

mehmetned wrote:

why was this error produced?

"The largest fire starts but with the smallest spark." - David Crow

"Judge not by the eye but by the heart." - Native American Proverb

Christian Graus wrote:

But a class *is* a struct.

Dave Schumann wrote:

So why can't B - which IS AN A - access the value member of another A?

"The largest fire starts but with the smallest spark." - David Crow

"Judge not by the eye but by the heart." - Native American Proverb

Maximilien wrote:

all members in structs should be public