Опрос: Используете ли вы Best Practice Check при разработке?

[Maxim Gorbunov]

Цитата:

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

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

Так, подождите. Декларируется ведь, что эта проверка только для локальных селектов. То есть, в случае, если курсор куда-то передаётся, то проверка не выполняется.

К сожалению, нет сейчас AX 2012 рабочей под рукой. Не могу проверить.

Цитата:

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

Такая оптимизация имеет смысл только при наличии покрывающего индекса, ну или если у тебя таблица очень большое количество полей имеет (Но в этом случае, правильнее соптимизировать за пределы организации самого разработчика этой таблицы).
Затаскивание этого в best practice - заведомый идиотизм...

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

Но заметь, наличие такой рекомендации как минимум должно заставить разработчиков задуматься о структуре данных - нет?

[trud]

Цитата:

Сообщение от Maxim Gorbunov

Так, подождите. Декларируется ведь, что эта проверка только для локальных селектов. То есть, в случае, если курсор куда-то передаётся, то проверка не выполняется.

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

А потом кто-то этот код дописывает

В целом по 2009 довольно несложно следовать BestPractice, в 2012 я потратил около 3 часов на приведение 30 часовой несложной модификации к 0 ошибкам проверок.
Т.е. опрос все же зависит от версии.

[Maxim Gorbunov]

Цитата:

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

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

А потом кто-то этот код дописывает

Ну, в таком случае Best Practice можно переписать так: выделяйте выборку данных и их обработку в разные методы сразу в первой же версии кода

Согласен, что рекомендация спорная. Но однозначно неправильной я бы тоже её не спешил называть.

Цитата:

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

В целом по 2009 довольно несложно следовать BestPractice, в 2012 я потратил около 3 часов на приведение 30 часовой несложной модификации к 0 ошибкам проверок.
Т.е. опрос все же зависит от версии.

Не совсем понял. Вы предлагаете дополнить опрос вариантом вроде "Использовал в 2009, но перестал в 2012?"

[S.Kuskov]

Цитата:

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

Разработчика принуждают писать select fieldlist.

А потом кто-то этот код дописывает

А для тех кто дописывает нужно ещё одну проверку сделать. Мол вы используете поле, которое отсутствует в списке полей select.

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