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

Рефакторинг метаданных
Atom
3/11/2011
Mikhail Sukhov


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

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

Tags:


Thanks:


1 2  >
aspirant

Avatar
Date: 3/11/2011
Reply


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

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


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

Простое имя таблицы (без имени стрима) мне нужно для создания схем. Потом отдельные свойства для имен таблиц и потоков пригодятся, если понадобится использовать Reflection. Например, чтобы отобрать все свойства-стаканы внутри PlazaColumns, можно сделать:

Code
var aggregations = PlazaColumns.GetAllMetadata().Where(x => x.SystemTableName == "orders_aggr");

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

Mikhail Sukhov

Avatar
Date: 3/11/2011
Reply


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

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


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

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


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

aspirant

Avatar
Date: 3/11/2011
Reply


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


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

Mikhail Sukhov

Avatar
Date: 3/11/2011
Reply


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


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


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

aspirant

Avatar
Date: 3/11/2011
Reply


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


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

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

Thanks:

Mikhail Sukhov

Avatar
Date: 3/11/2011
Reply


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


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

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



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

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


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

Code
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 добавлять (и добавлять ли вообще).
Thanks:

aspirant

Avatar
Date: 3/11/2011
Reply


Code
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;
}


Михаил, идея хорошая. Только в верхнем коде: таблице opt_vcb соответствует класс PlazaOptionColumns, а для PlazaVmFutureColumns внутри PlazaTableRegistry нужно будет создать переменную VmFuture.
Thanks:

aspirant

Avatar
Date: 3/13/2011
Reply


Я внес предложенные изменения:
  1. PlazaTable перенес в папку Metadata. Теперь этот класс только содержит метаданные для таблицы из стрима Плазы. Созданием и конфигурированием набора колоннок (полей) для получения данных он больше не занимается.
  2. Добавил класс PlazaTableRegister. Он будет содержать переменные типа PlazaTable для всех таблиц Плазы. Для наглядности добавил:
    Code
    public static PlazaTable RtsIndex = new PlazaTable(TableName.rts_index, ReplicationStream.RTS_INDEX_REPL, "Индекс РТС");

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

Жду ваши комментарии / пожелания, после чего начну рефакторить папку Metadata.
Thanks:

Mikhail Sukhov

Avatar
Date: 3/13/2011
Reply


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


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

aspirant

Avatar
Date: 3/13/2011
Reply


Mikhail Sukhov
Уже есть PlazaColumnList


OK, использую его.
Thanks:
1 2  >

Attach files by dragging & dropping, , or pasting from the clipboard.

loading
clippy