Рефакторинг метаданных

aspirant:

Mikhail Sukhov: Убрал перечисление PlazaTableTypes. Совершенно бесмысленная информация. Вместо этого за основу взял то что раньше называлось PlazaColumns.Table. Теперь везде именуется едино - SystemTableName. Оно равно НазваниеПотока/НазваниеТаблицы.

Не закончил до конца. Залил пока не компилябельное. Если есть желающие закончить - посмотрите как пример PlazaUsdOnlineColumns.

Михаил, посмотри, пожалуйста, на мою редакцию PlazaColumns. Комментарии внутри кода.

Простое имя таблицы (без имени стрима) мне нужно для создания схем.

Так может тогда 2 свойства ввести для PlazaColumn - StreamName и SystemTableName?

Mikhail Sukhov: Так может тогда 2 свойства ввести для PlazaColumn - StreamName и SystemTableName?

А что если: PlazaColumns owner?

aspirant:

Mikhail Sukhov: Так может тогда 2 свойства ввести для PlazaColumn - StreamName и SystemTableName?

А что если: PlazaColumns owner?

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

Mikhail Sukhov: Если дело в названии свойства - без разницы. Если другая концепция, то я не понял.

Концепция другая: в конструктор передается PlazaColumns owner. Потом, внутри класса PlazaColumn можно получить доступ к свойствам owner'а SystemTableName, ReplicationStream и т.д. Если потом что-то добавится, не нужно будет менять конструктор PlazaColumn. Все хорошо, только нюанс: в конструкторе PlazaColumn с owner'ом ничего нельзя будет сделать😢. Он будет еще пока равен null, потому что переменные типа PlazaColumn инициализируются в конструкторе наследника PlazaColumns. Например:

internal PlazaColumn(PlazaColumn owner, string name, Type dataType)
{
	//Следующая строчка вызовет исключение, потому мы все еще внутри конструктора PlazaColumns, и поэтому owner == null
	var tableName = owner.SystemTableName;
}

aspirant:

Mikhail Sukhov: Если дело в названии свойства - без разницы. Если другая концепция, то я не понял.

Концепция другая: в конструктор передается PlazaColumns owner. Потом, внутри класса PlazaColumn можно получить доступ к свойствам owner'а SystemTableName, ReplicationStream и т.д. Если потом что-то добавится, не нужно будет менять конструктор PlazaColumn. Все хорошо, только нюанс: в конструкторе PlazaColumn с owner'ом ничего нельзя будет сделать😢. Он будет еще пока равен null, потому что переменные типа PlazaColumn инициализируются в конструкторе наследника PlazaColumns. Например:

internal PlazaColumn(PlazaColumn owner, string name, Type dataType) { //Следующая строчка вызовет исключение, потому мы все еще внутри конструктора PlazaColumns, и поэтому owner == null var tableName = owner.SystemTableName; }

Весь смысл содержания SystemTableName в том, чтобы делать проверки на допустимость вставляемых колонок. А так, если owner только потом появиться, то это не очень хорошо... Есть предложение по owner. А почему бы его сразу не передавать в конструктор PlazaColumn? Тоесть, все эти таблицы, которые есть сейчас в Плаза, сразу создать в одном месте. Создаются без каких-либо колонок, просто название потока, таблицы и название человеческое. Например:

static class PlazaTableRegistry { public static PlazaTable OptVcb = new PlazaTable("FORTS_OPTINFO_REPL", "opt_vcb", "Опционы"); }

и уже потом при создании метаданных это учитывать:

public class PlazaVmFutureColumns: PlazaVmDerivativeColumns { internal PlazaVmFutureColumns() : base(PlazaTableRegistry.OptVcb) { VmReal = new PlazaColumn(PlazaTableRegistry.OptVcb, "vm_real", typeof(decimal), "d16.5", true); }

/// <summary>
/// Накопленная по сделкам вариационная маржа по фьючерсам, рассчитанная по текущей рыночной котировке.
/// </summary>
public readonly PlazaColumn VmReal;

}

А пользователь потом уже сам решит, что в OptVcb добавлять (и добавлять ли вообще).

aspirant: Для создания схем (ini файлов для получения данных из стримов Плазы) я добавил класс PlazaColumnSet. Внутри его конструктора проверяется, что все колонки принадлежат заданной таблице. В будущем его хочу дополнить, чтобы облегчить создание схем.

Лишний класс. Уже есть PlazaColumnList (кстати так же надо в Метадату перетащить), который проверяет на принадлежность. + проверять нужно не в самом конструкторе, а при добавлении колонок. Потому что колонки можно менять динамически (добавлять, удалять).

Mikhail Sukhov: Уже есть PlazaColumnList

OK, использую его.

aspirant: Сегодня планирую завершить рефакторинг метаданных. Когда вчера и позавчера правил файлы, столкнулся с таким расхождением: все классы фючерсов называются сокращенно, например, PlazaFutSessionContentsColumns, все классы опционов - полностью, например, PlazaOptionSessionSettlementColumns. Хочу привести все к единому стандарту. Что берем за основу: краткую версию или длинную?

Лучше длинное. Более правильное, что ли. Да и при такой длине слова как PlazaOptionSessionSettlementColumns не особо различимо.😀

aspirant: Проект теперь компилируется.

Good news everyone!

Mikhail Sukhov:

aspirant: Проект теперь компилируется.

Good news everyone!

Отрефакторил класс PlazaType.