Проход по классу модели внутри контроллера

Question

Эта публикация относится к Вложенным блокам Try / Catch

Вот моя модель:

 public class Benefits
    {
        public int Id { get; set; }
        public Guid? ResponseId { get; set; }
        public decimal? MedicalTotal { get; set; }
        public decimal? StdSicknessAccident { get; set; }
        public decimal? LtdWage { get; set; }
        public decimal? MedicalPremiums { get; set; }
        public decimal? DentalPremiums { get; set; }
        public decimal? VisionCare { get; set; }
        public decimal? RetireePremiums { get; set; }
        public decimal? LifeInsurance { get; set; }
        public decimal? Prescription { get; set; }
        public decimal? MedAdmin { get; set; }
        public decimal? MedOther { get; set; }
        public decimal? HsaFsa { get; set; }
        public decimal? PtoTotal { get; set; }
        public decimal? Holidays { get; set; }
        public decimal? Pto { get; set; }
        public decimal? Vacations { get; set; }
        public decimal? SickLeave { get; set; }
        public decimal? PtoOther { get; set; }
        public decimal? RetirementTotal { get; set; }
        public decimal? X401k {get; set; }
        public decimal? DefinedBenefit { get; set; }
        public decimal? CashBalance { get; set; }
        public decimal? RetirementAdmin { get; set; }
        public decimal? RetirementOther { get; set; }
        public decimal? MiscTotal { get; set; }
        public decimal? Severance { get; set; }
        public decimal? Dependent { get; set; }
        public decimal? Tuition { get; set; }
        public decimal? Relocation { get; set; }
        public decimal? Total { get; set; }
    }

Пользователи загружают файл Excel со строками в том же порядке (они не загружают свои собственные Id или ResponseId. Я начал писать операторы if для обнаружения ошибок как таковых:

                    bool success = Decimal.TryParse(table[0], out decimal MedicalTotal);
                    if (success)
                    {
                        b.MedicalTotal = MedicalTotal;
                    }
                    else
                    {
                        model.ErrorList.Add("Medical total cell should be formatted as Currency, Accounting, or Number.");
                    }


                    bool success1 = Decimal.TryParse(table[1], out decimal StdSicknessAccident);
                    if (success1)
                    {
                        b.StdSicknessAccident = StdSicknessAccident;
                    }
                    else
                    {
                        model.ErrorList.Add("STD, Sickness, and Accident Insurance cell should be formatted as Currency, Accounting, or Number.");
                    }



                    bool success2 = Decimal.TryParse(table[2], out decimal LtdWage);
                    if (success2)
                    {
                        b.LtdWage = LtdWage;
                    }
                    else
                    {
                        model.ErrorList.Add("LTD & Wage Insurance cell should be formatted as Currency, Accounting, or Number.");
                    }

                    bool success3 = Decimal.TryParse(table[3], out decimal MedicalPremiums);
                    if (success3)
                    {
                        b.MedicalPremiums = MedicalPremiums;
                    }
                    else
                    {
                        model.ErrorList.Add("Medical Premiums cell should be formatted as Currency, Accounting, or Number.");
                    }

Это достаточно простой процесс, но я чувствую, что, учитывая его повторяющуюся природу, мог бы быть какой-то способ перебрать поля моей модели и выполнить ту же задачу. Я бы согласился с добавлением [Display(Name="Medical Total")] или некоторых других метаданных к каждому полю модели. Я также рассмотрел вопрос о добавлении в мой контекст новой таблицы, в которой хранятся все сообщения об ошибках (это лучший метод?).

Так есть ли альтернатива моему решению, которая была бы значительно короче? Что-то вроде:

List<bool> success = new List<bool>();
for(var i = 1; i<Benefits.FieldCount; i++)
{
    success[i] = decimal.TryParse(Table[i], out decimal Benefits.Field[i]
};
if(success[i])
{
    b.Field[i] = Benefits.Field[i];
}
else
{
    model.ErrorList.Add(Benefits.Field[i].Name "must be formatted as Accounting, Currency, or Number.");
}

Answer 1

Я бы отказался от всей сложности и просто использовал пользовательские Атрибуты проверки .

public class Foo : ValidationAttribute
{
    private readonly string _name;

    public Foo(string name)
    {
        _name = name;
    }

    protected override ValidationResult IsValid(object value, ValidationContext context)
    {
        // if the value is null, don't go further
        if (value == null) return ValidationResult.Success;

        // if the value cannot be parsed as a decimal (not valid)
        if (!Decimal.TryParse(value, out decimal d))
        {
            // return an error message
            return new ValidationResult($"{_name} cell should be formatted as...");
        }

        // if the parsed decimal is negative
        if (d < 0)
        {
            // return an error message
            return new ValidationResult($"{_name} cell cannot be negative.");
        }

        // if we got this far it was a success
        return ValidationResult.Success;
    }        
}

затем просто украсьте ваши свойства атрибутом проверки

[Foo("STD, Sickness, and Accident Insurance")]
decimal? StdSicknessAccident { get; set; }

[Foo("LTD & Wage Insurance")]
decimal? LtdWage { get; set; }

Answer 2

Вы можете использовать отражение для этого, вот пример:

    Type type = typeof(Benefits);
    PropertyInfo[] properties = type.GetProperties();
    foreach (PropertyInfo property in properties)
    {
         // your logic here
        Console.WriteLine("{0} = {1}", property.Name, property.GetValue(benefitsInstance, null));
    }

    Console.Read();