vectorを仮引数とした関数を作りたい

Question

前提・実現したいこと

vectorのポインタを引数とする関数を作成しています．

発生している問題・エラーメッセージ

引数としているvectorのところで以下のようなエラーが出ています。
visual studio codeで当該部分にマウスを当てると「式には，クラス型が必要です。」と表示されます。
しかしながら，どう対処すればよいかわからずにいます。解決法をご教示頂けますと助かります。

y2:max_min moto$ g++ -c max.cpp
max.cpp:12:60: error: member reference base type 'int *' is not a structure or union
    std::vector<int>::iterator itr_max = std::max_element(v.begin(), v.end());
                                                          ~^~~~~~
max.cpp:12:71: error: member reference base type 'int *' is not a structure or union
    std::vector<int>::iterator itr_max = std::max_element(v.begin(), v.end());
                                                                     ~^~~~
max.cpp:15:39: error: member reference base type 'int *' is not a structure or union
    size_t max_index = std::distance(v.begin(), itr_max);
                                     ~^~~~~~
3 errors generated.

該当のソースコード

#include <iostream>
#include <vector>
#include <algorithm>

using namespace std;

//import vector v
void max(int* v)
{

//max_element::
    std::vector<int>::iterator itr_max = std::max_element(v.begin(), v.end());

    // size_t min_index = std::distance(v.begin(), itr_min);
    size_t max_index = std::distance(v.begin(), itr_max);

    std::cout << "max element id=" << max_index << endl;
    std::cout << "max element (hairetsu):" << v[max_index] << std::endl;
    std::cout << "max element (pointer):" << *itr_max << std::endl;
}

Answer 1

引数としているvectorのところで以下のようなエラーが出ています。
void max(int* v) { ... }

もしかして: int* と vector<int> は同じものと思っていますか?

void max(const std::vector<int>& v) {

    std::vector<int>::const_iterator itr_max = std::max_element(v.begin(), v.end());
    size_t max_index = std::distance(v.begin(), itr_max);

    std::cout << "max element id=" << max_index << endl;
    std::cout << "max element (hairetsu):" << v[max_index] << std::endl;
    std::cout << "max element (pointer):" << *itr_max << std::endl;
}

Answer 2

引数の型をvector<int>にすれば良いと思います。

C++ 値渡し、ポインタ渡し、参照渡しを使い分けよう

追記：
余談ですが関数名max()はstd::maxと被るので避けたほうが良いと思います。
回答が読みやすくなるようにあえてusing namespace std;を使用しています。

#include <bits/stdc++.h>
using namespace std;

// import vector v
void func_max(vector<int>* v) {
  // max_element::
  vector<int>::iterator itr_max = max_element(v->begin(), v->end());

  // size_t min_index = std::distance(v.begin(), itr_min);
  size_t max_index = distance(v->begin(), itr_max);

  cout << "max element id=" << max_index << endl;
  cout << "max element (hairetsu):" << v->at(max_index) << endl;
  cout << "max element (pointer):" << *itr_max << endl;
}

int func(vector<int>* v) {
  int result = 0;
  if (!v->empty()) {
    sort(v->begin(), v->end());
    result = v->back();
  }
  return result;
}

int main() {
  vector<int> v({3, 2, 1});
  cout << "sort:" << func(&v) << endl;
  func_max(&v);
  getchar();
}

// output
sort:3
max element id=2
max element (hairetsu):3
max element (pointer):3

Comments

[LenItsuki]

うまくいきました。どうもありがとうございます。

Answer 3

vectorのポインタを引数とする関数を作成しています．

void max(int* v)

どう見てもintのポインタです。