C / C++ / MFC

void add_number_of_subunits (int& data)
{
    char buff[1024]; // stores whatever the user types in
    unsigned int value;
    int charsRead;

	
    bool validEntry = false; // sets a flag so that the
                            // validation can be performed
                             //in the loop below
	while (!validEntry)	
	{ 	
	  cout << endl;
	  cout << "Please enter number of subunits: ";
	  cin >> buff; 
		
	  if (sscanf(buff,"%u%n", &value,&charsRead)==1 && value >=1 && value <= 12)	
		validEntry = true;	// test to see if the user enter a valid integer
		                        // If it is valid, the flag is changed which allow the 
		                        // program  to exit the loop. If not, the flag remains
		                        // as is until the user enter a valid number.
	}

	data = value; // The value entered by the user is copied into "data" and returned to main()
	              // which determines the number of cycles the user will have to enter per
	              // subunit

}

void add_node_at_end(data& item, int& increment)
{    
	     // what we are adding to
	char buffer[501] = {0}; // buffer for input

	cout << "Please enter the length of the protein sequence for Subunit " << increment << "> ";

        cin >> buffer; // I had to enter 2 cin statements since  the compiler for whatever reason
	cin.getline(buffer, 500, '\n');               // was not accepting one of them.

	item.Length = buffer;
        memset(buffer, 0, 501);

	cout << "Please enter the protein sequence for Subunit " << increment << "> ";
	cin.getline(buffer, 500, '\n');

	while (buffer == "\n")
    {
		cout << "A sequence is required!" << endl;
		cout << "Please enter a sequence." << endl;
                cin.getline(buffer, 500, '\n');
    }

    item.Sequence = buffer;
    memset(buffer, 0, 501);
    item.number = item.Sequence.size();


    cout << "Enter brief status on the presence of a modified N_Terminal: ";
    cin.getline(buffer, 500, '\n');
    item.N_Terminal = buffer;
    memset(buffer, 0, 501);

    cout << "Enter brief status on the presence of a modified C_Terminal: ";
    cin.getline(buffer, 500, '\n');
    item.C_Terminal = buffer;
    memset(buffer, 0, 501);

    g_DataList.push_back(item);   // add to list

    cout << endl;
	
	increment++;
}

Harold_Wishes wrote:

cin >> buffer; // I had to enter 2 cin statements since the compiler for whatever reason
cin.getline(buffer, 500, '\n'); // was not accepting one of them.

Harold_Wishes wrote:

while (buffer == "\n")

void add_number_of_subunits (int& data)
{
    string buff;
     
    do
    { 	
        cout << endl;
        cout << "Please enter number of subunits: ";
        cin >> buff; 
		
    } while (atoi(buff.c_str()) < 1 || atoi(buff.c_str()) > 12);	
 
    data = atoi(buff.c_str()); 
}
 
void add_node_at_end(_data& item, int& increment)
{    
    string buffer;
 
    cout << "Please enter the length of the protein sequence for Subunit " << increment << "> ";
    cin >> buffer;
 
    item.Length = buffer;
 
    cout << "Please enter the protein sequence for Subunit " << increment << "> ";
    cin >> buffer;
 
    item.Sequence = buffer;
    item.number = item.Sequence.size();
 
    cout << "Enter brief status on the presence of a modified N_Terminal: ";
    cin >> buffer;
    item.N_Terminal = buffer;
 
    cout << "Enter brief status on the presence of a modified C_Terminal: ";
    cin >> buffer;
    item.C_Terminal = buffer;
 
    cout << endl;
	
    increment++;
}

DavidCrow wrote:

void add_number_of_subunits (int& data)
{
string buff;

do
{
cout << endl;
cout << "Please enter number of subunits: ";
cin >> buff;

} while (atoi(buff.c_str()) < 1 || atoi(buff.c_str()) > 12);

data = atoi(buff.c_str());
}

void add_number_of_subunits (int& data)
{
    int check = 0;
     
    do
    { 	
        cout << endl;
        cout << "Please enter number of subunits: ";
        cin >> check; 
    } while (check < 1 || check > 12);	
 
    data = check; 
}

Zac Howland wrote:

This code looks vaguely familiar ... someone else was asking about a similar problem a couple weeks ago.

Zac Howland wrote:

int check = 0;
...
cin >> check;

DavidCrow wrote:

Same person. Still using sscanf() for validation, even though you provided him an alternative!

DavidCrow wrote:

I thought about this (i.e., using an int), but since he was storing the input string for later processing, I left it as a string.

Harold_Wishes wrote:

1) I want the user to be able to bypass keyboard entrys when they hit ENTER if they choose to skip a question. Now when I hit ENTER, the program will not let me go to the next cout prompt. It is waiting for something other than "\n" character. I believe this is why I used my getline implementation.

Harold_Wishes wrote:

2)Also, if I type in something like "new terminal", such entrys delimited by a space seem to cause the next cin input to read in the second string. But I need to test the output some more with this second issue since I'm getting other strange results.

char buffer[501] = {0}; // buffer for input

cout << "Please enter the length of the protein sequence for Subunit;
cin.getline(buffer, 500, '\n'); 
cin.getline(buffer, 500, '\n');

char buffer[501] = {0}; // buffer for input<br />
char buffer1[501] = {0}; // buffer for clearing<br />
<br />
cout << "Please enter the length of the protein sequence for Subunit;<br />
cin.getline(buffer, 500, '\n'); <br />
cin.getline(buffer1, 500, '\n');

char buffer[501] = {0}; // buffer for input

cout << "Please enter the length of the protein sequence;
cin >> buffer; 
cin.getline(buffer, 500, '\n');

Harold_Wishes wrote:

1) I want the user to be able to bypass keyboard entrys when they hit ENTER if they choose to skip a question. Now when I hit ENTER, the program will not let me go to the next cout prompt. It is waiting for something other than "\n" character. I believe this is why I used my getline implementation.

do
{
    cin.clear();           // reset cin, so more input can be done
    cin.ignore(100, '\n'); // remove bad data, up to 100 characters
                           // or until the end of line is reached
    cout << "Please enter number of subunits: ";
    cin >> data;
} while (! cin && (data < 1 || data > 12));

Harold_Wishes wrote:

2)Also, if I type in something like "new terminal", such entrys delimited by a space seem to cause the next cin input to read in the second string. But I need to test the output some more with this second issue since I'm getting other strange results.

string buffer;
cout << "Please enter the length of the protein sequence for Subunit " << increment << "> ";
getline(cin, buffer);

ningthemcha wrote:

when i Try to do same thing with DoModal
it is nit working .