12

我的 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 打印到屏幕上。有什么帮助吗?

4

3 回答 3

11

您的函数声明和定义不一致,您想从中生成向量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;
  }
}
于 2013-02-01T04:35:32.253 回答
7

您必须通过 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()功能不像您期望的那样工作。它似乎只是制作一个副本然后丢弃它,而不是修改现有向量的任何值。

于 2013-02-01T04:35:20.590 回答
0

利用 c++ std 的 11 及更高版本,这可以在一行中实现,如下所示。std::for_each(v.begin(),v.end(),[](const int &i) { std::cout << i; });

此代码使用从 begin() 到 end() 的 std::vector 迭代器运行 for 循环,连续地为 lambda/[] 函数提供要打印的所需参数。

于 2021-09-02T22:29:00.913 回答