C# 与えられた文字列が定義されてた列挙型に変換できない場合はnullを返したい

Question

実現したいこと

• 与えられた文字列が定義されてた列挙型に変換できない場合はnullを返したい
• 型パラメータ制約がEnumの場合はnull許容値でもnullが返せないのか確認したい

前提

文字列で与えられたコード（数値）を定義されている列挙型に変換する処理を書いています。
いろんな列挙型に対応する必要があるのでジェネリックメソッドを定義しています。
戻り値がnull許容値型ならdefaultでnullが返るものだと思っていたんですが、列挙型は0で返ってしまいます。
列挙型の値に0を定義しているものがあり、これではまずいのでどうしてもnullで返したいです。
戻り値nullはそもそも書けなくてどうしたらいいんだって感じです。

該当のソースコード

namespace Main;

class Program
{
    public static void Main()
    {
        var fruitsList = new List<Fruits?>();
        for(var i = 0; i < 10; i++)
        {
            fruitsList.Add(GetEnum<Fruits>(i.ToString()));
        }

        var nameList = new List<Name?>();
        for (var i = 0; i < 10; i++)
        {
            nameList.Add(GetEnum<Name>(i.ToString()));
        }

        var fruitsList2 = new List<Fruits?>();
        for (var i = 0; i < 10; i++)
        {
            fruitsList2.Add(GetStruct<Fruits>(i.ToString()));
        }

        var nameList2 = new List<Name?>();
        for (var i = 0; i < 10; i++)
        {
            nameList2.Add(GetStruct<Name>(i.ToString()));
        }

        var dates = new List<DateTime?>();
        for (var i = 0; i < 10; i++)
        {
            dates.Add(GetStruct<DateTime>(i.ToString()));
        }

    }

    enum Fruits
    {
        Apple = 0,
        Banana,
        Orange,
    }

    enum Name
    {
        John = 1,
        Mike,
        Kate,
        Jully
    }

    // 数値の文字列をT型に変換する
    public static T? GetEnum<T>(string? s) where T : Enum
    {
        if(int.TryParse(s, out var value))
        {
            var temp = (T)(object)value;

            if(Enum.IsDefined(typeof(T), temp))
            {
                return temp;
            }

        }

        return default;
    }

    // 数値の文字列をT型に変換する
    public static T? GetStruct<T>(string? s) where T : struct
    {
        Type type = typeof(T);
        if (!type.IsEnum) throw new ArgumentException(nameof(T));

        if (int.TryParse(s, out var value))
        {
            var temp = (T)(object)value;

            if (Enum.IsDefined(typeof(T), temp))
            {
                return temp;
            }

        }

        return default;
    }
}

試したこと

あと型パラメータ制約をstructにしてnullを返す方法を見つけたんですけど、これでもいいのか見てほしいです。

Comments

[KOZ6.0]

public enum Hoge1 : long, public enum Hoge2 : ulong
とか定義できるので、int.TryParse のところは long.TryParse でだめなら ulong.TryParse という感じにするといいと思います。
あと、[Flags] はどう扱うのか気になります。

[KOZ6.0]

>int.TryParse のところは long.TryParse ～
あ！ Enum.TryParse(type, s, out var value) でよかったみたいです。

[KOZ6.0]

string? が使えるということは、.NET 系だと思いますが、バージョンによって回答が変わるかもしれないので、明記してください。

[yukio2022]

>int.TryParse のところは
int.TryParseメソッドは数値に変換できたら何でもいいと思って使ってます。
>[Flags]はどう扱うのか
聞かれている内容がピンと来ていないんですけどビットフィールドのことは管変えていません。
>Enum.TryParse(type, s, out var value) でよかったみたいです。
Enum.TryParseメソッド使ってみたですんけど、文字列からの変換もいけるんですね。ここには書いていないんですけど、受け付けるコードは数値の文字列のみという仕様なのでダメみたいです。

[KOZ6.0]

enum に使えるのは SByte/Byte/Int16/UInt16/UInt32/Int32/UInt64/Int64 です。int 以外だと問題が出る可能性があります。
Flags というのは組み合わせて使うものなので(FontStyle.Regular | FontStyle.Italic) のような値は許可するのか、しないのかってことです。
数値しか許可しないのであれば、事前に数値であるかチェックしておけばいいと思います。
まぁそこまで考える必要がないのならOKです。

[yukio2022]

列挙型をByteで定義して256を引数に与えたらおっしゃる通りエラーが出たので正規表現で数値チェックをした後にEnum.TryParseメソッドにかけるようにしました。
教えていただきありがとうございます。

Answer 1

型パラメータ制約がEnumの場合はnull許容値でもnullが返せないのか確認したい

返せません。制約を where T : struct, Enum にすれば返せるようになります。

質問からは外れますが、
nullを返す実装よりは 例外スローかFruits.Unknownのようにエラーを示す値を追加したほうが良いかもしれません。

Comments

[yukio2022]

上記の方法でnullを返すことができスッキリしました。

これは今実装している検索エンジンの話になるんですけど、リクエストパラメータはすべて文字列型なのでそれを適切なデータ型に変換するんですけど、文字列から適切に変換できない値はパラメータが指定されていないものと同じ扱いにするという仕様なのでそれをnullということにしていて、nullかそうじゃないかでSQLクエリを組み立てています。
nullが良くないというのはNullReferenceExceptionが怖いからですか？
Fruits.Unknownの考えはありませんでしたが、複数のコードをカンマ区切りで指定する（"0,1,2"とか）仕様があってこれを列挙型のリストに変換して、変換したリストが空かどうかでSQLクエリに条件を加える処理にしているので難しいです。
```
if (query.PlanBags != null && query.PlanBags.Any())
{
mainQueryBuilder.AppendLine(@" AND (
1 = 0 ");

foreach (var item in query.PlanBags!)
{
switch (item)
{
case BagType.TwosomeGuarantee:
mainQueryBuilder.AppendLine(" OR gp.IsTwoSomeGuaranteed IN (1, 2) ");
break;
case BagType.NoTwosomeExtra:
mainQueryBuilder.AppendLine(" OR gp.IsTwoSomeExtraCharge = 0");
break;
case BagType.NoThreeBagsExtra:
mainQueryBuilder.AppendLine(" OR gp.IsThreeBagsExtraCharge = 0");
break;
case BagType.LimitedFourBags:
mainQueryBuilder.AppendLine(" OR gp.IsFourBagLimitPlan = 1");
break;
case BagType.ReservableTwosome:
mainQueryBuilder.AppendLine(" OR gp.IsTwoSomeAccept = 1");
break;
}
}

mainQueryBuilder.AppendLine(@" ) ");
}
```