📎 Скрытая ловушка в енамках

Метод Enum.TryParse кажется идеальным инструментом для безопасного парсинга строк в enum — он не бросает исключения и возвращает bool, сигнализируя об успехе или неудаче операции.

Но у этого метода есть неочевидное поведение, которое может привести к багам.

Представьте ситуацию: пользователь передаёт статус заказа через API, вы парсите его через TryParse, получаете true, уверенно обрабатываете заказ... и внезапно обнаруживаете в базе статус со значением 999, которого в вашем енаме вообще не существует.

Enum.TryParse возвращает true даже для несуществующих значений enum:

public enum OrderType
{
    Cool = 0,
    NotCool = 1
}

// Парсим значение, которого НЕТ в enum
Enum.TryParse("999", out OrderType type);
// ✓ Вернёт TRUE
// ✓ day = (OrderType)999 
// ✗ Но 999 не определён в OrderType!

Console.WriteLine($"Результат: {type}"); // Вывод: 999

TryParse проверяет только возможность конвертации строки в числовой тип, а не валидность значения для конкретного enum.

Решение

Добавьте проверку через Enum.IsDefined:

if (Enum.TryParse("999", out OrderType type) && 
    Enum.IsDefined(typeof(OrderType), type))
{
    // Здесь значение гарантированно валидно
} else {
    // 999 будет правильно отклонено
}

Enum.IsDefined использует рефлексию и может быть медленным в hot path.

Альтернативы:

// Для hot path: кешируем валидные значения
private static readonly HashSet<OrderType> ValidValues = 
    new(Enum.GetValues<OrderType>());

public static bool IsValid(OrderType value) => 
    ValidValues.Contains(value); // Быстрее IsDefined

// Для непрерывных enum: проверка диапазона
public static bool IsValid(OrderType value) => 
    (int)value >= 0 && (int)value <= 1; // Самый быстрый

Microsoft спроектировали это так намеренно, поскольку C# позволяет приводить любое число к типу енамки без ограничений. Это даёт гибкость, но требует от разработчика дополнительной бдительности.

🐸 Библиотека шарписта

#sharp_view