文字列に関するコンパイルエラー。

Question

###実現したいこと
ターミナルのPATH環境変数を;が出てきたら改行するといった感じで出力したい。
/usr/bin;/bin;/usr/sbin;/sbin;/usr/local/bin
だったら

(1) /usr/bin
(2) /bin
(3) /usr/sbin
(4) /sbin
(5) /usr/local/bin

###問題点
エラー

kadai35.cpp: In function 'int main()':
kadai35.cpp:30:39: error: cannot convert 'std::__cxx11::basic_string<char>' to '__gnu_cxx::__alloc_traits<std::allocator<char>, char>::value_type' {aka 'char'} in assignment
   30 |         path[i]=disrectry_names.substr(cnt[i],cnt[i+1]); //;    
             ~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
      |                                       |
      |                                       std::__cxx11::basic_string<char>
kadai35.cpp:32:35: error: cannot convert 'std::__cxx11::basic_string<char>' to '__gnu_cxx::__alloc_traits<std::allocator<char>, char>::value_type' {aka 'char'} in assignment
   32 |     path[0]=disrectry_names.substr(0,cnt[i]); //             ~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~
      |                                   |
      |                                   std::__cxx11::basic_string<char>

調べたらcharが関係しているようですがどこでcharが出てきたのかわかりません。
※文字列に関して学習したばかりです。
エラーの治し方を教えてください。
###コード

//PATH環境変数

#define _CRT_SECURE_NO_WARNINGS
#include<iostream>
#include<cstdlib>
#include<string>

int main()
{
    std::string disrectry_names=std::getenv("PATH"); //PATH環境変数を呼び出す
    char separator;
    std::string path;
    int size=disrectry_names.size(); //文字列の長さ
    int i;
    int cnt[10];
    int cnti=0;
    /*if(disrectry_names[0]=='/'){ //WindowsのVisual Studioのターミナルでの対応
        separator=':'; //:は区切り文字
    }else{
        separator=';';
    }*/
    for(i=1;i<=size;i++){
        if(disrectry_names[i]==';'){ //:が出てきたら改行する
            cnt[i]=i; //;が出てきたらその添え字を保存する
            cnti++; //改行の数計測
        }
        path[i]=disrectry_names.substr(cnt[i],cnt[i+1]); //;から次の;までの文字列取得
    }
    path[0]=disrectry_names.substr(0,cnt[i]); //最初だけ別

    for(i=0;i<cnti;i++){
        std::cout<<"("<<i+1<<") "<<path[i]<<"\n";
    }

    return 0;
}

###環境

• Windows10
• Visual Stdio Code

Answer 1

質問者のソースを活かしつつ動くようにしたもの。

• pathは配列にしないと、複数のstringを保持できない。エラーはそれが原因
• substrの2つめの引数は位置ではなく長さなので、切り取る長さを計算しないといけない
• forループ中のiは1文字ずつ先頭から見ていくためのインデックスで、cntやpathの添え字には使えない。代わりにcntiを使う。
• PATH内のディレクトリの個数は10個以上あるかも(試したPCでは18個あった)なので、cntやpathの要素数はそれ以上必要になる

#define _CRT_SECURE_NO_WARNINGS
#include <iostream>
#include <cstdlib>
#include <string>

int main(void)
{
    std::string disrectry_names = std::getenv("PATH");
    int size = disrectry_names.size();
    const int max_count = 1024;
    std::string path[max_count];
    int cnt[max_count];
    
    int cnti = 1;
    cnt[0] = -1;
    for (int i = 1; i <= size; i++) {
        if (disrectry_names[i] == ';') {
            cnt[cnti] = i;
            path[cnti] = disrectry_names.substr(cnt[cnti - 1] + 1, cnt[cnti] - cnt[cnti - 1] - 1);
            cnti++;
        }
        
    }

    for (int i = 1; i < cnti; i++) {
        std::cout << "(" << i << ") " << path[i] << "\n";
    }

    return 0;
}

Comments

[langhtorn]

詳しい解説をありがとうございました。

[Daregada]

これ、PATHの末尾にも「;」が付いてる前提なので、ついてない場合の処理を付け加えないといけない。

Answer 2

#define _CRT_SECURE_NO_WARNINGS
#include <iostream>
#include <sstream>
#include <cstdlib>
#include <string>

int main() {
    std::string disrectry_names = std::getenv("PATH"); //PATH環境変数を呼び出す
    std::istringstream stream(std::getenv("PATH"));
    std::string path;
    int index = 0;
    while ( std::getline(stream, path, ';') ) {
      std::cout << '(' << ++index << ") " << path << std::endl;
    }
}

Comments

[langhtorn]

ありがとうございます。

Answer 3

こんんちは。

path変数は、std::string型ですので、path[i]はchar型への参照となります。
そこへstd::string型の値を代入しようとしてエラーになっているようです。

Comments

[langhtorn]

stringを学習しなおします。ありがとうございます。