C++ mainで関数を呼び出してリストと比較する方法

Question

ソースコード2を参考にして、1のソースコードを書き直したいと思っています。
アドバイスなど、ご教授いただければ幸いです。

参考にしたいソースコードは抜粋してきたため、分かりにくいかもしれませんがよろしくお願いします。

概要としては、コマンドラインで入力された引数引数とフォーマットとしてリストに入っている文字を比較して出力するソースコードです。

やりたい事：mainでCheck関数を呼び出せるようにしたいです。

/* 1,コマンドラインで入力された引数とリストの文字を比較するコード */
#include <iostream>
#include <string>
#include <sstream>
#include <unordered_set>

using namespace std;

int main(int argc, char* argv[])
{
    /* 引数なしの場合、エラー表示 */
    if (argc < 2)
    {
        cout << "error : 引数がありません";
        return 1;
    }
    /* 使用可能な文字タイプ一覧 */
	/* 追加が発生した場合に追加する */
    unordered_set<string> chara_types =
    {
        "A",
        "B",
        "C"
    };
    /* 引数を読み込む */
    istringstream iss(argv[1]);
    for (string type; getline(iss, type, ',');)
    {
        if (chara_types.erase(type) != 1)
        {
            /* 文字タイプ一覧に存在しない場合 */
            cout << "error: 入力間違いです";
            return 2;
        }
    }
        cout << "入力された文字 : " << argv[1];
    return 0;
}

/* 2,参考にしたいコード　Check関数 */
/* check_listはコマンドラインで入力された文字列です */
bool Check( std::string input, const char delimiter, std::vector<std::string>& chek_list )
{
    std::istringstream stream(input);
    std::string field, result;

    while (std::getline(stream, field, delimiter)) 
    {
        auto result_find = std::find( check_list.begin(), chek_list.end(), field);

        /* 引数に重複がある */
        if (result_find != check_list.end())
        {
            printf("--check error. duplicate check-name=[%s]", field.c_str());
            return false;
        }
        // チェックリストに追加
        check_list.push_back(field);
    }
	return true;
}

Answer 1

アドバイスなど、ご教授いただければ幸いです。

僕ならこうする:

#include <iostream>
#include <string>
#include <sstream>
#include <set>
#include <algorithm>

bool Check(const std::string& input, const char delimiter, std::set<std::string>& check_list) {
    std::istringstream stream(input);
    std::string field, result;
    std::set<std::string> input_list;

    // delimiterを区切り文字として分割し、input_listに格納する
    // 重複があれば失敗
    while (std::getline(stream, field, delimiter) ) {
      if ( !input_list.insert(field).second ) return false; 
    }

    // input_listが空なら失敗
    if ( input_list.empty() ) return false;

    // check_listがinput_listを包含していれば成功
    return std::includes(check_list.begin(), check_list.end(),
                         input_list.begin(), input_list.end());
}

int main(int argc, char* argv[]) {
    using namespace std;

    /* 引数なしの場合、エラー表示 */
    if (argc < 2) {
        cout << "error : 引数がありません";
        return 1;
    }

    /* 使用可能な文字タイプ一覧 */
    /* 追加が発生した場合に追加する */
    set<string> chara_types = {
        "A",
        "B",
        "C"
      };

    /* 引数を読み込む */
    if ( !Check(argv[1], ',', chara_types) ) {
        cout << "error: 入力間違いです";
        return 2;
    }
    cout << "入力された文字 : " << argv[1];

    // おまけ: いくつかのテスト・ケース
    for ( string arg : { "A,B,C", "A,B", "C,A", "C,B", "", ",,", "A,B,C,A", "A,C,D" }) {
      cout << "arg: [" << arg << "] result:" 
           << boolalpha << Check(arg, ',', chara_types) << endl;
    }
    return 0;
}

Comments

[alice331]

ご回答ありがとうございます。
std::istringstream stream(input);
std::string field, result;
std::set<std::string> input_list;
上記の3つの意味を教えていただけますでしょうか？
調べてのですが、よく理解できませんでした。
よろしくお願いいたします。

[episteme]

それぞれ、なにがわからんですか？

[alice331]

std::istringstream stream(input);
==>コマンドラインから入力される引数という認識で合っていますでしょうか？
std::string field, result;
==>field,result が指している内容がわかりません。
std::set<std::string> input_list;
==>check_listは引数の文字で input_listは何を指していますでしょうか？
よろしくお願いいたします。

[episteme]

> std::istringstream stream(input);
> ==>コマンドラインから入力される引数という認識で合っていますでしょうか？
コマンドラインから得られた文字列:input から作られたストリーム

> ==>field,result が指している内容がわかりません。
field : コマンドラインから(delimiterで区切って)切り出された文字列
result : ごめん使ってないわ

> input_listは何を指していますでしょうか？
コマンドラインから(delimiterで区切って)切り出された文字列の集合から重複を取り除いたもの

[alice331]

回答ありがとうございました。理解できました！
もう一つよろしいでしょうか？
if ( !input_list.insert(field).second ) return false;
この一文で.secondは何を指しているのでしょうか？

[episteme]

重複のため挿入に失敗したらfalseになるbool値。