引数で受け取った配列の中身を、ポインタを使って直接書き換えるにはどうしたら良いですか？

Question

void test(vector<int> *a){

	*a[0] = 10;

}

このように書こうとすると、
「このオペランドと一致する演算子”*”はありません」と言われてしまうのですが、
タイトルのようにするにはどうしたら良いのでしょうか？
引数で受け取った配列の中身を直接書き換えたいです。

Answer 1

エラーを取るだけなら

ポインタ外しを先にすれば良いです。

void test(vector<int> *a){
    (*a)[0] = 10;
}

本来なら

ポインタではなく参照を使った方が良い場面かと思います。

void test(vector<int>& a){
    a[0] = 10;
}

Comments

[Hacchi_]

なるほど、参照を使うべきなのですね！

[rtr1950x]

参照渡しはメリット・デメリットがありまして、
・test()の呼び出し元で、&演算子を書かなくて済むメリット。
・test()の呼び出し元で、test()が引数を変更する可能性があるかどうか判断出来ないデメリット。
という点を考慮すべきかと思います。
ちなみに私は、引数を変更する関数は全てポインタ渡しにしています（好みの問題ですが）。

[maisumakun]

「test()の呼び出し元で、test()が引数を変更する可能性があるかどうか判断出来ない」ですが、変更しないものであればconstな参照を取ればいいわけなので、「constでない参照を取る=書き換える可能性がある」と解釈してそう問題があるわけでもないかと思います。

[LouiS0616]

@rtr1950x さん
> 引数を変更する関数は全てポインタ渡しにしています（好みの問題ですが）

私も以前は全く同じ方針でした。
しかし、
・nullptrの存在を常に気にかけなくてはならないこと
・constを厳に付けるルールがあれば、シグネチャを見れば書き換えの有無が判断できること
・ポインタをやり取りする際には、&の有無という明確な目印がなくなってしまうこと
・スマートポインタがあるために、生のポインタを見るとその寿命がどうしても気になってしまうこと

これらの理由から、参照渡しを常に使った方が良いのだと思うようになりました。

---
けっきょく『どちらが正しい』ということは無いわけですが、コメントを頂いたことでより深く問題を考えられたように思います。
コメントありがとうございます。

[rtr1950x]

@maisumakun さん
>「constでない参照を取る=書き換える可能性がある」と解釈してそう問題があるわけでもないかと思います。
それは関数のシグネチャを確認する必要が有りますよね。
例えば「test( intVector );」というコードだけを見て判断できないので、私はポインタで渡す方針でいます。
処理の流れをザッと眺める時に、ポインタ渡しの方が簡単に判断できるので、私はそっちのが好みです。
（正しい・間違っているという議論ではなく、単に私の好みの主張ですが。）

Answer 2

こんにちは。

演算子の優先順位の問題です。
*と[]では[]の方が優先順位が高いので、*(a[0])と解釈されています。
(*a)[0]=10;でいけると思います。