第一次迭代失败

Question

我对 C++ 还很陌生，但从逻辑上讲，我认为这应该可行。

我有一个函数，它接受用户的输入来设置数组的内存分配：

void setarraynum(){
  string mystr;
  cout<<"Please enter the size of your array: ";
  getline(cin, mystr);
  stringstream(mystr)>>arraynum;
  array = new int [arraynum];
  cout<<"\n";
}

arraynum 和 array 是全局变量：

int arraynum;
int * array;

然后我有一个接受数组输入的函数：

void setarray(){
  string mystr;
  cout<<"Please enter "<<arraynum<<" numbers:\n";
  for(int n=0; n<arraynum; n++){
    getline(cin, mystr);
    stringstream(mystr)>>array[n];
  };
cout<<"\n";
}

在输入数组编号时会出现问题，数组的第一个实例 (array[0]) 会自动设置为 0。就好像循环在不询问用户输入的情况下迭代第一次迭代一样。然后它会照常询问用户。

有什么想法吗？

谢谢。

*编辑（根据要求，全部代码）：

#include <iostream>
#include <sstream>
#include <string>
using namespace std;

int arraynum;
int * array;
bool isin = false;
int input;
char order;

void insertionsortdesc(int a[]);
void insertionsortasc(int a[]);
void getarray(int a[]);
void isinarray(int a[], int b);

void setsearch();
void setarray();
void setarraynum();
void setorder();

int main(){

 cout<<"\n//// INPUT ////\n\n";

 setarraynum();
 setorder();
 setarray();
 setsearch();

 if(order=='a'){insertionsortasc(array);
 }else if(order=='d'){insertionsortdesc(array);};

 cout<<"//// OUTPUT ////\n\n";
 getarray(array);
 isinarray(array, input);

 return 0;
}

void setorder(){
 bool isvalid = false;

 while(!isvalid){
  cout<<"Ascending or Descending [a/d]: ";
  cin>>order;
  if(order=='a'||order=='d'){isvalid = true;
  }else{cout<<"Please enter a valid option!\n";};
 };

cout<<"\n";
}

void setarraynum(){
 string mystr;
 cout<<"Please enter the size of your array: ";
 getline(cin, mystr);
 stringstream(mystr)>>arraynum;
 array = new int [arraynum];
 cout<<"\n";
}

void setsearch(){
 string mystr;
 cout<<"Search for (int): ";
 getline(cin, mystr);
 stringstream(mystr)>>input;
 cout<<"\n";
}

void setarray(){
 string mystr;
 cout<<"Please enter "<<arraynum<<" numbers:\n";
 for(int n=0; n<arraynum; n++){
  getline(cin, mystr);
  stringstream(mystr)>>array[n];
 };
cout<<"\n";
}

void insertionsortdesc(int a[]){
 for(int n=1; n<arraynum; n++){
  int key = a[n];
  int j = n-1;
  while((j>=0)&&(a[j]<key)){
   a[j+1] = a[j];
   j -= 1;
  };
  a[j+1]=key;
 }
}

void insertionsortasc(int a[]){
 for(int n=1; n<arraynum; n++){
  int key = a[n];
  int j = n-1;
  while((j>=0)&&(a[j]>key)){
   a[j+1] = a[j];
   j -= 1;
  };
  a[j+1]=key;
 }
}

void isinarray(int a[], int b){
 for(int n=0; n<arraynum; n++){
  if(a[n] == b){
   isin = true;
   break;
  };
 };

 if(isin){
  cout<<b<<" is present in the given array.";
 }else{
  cout<<b<<" is not present in the given array.";
 };

 cout<<"\n\n";
}

void getarray(int a[]){
 cout<<"Sorted array sequence: ";
 for(int n=0; n<arraynum; n++){
  cout<<a[n]<<", ";
 };
 cout<<"\n\n";
}

Answer 1

我强烈建议您直接使用cin >> arraynum;，而不是在stringstream(mystr) >> arraynum; 范式中读取整数。

来自 C++ 参考 http://www.cplusplus.com/reference/string/string/getline/ for getline

如果找到分隔符，则将其提取并丢弃，即它是 没有存储，下一个输入操作将在它之后开始。

所以这就是问题所在。 正如@Vaughn 所指出的，您在setorder 函数中使用cin >> order;，这会将拖尾\n 留在输入缓冲区中。

您的代码学到的教训是，在处理 I/O 问题时，请确保以相同的范例调用函数。

Answer 2

问题就在这里：

cout<<"Ascending or Descending [a/d]: ";
cin>>order;

您正在阅读“a”或“d”，但不是回车。

当你到达这里时：

cout<<"Please enter "<<arraynum<<" numbers:\n";
for(int n=0; n<arraynum; n++){
  getline(cin, mystr);

getline 会将您在 'a' 或 'd' 之后和回车之前输入的任何内容视为它所获得的行 - 可能是空白。

解决它的一种方法是在阅读 order 后添加另一个 getline：

cout<<"Ascending or Descending [a/d]: ";
cin>>order;
string rest_of_line;
getline(cin,rest_of_line);

Answer 3

我稍微修改了您的代码，它的行为符合预期。我建议您向用户表明他是否应该对他的输入进行定界，或者他应该如何定界。我宁愿使用向量而不是数组。您已经没有取消分配您的数组 - 这并不重要，但它可能在另一个程序中。

另外，我不知道你为什么在 cin 已经允许你获取整数时使用 getline（见代码）：

#include <iostream>
#include <sstream>
#include <string>

int arraynum;
int * array;
bool isin = false;
int input;
char order;

void insertionsortdesc(int a[]);
void insertionsortasc(int a[]);
void getarray(int a[]);
void isinarray(int a[], int b);

void setsearch();
void setarray();
void setarraynum();
void setorder();

int main(){

 std::cout<<"\n//// INPUT ////\n\n";

 setarraynum();
 setorder();
 setarray();
 setsearch();

 if(order=='a'){insertionsortasc(array);
 }else if(order=='d'){insertionsortdesc(array);};

 std::cout<<"//// OUTPUT ////\n\n";
 getarray(array);
 isinarray(array, input);

 return 0;
}

void setorder(){
 bool isvalid = false;

 while(!isvalid){
  std::cout<<"Ascending or Descending [a/d]: ";
  std::cin>>order;
  if(order=='a'||order=='d'){isvalid = true;
  }else{std::cout<<"Please enter a valid option!\n";};
 };

std::cout<<"\n";
}

void setarraynum(){
 std::cout<<"Please enter the size of your array: ";
 while( !( std::cin >> arraynum ) )
 {
   std::cout << "Please enter integer!";
 }
 array = new int [arraynum];
 std::cout<<"\n";
}

void setsearch(){
 std::cout<<"Search for (int): ";
 while( !(std::cin >> input) )
 {
   std::cout << "Please enter integer!" << std::endl;
 }
 std::cout<<"\n";
}

void setarray(){
 std::ostringstream msgStream;
 msgStream << "Please enter "<<arraynum<<" numbers:";
 std::cout << msgStream.str() << std::endl;

 for(int n=0; n<arraynum; n++){
   if( !(std::cin >> array[n]) ){
     n = 0;
     std::cout << msgStream.str() << std::endl;
   }
 }
std::cout<<"\n";
}

void insertionsortdesc(int a[]){
 for(int n=1; n<arraynum; n++){
  int key = a[n];
  int j = n-1;
  while((j>=0)&&(a[j]<key)){
   a[j+1] = a[j];
   j -= 1;
  };
  a[j+1]=key;
 }
}

void insertionsortasc(int a[]){
 for(int n=1; n<arraynum; n++){
  int key = a[n];
  int j = n-1;
  while((j>=0)&&(a[j]>key)){
   a[j+1] = a[j];
   j -= 1;
  };
  a[j+1]=key;
 }
}

void isinarray(int a[], int b){
 for(int n=0; n<arraynum; n++){
  if(a[n] == b){
   isin = true;
   break;
  };
 };

 if(isin){
  std::cout<<b<<" is present in the given array.";
 }else{
  std::cout<<b<<" is not present in the given array.";
 };

 std::cout<<"\n\n";
}

void getarray(int a[]){
 std::cout<<"Sorted array sequence: ";
 for(int n=0; n<arraynum; n++){
  std::cout<<a[n]<<", ";
 };
 std::cout<<"\n\n";
}