Приложение Б. Описания правил проверки кода

• Использование арифметических операций при конкатенации со строками

• Установка переменной "Параметры" в управляемой форме

• Вызов асинхронных методов в событии ПриЗавершенииРаботыСистемы

• Вызов асинхронных методов в событии ПередЗавершениемРаботыСистемы без установки Отказа

• Асинхронные конструкции доступны с версии платформы 8.3.18

• Небезопасное подключение внешней компоненты

• Использование устаревшего параметра "АвтоТест"

• Использование "Ждать" в не "Асинх" методах

• Использование некорректного символа

• Метод "НачатьТранзакцию" должен располагаться непосредственно перед оператором "Попытка"

• Сравнение с логической константой

• Сложные условия

• Вызов ОбщегоНазначения.WSПрокси() без таймаута

• Запуск внешних приложений

• Обращение к несуществующему элементу формы

• Работа с параметром "Отказ"

• Приведение типов в Попытке

• Приведение объекта к строке

• Изменение свойств только для чтения на управляемой форме

• Код должен быть расположен внутри области

• Блоки кода не должны быть "закомментированы"

• Опечатки в комментариях

• ЗафиксироватьТранзакцию находится вне конструкции ПопыткаИсключение

• Рекомендуется избегать в названии общего модуля таких общих слов как "Процедуры

• Клиентский общий модуль должен именоваться с постфиксом "Клиент"

• Клиент-серверный общий модуль должен именоваться с постфиксом "КлиентСервер"

• Глобальный общий модуль должен именоваться с постфиксом "Глобальный"

• Для глобальных модулей постфикс "Клиент" добавлять не следует

• Модули, выполняющиеся в привилегированном режиме, именуются с постфиксом "ПолныеПрава"

• Общий модуль с повторно используемыми значениями должен именоваться с постфиксом "ПовтИсп"

• Общий модуль с вызовом сервера должен именоваться с постфиксом "ВызовСервера"

• ComОбъект вне конструкции "Попытка"

• Использование директив компиляции

• Последовательные пустые строки

• Использование Если/ИначеЕсли с константным выражением

• Использование в конструкторе структуры конструкторов других объектов с параметрами

• Безумные многострочные литералы

• Нельзя создавать прикладные объекты с использованием конструктора

• Получение контейнера ключей из файла

• Циклические ссылки в коллекциях

• Использование строк подключения к БД

• Наличие обязательной конструкции "Если ОбменДанными.Загрузка Тогда ..."

• Удаление временного файла после использования

• Использование устаревшего метода "Найти"

• Использование устаревшего метода глобального контекста

• Устаревшие синхронные методы объекта HTTPСоединение

• Использование устаревшего типа "УправляемаяФорма"

• Нерекомендуемый вызов устаревшего метода

• Использована конструкция "ДанныеФормыВЗначение()"

• Использование метода ПолучитьФорму()

• Использование метода "РольДоступна()"

• Использование метода "Сообщить()"

• Использование устаревшего свойства "ЭтаФорма"

• Избыточное наличие описания параметра в комментарии к процедуре (функции), отсутствующего в сигнатуре вызова

• Избыточное наличие секции "Параметры" в комментарии процедуры (функции) без параметров

• Наличие всех параметров в комментарии экспортной процедуры (функции)

• Наличие секции параметров в описании экспортной процедуры (функции)

• Наличие секции возвращаемого значения в описании экспортной функции

• Секция "Возвращаемое значение" должна располагаться после секции "Параметры" в комментарии к функции

• Знак сложения дублируется

• Строковые литералы не должны дублироваться

• Пустой блок кода

• Пустая процедура/функция

• Код не должен содержать пустых областей

• Пустой оператор

• Обращение к несуществующему значению перечисления

• В вызове глобального метода 1С совпадают значения параметров

• Одинаковый код во взаимоисключаемых ветках ветвления кода

• Одинаковые условия в последовательности вида "Если (....) ИначеЕсли (....) ИначеЕсли (....)".

• Совпадающие выражения в вычислениях

• Совпадающие/одинаковые тела методов

• Одинаковые выражения (вызов методов) в соседних условиях/ветках/блоках кода

• Функция всегда возвращает одно и то же константное значение

• Дублирование одинаковых/похожих блоков кода

• Одинаковые части условий в разных условиях блока "Если ИначеЕсли"

• Отсутствие логирования ошибки

• Слишком много возвратов внутри функции

• Ограничения на использование "Выполнить" и "Вычислить" на сервере

• Экспортный метод в модуле команды

• Экспортная процедура или функция в модуле формы

• Код с программным интерфейсом должен быть расположен внутри специальных областей

• Экспортная процедура в модуле с повторным использованием

• Использование экспортной переменной

• Экспортные переменные должны содержать описание

• Таймауты при работе с внешними ресурсами

• Использование НайтиПоРеквизиту()

• Использование НайтиПоКоду()

• Использование НайтиПоНаименованию()

• Комментарий с тегом "FIXME"

• Использование метода "ТекущаяДата()

• Обработчик события формы указывает на несуществующий метод

• Проверка заполненности/пустоты коллекции

• Проверка, что функция является процедурой

• Ограничение на количество параметров функции в конструкторе структуры

• Имя функции начинается с глагола

• Функция должна возвращать значение

• Ограничение на использование оператора "Перейти"

• Использование сертификатов в коде

• Прямое указание GUID в коде

• Код не должен содержать указание паролей в явном виде

• Конструкторы объектов не должны содержать указания паролей в явном виде

• Вызовы методов не должны содержать указания паролей в явном виде

• Прямое указание пути в коде

• В коде не должно содержаться URL-ссылок

• В коде не должны содержаться e-mail

• IP адреса не должны храниться в коде

• В выражении "Если" отсутствует ветка "Иначе"

• Удаление из коллекции внутри цикла по коллекции

• Локальные переменные не должны инициализироваться и тут же возвращаться из функции

• Неверные отступы в коде

• Прекращение работы приложения 1С с запуском другого приложения

• Использование итератора или счетчика цикла вне тела цикла

• Подозрительное продолжение цикла

• Каноническое написание ключевых слов

• Операторы перед КонецЕсли, КонецЦикла и т.д. должны заканчиваться точкой с запятой

• Термин "1C:Предприятие" ошибочно задан с латинской "C"

• Длина имени метода

• Ограничение на длину строки

• Изменение итератора/счетчика цикла

• Цикл не должен быть бесконечным

• Инициализированы не все параметры запроса

• Магические даты

• Магические числа не должны использоваться

• Когнитивная сложность методов не должна быть слишком высокой

• Цикломатическая сложность методов не должна быть слишком высокой

• Соответствие имен процедур и функций соглашению о кодировании

• Все методы на управляемой форме должны иметь директиву компиляции

• Ограничение на количество строк Процедуры/Функции

• Ограничение на количество строк кода процедуры/функции

• Пропущено указание последнего параметра при вызове метода

• Вызов вложенного конструктора Структуры при вызове функций

• Чрезмерный уровень вложенности конструкций "Если

• Вложенные вызовы функций

• Стандартные области не должны быть вложенными в другие области

• Использование вложенного тернарного оператора

• Неразрешимые ссылки на объекты метаданных в элементах форм

• Несколько вложенных и последовательно идущих "Если"

• Вызов несуществующего метода объекта

• Неэкспортная процедура (функция) в разделах "ПрограммныйИнтерфейс" или "СлужебныйПрограммныйИнтерфейс"

• Сообщение содержит нерекомендуемое местоимение ("Вы

• Использование нестандартных имен областей верхнего уровня

• Использование тега "NOSONAR"

• Использование Асинх методов вне клиентского контекста

• Наличие метода, объявленного как обработчик при создании описания оповещения

• Директива компиляции и определение метода на одной строке

• Один оператор в одной строке

• Имя переменной не должно состоять из одного символа

• Событие "ПриОткрытии" не должно содержать код открытия других форм

• Перекрытие зарезервированных имен

• Для вызова НачатьТранзакцию отсутствует парный вызов ЗафиксироватьТранзакцию

• Для вызова НачатьТранзакцию отсутствует парный вызов ОтменитьТранзакцию

• Неверный синтаксис 1С

• Отсутствует использование пароля при работе с менеджером криптографии

• Разрываемые тексты запросов - Невозможно разобрать строку, которая выглядит как запрос

• Процедура не должна возвращать значения

• Экспортный метод в модуле с повторным использованием значений

• Знаки пунктуации должны отделяться пробелом

• Количество параметров процедуры/функции более 7

• Количество необязательных параметров процедуры/функции более 3

• Присвоение значения свойству доступному только для чтения

• Соответствие имен областей соглашению о кодировании

• Стандартные области не должны дублироваться

• Ограниченное использование оператора "Перейти"

• Нельзя возвращать изменяемые значения из методов модулей с повторным использованием возвращаемых значений

• Нельзя возвращать МенеджерВременныхТаблиц, Запрос, объекты базы данных из методов модулей с повторным использованием возвращаемых значений

• Возврат предопределенных элементов из методов модулей повторного использования

• Возврат значения примитивного типа из модуля повторного использования

• Функция должна заканчиваться возвратом

• Перезапись параметров метода, переданных по значению

• Потерянная/затираемая переменная

• Неверное использование метода БезопасныйРежим

• Чтение свойства доступного только для записи

• В начале процедуры обработки регламентного задания отсутствует вызов метода "ОбщегоНазначения.ПриНачалеВыполненияРегламентногоЗадания();"

• Присвоение переменной самой себе

• Сравнение выражения с самим собой

• Использование точки с запятой в конце метода

• Необязательные параметры процедуры/функции расположены перед обязательными

• Не задан префикс "Подключаемый" для методов модуля формы, подключаемых через "УстановитьДействие"

• Несколько директив компиляции

• Несколько серверных вызовов внутри клиентского метода

• Наличие пробела в начале комментария

• Ключевые слова должны быть отделены пробелом

• Код должен быть написан без опечаток

• В модуле должны быть определены стандартные области

• Некорректный порядок программных выражений

• В коде не должны содержаться e-mail адреса неизвестных доменов

• Конкатенация строк может быть заменена на СтрСоединить или СтрШаблон

• Строковые функции используются для вычисления константных выражений

• Использование унарного плюса в конкатенации строк

• Некорректное использование СтрШаблон

• При создании объекта типа Структура не рекомендуется передавать в конструктор более 3-х значений свойств

• Использование конструкторов элементов стиля у элементов управления

• Использование латиницы и кириллицы в одной строке

• Смешивание русских и английских символов в именах переменных и названиях методов

• Использование синхронных методов

• Использование синхронных методов в транзакции

• Использование функции "КаталогВременныхФайлов"

• Комментарий с тегом "TODO"

• Избыточное использование перечисления параметров

• Пустой блок обработки исключения

• Запрет использования пустых и мусорных комментариев

• Метод недоступный в веб-клиенте

• Безусловное прерывание/продолжение цикла

• Возможное использование неинициализированной переменной

• Имена переменных не должны начинаться с нижнего подчеркивания

• Процедуры и функции программного интерфейса должны иметь описание

• Использование несуществующих директив компиляции

• Неизвестная команда препроцессора

• Обращение к несуществующей роли

• Запрет на лишние пробелы

• Недостижимый код

• Неиспользуемый реквизит формы

• Неиспользуемый итератор цикла

• Неиспользуемая процедура/функция

• Неиспользуемый модуль

• Неиспользуемый параметр процедуры/функции

• Неиспользуемая переменная

• Тернарные операторы не рекомендуются к использованию

• Обращение к объектной модели при обходе выборки

• Использование имени переменной "Параметры" в методах управляемой формы

• Бессмысленный асинхронный метод

• Бесполезные скобки в выражениях

• Использование запросов внутри циклов

• Запрет пропуска обязательных параметров вызываемых методов

• Опасность использования операторов сбора системной информации системы

• Опасность использования информации о пользователях операционной системы

• Использование маски .

• Запрет на использование обработки исключений для преобразования значений к числу

• Взаимодействие с внешними ресурсами

• Использование внешних объектов без безопасного режима

• Использование модальных окон

• Использование не кроссплатформенных объектов

• Использование НЕ в названии переменной/функции

• Использование инструкций препроцессора в клиент-серверных общих модулях

• Использование зарезервированных имен

• Использование недоступных асинхронных методов

• Использование буквы "ё" в комментариях

• Использование буквы 'ё'

• Правила образования имен переменных

• Переменные/функции всегда используются в отрицании с НЕ

• Переменные должны содержать описание

• Проверка существования метаданных в коде

• Ожидание завершения фонового задания без указания таймаута

• Перенос параметров процедур, функций и методов

• Перенос арифметических выражений

• Перенос логических выражений

• Запись константы в модуле объекта

• Запись файлов в каталог "КаталогПрограммы()"

• Нерекомендуемое использование метода "ЗаписьЖурналаРегистрации"

• Неверное и бессмысленное арифметическое выражение

• Неверная передача параметров в конструкторы объектов 1С

• Неверные способы параметризации формы при открытии формы

• Неверная конкатенация строк

• Неправильное использование метода "ОтменитьТранзакцию"

• Неверная проверка на пустой результат выполнения запроса

• XPath-правило