ReactiveBIM/RxBim.Tools

TableBuilder Refactoring 2

Closed this issue · 5 comments

vildar82 commented
  1. Можно упростить интерфейсы
    ICellBuilder<Cell>, IColumnBuilder<Cell>, IRowBuilder<Cell>, ITableBuilder<Cell>
    без указания Cell?
  2. Упростить неймспейсы: namespace RxBim.Tools.TableBuilder.Builders; RxBim.Tools.TableBuilder.Styles; RxBim.Tools.TableBuilder.Content;
  3. Исчезли ObjectForBuild и ToTable() у билдеров строки и колонки.
  4. CellFormatStyleBuilder SetBorders(b => b.SetBorders()) - не красиво
pavlovek commented

по первому пункту не очень понятно что имеешь в виду

vildar82 commented

форматирование съело немного. поправил.

vildar82 commented

CellsSetBuilder смущает (

pavlovek commented
  1. убрать ObjectForBuild, заменив на реализацию IBuilder<T>
  2. в конструкторе билдеров делать копию объектов
vildar82 commented
  1. ObjectForBuild сделал protected. добавил IBuilder.
  2. Добавил методы Clone в объекты билдеров.