gifts2017

Типичные ошибки, некоторые вопросы качества и эффективности работы при разработке в 1С

Опубликовал ivanov660 ivanov660 (ivanov660) в раздел Программирование - Практика программирования

В этой статье мы приведем набор типичных и часто встречающихся ошибок при разработке в 1С (скорее всего особенно актуально для начинающих программистов). Предложим набор советов и рекомендаций по улучшению качества кода и работы при использовании типового инструментария. Это первая часть из 24 пунктов. Бонусом к каждому пункту мы привели разъяснения и комментарии.

На основании определенного опыта и работы с различными проектами нам удалось собрать определнный список типичных ошибок, рекомендаций и советов при работе в программе 1С. Как ни странно подобные ошибки встречаются иногда не только у начинающих разработчиков и их результатах работы, но и в типовых конфигурациях. Есть ряд советов, которые вроде широко известны, но почему-то не используются.

Ниже мы приведем список пунктов с примерным категорированием для акцентирования внимания (ошибки, запросы, код, хранилище, советы):

1. Деление на 0. (ошибки, запрос, код)

Используйте проверку деления на ноль в запросах и в коде. Классическая ошибка №1.

2. Копипаст или Ctrl+C & Ctrl+V. (ошибки, код)

Мы считаем, что это одна из самых коварных ошибок, которую допускают практически все.

3. Ошибки отсутствия проверки на ЭтоГруппа(ошибки, запрос, код)

Отсутствии проверки на элеметы и группы. У элементов и групп возможно различное использование реквизитов. Т.е. если вы будете пытаться получить реквизиты папки, которые доступны для элемента, то скорее всего получите ошибку.

4. Определение переменных. (ошибки, код)

Т.к. в 1С нет необходимости определять переменные как в Си, C# и др., то разработчики порой допускают ошибку определяя или устанавливая значения переменной в ветвях условий. Могут возникнуть точки когда переменная будет не объявлена или содержать недопустимое значение. Пример ошибки:

Функция ПолучитьЧтоТо(Вариант,Количество)
  Если Вариант=0 Тогда
        Коэффициент = 1;
  ИначеЕсли Вариант=1 Тогда
        Коэффициент = 0;
  КонецЕсли;
  Возврат Коэффициент*Количество; // получим ошибку произведение значения Неопределено на количество, если значения варианта будут отличны от 1 или 0
КонецФункции

5. Проверка в условии на истину переменной со значением "Неопределено"(ошибки, код)

В случае, когда значение переменной может принимать значения "Неопределено" даже вполне неожиданно необходимо включать это условие в проверку. К примеру,

… Если НЕ Результат=Неопределено // перенос строки
И Результат Тогда … // правильно
ИЛИ
...Если Результат=Истина Тогда ... // правильно
… Если Результат Тогда ... // не правильно

 6. Формат строки(ошибки, код)

Не забывайте про формат строки, т.к. 1С число преобразует в строку с пробелом между разрядами.

7. Не используйте в названиях переменных имена реквизитов формы(ошибки, код)

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

8. Предположение наличия результата выполнения запроса или наличие в массиве/таблице хотябы одной записи(ошибки, код)

Иногда результат запроса может вернуть ноль значений и предположение наличия хотябы одной записи в таблице результата приводит к ошибке. Т.е. конструкция вида РезультатЗапросаТаблица[0] может привести к ошибке, используйте условия проверки РезультатЗапроса.Пустой() или Выборка.Следующий(). Пример:

...
Результат = Запрос.Выполнить();
Если НЕ Результат.Пустой() Тогда
   Выборка = Результат.Выбрать();
    ...
КонецЕсли;
....

9. Неправильное использование функции языка запросов "Выразить" - несовместимые типы(ошибки, запросы)

 Функция "Выразить" языка запросов 1С не может преобразовать несовместимые типы, т.е. если вы попробуете преобразовать, к примеру, строку в справочник, то получите ошибку. Данная ситуация может быть особенно коварной для составного типа данных, при передаче в запрос таблицы значений или простых параметров.

10. Используйте отборы внутри виртуальных таблиц(ошибки, запросы)

Используйте это условие, кроме случаев, когда иное требуется логикой

11. Запрос в цикле(ошибки, код, запросы)

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

12. Роли/Доступ(ошибки, код)

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

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

13. Недостаточно данных в тесте при разработке или проверке алгоритма. (ошибки)

Подобная ошибка встречается довольно часто. К примеру, при разработке запроса вы используете в качестве входных данных одну строку, то при наличии левого соединения в запросе вы можете пропустить "задваивание", "затраивание" строк в результирующей таблице. Или другой пример: почему-то не предполагаете что пользователь может методом копирования создать полностью зеркальную строчку.

14. Ограниченное мышление(ошибки)

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

15. Не изобретайте велосипед/"индусский код". (ошибки)

Кроме специализированных функций самой конфигурации, 1С в своих типовых конфигурациях использует БСП, в которй достаточно широкий набором функциональности. В результате появляется множество полуфункций и полупроцедур, работающих по части условий и не всегда; разобраться в итоговом коде получается очень сложно.

16. Получение данных через точку (объектную модель). (совет, код)

Не получайте данные через точку. Получение данных через объектную модель может достаточно сильно снизить производительность системы.

17. Использование директивы БезКонтекста(совет, код)

Используйте &БезКонтекстаНаСервере и &НаСервере в зависимости от необходимости работы с контектом приложения на сервере особенно в случае вызова с клиента и при большом количестве данных на форме. Пример, была создана обработка, в которой на форме находилась большая таблица данных, в этой обработке была использована кнопка для анализа данных, которая проходила по таблице на клиенте и в случае срабатывания условий отправляла запрос на сервер с контекстом, в результате длительность работы этой процедуры оказалось безмерно завышена из-за частой передачи большого объема данных: до рефакторинга процедура выполнялась 10-15 минут, после 2-3 минуты.

18. Хранение в реквизитах справочника, документа типа данных «Хранилище Значения»(совет, код)

Старайтесь не использовать в реквизитах справочников, документов типов данных «Хранилище значения» иначе при работе с объектной моделью вы получите необоснованное снижение производительности.

19. Используйте в конструкторе запросов конструктор запросов. (совет, запросы)

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

20. Используйте расширенный синтаксис оператора языка запросов "В".  (совет, запросы)

Часто требуется определить условие вхождения по нескольким реквизитам и это довольно просто реализуется. Пример:

… (Организация,Контрагент) В (ВЫБРАТЬ Т.Организация,Т.Контрагент ИЗ ВтТ как Т) …

21. Испольуйте предопределенные значения в запросах(совет, запросы)

К примеру, вместо создания параметра передающего пустую ссылку используйте следующий код

… НЕ Контрагент=Значение(Справочник.Контрагенты.ПустаяСсылка) …

22. Оптимальность запросов. (совет, запросы)

1. Испольуйте пакетные запросы и временные таблицы вместо вложенных запросов

2. Для объединения нескольких таблиц используйте объединить, а не левое соединение

3. Для отладки сложных запросов используйте дам запроса

4. Инедксируйте реквизиты/устанавливайте индекс для реквизитов, по которым используете отбор, поиск, соединение.

5. И другие рекомендации, которые можно найти в сети если поставить цель. 

23. Передача между клиентом и сервером таблицы значений. (совет)

На текущий момент между клиентом и сервером не возможна передача типа "Таблица значений", но если очень хочется, то ситуацию можно решить. Для этого на форме создайте реквизит с типом значений с определенным составом полей или определяемым динамически и используйте.

24. Не используйте повторяющийся код. (совет)

Завершающий совет, но очень злободневный. Любой повторяющийся код старайтесь выносить в отдельную функцию. Это позволит в дальнейшем существенно облегчить работу - читаемость, рефакторинг, исправление ошибок.

 

Продолжение ожидается...

См. также

Подписаться Добавить вознаграждение

Комментарии

1. борян петров (TODD22) 15.02.15 08:25
п.8 Какой то не очень удачный пример.
используйте условия проверки РезультатЗапроса.Пустой() или Выборка.Следующий().

С начало вроде как проверяется результат на пустоту. И если результат не пустой тогда уже делается Выборка.
2. Сергѣй Батанов (baton_pk) 15.02.15 11:59
… Если НЕ Результат=Неопределено ИЛИ Результат=Истина Тогда … // правильно

ААА! Можно мне развидеть это?!
Stim213; SeiOkami; Yashazz; +3 Ответить 1
3. Петр (peterxx) 15.02.15 12:11
IRL не все, и не так уж очевидно.
Например:
п. 11. Сами разработчики 1С зачастую забивают на это правило. Кроме того если речь идет об обработке, которая запускается раз в месяц, не имеет значение будет ли она выполняться 30 секунд или 10 минут.
п. 15. БСП тоже не панацея. Попытка разобраться во взаимосвязях между частями БСП, чтобы вклиниться туда, может вам стоить трех-пяти дневного квеста без гарантии результата.
п. 24. Самый цимес получается, когда алгоритмы обработки данных, обращающихся к одной и той же функции/процедуре начинают "расходиться" в процессе разработки/эксплуатации. Вот тогда начинают появляться "индусские" куски кода в процедурах "обработки всего и всея".
RainyAugust22; +1 Ответить 1
4. борян петров (TODD22) 15.02.15 13:06
(2) baton_pk,
ААА! Можно мне развидеть это?!

Предложите ваш вариант проверки если одно из возможных значений Неопределено, а другие например булево.
5. ivanov660 ivanov660 (ivanov660) 15.02.15 13:31
(4) TODD22, мы не старались описать все возможные варианты и привести решения, на самом деле показан один из примеров реализации. Для случая когда результат не только булево решение такое но более элегантное:
...
Если НЕ Результат=Неопределено // не забыть перенос на другую строку :-)))
 И Результат Тогда
...
КонецЕсли;
...
...Показать Скрыть
6. ivanov660 ivanov660 (ivanov660) 15.02.15 13:34
(1) TODD22, Можно привести совсем идеальный правильный вариант:
...
Результат = Запрос.Выполнить();
Если НЕ Результат.Пустой() Тогда
  Выборка = Результат.Выбрать();
  Пока Выборка.Следующий() Цикл
  ...
  КонецЦикла;
КонецЕсли;
...
...Показать Скрыть
7. ivanov660 ivanov660 (ivanov660) 15.02.15 13:40
(3) peterxx,
Сами разработчики 1С зачастую забивают на это правило.

Давайте стремиться к лучшему, а то получается какая-то "круговая порука".
БСП тоже не панацея. 

Никто не говорит, что она идеальна, но очень много функций уже в ней есть.
Самый цимес получается, когда алгоритмы обработки данных, обращающихся к одной и той же функции/процедуре начинают "расходиться" в процессе разработки/эксплуатации.

А вот тут надо стараться хотя бы разрабатывать функционал сверху вниз, а не снизу вверх, тогда избежать такой ситуации гораздо проще. Хотя не всегда подобное получается.
8. борян петров (TODD22) 15.02.15 13:53

(7) ivanov660,
Никто не говорит, что она идеальна, но очень много функций уже в ней есть.

Сталкивался пару раз с тем что внезапно менялись функции и приходилось устранять возникающие при этом ошибки. Это было в БСП.
Так что теперь стараюсь всё выносить максимально в свои модули.
9. Сергѣй Батанов (baton_pk) 15.02.15 14:06
(4) (5)

Зачем эти ДВА условия?!

Если Результат = Истина Тогда // Этого вполне достаточно
Если Результат = ДокументОснование Тогда // Зачем тут ещё одно сравнение с неопределено?


К тому же возмутился я тем, что условие отработает неверно в случае когда Результат=Ложь.
10. ivanov660 ivanov660 (ivanov660) 15.02.15 16:58
(9) baton_pk,
Согласен поспешил с ответом. Поправил комментарий, теперь условие отражает суть ответа )
11. Сергѣй Батанов (baton_pk) 15.02.15 20:14
(10) ivanov660,
Опять же, зачем
Если НЕ Результат=Неопределено // не забыть перенос на другую строку :-)))
 И Результат Тогда
...
КонецЕсли;
...Показать Скрыть

Если можно просто
Если Результат = Истина Тогда // ...
Gavrik; SirYozha; so-quest; Nuobu; fancy; spetzpozh; +6 1 Ответить 1
12. Sergei Disev (viramen) 15.02.15 20:38
6. Формат строки. (ошибки, код)

Не забывайте про формат строки, т.к. 1С число преобразует в строку с пробелом между разрядами.

Вид получаемой строки зависит от локализации решения. Если лень прочитать книги по разработке и проектированию решений - такие статьи не помогут.
Fox-trot; +1 Ответить
13. bonv (bonv) 16.02.15 07:44
5. В первую очередь надо избегать таких случаев. Либо метод возвращает булево, либо надо использовать объект (структуру) для возврата.

21. >
НЕ Контрагент=Значение(Справочник.Контрагенты.ПустаяСсылка)

В 1С есть знак "не равно".
Контрагент <> Значение(Справочник.Контрагенты.ПустаяСсылка)
14. Сергѣй Батанов (baton_pk) 16.02.15 08:07
(13) bonv,
для "не равно" надо раскладку переключать простым смертным (у кого нет снегопатов и шаблонов). так что "не .. = .." имеет право на существование :)
kao_andi; Zord; Fox-trot; AlexAuto; +4 Ответить 3
15. bonv (bonv) 16.02.15 10:55
(14) baton_pk, сильный аргумент :)
Но с другой стороны это же запрос, а в конструкторе мышкой "не равно" проще выбрать, чем вписать "НЕ .. = .." ;)
16. Сергей Семченков (BigClock) 16.02.15 11:14
Хотелось бы узнать, корректно ли использовать конструкцию вида
Если НЕ Результат=Неопределено

Ведь если Результат равен Неопределено, то выражение Не Результат при вычислении дает ошибку "{(1)}: Преобразование значения к типу Булево не может быть выполнено"
Аналогичную ошибку дает код
Стр = ТабЗнч.Найти("Значение", "Колонка");
Если Не Стр = Неопределено Тогда
    ...........
КонецЕсли
17. Сергѣй Батанов (baton_pk) 16.02.15 11:22
(16) BigClock,
А = Неопределено;
Если Не А = Неопределено Тогда
	
КонецЕсли;
...Показать Скрыть

Отрабатывает без ошибок (8.2.19.83).
В любом случае, лучше использовать скобки :)
18. Сергѣй Батанов (baton_pk) 16.02.15 11:23
(15) bonv,
а в конструкторе мышкой

Открою маленький секрет: далеко не всегда люди пользуются конструктором запросов. ;-)
19. Jey Rico (jeyrico) 16.02.15 11:23
(14) Чтобы не переключаться хорошо помогает это специальная раскладка клавиатуры с сайта Павла Чистова http://1c.chistov.pro/2012/11/1.html

Где-то, вроде даже, на сайте ИТС или Евгения Гилева, была статья о том, что с <> нужно пользоваться осторожно, и рекомендуют использовать именно конструкцию (Не ... = ...)
20. Сергѣй Батанов (baton_pk) 16.02.15 11:25
(19) jeyrico,
"<>" от лукавого. Не понимаю, почему 1С до сих пор не сделали "!=".
21. Jey Rico (jeyrico) 16.02.15 12:06
(20) Может потому, что это будет знатный холивар насчет было \ стало.
22. bonv (bonv) 16.02.15 12:24
(18) baton_pk,
Открою маленький секрет: далеко не всегда люди пользуются конструктором запросов. ;-)

Чорт, правда что ли :)

Ну а если серьезно, то исходить все же надо из того что код пишется для людей. И прямое условие читать все же проще.
Хотя мне тоже более привычен "!=".
23. friend0 16.02.15 19:00
Начиналось все хорошо, но последние пункты весьма спорные. Использование "В" и "Объединить" в запросах лично меня напрягают. Так же как "ИЛИ". В свое время оптимизировал тормозной запрос, пробовал разные варианты и в итоге соединение оказалось самым быстродейственным вариантом замены "ИЛИ".

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

В общем совет для запросов - смотреть профайлер и планы исполнения. Почитать книжки по тому как там все устроено, ужаснуться как все сложно и непредсказуемо, как один вариант увеличивает количество операций чтения, а другой загрузку процессора... Попаниковать, расслабиться и начать делать по принципу "А ладно, не тромозит и фиг с ним". Шутка. :)

По поводу временых таблиц я бы тоже уточнил, что они в любом случае пишутся (тормоза зависимые от объема данных), а будет ли выигрыш по сравнению с вложенным запросом - надо смотреть.
Я бы тут порекоммендовал следующие правила (общий шаблон поведения с возможными отклонениями на конкретике):
1. Необходимое уточнение: виртуальная таблица - это неявный вложенный запрос. Кроме прочего это значит, что в некоторых ситуациях лучше использовать не ее, а исходный регистр. "ОстаткиИОбороты" насколько помню преобразуются аж в три запроса к реальным данным.
2. Не использовать соединения с вложенным запросом.
3. Не запихивать во временную таблицу слишком много данных.
4. Для группировки данных не класть исходные данные во временную таблицу (если эти данные не понадобятся в другом запросе пакета) - вложенный запрос работает на ура.
5. Не выбирать и не таскать по пакету ненужные поля. Особенно неиндексированые. Если вытаскиваются только индексированные поля (особенно если только ссылка), то велик шанс, что индексы будут использованы и не потребуется обращения к таблицам данных. В противном случае сначала будут выбираться индексы, а потом по ним искаться данные в таблицах данных. А если выбираемых записей много, то движок может решить, что резона использовать индексы нет и сразу начнет шерстить данные (и будет прав).
6. Для построителя отчетов (и возможно СКД) финальный запрос надо страться делать максимально простым. Вплоть до того, что поместить результат во временную таблицу и в финальном запросе просто оттуда читать. Но это надо смотреть по месту и опять же п.3
24. Яков Коган (Yashazz) 16.02.15 19:06
Особенно позабавил пункт 14. Это всё равно, что написать: "включайте мозги, когда проектируете и кодите" )))
25. Константин Юрин (kostyaomsk) 16.02.15 20:48
Бесполезно про ошибки писать. Все равно только практика подскажет.
26. ivanov660 ivanov660 (ivanov660) 17.02.15 09:13
(25) kostyaomsk, лучше учиться на чужих ошибках чем на своих.
27. ivanov660 ivanov660 (ivanov660) 17.02.15 09:15
(24) Yashazz, как это не удивительно звучит, но по результатам работы с франчами, начинающими и даже некоторыми опытными спецами (возможно невнимательность или еще что-то) этот пункт достаточно актуален. Все приведенные тут примеры взяты из практики.
28. ivanov660 ivanov660 (ivanov660) 17.02.15 09:18
(20) baton_pk, в также можно задать вопрос: почему еще нет классов, наследования, инкапсуляции и др.? Думаю, что этого не будет и в 8.4, хотя эклипс вроде обещали.
29. Максим Кошарный (demrak) 18.02.15 12:13
(13) bonv, в какой-то момент знак <> начинает наоборот отвлекать, уже года как 3 использую в сравнениях только НЕ = и в запросах и в коде. Тренирует логическое мышление, как мне кажется.
30. Вадим Латышев (pro1c@inbox.ru) 18.02.15 15:31
Стараюсь придерживаться правильному подходу, принятому, но:
1. Видел правильный хорошо выверенный код, но не решающий задачу.
2. Видел "спагетти" кода, но работает как часы. Разобраться сложно но можно.
Так что все относительно.
Тут недавно показывали код некоторых модулей исходников Windows, так там используют даже GOTO!!!!
и ничего! спор шел по-поводу целесообразности выхода из цикла 'Break' или 'GoTo Метка'?
Так вот сошлись на том, что Break, что GoTo сути не меняет, но для некоторых понятнее куда переходим из цикла при прерывании, а не просто прерываем цикл (break) и следуем далее по ходу выполнения после тела цикла!
Как-то так...
Светлый ум; SeiOkami; 1coder; SirYozha; +4 Ответить
31. Марат Хафизов (Painted) 18.02.15 22:02
15. Не изобретайте велосипед
Где бы еще найти описание функций общих модулей. Бывает и нередко, что действительно изобретаю велосипед.
32. Сергей Гуков (SirYozha) 19.02.15 08:33
21. Испольуйте предопределенные значения в запросах. (совет, запросы)


С пунктом 21 не согласен.

Так как использование функций (ЗНАЧЕНИЕ() и т.п.) в запросе ведет к дополнительным преобразованиям на стороне СУБД. Соответственно, может увеличиться время исполнения запроса.
Сам экспериментировал с использованием предопределенных значений в запросе на больших объемах данных. Выигрыш в скорости выполнения дает использование параметров для передачи предопределенных значений в запрос.


p.s. кстати, буква "з" пропущена в слове ИспольЗуйте
33. ivanov660 ivanov660 (ivanov660) 19.02.15 08:45
(32) SirYozha,
1. Плюс от такого использования в запросах - более простая читаемость кода и уменьшение количества кода
2. Быстродействие зависит от платформы к платформе и оптимизатора 1С
34. ivanov660 ivanov660 (ivanov660) 19.02.15 08:54
(31) Painted, на ИТС есть документация библиотеки стандартных подсистем от фирмы 1С - описание использования набора универсальных функциональных подсистем, предназначенных для использования в прикладных решениях на платформе "1С:Предприятие" Стандартные библиотеки. Документация..
35. ivanov660 ivanov660 (ivanov660) 19.02.15 14:22
(23) friend0,
1. на самом деле подобные вещи про оптимизацию запросов собирались из разных источников: настольная книга 1с: эксперта по технологическим вопросам; в интернете - на текущем сайте, других формах; про особенности работы MS SQL оптимизацию на technet.microsoft.com; в некоторых случаях проводились практические замеры.
К примеру, индексировать временную таблицу при наличии 10 записей не эффективно.
2. Вопрос про использование временных таблиц в отличи от вложенных связан с созданием плана запросов. В случае временных таблиц и пакетного запроса выше вероятность гарантирования наличия оптимального и однотипного плана запросов, да и оптимизатор будет угадывать лучше. В случае с вложенными запросами на план будут влиять нагрузка сервера, количество данных, индексация и др.
3. Вопрос: про неиспользование виртуальных таблиц - при создании запроса по реальной таблице Вы сразу вспомнили и подумали про поле активность?
36. Андрей Оспичев (andrey3d) 26.02.15 12:06
(11) baton_pk,
Можно еще проще

Если Результат Тогда // ...
37. ivanov660 ivanov660 (ivanov660) 26.02.15 12:52
(36) andrey3d, если Вы про пункт 5, то нельзя (читайте внимательнее).
38. Денис Соломасов (Denis S) 28.02.15 15:29
(14) baton_pk,
Можно еще использовать команды:
< - ALT + 60;
> - ALT + 62.
Сам только так и делаю.
Еще пример:
& - ALT + 38
39. Сергѣй Батанов (baton_pk) 05.03.15 20:58
(38) Denis S,
раз уж рука дотянулась до альта, то лично мне проще дважды раскладку переключить: alt+shift [] alt+shift
цифрами я только | (alt+124) набираю и # (alt+35).

к тому же альт+... плохо работает при работе через TeamViewer, особенно под Убунту. а если ещё ноутбук без нумпада - это вообще в ад превращается :)
Для написания сообщения необходимо авторизоваться
Прикрепить файл
Дополнительные параметры ответа