スコープに関すること

Question

自作関数でファイル入力を行いmain関数で実際に入力されたものを出力していこうと思ったのですが、スコープの関係上main関数で出力できず困る羽目になってしまいました。アドバイスしてていただけると幸いです。

.\uho16.cpp:33:17: error: 'uhokun' was not declared in this scope
         cout << uhokun.bigin() + j << endl;

#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; 
    }
}

Answer 1

いや、cout << &y + j << endl; はだめです、なぜなら読み込んだファイルが3行以上のとき、範囲の直後の要素のさらに次の要素にポインタを向けているため、未定義動作になるからです(有効でない領域へのポインタは配列の最終要素のすぐ次へを除いて存在するだけで未定義動作)。

というかポインタを表示するとかやっていることが謎。

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

std::vector<std::string> read_f(const std::string& uho){
    std::ifstream ifs(uho);
    std::vector<std::string> re;
    if(!ifs){
        throw std::runtime_error("file no exist");
    }
    for(std::string str; std::getline(ifs,str,'\n');){
        re.emplace_back(std::move(str));
    }
    return re;
}
int main(){
    std::string x;
    std::cin >> x;
    auto y = read_f(x);
    for(auto&& y_e : y){
        std::cout << y_e << std::endl; 
    }
}

https://wandbox.org/permlink/WK203Y0JiPo5cUtK

やるならこうでは？

Comments

[yumetodo]

もっというとstd::getlineの第三引数の区切り文字はデフォルトで'\n'なので省略していいな

[退会済みユーザー]

そうなんですね、了解です

Answer 2

こんな感じにしたらコンパイル通りました！

#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 << &y + j << endl; 
    }
}

Comments

[yumetodo]

未定義動作です