AX2012. Цель атрибутов в расширении наследования классов

[mazzy]

Цитата:

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

"вызывать его напрямую вообще говоря моветон в Аксапте" ? Ты предлагаешь через menuItem?

я предлагал? покажи мне это место, пожалуйста.

я говорил, что МС мог бы сделать более универсальный механизм, который учитывает инфраструктуру Аксапты. И мог бы насыпать синтаксического сахара. А также привел примеры того, как это делают другие.

Цитата:

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

Программировать? Для расширения - просто пометить атрибутом, для того места, которое надо расширить - вызов метода. Если нужен новый тип данных для ключа, добавить атрибут для него.

Прекрасно. Только мы говорили про вызывать. Ты довел мысль только до расширения.
доведи свою мысль до вызова функцонала класса, пожалуйста.

можно на примере расширить периодическое сопоставление по клиентам, например. Некоторым пользователям можно запускать специальное сопоставление по клиентам. со специальными полями в диалоге, которые устанавливает специальное условия в query. Остальные поля - стандартные.

как ? как дать пользователю пользователю? как проверить права?
как обеспечить, чтобы это расширение работало и в пакетных заданиях.

для определенности, давай сформулирую условие на старо-программистком:
= создать класс-потомок от CustVendAutoSettlement_Cust_RU (см. скриншот)
= ovverride метод dialog*
= ovverride метод query
= новый функционал должен работать как в пакетном задании, так и при непосредственном запуске пользователем
= новый функционал должен быть доступен только некоторым пользователям (желательно, чтобы система доступа была построена на стандартных правах аксапты)

=============
говоришь, кто-то указал, что это final класс?
ничего-ничего, на проекте можно [было] убрать это слово.
давай будем считать, что этого слова там нет.
=============

Цитата:

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

И это правильно - у нее четкое предназначение.

И какое же на твой взгляд?
А какое предназначение было у конструкторов, которых не будет?

Цитата:

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

Расскажи чем оно так сильно портит жизнь партнеру?

Макс, легко.
Обещаю, я сделаю это, после того, как ты покажешь как решить типовую задачу на типовом внедрении - как предоставить пользователю возможность запуска расширенного функционала тривиального семейства из 7-8 классов "просто пометив атрибутом".

[belugin]

Цитата:

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

я предлагал? покажи мне это место, пожалуйста.

- "Вызывать класс напрямую - это моветон"
- "Info.add" это вызов класса
=> просто писать info.add это моветон.
Я прошу показать бон тон.

Цитата:

можно на примере расширить периодическое сопоставление по клиентам, например. Некоторым пользователям можно запускать специальное сопоставление по клиентам. со специальными полями в диалоге, которые устанавливает специальное условия в query. Остальные поля - стандартные.

как ? как дать пользователю пользователю? как проверить права?
как обеспечить, чтобы это расширение работало и в пакетных заданиях.

Вариант 1. Унаследоваться и перекрыть необходимые методы. В случае, если проверка прав не проходит, вызывать методы предка. Case при создании заменить на SysExtension.

Вариант 2: Использовать SysIConditionalExtension

Цитата:

говоришь, кто-то указал, что это final класс?

Не говорю - это часть другой дискуссии - запирать код или нет. Это оотогональное решение.

Цитата:

И какое же на твой взгляд?
А какое предназначение было у конструкторов, которых не будет?

1) Выше ссылка на пост Вани о том как использовать конструкторы.
2) Без конструкторов легко как и практически во всех компонентный моделях (метод init - те же самые RunBase получают параметры не из конструктора)

[mazzy]

Цитата:

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

- "Вызывать класс напрямую - это моветон"
- "Info.add" это вызов класса
=> просто писать info.add это моветон.
Я прошу показать бон тон.

Макс, я в прошлый раз промолчал. Но раз ты настаиваешь...

info.add - это вызов метода класса через глобальную переменную infolog.
в аксапте есть три общеизвестных глобальных предопределенных переменных - infolog, session, appl.
ты точно хочешь продолжать защищать технологию атрибутов именно на этом примере?

если хочешь продолжать, то обрати внимание, что метод info.add по идее должен был бы быть protected + нужно было бы объявлять friend классы (error, warning, info) для доступа к add. но в аксапте нет friend.
Поэтому используется соглашение, что разработчики НЕ вызывают info.add, а должны пользоваться статическими методами Global::error, Global::warning, Global::info.

Поэтому приведенный тобой пример предельно НЕ типичен для обычной работы обычных людей...

Макс, я повторю вопрос - ты точно настаиваешь на этом примере в качестве обоснования технологи атрибутов?

=====================
А бон тон очень просто - класс, у которого есть статический метод main с арзументом Agrs. для такого класса должен быть создан menuItem. вызвать класс через стандартную конструкцию
new MenuFunction(menuitemDisplayStr(myMenuItem), MenuItemType::Display).run(args)

К сожалению, среди разработчиков МС не принято беспокоится о правах доступа, поэтому часто вызывают класс напрямую через конструктор + run. Этот способ не назвать бон тоном, но по факту игнорировать этот способ нельзя - нужно думать о нем и поддерживать его тоже.

Цитата:

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

Вариант 1. Унаследоваться и перекрыть необходимые методы. В случае, если проверка прав не проходит, вызывать методы предка. Case при создании заменить на SysExtension.

Вариант 2: Использовать SysIConditionalExtension

Макс, а можно проектик? Все участники тебе спасибо скажут. Особенно те, кто интересуется акс7, но у них еще нет возможности попробовать аксапту вживую.

Ты же сам сказал, что это "просто проставить атрибуты"

Цитата:

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

Не говорю - это часть другой дискуссии - запирать код или нет. Это оотогональное решение.

Согласен. Поэтому никаких ограничений на final и protected - да, их придется учитывать разработчикам на клиенте и заказчике. Но, согласен, это отдельная ветка обсуждения. Поэтому в этой ветке пропускаем.

Цитата:

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

1) Выше ссылка на пост Вани о том как использовать конструкторы.
2) Без конструкторов легко как и практически во всех компонентный моделях (метод init - те же самые RunBase получают параметры не из конструктора)

Мы все читать умеем. И все про конструкторы прочитали. С момента выхода статьи прошло несколько лет, поэтому время было.

Обрати внимание, что во всех статьях на эту тему рассматривается плоское одноуровневое наследование, а я спрашиваю про иерархию классов. Людям тоже придется иметь дело с иерахиями - их много в стандартной аксапте, их много в кастомизациях. Я думаю, что все уже давно увидели эту разницу и с нетерпением ждут ответа. Уверен, что ты легко приведешь изящный способ.

Также обрати внимание, что ты отвечаешь про конструкторы.
людям же нужно добавить функциональность, которую могут использовать пользователи.

Нет, Макс, это не одно и то же.
Нет, участники, это не придирка с словам и правописанию.

Итак, формулировка:

Цитата:

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

Просто продемонстрируйте проектик с добавленной функциональностью, которую могут использовать пользователи со специальными правами,
в которой используется мощнейшая технология атрибутов и SysExtensionAppClassFactory::getClassFromSysAttribute.

Ведь это ж "просто пометить атрибутом".

[ax_mct]

Цитата:

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

Мы все читать умеем. И все про конструкторы прочитали. С момента выхода статьи прошло несколько лет, поэтому время было.

Обрати внимание, что во всех статьях на эту тему рассматривается плоское одноуровневое наследование, а я спрашиваю про иерархию классов.

Приведенная ссылка
http://kashperuk.blogspot.co.uk/2017...extension.html
является статьей двухмесячной давности.

Основная описанная проблема - игнорирование аргументов с dictClass.makeObject().
SysExtension framework will instantiate a new instance of the corresponding class (dictClass.makeObject()), which ignores the constructor arguments.


Предложенное решение - передавать обьект третьим параметром.
In order to account for the constructor arguments we need to use an Instantiation strategy, which can then be passed in as the 3rd argument when calling SysExtensionAppClassFactory.

Если посмотреть на сигнатуру SysExtensionAppClassFactory::getClassFromSysAttribute
https://msdn.microsoft.com/en-us/lib...attribute.aspx
то этот третий параметр существует как
[SysExtAppClassDefaultInstantiation _defaultInstantiation])
Если проблема работы с многоуровневой иерархией решается то решается она через class SysExtAppClassDefaultInstantiation implements SysExtensionIInstantiationStrategy

И вот эта самая Instantiation Strategy как-то и призвана решить все проблемы.

public class BaseClassWithArgInConstructor
{
    //...
    public static BaseClassWithArgInConstructor construct(NoYesUnchanged _factoryType, str _argument)
    {
        NoYesUnchangedFactoryAttribute attr = new NoYesUnchangedFactoryAttribute(_factoryType);
        BaseClassWithArgInConstructor instance = SysExtensionAppClassFactory::getClassFromSysAttribute(
            classStr(BaseClassWithArgInConstructor), attr, [B]InstantiationStrategyForClassWithArg::construct(_argument)[/B]);
 
        return instance;
    }
}.

[mazzy]

Цитата:

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

Приведенная ссылка
http://kashperuk.blogspot.co.uk/2017...extension.html
является статьей двухмесячной давности.

Да. Приведенная по ссылке статья является статьей двухмесячной давности.
Извините.

Цитата:

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

Если проблема работы с многоуровневой иерархией решается то решается она через class SysExtAppClassDefaultInstantiation implements SysExtensionIInstantiationStrategy

Ребяты, вы опять и снова пытаетесь тыкамть меня носом в то, что это работает.
Ок. Верю.

Вы утверждаете что "просто". Ок. Можно проектик?

[ax_mct]

Цитата:

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

Ребяты, вы опять и снова пытаетесь тыкамть меня носом в то, что это работает.
Ок. Верю.

Вы утверждаете что "просто". Ок. Можно проектик?

Чур не я. Очень надеюсь что выше AX2012R3 не полезу.
Но если в рамках темы то бишь AX2012 то не вижу смысла заморачиваться c этим фреймворком.

Если AX7 то это лучше к Ване. У него там даже код есть.
Проект: https://1drv.ms/f/s!ApLM1du7Goflnpl_RQlhOuzEW6fL-A

Вот его продолжение про Instantiation strategy.

using an instantiation strategy in combination with the SysExtensionIAttribute attribute was not supported.
We have updated the SysExtension framework in Platform Update 5, so now you can rip the benefits of both worlds, using an instantiation strategy and implementing the SysExtensionIAttribute interface on the attribute.

Development tutorial: SysExtension framework with SysExtensionIAttribute and an Instantiation strategy
http://kashperuk.blogspot.co.uk/2017...ension_31.html

[mazzy]

Цитата:

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

Очень надеюсь что выше AX2012R3 не полезу.

а... это вы теоретически...

я уже говорил, что

Цитата:

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

А патентованная технология замечательная. Да-да.

Повторю - замечательная патентованная технология.

[belugin]

Цитата:

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

Поэтому используется соглашение, что разработчики НЕ вызывают info.add, а должны пользоваться статическими методами Global::error, Global::warning, Global::info.

ok, меняем пример на global::info

Цитата:

А бон тон очень просто - класс, у которого есть статический метод main с арзументом Agrs. для такого класса должен быть создан menuItem. вызвать класс через стандартную конструкцию
new MenuFunction(menuitemDisplayStr(myMenuItem), MenuItemType:isplay).run(args)

Ты предлагаешь это для добавления каждого сообщения в инфолог?

Цитата:

К сожалению, среди разработчиков МС не принято беспокоится о правах доступа, поэтому часто вызывают класс напрямую через конструктор + run. Этот способ не назвать бон тоном, но по факту игнорировать этот способ нельзя - нужно думать о нем и поддерживать его тоже.

Мне кажется, что ты немного обще свормулировал - мы не говорим про ВСЕ вызовы. Мы говорим в данном случае про вызовы из UI. В каком виде там внутри будут работать маленькие классы с деталями реализации не важно с точки зрения security.

Цитата:

Обрати внимание, что во всех статьях на эту тему рассматривается плоское одноуровневое наследование, а я спрашиваю про иерархию классов. Людям тоже придется иметь дело с иерахиями - их много в стандартной аксапте, их много в кастомизациях. Я думаю, что все уже давно увидели эту разницу и с нетерпением ждут ответа. Уверен, что ты легко приведешь изящный способ.

Тут вообще на важно иерархия или нет. Важно каким образом определяется элемент этой иерархии - если этот способ легко положить лукап по ключу, то SysExtension справится.

Цитата:

Также обрати внимание, что ты отвечаешь про конструкторы.
людям же нужно добавить функциональность, которую могут использовать пользователи.

SysExtension это способ добавить такую функциональность.

[mazzy]

способ опосредованный и не оптимальный. отвечу.
также напомню, что хотелось бы увидеть от тебя проект )

Цитата:

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

способ легко положить лукап по ключу

ты щас очень важный вещь сказал - надо подумать.

а что будет, если ключ не уникальный?
фреймворк создаст несколько инстансев с разными классами, но с одинаковым ключом? и как он будет работать с этими инстансами? типа параллельно? может, это в сторону асинхронной работы? бред, конечно. но вдруг? кто следит за уникальностью? и что делает с неуникальными ключами?

надо подумать. и в код посмотреть. и подумать.
на остальное отвечу позже.

[belugin]

Цитата:

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

а что будет, если ключ не уникальный?

Это зависит по идее от логики того. что расширяется. Если ему нужно что-то одно, то должен быть exception так как ситуация неоднозначная.

Если несколько, то надо дальше как-то их дергать. Например если у нас какие-ниьудь протоколяторы - один в файл. другой в инфолог, третий в EventLog то можно им раздавать информацию параллельно.