引数をどのようにとればいいか

Question

---read_f関数を呼び出すとき、引数はどのようなものをあたえればいいのか？
教えてほしいです
read_f(この中に、引数をいれるべきなのですが)どのようにいれていけばいいのかわからないです
例えば
void sum(int x,int y){
}
int main(){
sum(4,3)
}
こんな感じにsumの引数は簡単に書くことはできるのですが、今のところ上記のやつは難しくてわからないです

#include <iostream>
#include <fstream>
#include <sstream>
#include <vector>
#include <string>

using namespace std;

int i(0),x;


void read_f(const string uho,vector<string> &uhokun){

    ifstream ifs(uho);

    if(!ifs){
        cout << uho << "file no exist" << endl;
        return ;
    }
    string str;
    while(getline(ifs,str,'\n')){
        uhokun.push_back(str);
    i++;
    }
    ifs.close();
}
int main(){
    string x;
    vector<string> &y;
    cin >> x;
    read_f(x,&y);
    for(int j(0);i != j;j++){
        cout << uhokun.bigin() + j << endl; 
    }
}

Comments

[KoichiSugiyama]

ソースコード自体には問題ないように見受けられます。実行した結果予想と違う動きをしてしまうので、原因を知りたい、とか、確認のためにソースに実際の値を埋め込みたいが、どう記述してよいのか判らない、など疑問点をもっと具体的に書いてもらった方がアドバイスしやすいです。

[退会済みユーザー]

アドバイスありがとうございます

Answer 1

こんにちは。

ざっと見て、一箇所問題に気が付きました。（他にもあるかもです。）

read_f(x,&y);

この呼出では y変数（vector<string>型変数）へのポインタを渡しています。
受け取っているのはvector<string>型への参照ですから、そのポインタを渡すことはできません。
y変数をそのまま書けば自動的に参照へ変換されますので、下記で良いはずです。

read_f(x,y);

Comments

[退会済みユーザー]

おかげさまで、今回の問題は解決しましたが別の問題としてまた出てきた部分があるので、また質問を挙げるつもりですので見てもらえたら幸いです。
ありがとうございました。

Answer 2

意味がわからない。
そのまま読めばいいんじゃないの?
何がわからんのか不明瞭だから回答できない...

---

[追記1]

もしかして「どういうデータを渡すべきか、わからん」ってことでしょうか?

定義を見る限り、
vectorのほうは constがついていないので戻り値として使う可能性がある。(コード的にもそうだし。)

だから今回の場合は、

第一引数: 文字列( 直接でもいいかもしれんが。 )
第二引数: 戻り値扱いの引数で、vector<string> のオブジェクトを渡す

でしょ。

Comments

[退会済みユーザー]

わかりやすく追記してくださって助かりました。
ありがとうございました。

Answer 3

lvalue referenceとはなにか勉強しましょう。

そうすることで、他の方の回答がようやく理解できると思います。

さしあたり
C++マニアック,リファレンス＝参照の使い方,how to use reference,C++入門,C++言語講座
とか。