使用矢量C++

Question

复杂的问题，可能很简单的答案。所以我需要制作的程序不能包含除String，iostream和vector之外的任何库。我需要创建一个具有3个功能的程序。一个创建一个整数矢量，一个反转矢量，一个打印矢量。为了获取值，我需要使用getline来获取字符串，如果字符串状态退出，我们会停止向其中添加新值。其他方面，我们需要测试一个整数（正数或负数）并将其添加到矢量中。我的代码开始变得复杂，所以我真的需要一些帮助。以下是我到目前为止。如果这很重要，我也使用Visual Studio。感谢您提前提供任何帮助！我的问题是当我运行该程序时，它只会输出第一个数字。我不知道为什么，并想知道我做错了什么。

#include <iostream> 
#include <string> 
#include <vector> 
using namespace std; 
vector<int> CreateVector() 
{ 
    string tempvariable; 
    bool quit = false; 
    vector<int> userinput; 
    cout << "Please enter in an integer, type 'quit' to exit " << endl; 
    while (!quit) 
    { 
     getline(cin, tempvariable); 
     if (tempvariable == "quit") 
      quit = true; 
     else 
     { 
      bool isaninteger = true; 
      for(int i = 1; i <= tempvariable.size(); i++) 
      { 
       if (tempvariable[i] = "-" || isdigit(tempvariable[i])) 
        continue; 
       else 
       { 
        cout << "You entered in an incorrect option, please enter in a correct option" << endl; 
        cin.clear(); 
        cin.ignore(); 
        isaninteger = false; 
        break; 
       } 
      } 
      if (isaninteger) 
       userinput.push_back(stoi(tempvariable)); 
      cout << "Please enter in an integer, type 'quit' to exit " << endl; 
     } 
    } 
    return userinput; 
} 
void printVector(vector<int> userinput) 
{ 
    int amountofspots = userinput.size(); 
    cout << "Your Vector is "; 
    for (int i = 0; i < amountofspots; i++) 
    { 
     if (i = (amountofspots - 1)) 
      cout << userinput.at(i) << endl; 
     else 
      cout << userinput.at(i) << " , "; 
    } 
} 
void reverseVector(vector<int>& userinput) 
{ 
    int amountofspots = userinput.size(); 
    vector<int> tempvector; 
    for (int i = 0; i < amountofspots; i++) 
     tempvector.push_back(userinput.at(amountofspots - i)); 
    for (int i = 0; i < amountofspots; i++) 
     userinput.pop_back(); 
    for (int i = 0; i < amountofspots; i++) 
     userinput.push_back(tempvector.at(i)); 
} 
int main() 
{ 
    vector<int> CreatedVector = CreateVector(); 
    printVector(CreatedVector); 
    reverseVector(CreatedVector); 
    printVector(CreatedVector); 
    system("pause"); 
    return 0; 
} 

Answer 1

1. 变化if (i = (amountofspots - 1))到if (i == (amountofspots - 1))在printVector()循环。

2. 将tempvariable[i] = "-"更改为tempvariable[i] == '-'的CreateVector()。

=是赋值运算符，==是比较运算符。此外，单个字符被单引号包围，而不是双引号。

Answer 2

void reverseVector(vector<int>& userinput) 
{ 
    int amountofspots = userinput.size(); 
    vector<int> tempvector; 
    for (int i = 0; i < amountofspots; i++) 
     tempvector.push_back(userinput.at(amountofspots - i)); 

也许应该

void reverseVector(vector<int>& userinput) 
{ 
    int amountofspots = userinput.size(); 
    vector<int> tempvector; 
    for (int i = 1; i < amountofspots+1; i++) // Index error 
     tempvector.push_back(userinput.at(amountofspots - i)); 

Answer 3

以下：

 for(int i = 1; i <= tempvariable.size(); i++) 
     { 
      if (tempvariable[i] = "-" || isdigit(tempvariable[i])) 

必须成为：

 for(int i = 0; i < tempvariable.size(); i++) 
     { 
      if (tempvariable[i] == '-' || isdigit(tempvariable[i])) 

说明：

• 字符串索引从0 开始和尺寸（）结束 - 1。
• [i]返回单个char。 "-"不是一个单一的char，而是一个完整的字符串文字。您不能直接比较单个char与整个字符串。然而，-是一个单独的char，所以比较会起作用。
• =不是比较而是分配。 ==用于比较。由于你的编译器应该警告你！

---

注意，有更多的问题与您的代码：

if (i = (amountofspots - 1)) 

你又混合赋值和比较。做那==。并注意编译警告！

最后，isdigit不是一个很好的功能。为了正确使用它，首先必须将操作数强制转换为int，并同时确保它不是无效值，如文档here所述。

如果指定的字符串不能被解析，为什么不抓住stoi抛出的异常呢？