Почему следующая программа возвращает правильное значение типа только в 3-м случае?

Question

[Вадим]

Почему следующая программа возвращает правильное значение (высокого уровня) типа только в 3-м случае и как еще можно вернуть тип объекта в List

using System;
using System.Collections.Generic;
using System.Reflection;

public class Program
{
    public static void Main()
    {
        List<dynamic> idChecksList = new List<dynamic>();
        
        List<ClientViewModelNew> mappedClientsNew = new List<ClientViewModelNew>(); 
        List<ClientViewModelOld> mappedClientsOld = new List<ClientViewModelOld>();
        var obj1 = new ClientViewModelNew();
        var obj2 = new ClientViewModelOld();

        // Console.WriteLine(obj1.EXa);

        mappedClientsNew.Add(obj1);
        mappedClientsOld.Add(obj2);

        bool useNewClientObject = true; 
           
        if (useNewClientObject == true)
        {
            foreach (ClientViewModelNew item in mappedClientsNew)
            {
                idChecksList.Add(item);
            }
                
        }
        else
        {
            foreach (ClientViewModelOld item in mappedClientsOld)
            {
                idChecksList.Add(item);
            }
        }

        String typeArg0 = idChecksList.GetType().GetGenericArguments()[0].ToString(); //  GenericTypeArguments[0].ToString();
        String typeItem1 = idChecksList.GetInternalType().ToString();
        String typeItem2 = idChecksList[0].GetType().ToString();

        Console.WriteLine("internal type of List, Method 1:" + typeArg0);
        Console.WriteLine("internal type of List, Method 2:" + typeItem1);
        Console.WriteLine("internal type of List, Method 3:" + typeItem2);
    }
}

public static class GenericListExtensions
{
    public static Type GetInternalType<T>(this List<T> list)
    {
        return typeof(T);
    }
}

public class ClientViewModelNew
{
    public String EXa { get; set; } = "Initial Name for New Class";
}

public class ClientViewModelOld
{
    public String EXa { get; set; } = "Initial Name for Old Class";
}

возвращает

internal type of List, Method 1:System.Object
internal type of List, Method 2:System.Object
internal type of List, Method 3:ClientViewModelNew

Почему первые два возвращают System.Object и как это можно исправить?

Answer 1

[Василий Банников]

почему первые два возвращают System.Object и как это можно исправить?

Потому-что dynamic под капотом - это на самом деле Object и куча рефлексии.
List<dynamic> - это на самом деле List<object> (можешь загнать в sharplab и убедиться в этом)

Исправить это можешь только отказавшись от dynamic (а лучше вообще забыть о его существовании. Крайне сломанная фича, которая путём говнокода позволяет в некоторых случаях сэкономить строчки)

Например, если нельзя трогать ClientViewModelNew и ClientViewModelOld, то можно попробовать так:

public interface IClientViewModel {
  public string EXa { get; set; }
}

public class ClientViewModelNewWrapper: IClientViewModel {
  private readonly ClientViewModelNew _internal;
  public ClientViewModelNewWrapper(ClientViewModelNew obj) => _internal = obj;
  public string EXa { get => _internal.EXa; set => _internal.EXa = value;}
}

public class ClientViewModelOldWrapper: IClientViewModel {
  private readonly ClientViewModelOld _internal;
  public ClientViewModelNewWrapper(ClientViewModelOld obj) => _internal = obj;
  public string EXa { get => _internal.EXa; set => _internal.EXa = value;}
}

Comments

[Вадим]

спасибо вам большое, да эти классы ClientViewModelNew и ClientViewModelOld трогать нельзя, но не совсем понимаю что теперь надо изменить в нижеприведенном коде, у нас же idChecksList должен забирать список обьектов одного или другого типа, а если кормить его ClientViewModelOldWrapper или ClientViewModelNewWrapper, то что изменится? Не хочется менять idChecksList, так как он передается в качестве параметра другому классу. Или теперь idChecksList надо обьявлять от интерфейса? Если так, то как передавать его другому классу, чтобы было без изменений и как получить внутренний класс List?

List<dynamic> idChecksList = new List<dynamic>();
        
        List<ClientViewModelNew> mappedClientsNew = new List<ClientViewModelNew>(); 
        List<ClientViewModelOld> mappedClientsOld = new List<ClientViewModelOld>();
        var obj1 = new ClientViewModelNew();
        var obj2 = new ClientViewModelOld();

        // Console.WriteLine(obj1.EXa);

        mappedClientsNew.Add(obj1);
        mappedClientsOld.Add(obj2);

        bool useNewClientObject = true; 
           
        if (useNewClientObject == true)
        {
            foreach (ClientViewModelNew item in mappedClientsNew)
            {
                idChecksList.Add(item);
            }
                
        }
        else
        {
            foreach (ClientViewModelOld item in mappedClientsOld)
            {
                idChecksList.Add(item);
            }
        }

с уважением, Вадим

[Василий Банников]

Вадим,

если кормить его ClientViewModelOldWrapper или ClientViewModelNewWrapper, то что изменится?

Запихивай туда интерфейс IClientViewModel.
И убери динамик

[Вадим]

ругаются они... извиняюсь, туплю и не нагуглил ничего (C# assign list of objects to lis of interfaces)

List<IClientViewModel> idChecksList = new List<IClientViewModel>();
            
            List<ClientViewModelNew> mappedClientsNew = new List<ClientViewModelNew>(); 
            List<ClientViewModelOld> mappedClientsOld = new List<ClientViewModelOld>();
            var obj1 = new ClientViewModelNew();
            var obj2 = new ClientViewModelOld();

            // Console.WriteLine(obj1.EXa);

            mappedClientsNew.Add(obj1);
            mappedClientsOld.Add(obj2);

            bool useNewClientObject = true; 
               
            if (useNewClientObject == true)
            {
                foreach (ClientViewModelNewWrapper item in mappedClientsNew) // Suspicious cast: there is no type in the solution
                    // which is inherited from both 'ConsoleApp1.ClientViewModelNew' and 'ConsoleApp1.IClientViewModel'
                
                {
                    idChecksList.Add(item);
                }
                    
            }
            else
            {
                foreach (ClientViewModelOldWrapper item in mappedClientsOld) // Cannot convert element type
                    // 'ConsoleApp1.ClientViewModelOld' to iterator type 'ConsoleApp1.ClientViewModelOldWrapper'
                {
                    idChecksList.Add(item);
                }
            }

[Василий Банников]

Вадим,

foreach (var item in mappedClientsNew)
{
  idChecksList.Add(new ClientViewModelNewWrapper(item));
}
// ...
foreach(var items in mappedClientsOld)
{
  idChecksList.Add(new ClientViewModelOldWrapper(item));
}

[Вадим]

спасибо, теперь правда возвращается интерфейс, но хотя бы не систем обьект как ранее )) Но наверное это фундаментальный момент и избавиться от него невозможно

internal type of List, Method 1:IClientViewModel
internal type of List, Method 2:IClientViewModel
internal type of List, Method 3:ClientViewModelNew

[Василий Банников]

Вадим, да, конечно. А зачем вам вообще знать конкретный объект?

[Вадим]

в зависимости от того, какой обьект находится в Лист, вызываются два разных метода для его обработки другим классом... я думал в начале сделать

if (Startup.EnvVar == "true")
            {
                List<ClientViewModelNew> idChecksList = new List<ClientViewModelNew>();               
            }
            else
            {
               List<ClientViewModelOld> idChecksList = new List<ClientViewModelOld>();  
            }

но из за "дурацких"... странных особенностях языка C# он не видит переменных, обьявленных внутри conditionals

[Василий Банников]

в зависимости от того, какой обьект находится в Лист, вызываются два разных метода для его обработки другим классом... я думал в начале сделать

Это можно через полиморфизм разрулить.

но из за "дурацких"... странных особенностях языка C# он не видит переменных, обьявленных внутри conditionals

А что в этом странного? Область видимости переменной ограничивается фигурными скобками. И двух переменных с одним именем быть никак не может

[Вадим]

Василий Банников, Область видимости переменной ограничивается фигурными скобками. - а если тернарный оператор использовать?

[Василий Банников]

Вадим, тогда будет ошибка из-за несовпадения типов)