C++の配列の関数定義と呼び出し方について

Question

C++のヘッダーで

c.h

class bun {
void erase(vector<float> pp);
vector<float> num;
}

があったとき、c.cppでは

void bun::erase(vector<float> pp){
//ここの引数にnum[3]とかnum[4]とかの引数を入れたいのですが、その場合のこの関数内でのnum[~]の値はどのように記述するのでしょうか？
//ここの num[]+５ とか書くとエラーになってしまうのですが、正式な書き方を教えてください

}

例えば、num[3] = 5.5;　の場合
erase()をするとその値は-3された値がnum[3]に代入されるような関数の場合の書き方を教えて欲しいです。
erase(num[3])と書くと num[3] は 2.5 になってるというような感じです。

Comments

[LouiS0616]

引数を『入れる』という表現がなんとも漠然としている印象なのですが、eraseの実行前後でどのような変化が生じることを期待しているのでしょうか？

[LouiS0616]

また、C#とCのタグは外してください。

Answer 1

こんにちは。

erase(num[3])

とは書けないですよ。num[3]はfloat型ですが、erase()が受け取るのは、vector<float>型ですので型が異なります。

もしかして、次のような意味でしょうか？

void bun::erase(int index)
{
    num[index] -= index;
}

erase(3)と呼び出すとnum[3]の値が3減ります。

---

しかし、どうも外している気がします。
3が2箇所に使われているので、それがインデックスなのか減算する値なのか読み取れませんでした。

Comments

[santaro]

すいません、num[2]でもnum[3]でもnum[4]でも-3されるという意味でした。
それとerase()が受け取る値はnumに限らなく、num_other[]という配列(他の名前の配列)も受け取ることができる場合、
void bun::erase(~~)この~~の部分はどう書くのでしょうか？

[Chironian]

ごめんなさい。何を言っているのか判らないです。
numはメンバ変数ですから、わざわざメンバ関数へ渡す必要ないですよ？

erase関数は何らか型の動的配列std::vector<T>を受け取り、その要素を全て -3 したいという意味でしょうか？
YESであればお手軽には下記ですが、やはり外しているような予感です。

template<typename T>
void erase(std::vector<T>& oArray)
{
　　for(auto& elem : oArray)
　　{
　　　　elem -= 3;
　　}
}

[Chironian]

すいません。別の名前でしたね。分かりました！！
きっと下記です。

void erase(std::vector<float>& oArray)
{
　　for(auto& elem : oArray)
　　{
　　　　elem -= 3;
　　}
}

[santaro]

説明下手ですいません、、

例えば
num[0] = 4;
num[1] = 7;
num[2] = 2;
num[3] = 9;
num_o[0] = 1;
num_o[1] = 5;
numnum[0] = 3;
numnum[1] = 6;

みたいなときに

erase(num[2]);
とかくと
num[2] == -1 ;

になるし

erase(numnum[1]);
とかくと
numnum[1] == 3

になるみたいなイメージです、、、

[episteme]

void erase(int index) {
num[index] から 3 を引く
}

ではアカンのですか?

[Chironian]

santaroさん。ならば下記ですね。
void erase(float& ioElement)
{
　　ioElement -= 3;
}
しかし、その内容でしたら、bunクラスの非staticなメンバ関数として実装する意味はほぼありません。
staticメンバ関数の方がよいと思います。

class bun
{
　　static void erase(float& ioElement)
　　{
　　　　ioElement -= 3;
　　}
　　vector<float> num;
}

Answer 2

こういうことでしょうか？

#include <iostream>
#include <vector>

#include "header.hpp"

void Bun::subtract_at(int index, int diff) {
    nums_.at(index) -= diff;
}
void Bun::print(void) {
    for(auto num: nums_) {
        std::cout << num << " ";
    }
    std::cout << std::endl;
}

int main(void) {
    Bun bun{1.5, 2.5, 3.5, 4.5};

    bun.print();
    bun.subtract_at(3);
    bun.print();

    return 0;
}

実行結果 Wandbox

1.5 2.5 3.5 4.5 
1.5 2.5 3.5 1.5 

erase(num[3])と書くと num[3] は 2.5 になってるというような感じ

目的の変数を参照渡しする方法も無いことは無いですが、
参照渡しをむやみに使うよりインデックスで指定した方が無難だと思います。

メンバ変数へのアクセスも狭められますし。

追記

どうやらこういうことみたいですね。

void subtract(double& num, double diff=3) {
    num -= diff;
}

int main(void) {
    Bun bun{1.5, 2.5, 3.5, 4.5};
    auto& nums = bun.getNums();

    bun.print();
    subtract(nums[3]);
    bun.print();

    return 0;
}

実行結果 Wandbox

1.5 2.5 3.5 4.5 
1.5 2.5 3.5 1.5 

でもこれ、正直悪手だと思います。
せっかくクラスにvectorを閉じ込めたのに、外部からいくらでも操作出来てしまうので。

この方法を選ぶしかない状況ってあるんでしょうか。ちょっと考えなおした方が良いような。