Неразумно ли использовать анонимный объект + динамический?

Question

У меня есть класс, где одно из свойств возвращает List<object>. Внутри этого списка я помещаю набор анонимных объектов.

После этого у меня есть цикл, использующий этот объект как динамическую переменную.
Так что мой код выглядит следующим образом:

private List<object> BookerTypes 
{ 
    get 
    { 
     if (this.bookerTypes == null) 
     { 
      this.bookerTypes = new List<object>(); 

      var com = new SqlConnection(functions.ConnectionString).CreateCommand(); 
      com.CommandText = @" 
SELECT 
    BT.id 
    , BT.name 
FROM dbo.BookerTypes AS BT 
ORDER BY BT.name ASC 
"; 
      com.Connection.Open(); 
      try 
      { 
       using (var dr = com.ExecuteReader()) 
       { 
        while (dr.Read()) 
        { 
         this.bookerTypes.Add(new { id = dr.GetInt32(0), name = dr.GetString(1) }); 
        } 
       } 
      } 
      finally 
      { 
       com.Connection.Close(); 
      } 
     } 

     return this.bookerTypes; 
    } 
} 

[...]

this.cblSBT.Items.Clear(); 
    foreach(dynamic bt in this.BookerTypes) 
    { 
     this.cblSBT.Items.Add(new ListItem() 
     { 
      Value = bt.id.ToString() 
      , Text = bt.name 
      , Selected = this.competition.SubscriptionTypes.Contains((int)bt.id) 
     }); 
    } 

Помимо очевидного потерянное сильно типизированных типа, есть ли причина, почему я не должен этого делать?

Answer 1

Основная причина не в том, что, как вы сказали, вы потеряли статическую типизацию. Существуют также затраты на производительность, связанные с этим, но они менее важны, чем проблема, которую этот код имеет с точки зрения удобочитаемости и ремонтопригодности.

Если окажется, что вы ошибаетесь или ошибочно используете имя переменной, вы не можете проверять время компиляции (и это проще сделать без поддержки завершения кода). У вас также нет эффективных средств знать, во время компиляции, какие переменные могут существовать в List<object>, которые вам даны. Становится нетривиальной задачей отслеживать источник этого списка, чтобы выяснить, какие переменные могут там использоваться.

Почти наверняка стоит потратить время и усилия на создание нового именованного типа вместо использования анонимного типа, когда вы находитесь в этой ситуации. Небольшая начальная стоимость создания нового класса практически всегда окупается.

Answer 2

В верхней части типа безопасности потери & Другие проблемы, о которых уже было указано, я чувствую, что использование dynamic здесь просто неправильно.

Общий прецедент для dynamic предназначен для использования данных из внешних источников, например. API/COM и т. Д., В основном сценарии, где тип информации еще не определен. В вашем сценарии у вас есть контроль над данными, которые вы запрашиваете, и вы знаете, что типа данных, чтобы ожидать поэтому я не могу на самом деле оправдать, почему вы хотели бы использовать его над преимуществами, полученными при наличии четко определенного типа безопасная модель.

Неразумно ли использовать анонимный объект + динамический?

В вашем случае я бы сказал, что да.