Понял. Я не думаю, что тебе имеет смысл терять время на то, чтобы все такие места найти и исправить.
Это абсолютно верно, что такие обращения не допустимы, т.к. кэшируются весь объект при первом вызове, правильно использовать модуль ОбщегоНазначения и соответвующие функции "ЗначениеРеквизитаОбъекта" и т.д., там собственно под квпотом и будет запрос.
Особенно плохо будет такое обращение внутри цикла к разным ссылкам, т.к. это будет запрос в цикле.
Еще нельзя использовать функцию Строка(Ссылка), т.к. тоже будет запрос к БД за представлением.
Твое намерение абсолютно позитивное, но я бы советовал:
1) Не писать такой код самому
2) Передавать опыт коллегам и обсжудать стандарты (у нас например есть ежедневная рассылка стандартов разработки)
3) Исправлять такие места по мере обнаружения, где видишь их, где работаешь по другой задаче не лениться сразу исправлять
4) Раз уж ты полез в анализ код, наверное тебе нужно изучить проект BSL LS, это как раз про анализ кода на ошибки, там много чего более интересного
5) Смотреть где тормозит и там проводить самые срочные оптимизации в первую очередь, т.к. может быть много чего другого интресного
У нас страшный многолетний легаси, мы именно так и делаем и хорошо живем.
За исключением того, что я все никак не доберусь до BSL LS