ExportXPO - замечания и предложения по проекту

[mazzy]

решил выделить в отдельную тему отсюда
Как правильно выкладывать проекты по Аксапте на github, например?

насчет тулзы.
во-первых, стоит посмотреть https://github.com/Go-ERP/Ax2Git
)

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

Цитата:

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

Пока придумал только выгрузку XPO.

придумайте загрузку. сразу многие вопросы отпадут.

Цитата:

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

Прошу профессионал посмотреть и дать советы, как и что можно улучшить в моем проекте.

Посмотрел чисто внешне, сам проект.

==========================
таблица ExportXPOParameters
во-первых, прочитайте bestPractice по поводу наименований.
в аксапте таблицы принято называть в единственном числе.
кроме того, суффикс parameter предполагает, что таблица полностью кэшируемая,
в таблице хранится одна запись на каждую компанию.
и параметр достается методом find

делать Query на параметры - дикость какая-то в аксапте.
скорее всего, нужно дать другой суффикс.

==========================
хранить параметры по пользователю...
ну, не знаю... есть механизм sysLastValue. лучше его задействовать, нежели вручную что-то сортировать по пользователю. мне кажется, что переключатель FilterUser какой-то не логичный.

==========================
box::info(strFmt('Проект %1 импортирован', UtilElements.name))
нихт! никаких модальных диалогов!
если уж так хотите рассказать, то пользуйтесь инфологом - info, warning, error

=========================
класс ExportAOTtoXPO ничё так.
но опять же, лучше завязаться на правую кнопку мыши с объектами AOT.
и программисту привычнее.

опять же, посмотрите https://github.com/Go-ERP/Ax2Git

Кстати, обратите внимание, что и в AOT, и в проекте можно выделить несколько объектов и нажать на них правую кнопку мыши )

кроме того, программист может захотеть выгрузить в СКВ несколько проектов, а не один )

[demianimp]

Спасибо большое за рецензию.

Цитата:

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

...
во-вторых, лучше ориентироваться не на отдельный диалог (хотя и он важен),
а на правую кнопку мыши в AOT и в проекте.
...
но опять же, лучше завязаться на правую кнопку мыши с объектами AOT.
и программисту привычнее.
...

Вот если найдется добрый человек, который ткнет меня носом как это сделать, то буду бесконечно благодарен.

Цитата:

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

...
таблица ExportXPOParameters
во-первых, прочитайте bestPractice по поводу наименований.
в аксапте таблицы принято называть в единственном числе.
кроме того, суффикс parameter предполагает, что таблица полностью кэшируемая,
в таблице хранится одна запись на каждую компанию.
и параметр достается методом find
...

В данной таблице параметры хранятся для каждого пользователя.
Это сделано для того, если нет возможности развернуть AOS для каждого разработчика.

Цитата:

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

...
делать Query на параметры - дикость какая-то в аксапте.
скорее всего, нужно дать другой суффикс.
...

Вот здесь ничего не понял, можно подробнее объяснить?

Цитата:

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

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

Просто я не знаю другого способа работы отсортировать проект, который создал пользователь.

Цитата:

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

...
box::info(strFmt('Проект %1 импортирован', UtilElements.name))
нихт! никаких модальных диалогов!
если уж так хотите рассказать, то пользуйтесь инфологом - info, warning, error
...

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

Цитата:

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

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

кроме того, программист может захотеть выгрузить в СКВ несколько проектов, а не один )

Архитектура задумывалась из идеи 1 проект = 1 коммит

[mazzy]

Цитата:

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

Вот если найдется добрый человек, который ткнет меня носом как это сделать, то буду бесконечно благодарен.

уже неоднократно )
https://github.com/Go-ERP/Ax2Git

Цитата:

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

В данной таблице параметры хранятся для каждого пользователя.

в аксапте есть паттерн для решения задачи "хранить параметры для каждого пользователя". называется sysLastValue. поищите по форуму.

Цитата:

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

Вот здесь ничего не понял, можно подробнее объяснить?

https://msdn.microsoft.com/EN-US/library/aa846438.aspx

Цитата:

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

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

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

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

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

Цитата:

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

Архитектура задумывалась из идеи 1 проект = 1 коммит

эээээээ.....
без комментариев. только анекдот:

Цитата:

Приходит любовник к своей даме и с порога говорит ей:
- Узнал новую позу, давай попробуем!
- Хорошо, дорогой, только давай клетку с попугаем вынесем из спальни, иначе все мужу вечером разболтает...
- Давай лучше салфеткой накроем клетку и пусть сидит.
- А что за поза?
- Ты раздеваешься, стоишь у окна, я раздеваюсь, залезаю на шкаф, оттуда прыгаю на штору, со шторы - на люстру, оттуда лечу к тебе....
Попугай не выдержал: "Отрежьте мне язык, но я должен ЭТО видеть!!!"

======================================
вообще говоря, очень рекомендую ознакомиться
https://msdn.microsoft.com/EN-US/library/aa658028.aspx
в частности
https://msdn.microsoft.com/EN-US/library/aa855488.aspx
https://msdn.microsoft.com/EN-US/library/aa632638.aspx
https://msdn.microsoft.com/EN-US/library/aa622478.aspx