Выделение библиотеки annotations #65
Conversation
nixel2007
force-pushed
the
feature/annotations-lib
branch
from
add685b
to
708624a
Compare
| &ВремениИнициализации | ||
| Функция РазворачивательАннотаций(&Пластилин Поделка) Экспорт | ||
| Рефлектор = Новый Рефлектор(); | ||
| КонтейнерАннотаций = Рефлектор.ПолучитьСвойство(Поделка, "КонтейнерАннотаций"); |
There was a problem hiding this comment.
Знаю, что грязновато, но контейнер в публичный апи я пока отдавать не хочу
There was a problem hiding this comment.
А завязь всегда снглтоны регает?
There was a problem hiding this comment.
А у тебя же в поделке Разворачиватель сразу закешированный лежит, зачем через контейнер обращаться? Или я опять не понял?
There was a problem hiding this comment.
Синглтон завязи зависит от характера же.
There was a problem hiding this comment.
Я убрал разворачиватель из полей поделки, там теперь контейнер только, а разворачиватель локально достаётся в одном месте
There was a problem hiding this comment.
Типа
&Завязь
&Характер()
?
Т.е по дефолту синглтон?
There was a problem hiding this comment.
Я убрал разворачиватель из полей поделки, там теперь контейнер только, а разворачиватель локально достаётся в одном месте
Может ты конечно подумал что так сделал, но не сделал
There was a problem hiding this comment.
Да, по дефолту синглтон, как и жёлудь.
There was a problem hiding this comment.
Это ж не поле, а локальная переменная)
There was a problem hiding this comment.
Это ж не поле, а локальная переменная)
Перепутал, в фабрике поле.)
| @@ -111,6 +112,18 @@ | |||
| Ожидаем.Что(Верховный).ИмеетТип("Булево"); | |||
| Ожидаем.Что(ВремениИнициализации).ИмеетТип("Булево"); | |||
|
|
|||
| Если ВремениИнициализации Тогда | |||
| ТекстСообщения = СтрШаблон( | |||
| "К желудю времени инициализации %1 можно прилеплять только детальки.", | |||
There was a problem hiding this comment.
- По идее можно разрешить еще и прилепление других желудей времени инициализации, не вижу тут противоречий
- Мне не очень нравится доп проверка в конструкторе ДТОхи, кажется, что это должно быть в фабрике желудей. А в фабрике ОпределениеЖелудя создается два раза, что меня тоже уже очень давно напрягает... Помощь с рефакторингом приветствуется!
There was a problem hiding this comment.
- Йеп
- Не понял, а где смотреть что оно два раза создаётся?
There was a problem hiding this comment.
- В фабрике, два раза конструктор вызывается - из жёлудя и из завязи
There was a problem hiding this comment.
- В фабрике, два раза конструктор вызывается - из жёлудя и из завязи
А, увидел, ага
There was a problem hiding this comment.
- В фабрике, два раза конструктор вызывается - из жёлудя и из завязи
Но ты имеешь ввиду что просто в двух местах конструктор используется, а не то что он прям два раза вызывается на одном и том же желуде?
There was a problem hiding this comment.
Да, используется в двух местах. Постоянно надо синхронизировать изменения и тут и там.
There was a problem hiding this comment.
- Предлагаю оставить пока как есть и решить на уровне BFPP
Analysis Details3 Issues
Coverage and Duplications
|
Analysis Details9 Issues
Coverage and Duplications
|
https://github.com/autumn-library/annotations