-2

C++のstd::mapのインスタンス(下記例では::table)の初期化コードをもう少し綺麗にしたいと思っています。
お知恵をお貸しください。
※C++17でお願いします

詳細

magic_enumというenumを便利に扱えるOSSを使用して、std::mapの静的インスタンス(下記例では::table)をグローバル関数(下記例では::trash::initialize)から汚らしく初期化しています。コンストラクタや初期化リストを使って綺麗に書きたいのですが、どうにも難しいので、書き方を教えてください。

型やデータは増えても構いませんし、同じ使い方が出来ればstd::mapでなくても構いません(でも出来れば依存は増やしたくないです)。

追記
(他でも使えるように)::tableの要素のfirstメンバの値を生成するロジックは今後変更される可能性があり、そのときでも有用となる書き方でお願いします。

コード

#include <iostream>
#include <map>
#include <sstream>
#include <iomanip>
#include "magic_enum.hpp" // https://github.com/Neargye/magic_enum/releases/download/v0.8.1/magic_enum.hpp
using namespace std;
enum TCPSTATUS {
    UNKNOWN, ESTABLISHED, SYN_SENT, SYN_RECV, FIN_WAIT1, FIN_WAIT2, TIME_WAIT, CLOSE, CLOSE_WAIT, LAST_ACK, LISTEN, CLOSING,
};
map<string, TCPSTATUS> table;
namespace trash {
    void* initialize() {
        magic_enum::enum_for_each<TCPSTATUS>([&] (auto val) {
            constexpr TCPSTATUS status = val;
            stringstream ss;
            ss << hex << uppercase << setw(2) << setfill('0') << magic_enum::enum_integer(status);
            table[ss.str()] = status;
        });
        return nullptr;
    }
    void* dummy = initialize();
}
int main(int argc, char* argv[]) {
    for (const auto& p: table) {
        cout << p.first << "->" << magic_enum::enum_name(p.second) << endl;
    }
    return 0;
}

出力(参考)

00->UNKNOWN
01->ESTABLISHED
02->SYN_SENT
03->SYN_RECV
04->FIN_WAIT1
05->FIN_WAIT2
06->TIME_WAIT
07->CLOSE
08->CLOSE_WAIT
09->LAST_ACK
0A->LISTEN
0B->CLOSING
15
  • 4
    綺麗は主観が入るのでオイラにとってのきれいとあなたにとってのきれいは全く違う可能性があるです。客観的表現ができると回答できる人が増えそう。
    – 774RR
    Commented 2022年10月24日 2:29
  • あなたの綺麗と私の綺麗は違います。「コンストラクタや初期化リストを使って綺麗に書きたい」とは書いているので、その範囲であなたが綺麗だと思うコードを書いて頂ければOKです。
    – dameo
    Commented 2022年10月24日 2:39
  • enumの整数値の16進数2桁の文字列からenumの整数値への対応ができればよいのでしょうか?
    – int32_t
    Commented 2022年10月24日 2:52
  • 3
    質問のクローズはあくまで "一時保留" の状態であり、(基本的には) モデレーターの独断ではなくこのサイトの参加者による投票で判断されます。 / dameo さん自身が 「あなたの綺麗と私の綺麗は違う」と書いている通り、"綺麗な" という修飾語によって「主観的な」答えしか集まらないと判断されている可能性があります。もしコンストラクタ等の書き方が分からないという事であれば、そちらを目立つような文書に見直してもらった方が意図が伝わりやすいと思います。 / クローズされたからといって (ほぼ同じまま) 別の質問をしても、懸念点が改善されないままだと同じようにクローズされてしまう可能性があります。まずは既存の質問を編集することを優先してみてください。
    – cubick
    Commented 2022年10月25日 12:07
  • 2
    ja.stackoverflow.com/help/dont-askja.stackoverflow.com/help/closed-questions です。stackoverflow.blog/2010/09/29/good-subjective-bad-subjectivestackoverflow.blog/2011/01/17/real-questions-have-answers もこの問題についてよく参考にされる記事になります。
    – nekketsuuu
    Commented 2022年10月25日 14:07

1 件の回答 1

5

C++17の範囲でとなるとインライン変数を使って

static inline auto const table = [] {
    std::map<std::string, TCPSTATUS> table;
    for (auto e : magic_enum::enum_values<TCPSTATUS>()) {
        std::stringstream ss;
        ss << std::hex << std::uppercase << std::setw(2) << std::setfill('0') << e;
        table[ss.str()] = e;
    }
    return table;
}();

これぐらいですかね。C++20のstd::format()を使用できれば

static inline auto const table = [] {
    std::map<std::string, TCPSTATUS> table;
    for (auto e : magic_enum::enum_values<TCPSTATUS>())
        table[std::format("{:02X}", int(e))] = e;
    return table;
}();

もっと簡単に済ませられます。

3
  • gcc 10.3.0でビルドしたところ、stringstreamのstr()が失敗するようです。<<演算子の戻り値の型がstringstreamではないからなのかもしれません。 もし、私の勘違いでなければstringstreamのインスタンスを宣言しても問題ないと思うので、修正お願いします。 あと今回の要件でインライン変数は別に使用しなくても大丈夫です(したほうがいいですが)。
    – dameo
    Commented 2022年10月24日 12:49
  • 修正確認しました。 C++17でもオプション指定して使ってるときがあるくらいなのでC++20はまだ…。 個人的にはmapインスタンスを別に作らないと構築できないのが少し気になりますが、実装が簡潔=綺麗というところなのでしょうね。
    – dameo
    Commented 2022年10月24日 13:24
  • クローズされた質問のコメント欄で新しく議論が生まれていたため、この部分をチャットに移動しました。新しい疑問点がある場合は別途新しい質問としてご投稿ください。コメント欄は長い議論をするための場所ではないためです。 chat.stackexchange.com/rooms/140888/…
    – nekketsuuu
    Commented 2022年11月27日 13:33

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。