AXForum  
Вернуться   AXForum > Microsoft Dynamics AX > DAX: Прочие вопросы
All
Забыли пароль?
Зарегистрироваться Правила Справка Пользователи Сообщения за день Поиск Все разделы прочитаны

 
 
Опции темы Поиск в этой теме Опции просмотра
Старый 25.08.2016, 10:14   #1  
Pokersky09 is offline
Pokersky09
Участник
 
18 / 45 (2) +++
Регистрация: 15.11.2012
Адрес: Казань
Показать/Скрыть комментарии кода
Есть ли возможность добавить кнопку в редакторе кода, например рядом с Toggle change tracking margin,
функция которой будет скрытие и отображение всего что закомментировано? Т.е. скрыть весь зеленый текст.


Возникла проблема, в ходе постоянных доработок кода разными разработчиками, соотношение по количеству строк кода/комментарий примерно 60/40.

Соответственно сильно страдает читабельность кода.

Удалять комментарии, либо переписывать в новые методы, не вариант, все изменения требуется сохранять.
Старый 25.08.2016, 10:17   #2  
Stitch_MS is offline
Stitch_MS
Участник
Аватар для Stitch_MS
Соотечественники
 
212 / 201 (7) ++++++
Регистрация: 27.02.2006
Адрес: Дания
Т.е. у вас не настроена система контроля версий?
Старый 25.08.2016, 10:43   #3  
pitersky is offline
pitersky
северный Будда
Аватар для pitersky
Ex AND Project
Соотечественники
 
1,251 / 247 (11) ++++++
Регистрация: 26.09.2007
Адрес: Солнечная система
Цитата:
Сообщение от Pokersky09 Посмотреть сообщение
Возникла проблема, в ходе постоянных доработок кода разными разработчиками, соотношение по количеству строк кода/комментарий примерно 60/40.
................
Удалять комментарии, либо переписывать в новые методы, не вариант, все изменения требуется сохранять.
если вы работаете на клиенте то кмк комментарии полугодовалой и более давности совершенно не нужны
__________________
С уважением,
Вячеслав
Старый 25.08.2016, 11:29   #4  
Pokersky09 is offline
Pokersky09
Участник
 
18 / 45 (2) +++
Регистрация: 15.11.2012
Адрес: Казань
Цитата:
Сообщение от Stitch_MS Посмотреть сообщение
не настроена система контроля версий?
К сожалению, нет

Цитата:
Сообщение от pitersky Посмотреть сообщение
комментарии полугодовалой и более давности совершенно не нужны
Удалять чужой код полностью запрещено, регламент работ требует сохранения всех изменений, с внесением комментариев
Старый 26.08.2016, 02:30   #5  
Alex_KD is offline
Alex_KD
Участник
AxAssist
MCBMSS
Соотечественники
 
497 / 255 (9) ++++++
Регистрация: 06.07.2006
Адрес: Melbourne, Down Under
Цитата:
Сообщение от Stitch_MS Посмотреть сообщение
Т.е. у вас не настроена система контроля версий?
Cистема контроля версий это хорошо, но это не спасет в случае когда коменты перед телом метода (стандартный <summary> иногда строк 100, в 7ке это убивает читабельность да и вообще удобство).

Без кастомизации не скрыть.

Для AX2012, в AxAssist можно поменять цвет коментов на белый.
Для AX7, можно поменять цвет в VS.
AxAssist для AX7 (который пока не public) схлопывает коменты как раз как вы описали.
__________________
AxAssist 2012 - Productivity Tool for Dynamics AX 2012/2009/4.0/3.0
За это сообщение автора поблагодарили: mazzy (2).
Старый 26.08.2016, 10:39   #6  
Владимир Максимов is offline
Владимир Максимов
Участник
КОРУС Консалтинг
 
1,331 / 811 (30) +++++++
Регистрация: 13.01.2004
Когда количество комментариев в коде начинает превышать сам код или серьезно страдает "читабельность" кода, то я делаю копию метода с окончанием "_old" и в нем пишу в шапке, что это старая версия метода XXX. В шапке самого метода также пишу, что старая версия сохранена в методе XXX_old.

В результате, и история сохранена и лишние комментарии можно удалить. "... И пастуху вечная память" (с)
__________________
- Может, я как-то неправильно живу?!
- Отчего же? Правильно. Только зря...
За это сообщение автора поблагодарили: dn (1), Dreadlock (1).
Старый 26.09.2016, 12:58   #7  
Pokersky09 is offline
Pokersky09
Участник
 
18 / 45 (2) +++
Регистрация: 15.11.2012
Адрес: Казань
Цитата:
Сообщение от Alex_KD Посмотреть сообщение
Для AX2012, в AxAssist можно поменять цвет коментов на белый.
Для AX7, можно поменять цвет в VS.
AxAssist для AX7 (который пока не public) схлопывает коменты как раз как вы описали.
Вопрос по прежнему актуален, нужно для Ах2012.

AxAssist очень хорошая штука, но научить бы ее "схлопывать" комментарии, вопрос был бы решен Это реализуемо?

По теме АхAssist возникает проблема "скрытой" выборочной установки на терминалку, ибо части коллег не нравится изменение стнд среды, а на одной терминалке больше 10 разных приложений. Соответственно установив AxAssist, требуется также устанавливать ХРО на все приложения(что уже не есть гуд), и всем разрабам раздавать конфиги, где по умолчанию все приведено к дефолтовому виду.

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

Цитата:
Сообщение от Владимир Максимов Посмотреть сообщение
делаю копию метода с окончанием "_old" и в нем пишу в шапке, что это старая версия метода XXX. В шапке самого метода также пишу, что старая версия сохранена в методе XXX_old.

В результате, и история сохранена и лишние комментарии можно удалить. "
Такой вариант уже рассматривался, но как я упомянул в шапке, переписывать в новые методы нельзя

Последний раз редактировалось Pokersky09; 26.09.2016 в 13:04. Причина: Re: Владимир Максимов
Старый 26.09.2016, 19:16   #8  
Владимир Максимов is offline
Владимир Максимов
Участник
КОРУС Консалтинг
 
1,331 / 811 (30) +++++++
Регистрация: 13.01.2004
Цитата:
Сообщение от Pokersky09 Посмотреть сообщение
Такой вариант уже рассматривался, но как я упомянул в шапке, переписывать в новые методы нельзя
Вы не в новые методы переписываете, а старый код сохраняете. Метод остается тот же самый. С тем же именем. Просто очищенный от комментариев.

А вот старый код с комментариями как раз в новый метод и копируется. Именно "для истории". Причем имя "исторического" метода такое, что он оказывается рядом с текущим методом в списке методов.

Если Вы не поняли, то у Вас был метод

X++:
void MyMethod()
{
   int I;

   I++;
   /*
   Много комментариев
   */
}
А стало

X++:
// 29.09.2016
// Метод полностью переписан. Старая версия сохранена в методе MyMethod_old()
void MyMethod()
{
   int i;

   i++;
}

// 29.09.2016
// Старая версия метода MyMethod(). Не используется
void MyMethod_old()
{
   int i;

   i++;
   /*
   Много комментариев
   */
}
__________________
- Может, я как-то неправильно живу?!
- Отчего же? Правильно. Только зря...
Старый 27.09.2016, 13:11   #9  
Ivanhoe is offline
Ivanhoe
КОРУС Консалтинг
Аватар для Ivanhoe
Лучший по профессии 2014
Лучший по профессии AXAWARD 2013
Лучший по профессии 2011
КОРУС Консалтинг
 
3,469 / 1646 (61) ++++++++
Регистрация: 29.09.2005
Адрес: Санкт-Петербург
На крайняк - прямо внутри метода вначале пустить "чистый" метод, внизу оставить всю историю.
__________________
Ivanhoe as is..
Старый 27.09.2016, 13:38   #10  
DSPIC is offline
DSPIC
Боец
Аватар для DSPIC
Лучший по профессии 2014
MCP
Лучший по профессии 2009
 
956 / 934 (33) +++++++
Регистрация: 11.04.2008
Адрес: Минск
Цитата:
Сообщение от Pokersky09 Посмотреть сообщение
Вопрос по прежнему актуален, нужно для Ах2012.

AxAssist очень хорошая штука, но научить бы ее "схлопывать" комментарии, вопрос был бы решен Это реализуемо?

По теме АхAssist возникает проблема "скрытой" выборочной установки на терминалку, ибо части коллег не нравится изменение стнд среды, а на одной терминалке больше 10 разных приложений. Соответственно установив AxAssist, требуется также устанавливать ХРО на все приложения(что уже не есть гуд), и всем разрабам раздавать конфиги, где по умолчанию все приведено к дефолтовому виду.

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


Такой вариант уже рассматривался, но как я упомянул в шапке, переписывать в новые методы нельзя
Сюда не заглядывали? Microsoft Dynamics AX 2012 X++ Editor Extensions Там по аналогии с Outlining Extension, наверное, сделать не проблема.
__________________
Мой блог
Теги
кнопка, редактор кода, комментарий

 

Похожие темы
Тема Автор Раздел Ответов Посл. сообщение
Выдержки кода Axapta в интернете. fixer DAX: Прочие вопросы 3 02.02.2015 12:43
Обзор написанного кода MikeR DAX: Программирование 10 20.11.2014 07:59
скрыть/показать кнопку BokarevSS DAX: Программирование 3 16.10.2012 11:23
Убрать из Меню Скрыть\показать fialka DAX: Программирование 17 04.09.2012 12:31
проблема с настройкой штрихового кода Nic DAX: Функционал 4 27.12.2004 09:04
Опции темы Поиск в этой теме
Поиск в этой теме:

Расширенный поиск
Опции просмотра

Ваши права в разделе
Вы не можете создавать новые темы
Вы не можете отвечать в темах
Вы не можете прикреплять вложения
Вы не можете редактировать свои сообщения

BB коды Вкл.
Смайлы Вкл.
[IMG] код Вкл.
HTML код Выкл.
Быстрый переход

Рейтинг@Mail.ru
Часовой пояс GMT +3, время: 03:12.
Powered by vBulletin® v3.8.5. Перевод: zCarot
Контактная информация, Реклама.