Question

我的void print函数无法打印出这个向量。我不太确定它与“std :: allocator”有什么关系。我得到这些错误：

st1.cpp: In function ‘void Print(std::vector<int, std::allocator<int> >)’:
st1.cpp:51: error: declaration of ‘std::vector<int, std::allocator<int> > v’ shadows a      parameter

这是文件：

#include <iostream>
#include <string>
#include <vector>
#include <stack>
#include <algorithm>
using namespace std;

void Initialize();
void Print();

int main()
{
stack<string> s1, s2;
s1.push("b");
s2.push("a");

if (s1.top() == s2.top())
{
    cout << "s1 == s2" << endl;
}
else if (s1.top() < s2.top())
{
    cout << "s1 < s2" << endl;
}
else if (s2.top() < s1.top())
{
    cout << "s2 < s1" << endl;
}
else 
{
    return 0;
}

vector<int> v;
Initialize();
Print();
}

void Initialize(vector<int> v)
{
int input;
cout << "Enter your numbers to be evaluated: " << endl;
while(input != -1){
    cin >> input;
    v.push_back(input);
    //write_vector(v);
}
}

void Print (vector<int> v){
vector<int> v;
for (int i=0; i<v.size();i++){
    cout << v[i] << endl;
}
}

我只想将v打印到屏幕上。有什么帮助吗？

Answer 1

您的函数声明和定义不一致，您想从Initialize生成向量，您可以这样做：

void Initialize(vector<int>& v);

打印矢量：

void Print(const vector<int>& v);

现在你打电话：

vector<int> v;
Initialize(v);
Print(v);

请勿忘记更改Initialize，Print的功能定义以匹配我在上面提供的新签名。你也正在重新定义一个阴影函数参数的局部变量v，你只需要注释掉那一行，也可以通过const ref传递向量：

void Print (const vector<int>& v){
  //vector<int> v;
  for (int i=0; i<v.size();i++){
    cout << v[i] << endl;
  }
}

Answer 2

你必须传递const引用并删除无关的向量。

void Print(const std::vector<int>& v){
    for(unsigned i = 0; i< v.size(); ++i) {
        std::cout << v[i] << std::endl;
    }
}

打印出来的另一种方法是使用像这样的迭代器：

void Print(const std::vector<int>& v) {
    std::vector<int>::iterator it;
    for(it = v.begin(); it != v.end(); ++it) {
        std::cout << (*it) << '\n';
    }
}

或者在C ++ 11中你可以这样做：

void Print(const std::vector<int>& v) {
    for(auto& i : v)
        std::cout << i << '\n';
}

我认为你的Initialize()功能不像预期的那样有效。它似乎只是复制然后丢弃它，而不是修改现有矢量的任何值。

如何在向量c ++中打印元素

2 个答案: