'Более свободный' набор текста в C# путем отбрасывания дерева наследования

Вам действительно не следует обращаться к каким-либо производным классам из базового класса, поскольку это в значительной степени нарушает идею ООП. Читаемость, безусловно, имеет большое значение, но я бы не променял ее на возможность повторного использования. Рассмотрим случай, когда вам нужно добавить еще один подкласс - вам также потребуется соответствующим образом обновить базовый класс.

Если длина файла - это то, что вы часто извлекаете, почему бы не реализовать свойство в классе BDictionary (?) ... чтобы ваш код стал:

BDictionary torrent = BItem.DecodeFile("my.torrent");
int filelength = torrent.FileLength;

Таким образом, детали реализации скрыты от пользователя.

Думаю, я бы сделал виртуальные аксессоры this [int] и this [string] и переопределил их в BList / BDictionary. Классы, в которых методы доступа не имеют смысла, должны приводить NotSupportedException () (возможно, имея реализацию по умолчанию в BItem).

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

 (BInteger)torrent["info"][0]["files"]["length"];

по ошибке.

Вы не задумывались о том, чтобы разобрать простой «путь», чтобы можно было написать его так:

BDictionary torrent = BItem.DecodeFile("my.torrent");
int filelength = (int)torrent.Fetch("info.files.0.length");

Возможно не лучший способ, но читаемость увеличивается (немного)

• Если у вас есть полный контроль над своей кодовой базой и своим мыслительным процессом, обязательно сделайте это.
• Если нет, вы пожалеете об этом в тот день, когда какой-то новый человек внедрит производную BItem, которую вы не заметили в ваш BList или BDictionary.

Если вам нужно это сделать, по крайней мере, оберните его (управляйте доступом к списку) в класс, который имеет строго типизированные сигнатуры методов.

BString GetString(BInteger);
SetString(BInteger, BString);

Принимайте и возвращайте BStrings, даже если вы храните их внутри BList BItems. (позвольте мне разделиться, прежде чем я сделаю свои 2 B или нет 2 B)

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

На мой взгляд, не все BItems являются коллекциями, поэтому не все BItems имеют индексаторы, поэтому индексатор не должен быть в BItem. Я бы унаследовал другой абстрактный класс от BItem, назовем его BCollection и поместим туда индексаторы, примерно так:

abstract class BCollection : BItem {

      public BItem this[int index] {get;}
      public BItem this[string index] {get;}
}

и сделайте BList и BDictionary наследованными от BCollection. Или вы можете приложить дополнительные усилия и сделать BCollection универсальным классом.

Если вы введете дженерики, вы сможете избежать приведения типов.

class DecodedTorrent : BDictionary<BDictionary<BList<BDictionary<BInteger>>>>
{
}

DecodedTorrent torrent = BItem.DecodeFile("mytorrent");
int x = torrent["info"]["files"][0]["length"];

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

Это только я

BDictionary torrent = BItem.DecodeFile("my.torrent");int filelength = (BInteger)((BDictionary)((BList)((BDictionary)             torrent["info"])["files"])[0])["length"];

Приведение BDictionary не требуется, torrent объявлен как BDictionary.

public BItem this[int index]{    get { return ((BList)this)[index]; }}public BItem this[string index]{    get { return ((BDictionary)this)[index]; }}

Они не дают желаемого результата, так как возвращаемый тип по-прежнему является версией abstrat, поэтому вам все равно нужно выполнить приведение.

Переписанный код должен быть

BDictionary torrent = BItem.DecodeFile("my.torrent");int filelength = (BInteger)((BList)((BDictionary)torrent["info"]["files"])[0])["length"];

Что так же плохо, как и первая партия

Я бы порекомендовал ввести больше абстракций. Меня сбивает с толку то, что у BItem есть DecodeFile (), который возвращает BDictionary. Я не знаю, это может быть разумным поступком в торрент-домене.

Однако я бы нашел более разумным api, подобный следующему:

BFile torrent = BFile.DecodeFile("my.torrent");
int filelength = torrent.Length;