web-dev-qa-db-ja.com

ベクトルc ++で要素を印刷する方法

このベクターを印刷する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を出力したいだけです。何か助けは？

c++arraysvector
9
TheNameHobbs

関数の宣言と定義に一貫性がなく、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;
  }
}
9
billz

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()関数が期待通りに機能するとは思いません。既存のベクトルの値を変更せずに、単にコピーを作成してからそれを破棄するようです。

7
Rapptz