17

Просматривая исходник AutoMapper, наткнулся на интересную вещь:

Класс Mapper:

public class Mapper : IRuntimeMapper, IMapper
{
//...

Интерфейс IRuntimeMapper:

public interface IRuntimeMapper : IMapper
{
//...

Вопрос

Зачем Mapper реализует IMapper, если IRuntimeMapper уже наследует его? Такая "ошибка" возникла в ходе расширения библиотеки или это нормальная практика?

2
  • 1
    в последней версии такого нет github.com/AutoMapper/AutoMapper/blob/v6.1.1/src/AutoMapper/… . Да и ошибкой не назовешь, скорее обычное дело в ходе рефакторинга, за всем не уследишь.
    – tcpack4
    4 авг 2017 в 20:41
  • @tcpack4, да действительно на github нет. Я это обнаружил через Visual Studio. Там в заголовке также значилось ...\packages\AutoMapper.6.1.1\lib\net45\AutoMapper.dll. Видимо не тот исходник компилировали в dll. 4 авг 2017 в 20:47

3 ответа 3

10

В исходниках автомаппера на гитхабе строчки

public class Mapper : IRuntimeMapper, IMapper

нет и никогда не было, судя по логам.

Декомпиляторы могут показывать всю цепочку наследования интерфейсов интерфейсов по достаточно странной причине - наследование классов и интерфейсов в C# работает по разному.

В случае иерархии классов каждый потомок имеет ровно одного явного родителя:

class A {}

class B : A {} 

class C : B {}

В случае интерфейса срабатывает механизм схлопывания иерархии:

A class or struct that directly implements an interface also directly implements all of the interface's base interfaces implicitly. This is true even if the class or struct doesn't explicitly list all base interfaces in the base class list.

Компилятор ищет все базовые интерфейсы, строит из них полный список и дописывает их в качестве реализуемых. Так что он превращает

interface A { }

interface B : A { }

public class SomeClass : B { }

в

public class SomeClass : B, A { }

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

8

Если вы просматривали декомпилированный исходник, то это нормально. Для сравнения: Исходники BlockingCollection и описание на МСДН. JetBrains decompiler показывает состояние как в справке, несмотря на то, что сорцы явно другие.

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

4

Хотя, как заметил @PashaPash строки

public class Mapper : IRuntimeMapper, IMapper

на GitHub никогда не было. Подобные ситуации далеко не редки. Например, стандартный класс List<T> реализует следующие интерфейсы:

public class List<T> : IList<T>, System.Collections.IList

при этом в документации указан гораздо больший список:

public class List<T> : IList<T>, ICollection<T>, IEnumerable<T>, 
                       IList, ICollection, IEnumerable, ...

причем что важно:

  • IList<T> наследует ICollection<T> и IEnumerable<T>
  • ICollection<T> наследует IEnumerable<T> и IEnumerable
  • IEnumerable<T> наследует IEnumerable

Казалось бы к чему все это, какая цель? Ответ на подобный вопрос Эрик Липперт объясняет так, когда похожий код встречается в:

  • коде другого разработчика, автор преследует цель (на взгляд автора) сделать код проще для понимания и более самодокументированным

  • когда дело касается документации преследуется цель предоставить максимум информации и избавить вас от головной связанной с самостоятельным выяснением цепочки наследования

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

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.