Хорошее MVC

[dimarik]

И еще. Можно вышестоящим родителем сеттить user, если он у него есть, конечно. Т.е. не обращаться к родителю, а он все сам сделает. Аха, да, это инжект и есть.

[incvizitor]

Хм, разве это дорогостоящая операция?

[etc]

Сообщение от incvizitor

Хм, разве это дорогостоящая операция?

Вполне себе да, особенно учитывая, что оно всплывающее.

[dimarik]

if (!this._user...
this._user = null;

Он вроде и так null;

[incvizitor]

@dimarik, не обязательно.
@etc, ок спасибо.

[dimarik]

UserData вполне себе может быть IUserAsset, кстати. И UserData#user может указывать, например, на персонажа текущего пользователя. Лишняя проверка убирается.

Добавлено через 1 минуту
ADDED/REMOVED еще продиагностировать нужно, от кого пришло. А если глубина вложенности большая, то только и будешь определять target.parent с currentTarget

[etc]

Сообщение от dimarik

UserData вполне себе может быть IUserAsset, кстати. И UserData#user может указывать, например, на персонажа текущего пользователя. Лишняя проверка убирается.

Персонаж это персонаж, а юзер это юзер. Последний ассетом самого себя не может быть.
ADDED/REMOVED слушают и диагностируют те, кому это нужно, просто в любом случае смена юзера означает изменение контейнера, а значит в представлении точно также поменяется отображаемый контейнер и это будет уже другой вьювер.

[dimarik]

Сообщение от etc

Персонаж это персонаж, а юзер это юзер. Последний ассетом самого себя не может быть.
ADDED/REMOVED слушают и диагностируют те, кому это нужно, просто в любом случае смена юзера означает изменение контейнера, а значит в представлении точно также поменяется отображаемый контейнер и это будет уже другой вьювер.

А я разве сказал, что this._user ассет самого себя? Он ассет кого надо ассет. Какого-то юзеря. Может текущего персонажа. Удобно с текущим пером меряться. Он под рукой всегда.

Ну поменяется отображаемый контейнер, если ты сверху вниз на стадии смены контейнера пересеттишь этот ассет. Я не вижу смысла тут пользоваться прослушиванием ADDED/REMOVED

Сообщение от etc

В модели таких вложенностей не бывает.

Это по стандарту так?

[dimarik]

this._user.contains(this) при глубоком (IUserAsset находится в энном колене) и широком (обладатель этого ленивого геттера находится на тысячной позиции) дереве будет чутка тормозить.

[etc]

Сообщение от dimarik

this._user.contains(this) при глубоком (IUserAsset находится в энном колене) и широком (обладатель этого ленивого геттера находится на тысячной позиции) дереве будет чутка тормозить.

В модели таких вложенностей не бывает.