Определить, если два разных PropertyInfo происходят из одного и того же интерфейса? - PullRequest
Новая
       14

Определить, если два разных PropertyInfo происходят из одного и того же интерфейса?

2 голосов
/ 15 марта 2019

Я использовал 

public static bool IsSameAsProperty(PropertyInfo first, PropertyInfo second) =>
     first.DeclaringType == second.DeclaringType && first.Name == second.Name;

Чтобы определить, соответствует ли информация отраженного свойства какому-либо свойству, которое я выбрал из базового класса.

Этот подход начал разваливаться, когда я пытаюсьссылочные свойства, определенные в интерфейсах.

Например, представьте следующий сценарий многоинтерфейсного наследования: 

interface IAnimal : { bool IsHungry { get; } }
interface IDog : IAnimal { }

abstract class Animal : IAnimal { public bool IsHungry { get; set; } }
class Dog : Animal, IDog { }

Если я создаю выражения свойств, все перечисленные ниже допустимы: 

Expression<Func<object, bool>> propertyExpression;
propertyExpression = (IAnimal animal) => animal.IsHungry
propertyExpression = (Animal animal) => animal.IsHungry
propertyExpression = (IDog dog) => dog.IsHungry
propertyExpression = (Dog dog) => dog.IsHungry

Поскольку каждый из этих типов определяет или наследует свойство IsHungry, все эти выражения допустимы.Можно даже утверждать, что все они ссылаются на одно и то же свойство (хотя я могу оценить тонкие различия между объявлением интерфейса и экземпляра).

Моя проблема в том, что я хочу каким-то образом программно обнаружить, что все эти свойства«приходят из» общего интерфейса IAnimal и совместимы.К сожалению, мой тест возвращает false, потому что:

  • IDog.IsHungry имеет DeclaringType == typeof(IAnimal), тогда как
  • Dog.IsHungry имеет DeclaringType == typeof(Animal)

IЯ не могу придумать простой способ сравнения выражений интерфейса и свойств конкретного типа, не прибегая к простому сравнению Name (которое склонно к ложным срабатываниям) - но я не могу придумать ничего, что не связано с перечислениемвсе интерфейсы, унаследованные этими двумя типами и ищущие что-либо с именем этого свойства, которое находится в обоих наборах.

В: Можем ли мы создать функцию, которая возвращает true при сравнении любого из данных PropertyInfo, полученныхиз приведенных выше 4 выражений свойств. (например, определить, все ли они представляют / реализуют одно и то же свойство базового интерфейса?)

Ответы [ 2 ]

0 голосов
/ 15 марта 2019

Решение, которое я придумал, сравнивает геттер / сеттер MethodInfo с InterfaceMapping.Он проходит все тесты, о которых я только мог подумать, но я не уверен на 100%, что не бывает странных угловых случаев, которые это не уловит. 

public static bool IsSameAsProperty(PropertyInfo first, PropertyInfo second)
{
    if (first.DeclaringType == second.DeclaringType && first.Name == second.Name)
        return true;

    bool firstIsSecond = second.DeclaringType.IsAssignableFrom(first.DeclaringType);
    bool secondIsFirst = first.DeclaringType.IsAssignableFrom(second.DeclaringType);

    if (firstIsSecond || secondIsFirst)
    {
        PropertyInfo baseProp = firstIsSecond ? second : first;
        PropertyInfo derivedProp = firstIsSecond ? first : second;

        MethodInfo baseMethod, implMethod;
        if (baseProp.GetMethod != null && derivedProp.GetMethod != null)
        {
            baseMethod = baseProp.GetMethod;
            implMethod = derivedProp.GetMethod;
        }
        else if (baseProp.SetMethod != null && derivedProp.SetMethod != null)
        {
            baseMethod = baseProp.SetMethod;
            implMethod = derivedProp.SetMethod;
        }
        else
        {
            return false;
        }
        // Is it somehow possible to create a situation where both get and set exist
        // and the set method to be an implementation while the get method is not?

        if (baseMethod.DeclaringType.IsInterface)
            return IsInterfaceImplementation(implMethod, baseMethod);
        else
            return IsOverride(implMethod, baseMethod);
    }
    return false;
}

private static bool IsInterfaceImplementation(MethodInfo implMethod, MethodInfo interfaceMethod)
{
    InterfaceMapping interfaceMap = implMethod.DeclaringType.GetInterfaceMap(interfaceMethod.DeclaringType);
    int index = Array.IndexOf(interfaceMap.InterfaceMethods, interfaceMethod);
    // I don't think this can ever be the case?
    if (index == -1)
        return false;
    MethodInfo targetMethod = interfaceMap.TargetMethods[index];
    return implMethod == targetMethod || IsOverride(implMethod, targetMethod);
}

private static bool IsOverride(MethodInfo implMethod, MethodInfo baseMethod)
{
    return implMethod.GetBaseDefinition() == baseMethod.GetBaseDefinition();
}
0 голосов
/ 15 марта 2019

Это, вероятно, приведет к ложным срабатываниям в случае использования ключевого слова new для скрытия унаследованных свойств, но: 

public static bool IsSameAsProperty(PropertyInfo first, PropertyInfo second) =>
    first == second || // If default equals implementation returns true, no doubt
    first.Name == second.Name && (
        first.DeclaringType == second.DeclaringType ||
        first.DeclaringType.IsAssignableFrom(second.DeclaringType) ||
        second.DeclaringType.IsAssignableFrom(first.DeclaringType));

Полагаю, я мог бы быть более конкретным и проверить, если first.DeclaringType.IsInterface и наоборот, но все же можно явно реализовать этот интерфейс и скрыть его свойство с новым с тем же именем, поэтому дополнительная проверка не сделает это более «безопасным».

Неуверен, могу ли я сказать, представляет ли один экземпляр PropertyInfo реализацию интерфейса другого.

...