gifts2017

Контур.EDI изнутри, или история командной разработки тиражного продукта на 1С

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

“Система - больше, чем просто совокупность ее элементов”; “Несколько разработчиков - это еще не команда разработки”. Расскажу, как мы прочувствовали эти очевидные утверждения на своей шкуре.

Исходные данные

Вкратце о EDI: это передача документов между организациями в виде структурированных файлов.  Подробнее написано здесь. Активнее всего в России EDI используется при взаимодействии проводольственных сетей (Магнит, Ашан, ...) и их поставщиков. Они обмениваются заказами, подтверждениями, счетами и прочими рабочими сообщениями для поставки товаров. Вот этим всем мы и занимаемся.

Наша команда разрабатывает модуль интеграции EDI для 1С. По ссылке доступна демо-база со встроенным модулем. Также есть короткое видео о том, как оно все работает. За термином “модуль интеграции” скрывается обычная внешняя обработка.

Изначально обработка должна была выполнять 4 действия:

  1. По существующим в системе документам формировать сообщения (файлы определенного формата).

  2. Отправлять эти сообщения по каким-то каналам связи.

  3. Получать по этим же каналам новые сообщения.

  4. Из принятых сообщений создавать новые документы в 1С.

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


Проблемы предметной области

Почти в каждом IT-проекте встречаются совершенно очевидные предположения, которые разбиваются о суровую реальность. Мне очень нравится пример с именами.

Чем же нас порадовал российский EDI?

  1. Покупатель может прислать заказ, а потом отменить его следующим сообщением.

  2. Поставщик в это время может уже успеть отправить машину с товаром.

  3. Покупатель может прислать один и тот же заказ 30 раз. Абсолютно одинаковый.

  4. Покупатель может прислать заказ с одним и тем же номером несколько раз с разными данными. В первом заказе 200 куриных тушек, во втором - 150. Сколько куриц убивать в итоге? А если 200 уже полегло, и тут прилетает заказ на 150?

  5. Уведомления о приемке товара очень часто НЕ СОДЕРЖАТ идентификатор отгрузки. Т.е. покупатель сообщает, что не досчитался 5 бутылок доброго виски при приемке машины, но не говорит, по какой накладной.

  6. Один покупатель может заказать 60 коробок вина по 20 бутылок в каждой. Другой может заказать 60 бутылок по 20 в коробке. При этом сообщения могут выглядеть одинаково. А у поставщика вообще учет в литрах. Хорошо, это не курицы, но сколько же вина привезти?

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

  8. Сообщений может быть ОЧЕНЬ много. По 300 заказов в день одному поставщику  от одной сети - не редкость. Обрабатывать их поштучно при таком объеме уже нереально. А теперь давайте добавим сюда предыдущий пункт с делением заказов.

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

  10. У покупателей постоянно меняются точки доставки и юр.лица. Например, торговая сеть X5 открывает/закрывает магазины несколько раз в неделю. Поставщик должен обладать актуальной информацией о том, куда везти товар. А модуль должен как-то предоставлять эти данные.

  11. Заполнить товары документа по сообщению очень просто? Получаем в сообщении код товара, находим по нему номенклатуру и закидываем в документ? Не торопитесь. Один и тот же товар покупателя (“Булочки в ассортименте”) может быть заведен у поставщика разными позициями (“Булочки с маком”, “Булочки с повидлом” и т.д.). Или наоборот.

И далее в том же духе.


Проблемы кода

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

За плечами у нас были франчи, ИТ-отделы компаний разных масштабов, фриланс. Каждый работал в одиночку, как это обычно и бывает с 1С-никами.

И понеслось.

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

Общее владение кодом падало.

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

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

В общем, мы столкнулись с тем, с чем ранее не имели дела: с разработкой тиражного продукта (и со всей ответственностью, вытекающей из этого), а также с необходимостью работы в команде.

Средства командной разработки от 1С нам не подошли. “Хранилище Конфигурации” решает другую задачу: оно в буквальном смысле хранит конфигурацию. И все. Ветвление и объединение веток, режим “blame” и возможность построчного комментирования кода отсутствуют. Да, в последних версиях 8.3 разработчики платформы дали возможность разложить конфигурации на исходники в виде текстовых файлов, которые уже можно хранить в современных системах контроля версий. И скомпилировать конфигурацию из этих исходников, разумеется. Нас эти возможности очень радуют, но воспользоваться мы ими не можем. Причина проста - нам приходится поддерживать версию для 8.2.

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


Как выживать?

Я не открою здесь Америку, многое давным-давно описано в учебниках ООП. Можно почитать  Фаулера. Если вы, как и я, не знаете Java, то пойдет эта статья. Но 1С8 - не совсем ООП, и мы долгое время писали по принципу “лишь бы работало” (и работало, конечно, до поры до времени).

На Инфостарте тоже лежит несколько статей с хорошими советами (пример, пример, еще пример), рекомендую к прочтению и осознанию. Опишу наш собственный опыт.


Технические решения

  1. Рефакторинг - наше всё

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

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

Например, встречаются такие нагромождения:

"Если ...

<170 строк чего-то>

ИначеЕсли...

<250 строк чего-то>

…(в том же духе)

КонецЕсли" (несколько экранов спустя).

И где-то там еще есть вложенные “Если”. Эти конструкции работают ровно до тех пор, пока в них не придется что-либо поменять (например, исправить ошибку, которая там наверняка встретится). Три-четыре уровня вложенности "Если" попахивают неприятностями. Пять-шесть уровней практически гарантируют баги. Больше всего времени при исправлении ошибок нами было убито именно на то, чтобы понять, да как же, черт возьми, попасть в эту ветку. Начали раскладывать такие нагромождения в линейные конструкции вроде

Если ДальшеДелатьНечего Тогда

Возврат

КонецЕсли;

И жить стало легче.

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

Рефакторинг проводим время от времени, без какого-либо плана, по настроению. А настроение обычно возникает на фоне необходимости добавления какой-либо фичи, затрагивающей запутанные участки кода.

  1. Чем меньше кода, тем лучше

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

Мы начали избавляться от функциональности, от которой в принципе можно было избавиться (имелась альтернатива). Это было довольно болезненно: взять и выбросить десятки тысяч строк кода, написанных когда-то совместными усилиями. Скрежет наших зубов раздавался по всему этажу. Самые значимые примеры:

  • Забросили развитие модуля для УФ, в который до этого вложили кучу сил, и стали вписывать управляемые формы в модуль для ОФ, сэкономив десятки тысяч строк кода.
  • Прекратили поддержку хранения данных в типовых объектах системы (справочник “Хранилище дополнительной информации” + регистр сведений “Значения свойств объектов” на конфигурациях вроде УТ). Сейчас все данные модуля (сообщения, настройки,...) хранятся непосредственно в текущей базе данных - для этого в базу добавляется 3 справочника и 3 регистра сведений. Альтернативой “Типовым объектам” стало хранение данных модуля “снаружи” текущей БД. Т.е. модуль разворачивает вспомогательную базу с почти такой же конфигурацией, и работает с ней через COM-соединение. За счет близкой структуры данных мы здорово сэкономили на коде и текстах запросов.
  • Написали с нуля новый интерфейс главного окна (и шесть раз переписали движок почти полностью). В последнем варианте удалось сделать построение списков документов максимально гибким, скоростным и универсальным. При этом кода стало меньше.

Кроме того, многие участки кода удалось вообще преобразовать в макеты. Это часть запросов, названия типов документов/справочников в разных типовых конфигурациях, списки настроек, XSD-схемы некоторых файлов и т.д.

Полтора года назад у нас было 2 обработки и 85 000 строк кода в сумме. Сейчас одна обработка и 70 000 строк кода. Разумеется, эти полтора года мы не только уничтожали код, но и писали много нового.

  1. Декомпозиция методов

Это тоже часть рефакторинга, но о ней я расскажу чуть подробнее.

Чем меньше разных действий выполняет метод, тем он читабельнее. Например, если в одной процедуре/функции встречается чтение файла с FTP и заполнение какого-либо документа, логичнее разбить его на два разных метода. Методу, который парсит XML, незачем знать, как устроена текущая конфигурация. Еще есть управляемые формы, где интерактивные действия недоступны на сервере.

Это аналог старых добрых классов в ООП и немного закос под mvc.

Мы стали делить длинные портянки кода на логические блоки, и средний размер одного метода упал с 40 до 30 строк кода. Стало гораздо проще объединять код разных веток перед релизом. Вероятность конфликтов свелась к минимуму, а их разрешение упростилось в разы.

Да, у нас еще остаются портянки строк на 300. Придет время - и их по возможности раскидаем.

  1. Автоматизированное тестирование

Мы внедрили 2 системы тестов: сценарные тесты и юнит-тесты на основе xUnitFor1c. Каждый коммит в системе контроля версий подхватывается скриптом и автоматически отправляется на тестирование в нескольких типовых конфигурациях 1С. Релиз не выпускаем до тех пор, пока все тесты не позеленеют, и пока тестировщики не скажут "ок", разумеется. (Да, у нас есть тестировщики, нет, они не программисты).

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

Второй не столько ловит ошибки (да, мы еще ленимся писать новые тесты, типичная проблема), сколько заставляет писать новый код с тем расчетом, чтобы его можно было прогнать через юнит-тесты. То есть его действие пока больше психологическое, но оно помогает более грамотно структурировать код. А значит, избегать появления ошибок еще на этапе разработки.

Стараемся придерживаться такого правила: если в релизной версии более 1-2 раз регистрируются ошибки, возникающие в одном и том же методе, то покрываем этот метод тестами. Чаще всего сначала его приходится потрошить и расчленять подвергать рефакторингу.

  1. Система контроля версий

Изначально использовали SVN с клиентом от Tortoise. Использовали его для блокировки предрелизной версии в момент заливки изменений и для хранения истории коммитов и релизов.

Затем на коленке собрали собственную систему контроля версий на 1С. Выглядит примерно так:

Король версий.png

Автотесты интегрированы с этой системой и зеленеют/краснеют прямо на коммитах. Столбец с зелеными кружками и заголовком “U” - юнит-тесты. Столбец правее с заголовком “F” - интеграционные тесты.

Иногда встречаются такие цепочки коммитов:


Сюда же встроен V8Reader с возможностью сравнить 2 произвольных коммита без открытия конфигуратора. Получается очень удобно.

Здесь же автоматизированы действия при выпуске релиза: формирование RSS, закрытие задач в трекере, расчет контрольной суммы файла и т.д.

Почитав статьи про современные системы контроля версий (раз, два), захотели перейти на связку Git+Precommit1c с компиляцией обработки из исходников. Увы, в платформе 8.x есть не очевидная особенность: при сохранении файла внешней обработки некоторые формы (не управляемые) хаотично перекодируются, выдавая совершенно другой файл, даже если вы просто пробел в коде добавили. Мы пытались как-то обмануть платформу, чтобы можно было объединять изменения разных веток и компилировать исходники обратно во внешнюю обработку. В конце концов пришлось смириться с реальностью и оставить эти попытки. Однако используем Git+Precommit1c для “ленивого” Code Review: каждый участник команды получает уведомления о новых коммитах и может оставить комментарии к любой строке изменившегося кода. В precommit1c внесли изменения:

  1. После декомпиляции удаляем файлы обычных форм, чтобы не засорять коммиты.

  2. Из управляемых форм вытаскиваем код отдельно.

  3. Каждый метод записывается отдельным файлом. Это сильно облегчает и Code Review, и просмотр в режиме Blame.

Кто не знает, режим Blame в Git отвечает на вопрос:

Кто это накодил.jpg

Но делает это более корректно:

blame.png

  1. Работа в нестандартных конфигурациях

Периодически возникавшая необходимость адаптировать модуль под нетиповую конфигурацию очередного крупного клиента заставила нас задуматься о каком-то инструменте, который позволит выполнять эту работу хотя бы с минимальными усилиями. Мы же программисты, самые ленивые люди в мире. Тем более что для 7.7 такой инструмент уже был готов: у нашего основного разработчика на этой платформе черный пояс по магии.

Изначально модуль 8.х разрабатывался на УТ 10.3. Весь код был пронизан конструкциями вида Заказ = Реализация.Сделка, например.

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

Был период, когда мы целенаправленно брали вне очереди задачи на настройку модуля в самописных конфигурациях и смотрели, в какие участки кода приходится вносить правки. Начинали с простого: с замены конструкций “Заказ = Реализация.Сделка” на функции-обертки: “Заказ = ПолучитьЗаказ (Реализация)”. В итоге получился довольно мощный инструмент, основная идея которого, надеюсь, будет понятна. Большую часть описания конфигураций удалось оформить декларативно (макеты, тексты запросов, без кода).

Получилось примерно так:

Шаблоны запросов.png

(Данные запросы разбираются на части и собираются заново в нужном виде. И не говорите, что СКД для этого не предназначена. Знаю, согласен.)


Для заполнения документов по сообщениям и наоборот хотим сделать двухсторонний маппинг полей “Сообщение - Документ”. Посмотрим, насколько это возможно, а пока пользуемся простой функцией-оберткой для доступа к реквизитам документов. Длина кода - 80 строк.

Эти вещи позволили нам легко выполнять 90% работы по кастомизации обработки.

После этого провели боевой тест на полностью самописной конфигурации, вот фрагмент метаданных для наглядности :

Снимок.PNG

Модуль взлетел быстро, большую часть изменений удалось вынести “сбоку”, что дает возможность с минимальной болью обновляться в будущем.

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

И еще один побочный эффект от проделанной работы: стало гораздо проще влить в модуль логику работы с 3 основными конфигурациями на управляемых формах.

Организационные решения

  1. Code Review

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

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

Пример “ленивого” Code Review:

git - holywar.png

  1. “Митинги” (Daily Scrum meeting)

Это  часть технологии “SCRUM”. Каждый день в 15:45 встаем возле доски и двигаем листочки с задачами, рассказывая друг другу, что сделали и что собираемся делать. Это здорово помогает быть в курсе того, куда вообще двигается продукт, и заодно позволяет избегать проблем с объединением кода.

  1. Циклы разработки

Изначально мы выпускали релизы по такой схеме:

  • Добавили новую фичу

  • Отправили в продакшн.

Сейчас действуем так:

  • Набираем задач на ближайшую неделю
  • Разрабатываем и тестируем в своих ветках
  • Проводим Code Review всех крупных изменений
  • Сливаем в предрелизную ветку, она же версия для тестирования
  • Тестировщики говорят свое "фи", исправляем баги
  • Когда тестировщики говорят "ок", готовим описание релиза для rss, выпускаем релиз (обычно это 2-3 недели от начала итерации, бывает и больше).
  • Критичные баги в случае их появления лечим в промежуточных релизах, дублируя фикс в предрелизную ветку.

Получается около 3-4 недель на итерацию. Если перед релизом есть ощущение, что что-то не так, значит, что-то действительно не так. Откладываем релиз и ищем причины неприятных ощущений, интуиция иногда спасает.

  1. Работа с багами.

Представьте, что перед вами большая очередь задач на разработку, вы увлеченно кодите, и тут к вам прибегают 2 пользователя. У одного при открытии какого-то документа аварийно завершается работа системы. У второго - в каком-то окне вылетает непонятное сообщение об ошибке, которое можно закрыть и спокойно продолжить работу. Когда вы приступите к исправлению первого бага? А второго?

Критичные/блокирующие баги исправляются везде и всегда в первую очередь. А мелкие, позволяющие продолжить работу, обычно копятся. Психологически очень тяжело заставить себя потратить час на исправление одной ошибки из известных 46, чтобы их осталось 45.

Однажды мы ввели “день багов”. Это один день в неделю, когда все, преодолев собственное сопротивление, откладывают в сторону все новые фичи и занимаются исключительно исправлением таких вот мелких ошибок. Сейчас их осталось 13, это самые редкие или неуловимые.

  1. Подбор задач из очереди

Были времена, когда мы лихорадочно пытались закрыть как можно больше задач за минимальное время — сказывался опыт работы во франчах и ИТ отделах компаний.

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

При работе фрилансом я сталкивался с тем, что клиент настаивал на придуманном им неоптимальном способе решения задачи. Если не удавалось его переубедить, в действие вступал простой алгоритм:

  1. Делаю, что попросили.

  2. Получаю оплату.

  3. Клиент просит "сделать обратно". Делаю.

  4. Получаю оплату.

Так вот, в тиражном продукте такой способ не подходит. Поток входящих задач всегда гораздо больше, чем способна переварить разработка. Со временем очередь задач выросла настолько, что нам пришлось привыкнуть к мысли: многие из этих хотелок мы будем делать только тогда, когда более важные закончатся. То есть никогда. Совсем. Было очень грустно это понимать, поскольку за каждой задачей обычно стоит какой-то клиент, которому без нее, возможно, будет плохо. Такова обратная сторона “массовости” продукта.

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

  • Потенциальная польза от выполнения задачи.

  • Соответствие ее общей стратегии развития.

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

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

Это дало интересный эффект: некоторые задачи мы стали выполнять еще до того, как они к нам поступали от отдела внедрения или от клиентов. Забавно на вопрос “ребята, нужна такая-то возможность, ее долго делать?” отвечать “мы 15 минут назад ее в релиз включили, скачивай”.


Что сейчас?

Не скажу, что год-два назад мы прямо ощущали какой-то большой дискомфорт. Нет, мы занимались разработкой в привычном для 1С-ника режиме (ну, вы понимаете, в каком именно). Более того, казалось, что по-другому и не бывает. Это как с курением: пока куришь, не понимаешь что что-то не так. Понимаешь только тогда, когда уже бросил.

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


См. также

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

Комментарии

1. Денис Павлов (pavlov_dv) 17.11.15 06:23
Достаточно интересная статейка получилась!

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

Кстати, настолько плавно и органично в статье осуществлен переход от темы "EDI" к теме про "командную работу" - читабельность поразительная! :-)

Спасибо!
kstukov; kiros; dj_serega; kuzyara; pt_olga; +5 Ответить 1
2. Артур Аюханов (artbear) 17.11.15 09:46
Интересная статья, спасибо!

Меня, как одного из мейнтейнеров продуктов precommit1c и V8Reader, заинтересовали доработки к нашим продуктам.
Вопросы
В precommit1c внесли изменения:

После декомпиляции удаляем файлы обычных форм, чтобы не засорять коммиты.

Из управляемых форм вытаскиваем код отдельно.

Каждый метод записывается отдельным файлом. Это сильно облегчает и Code Review, и просмотр в режиме Blame.

Можно получить эти доработки для основного проекта precommit1c?
Например, сделать pull-request ?

Мы пытались как-то обмануть платформу, чтобы можно было объединять изменения разных веток и компилировать исходники обратно во внешнюю обработку.

И по V8Reader можно ли также получить pull-request?
aduyunov_2015; kuntashov; Evil Beaver; +3 Ответить 1
3. Данила Володькин (skif47) 17.11.15 09:58
(2) artbear, Pull request на самом деле уже есть, мы списывались несколько месяцев назад:
https://github.com/xDrivenDevelopment/v8Reader/pull/30
Но ту идею я в конце концов забросил.
Сам V8Reader мы почти не меняли, только убрали в каком-то месте переименование макетов и добавили вызов нашего обработчика, который проводит дальнейшее разбиение разобранного модуля. Лежит здесь: https://github.com/volodkindv/v8Reader/tree/develop (скорее всего, устарело, давно не обновлял). Могу отправить на почту то и другое.
И спасибо за эти разработки, нам очень помогло!
4. Артур Аюханов (artbear) 17.11.15 10:02
(3) Было бы удобно получить последние версии с описанием доработок.
Можно и на почту.
Жду
спасибо за эти разработки

Всегда пожалуйста.

Предлагай свое, я вижу, что также интересные вещи делаете.
5. Вадим Миляев (PrinzOfMunchen) 17.11.15 10:03
Приходилось часто работать с этой штукой...Столько приходилось дописывать/переписывать, столько там "хорошего" кода, с кучей комментариев разных разработчиков, которые из одних косяков делают другие...Но в целом продукт годный и крайне полезный. Надеюсь избавитесь от мелких недочетов, если уже не избавились. ))
6. Андрей Овсянкин (Evil Beaver) 17.11.15 10:18
(5) PrinzOfMunchen, речь про xUnit, если я правильно понял? Есть статья про эволюцию этого инструмента, поясняющая почему там такой код и версия 4.0, в которой legacy-код из прототипа был удален.

Ну и сами же пишете:
в целом продукт годный и крайне полезный

А внутри может быть и страшно, но это называется инкапсуляция :)
7. Вадим Миляев (PrinzOfMunchen) 17.11.15 10:47
(6) Evil Beaver, нет, я про модуль интеграции EDI<->1С. Там внутри был ад перфекциониста, по крайней мере раньше. Как с текущими версиями дела обстоят, к сожалению не знаю. )
В целом-то продукт хороший, но когда надо заглядывать внутрь...
8. Большаков Павел Сергеевич (bolshakov_2015) 17.11.15 11:02
(7) PrinzOfMunchen, а как давно заглядывали?
9. Евгений Сосна (pumbaE) 17.11.15 11:10
(0) тесты прогоняете на всех версиях различных видов конфигурации? Отдельные виртуалочки для 8.3, 8.2 ? На скольких версиях 1с запускаете тесты?
10. Вадим Миляев (PrinzOfMunchen) 17.11.15 11:10
(8) bolshakov_2015, довольно таки давно, если честно. Где-то с полгода назад.
11. Данила Володькин (skif47) 17.11.15 11:22
(5) PrinzOfMunchen, Да, у нас там в комментариях иногда шекспировские страсти разыгрываются ))
Избавляемся постепенно.
Интересно было бы узнать подробнее, что вам приходилось переписывать/исправлять. Возможно, что-нибудь в коробку возьмем. Ну и если есть какие-то баги, о которых мы еще не знаем, то тоже будем работать. Если скинете вашу версию модуля, проанализируем.
12. Данила Володькин (skif47) 17.11.15 11:34
(9) pumbaE,

Для юнит-тестов список баз такой:

БП 1.6
БП 2.0
КА 1.1
УПП
УТ 10.2
УТ 10_3 (MS SQL и DB2 на всякий случай)
УТ 11.0
УТ 11.1

Платформы - 8.2.19.106 для УТ11 и 8.2.16.362 для остального.

Для сценарных:

БП 1.6
КА 1.1
УПП 1.3
УТ 10.2
УТ 11.1
БП 2.0 (в 2 режимах - "справочники добавлены в базу" и "справочники во внешней базе")
УТ 10.3 - так же, как БП 2.0.

Платформы те же.

Т.е. покрытие не полное. Автотесты пока не запускаем на конфигурациях с дополнительной защитой (Астор, Рарус и т.д.), на БП 3.0, ERP 2.0/2.1, платформу 8.3 тоже пока не задействовали. Различные релизы тех же БП 2.0 не трогаем. Это все в планах.
13. Вадим Миляев (PrinzOfMunchen) 17.11.15 12:00
(11) skif47, как у многих ))
То что избавляетесь, крайне радует. Судя по статье идеал не за горами. )))
Переписывал пока во франче работал, для разных конфигураций под разные нужды. Где-то адреса доставки некорректно заполнялись, где-то с единицами мест беды были при пересчете, сейчас все не упомню. Попробую поискать дописки. Не из багов, дописывал параметризацию проставления некоторых реквизитов, например складов. Что бы для каждого контрагента свои склады проставлялись.
14. Maxim Maleshov (maleshov.maxim) 17.11.15 12:08
Можно узнать поподробнее про собственный инструмент контроля версий на 1С?
hohaspb; aduyunov_2015; +2 Ответить 1
15. Евгений Сосна (pumbaE) 17.11.15 12:11
(14) maleshov.maxim, это больше не инструмент контроля версий, а инструмент для запуска тестов, т.к. в дальнейшем скрины показаны стандартного gitlab.
16. Данила Володькин (skif47) 17.11.15 12:23
(1) pavlov_dv, спасибо, статью тоже рефакторили )

У вас EDI в какой отрасли? Продуктовый ритейл или что-то другое?
17. Данила Володькин (skif47) 17.11.15 14:00
(14) maleshov.maxim, (15) pumbaE, это смесь СКВ и автотестов. Отдельная база на 1С, которая хранит все "дерево разработки" и версии модулей.
Основной сценарий такой: разработчик хочет внести изменения в модуль. Для этого он создает отдельную ветку. На его машине в определенной папке появляется файл с обработкой, которую он может править. Время от времени он "коммитит" - файл обработки сохраняется в базе и становится виден всем, при этом запускаются автоматические тесты. После окончания работ он блокирует предрелизную ветку. На его машине появляется второй файл с предрелизной версией модуля. Затем он вручную переносит свои изменения и сливает ветки. В git публикуются его изменения, снова запускаются автотесты, на машинах тестировщиков автоматически обновляется модуль для тестирования. В нее же встроена автоматизация выпуска релиза: обновление архива модулей, расчет контрольной суммы файла, обновление RSS и прочее.
В эту же базу ради интереса встроили интеграцию с YouTrack, чтобы работать с задачами в одном месте и привязывать их к коммитам.
Git и его веб-интерфейс Gitlab используем для Code Review, там очень удобно комментировать. Храним там только тексты модулей.
18. Алексей Лустин (lustin) 17.11.15 23:00
(0) один вопрос меня волнует - кто этот волшебный маг семерочник ;-). Можете озвучить никнейм - чувствую я, что никнейм будет известным.
19. Алексей Лустин (lustin) 17.11.15 23:05
(0) (18) имеется ввиду вот эта цитата

Тем более что для 7.7 такой инструмент уже был готов: у нашего основного разработчика на этой платформе черный пояс по магии


очень хочется узнать обладателя черного пояса ;-).
Evil Beaver; artbear; +2 Ответить 1
20. Владимир Поздняков (Red_Devil) 18.11.15 08:04
Пользуюсь вашей обработкой, молодцы! За подключаемый модуль отдельный +
21. Денис Павлов (pavlov_dv) 18.11.15 08:58
(16) skif47, торговля электронными компонентами, автоматикой, КИП и т.д.

Используем EDIFACT... Хотя слово "используем" -это за уши притянуто :-)
Кастомизация практически под каждого конкретного поставщика. От стандарта остается только заголовок :-)

Зато зарубежные (особенно стоки) с удовольствием соглашаются, но с ними тяжеловато работать - этапов тестирования не будет :-) Тренируемся на кошках на маленьких посылках.
Отечественные же при слове "Едиай" смотрят мутными глазами, переспрашивают "А шо это?" и отказываются: "Ышшо деды работали безо всяких буржуйских извращений, и мы не будем".
22. Данила Володькин (skif47) 18.11.15 09:48
(18) lustin, (19) lustin, Матвей Новоселов, он сказал, что вряд ли его кто знает )
23. Данила Володькин (skif47) 18.11.15 09:57
(20) Red_Devil, Спасибо, приятно слышать.
24. qazaz2 (qazaz2) 18.11.15 11:24
Спасибо автору, интересно и даже как-то мотивирует на хорошую работу)
25. Убийца Джедаев (vakham) 18.11.15 12:03
Притча.
Лет десять назад настроили с магазинами электронные заявки. Быстро, дёшево и сердито.
Спустя годы появились EDI-провайдеры с откровенно глючным сырьём и невменяемыми ценами за воздух. Посему все "выгодные мля буду" идеи были посланы в топку.
И вдруг... Буквально пару лет назад все крупные сети как с цепи сорвались. В ультимативном порядке сети потребовали всех подключить к своему EDI-провайдеру, которые разгубастились трясти деньги за каждый чих. Ниточка потянулась в Вашингтон. Погуглив "ху из ху", а так же "кто такие", "чё такие борзые", "с какого району", выяснилось, что по крайней мере один из "независимых" EDI-провайдеров был очень даже зависим от сети, которая требовала заявок через этот провайдер.
Итого: крупные торговые сети не только навели порядок со своими поставщиками и клиентами (что само по себе не плохо), но и хорошо нагрели руки, продавая воздух. Суки. Из-за них сломали что итак отлично работало, так ещё и деньги платим хер знает за что.
D-masterrr; terkin; vawd; sevushka; +4 Ответить 1
26. Иван Титов (Ibrogim) 18.11.15 13:15
(0) Статья Супер.
Очень интересует это
Данные запросы разбираются на части и собираются заново в нужном виде. И не говорите, что СКД для этого не предназначена. Знаю, согласен.

и возможность сделать так
часть изменений удалось вынести “сбоку”

Был бы признателен примерам этих собираний разбираний
27. Evgeny Sergeev (sea123) 18.11.15 13:25
(8) bolshakov_2015, заглядывали буквально только что
комментарии в стиле
Если Стр.СуммаНДС=0 Тогда //самый важный блок вроде бы
Если Стр.СуммаБезНДС<>0 Тогда
Стр.СуммаНДС=Окр(Стр.СуммаБезНДС*КоэффНДС,2);
ИначеЕсли Стр.СуммаСНДС<>0 Тогда
Стр.СуммаНДС=Окр(Стр.СуммаСНДС*КоэффНДС/(1+КоэффНДС),2);
ИначеЕсли Стр.ЦенаСНДС<>0 Тогда
Стр.СуммаНДС=Окр(Окр(Стр.ЦенаСНДС*Стр.Количество,2)*КоэффНДС/(1+КоэффНДС),2);//адовы округления детектед
ИначеЕсли Стр.ЦенаБезНДС<>0 Тогда
Стр.СуммаНДС=Окр(Окр(Стр.ЦенаБезНДС*Стр.Количество,2)*КоэффНДС,2);//адовы округления детектед
КонецЕсли;
КонецЕсли;

это нечто, визуально понижающее уровень программистов и показывающий их мощную работу мысли.
Ну и вот в этих самых краях, начиная с 10 ноября и кроется мощный косяк "адовых округлений"
Нам в компанию приходят заказы с ценами без НДС. Ну вот скажите, какая легкая наркомания заставила вашу команду сначала посчитать цену с НДС, ОКРУГЛИТЬ ее до копеек и потом считать НДС?

28. Alex (Crazy_kz) 18.11.15 13:44
Я не гуру 1С, с нового года перешли на эту систему, кучу времени потратил что бы вставить свои костыли, потому что просто не работает что то или работает но криво.
29. Данила Володькин (skif47) 18.11.15 13:51
(26) Ibrogim,
Если вкратце, то за основу связей между документами и типами сообщений мы берем запрос такого вида:
ВЫБРАТЬ
	докЗаказПокупателя.Ссылка КАК PORDERS,
	докЗаказПокупателя.Ссылка КАК ORDERS,
	докЗаказПокупателя.Ссылка КАК ORDRSP,
	докРеализацияТоваровУслуг.Ссылка КАК DESADV,
	докРеализацияТоваровУслуг.Ссылка КАК RECADV,
	СчетФактураВыданныйДокументыОснования.Ссылка КАК INVOIC
ИЗ
	(ВЫБРАТЬ
		1 КАК Поле1) КАК Заглушка
		{ПОЛНОЕ СОЕДИНЕНИЕ Документ.СчетФактураВыданный.ДокументыОснования КАК СчетФактураВыданныйДокументыОснования
			{ПОЛНОЕ СОЕДИНЕНИЕ Документ.РеализацияТоваровУслуг КАК докРеализацияТоваровУслуг
				{ПОЛНОЕ СОЕДИНЕНИЕ Документ.ЗаказПокупателя КАК докЗаказПокупателя
				ПО докРеализацияТоваровУслуг.Сделка = докЗаказПокупателя.Ссылка}
			ПО СчетФактураВыданныйДокументыОснования.ДокументОснование = докРеализацияТоваровУслуг.Ссылка}
		ПО (ИСТИНА)}
...Показать Скрыть

Под каждую конфигурацию и каждую цепочку документов свой.
Дальше функция ПолучитьТаблицуДляСборкиЗапросов() парсит данный запрос в ТаблицуЗначений. Затем для каждого конкретного случая функция ПолучитьТекстЗапросаДляТаблицы() преобразует эту таблицу в нужный запрос. Например, нам надо получить все Реализации, по которым не созданы Счета-Фактуры. Понятно, что надо взять таблицу РеализацияТоваровУслуг и сделать левое соединение со счетами-фактурами.
С реквизитами шапки документов немного сложнее, мы пока предполагаем, что там всегда есть Организация, Контрагент, СуммаДокумента и т.д. Случаи, когда это не выполняется, приходится уже кастомить вручную, но это уже мелочи на фоне всего остального, что приходилось делать раньше.
Наборы этих запросов как раз и лежат в макете СКД. Там их удобнее структурировать.

Еще есть так называемый "Подключаемый модуль". Это обработка, которая лежит рядом с основной и содержит в себе код, дополняющий или переопределяющий некоторые методы из основного модуля. Грубо говоря, в основном модуле лежит некая функция ЗаполнитьЗаказПокупателя(Сообщение,ОбъектДокумента), которая в инициализированном ДокументОбъект.ЗаказПокупателя заполняет Организацию, Контрагента, Товары и т.д. Она рассчитана на известные типовые конфигурации, но у клиента есть какое-то нестандартное поле в этом документе, и его надо тоже заполнить. Чтобы не добавлять код в основном модуле и не перетаскивать его потом в новые релизы, мы сделали вызов экспортного метода "ПослеЗаполненияЗаказаПокупателя" в дополнительной обработке ("подключаемый модуль"). Туда передаются те же самые параметры (Сообщение,ОбъектДокумента), и там уже можно писать произвольный код.

Это довольно долго рассказывать, проще будет в коде глянуть, вверху есть ссылка на демо-базу с модулем.
30. Иван Титов (Ibrogim) 18.11.15 14:04
(29) skif47,
вверху есть ссылка на демо-базу с модулем.

У меня похоже и модуль ваш есть, его купили для сильно переписанной УТ11. без моего ведома Всё ещё не взлетело, скоро продолжу попытки подружить с нашей конфой

мы сделали вызов экспортного метода "ПослеЗаполненияЗаказаПокупателя"
Чтобы не добавлять код в основном модуле и не перетаскивать его потом в новые релизы

Я подобную штуку использую и в своих разработках.
31. Данила Володькин (skif47) 18.11.15 14:28
(27) sea123, раз уж вы добрались до этого кода, то вы прочитали также и комментарий в начале метода, где этот код расположен.
Изначально я описывал этот случай в разделе "Проблемы предметной области", но потом решил, что как-то много получается.
В общем случае сеть может "забыть" указать любые поля в сообщении, в любой комбинации. В сообщении есть 6 числовых полей: цена с/без НДС, сумма с/без НДС, сумма НДС, ставка НДС. Теперь вычеркните любые из них в случайном порядке и попробуйте придумать алгоритм, который рассчитает недостающие.
Мы пытаемся их рассчитать за торговую сеть хоть как-то. Мне пока известен только один случай, когда этот алгоритм кому-то не подходил. Там, насколько я помню, мы сделали небольшой чит и использовали событие подключаемого модуля "ПринятьВходящееСообщение" не совсем по назначению, зато сработало на ура.
Конкретно по вашему случаю мне хотелось бы больше деталей. Вы можете обратиться к вашему внедренцу или на техподдержку, чтобы они зарегистрировали этот инцидент. Он рано или поздно попадет к нам. Или можете отправить мне в ЛС ссылку на подобное сообщение в мониторинге и скриншоты загруженного документа - как загрузил модуль и как должно быть. Постараемся смоделировать это у себя, если это действительно общий случай, то включим в коробку. Если он возникает только у вас, попробуем решить подключаемым модулем.
32. Данила Володькин (skif47) 18.11.15 14:29
(27) sea123, этому коду, кстати, года полтора примерно.
33. Данила Володькин (skif47) 18.11.15 14:35
(28) Crazy_kz, было бы интересно узнать, что именно не работало и с чем возникали сложности. Сможете выслать получившийся у вас модуль с доработками на почту?
Как раз в начале этого года мы начали работать над облегчением кастомизации модуля. Брали реальные примеры кастомов и искали общие места, которые чаще всего приходится костылить. Возможно, в вашем варианте есть что-то, чего мы еще не учли.
Смотрю историю версий, у вас, скорее всего, 4.02.001.02 или где-то рядом?
34. Данила Володькин (skif47) 18.11.15 14:37
(30) Ibrogim, будет интересно посмотреть, что у вас в конечном итоге получится и какие сложности возникали. Возможно, что-то возьмем в коробку, облегчим обновление модуля вам и тем, у кого подобные кастомы.
35. Большаков Павел Сергеевич (bolshakov_2015) 18.11.15 14:54
(30) Ibrogim, УТ 11 - как много в этом звуке
Вот только их 3 разных версии и изменения (11.0-11.1-11.2)там отнюдь не косметические. И даже в пределах 11.0 реквизиты того же заказа от релиза к релизу волшебным образом исчезали и появлялись вновь. ))
Доля сильно кастомизированных УТ 11 сейчас в природе крайне мала (боятся все-таки 1С ники УФ всё ещё)

и да, очень хотелось бы услышать фидбек по возникавшим проблемам.
36. Evgeny Sergeev (sea123) 18.11.15 15:19
(31) skif47,
Заявку мы уже оставили.
теперь разберемся с проблемами от сетей
====
В общем случае сеть может "забыть" указать любые поля в сообщении, в любой комбинации. В сообщении есть 6 числовых полей: цена с/без НДС, сумма с/без НДС, сумма НДС, ставка НДС. Теперь вычеркните любые из них в случайном порядке и попробуйте придумать алгоритм, который рассчитает недостающие.
====
Мой взгляд со стороны (работаю в УПП 1.3):
1. поле НДС в принципе мало кому интересно и выбирается из карточки номенклатуры, т.к. продавец лучше знает, по какой ставке НДС он продает.
2. При наличии полей сумма с/без НДС тоже ничего сложного, суммы то известны, и вы обратным ходом считаете цены.
3. Поле СуммаНДС обязательно будет идти совместно с полями сумма с/без НДС и мы опять легко выходим на цену.
4. При условии, что указаны поля в п.2, общая сумма накладной и сумма НДС высчитывается элементарно.
5. При наличии поля цена с НДС тоже вопросов нет. Создаем документ заявки в флагом "сумма включает НДС" и спим спокойно.
6. Если пришли оба поля цен (с и без НДС), то приоритетное поле идет с НДС
7. пришла сумма без НДС. Это как раз наш случай.
Пример такой: цена продукта 28,95, количество 2 646 единиц, ставка НДС 18% (0,18)
Расчет суммы НДС делается на мой взгляд просто: ЦенаБезНДС*Количество*СтавкаНДС, получаем сумму 13 788,31 (вообще получается 13 788,306, но мы округляем по арифметическим правилам)
Сумму по строке нашли по формуле ЦенаБезНДС*Количество+СуммаНДС, получаем 90 390,01
Смотрим ваш метод:
а. Находим цену с НДС = 28.95*1,18=34,161
б. Округляем цену с НДС = 34,16
в. Находим сумму НДС, Стр.СуммаНДС=Окр(Окр(Стр.ЦенаСНДС*Стр.Количество,2)*КоэффНДС/(1+КоэффНДС),2);//адовы округления детектед
получаем 13 787,9023728814
Все, приехали, сумма НДС не совпала
D-masterrr; sevushka; +2 Ответить 1
37. С К (kraynev-navi) 18.11.15 17:50
Приятно читать то, что написано человеком, который имеет талант к творческому изображению довольно-таки сухих технических тем )) спасибо
А к диадоку (http://www.diadoc.ru/integrations/1c) вы отношение имеете как разработчик?
38. Данила Володькин (skif47) 18.11.15 22:38
(36) sea123, все не так однозначно. Ставка НДС далеко не всегда выбирается из карточки номенклатуры. У кого-то она вообще не заполнена в справочнике. У кого-то есть ИП на УСН и рядом же ООО на ОСНО. Порядок расчета цен у каждой сети свой и иногда даже прописывается в договоре поставки. Одни считают суммы от цен, другие - цены от суммы, кто-то включает НДС в сумму, кто-то нет.
По вашему примеру - кусок XML файла мне было бы удобнее проанализировать. В любом случае, на нашем трекере сегодня появился тикет с таким же содержанием, и, видимо, это как раз ваш. Так что будем с вами связываться.
39. Данила Володькин (skif47) 18.11.15 22:53
(37) kraynev-navi, спасибо )
Диадок - это другой проект, у него своя команда разработки, лично я в нее не вхожу. Но мы часто общаемся и обмениваемся идеями, а иногда и кодом (а иногда даже сотрудниками )) ). Например, для группового подписания счетов-фактур мы используем компоненту Диадока. Архитектура модуля изначально была позаимствована у них, и лишь потом мы выстроили ту, которая больше подходила именно под специфику EDI. Диадок тоже иногда перенимает кое-какие наработки из EDI. Мир очень тесен.
40. vawd (vawd) 19.11.15 08:12
Как поставщик, "Проблемы предметной области" - абсолютная истина! Конфигурация полностью кастомная, основана на древней БП 1.6, потому "Подключаемый модуль" ОЧЕНЬ помогает.
41. Всеволод Соковиков (sevushka) 19.11.15 08:37
Спасибо за статью, читать было интересно. Увидел некоторые не очевидные для меня (как для пользователя вашей системы) моменты, даже немного "посочувствовал" программистам.
Хотелось бы уточнить некоторые нюансы (не все по данной статье). 4.03.004.03, сильно переделанная УТ на УФ.

1. При небольшом количестве заявок - с системой работать можно. Когда сетей становится несколько, и число заявок за день от одной сети измеряется сотнями - Контур.EDI это очень медленная и неудобная система. В (25) правильно написали, что до ЕДИ все отлично работало, было вылизано и работало годами. Когда мои пользователи в первый раз увидели Контур.EDI, то единственное впечатление которое было "как же это все безумно медленно работает, давай вернемся к тому, что было". И эта точка зрения не изменилась за последние полгода, в обработку стараются заходить как можно реже (хотя надо признать, что тормозов стало меньше). Скажите, а переделку логики работы для большого количества заказов вообще планируют? Выделение цветом измененных и отмененных заявок? Информирование работников о том, что при автообмене что-то не сработало и не провелось? Информация о том, что текущая версия документа не совпадает с той, что была отправлена заказчику? Какие-то механизмы информирования заказчика, что "извините, но заявки данному поставщику должны быть сделаны до 20:00, и ваша заявка или не будет принята, или перенесется на следующий день" и т.п.
Я понимаю, что это часть из этого можно сделать (и делается) в подключаемом модуле, но... Каков вектор развития системы, к чему вы движетесь?
2. От ваших конкурентов (название не принципиально, но в силу некоторых причин пользуемся и ими) время от времени приходят письма, что "в нашей программе добавились новые фичи, текущий номер релиза хххх, скачать можете по прямой ссылке". Когда-нибудь у Контура это планируется?
3. Хочется получить рид-онли доступ на вашу вики (или что там у вас используется). Далеко не всегда очевидно, в какой момент какими функциями подключаемого модуля надо пользоваться. От внедренцев этого не добиться, помогает только отладчик. Ну или хотя бы что-то типа best practices.
4. Нахожу некие мелкие баги в программе. Пишу внедренцам. Получаю: "спасибо, Ваша информация передана разработчикам". Все. То, что некоторые из этих багов исправили, я узнаю только через несколько релизов, когда мне присылают новую версию. Хотелось бы обратную связь (пусть и не через внедренцев, а что-то типа otrs или любой другой тикет-системы), что ваш баг исправлен в релизе ххх, скачивайте и пользуйтесь на здоровье.
4а. Хочется видеть хотя бы примерный список работы на квартал. Т.е. - что именно планируют разработчики выпустить в ближайшее время. Если моя задача входит в этот список - я могу и подождать, если нет - я четко понимаю, что быстрее ее реализовать самостоятельно.
Раз уж у вас все так технологично сделано внутри - ну сделайте что-то полезное для сторонних программистов, которые внедряют ваши системы, тот же форум и вики, хотя бы.

И еще. Спасибо вашим внедренцам, отвечают быстро и по теме. Спасибо тому, кто реализовал подключаемый модуль, с ним реально удобно. Спасибо вашим продажникам (или не знаю кому), что у вас есть нормальные тарифы.
zumik3; skif47; +2 Ответить 9
42. Данила Володькин (skif47) 19.11.15 09:26
(41) sevushka, спасибо за подробный комментарий. Я надеялся как раз вот такую развернутую обратную связь получить.
Буду отвечать постепенно.

1. Логика работы с большим количеством заказов в интерфейсе была заложена изначально: Ctrl+A, "Выполнить действия". Большую часть действий рекомендуем выполнять именно в таком ключе, т.е. не заходя в каждый документ по отдельности.
Измененные заявки отражаются в статусе "Обработать новую версию", вам это разве не подходит? Если нет, то почему, как бы вы хотели это видеть?
Информирование работников о том, что при автообмене что-то не сработало и не провелось?
: что именно не сработало?
Не срабатывать могут в большинстве случаев 2 вещи:
1) Перебои со связью, например. В этом случае вы, разумеется, не увидите новых заказов. Как бы вы хотели видеть сообщения об этом? Автообмен у вас выполняется регламентным заданием? Если да, то какого пользователя информировать?
2) Не провелись автоматически некоторые заказы. Например, нет остатков, не заполнены какие-то поля, не заданы соответствия для новых товаров.
Все это вы увидите в основном интерфейсе. Мы стараемся показывать пользователю именно то, что ему надо сделать в текущий момент времени. Непроведенные заказы - да, показываем. Непринятые сообщения (нет соответствий, например) - да, показываем. Пока не понимаю, чего именно вам не хватает, напишите, пожалуйста, в ЛС поподробнее.

"Информация о том, что текущая версия документа не совпадает с той, что была отправлена заказчику"
- про какой документ и какое сообщение? Если речь, например, о DESADV, то это непростая задача. Т.е. для этого надо каким-то образом собрать табличные части всех реализаций (каких именно, за какой период? тоже непонятно). Затем собрать отправленные сообщения, которые с ними связаны. Сравнить то и другое, отобразить все, где есть расхождения. Понимаете, как это нагрузит сервер? При обновлении списка документов мы этого точно делать не станем. Нас же все остальные клиенты после такого поубивают ))
А заходить в каждый документ и смотреть "документ не совпадает с сообщением", судя по количеству заявок в вашей базе, вам вряд ли подойдет.
Но вопрос такой: а зачем вы меняете документы после отправки сообщений? Какими особенностями ваших процессов это вызвано? Возможно, мы придумаем для вас какой-то инструмент, опять же, нужно больше данных от вас. Ответьте в ЛС по возможности.

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

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

Каков вектор развития системы, к чему вы движетесь?
О, векторов много. Например, сейчас мы продумываем автоматическое сопоставление товаров между поставщиком и торговой сетью. Заканчиваем объединение логики работы обычных и управляемых форм. Подумываем сделать третий вариант хранения данных - SQLite, чтобы можно было ставить его и поставщикам на базовых поставках 1С. Перевод работы с сообщениями на XDTO. Визуальный маппинг полей EDI<->1C для сильно кастомных конфигураций. И много других. Вам что было бы интересно? (можете предложить свои варианты).

Ну и насчет медленной работы. Более 90% случаев медленной работы модуля связаны с обновлением списка задач пользователя. Посмотрите по замерам время выполнения метода "ПостроитьСписокЗадач_Сервер". Вы говорите, что у вас сильно кастомная конфигурация. Проверьте, индексируются ли у в ней поля связи между документами (Заказ и Реализация, Реализация и СФ). Попробуйте добавить индексы на поля "Контрагент" в этих документах. В большинстве случаев это дает дикий прирост скорости работы.
Да, и проверьте интервал отбора в интерфейсе. Понятно, что список сообщений/документов за полгода будет строиться дольше, чем за неделю.

Если это не поможет, пишите в ЛС, будем думать дальше.
43. Данила Володькин (skif47) 19.11.15 09:42
(41) sevushka,
2. От ваших конкурентов (название не принципиально, но в силу некоторых причин пользуемся и ими) время от времени приходят письма, что "в нашей программе добавились новые фичи, текущий номер релиза хххх, скачать можете по прямой ссылке". Когда-нибудь у Контура это планируется?


В интерфейсе для обычных форм в меню ("шестеренка") есть пункт "Новое в релизе". Туда можно иногда посматривать, важную информацию выкладываем там.
На УФ это еще не перетягивали, было полно других задач. Со временем и туда прикрутим.
Можем приделать RSS, вам это насколько удобно будет?
44. Данила Володькин (skif47) 19.11.15 09:54
(41) sevushka,

3. Хочется получить рид-онли доступ на вашу вики (или что там у вас используется). Далеко не всегда очевидно, в какой момент какими функциями подключаемого модуля надо пользоваться. От внедренцев этого не добиться, помогает только отладчик. Ну или хотя бы что-то типа best practices.


Доступ на вики - сильно сомневаюсь, если честно. Там очень много внутренней информации, не только по проекту EDI.
Года полтора назад мы делали ветку в одном из форумов Контура как раз под эти цели. Т.е. хотели накопить некую базу знаний и организовать общение клиентов друг с другом. Что-то вроде партнерского форума 1С. Как это часто бывает с форумами, он помер от голода, туда почти никто не заходил. Непонятно было, а кому он вообще интересен, кому о нем рассказать, как на нем регистрироваться и т.д. Я попробую поднять этот вопрос.
Лично у меня вообще есть давняя мечта - выложить исходники на GitHub, включить там же Вики, и вперед. Там же и форки делать. Возможно, когда-нибудь к этому придем.

Насчет подключаемого модуля. У нас когда-то были развернутые инструкции, примеры, как его писать и какими событиями пользоваться. Они уже безнадежно устарели. Мы пошли другим путем: в сам шаблон ПМ выкладываем примеры обработчиков событий, их можно раскомментировать и настроить под себя. Там есть примеры для большей части событий.
Там же есть простенькая возможность трассировки событий, посмотрите по коду, где-то в самом начале.
45. Данила Володькин (skif47) 19.11.15 10:07
(41) sevushka,

4. Нахожу некие мелкие баги в программе. Пишу внедренцам. Получаю: "спасибо, Ваша информация передана разработчикам". Все. То, что некоторые из этих багов исправили, я узнаю только через несколько релизов, когда мне присылают новую версию. Хотелось бы обратную связь (пусть и не через внедренцев, а что-то типа otrs или любой другой тикет-системы), что ваш баг исправлен в релизе ххх, скачивайте и пользуйтесь на здоровье.


В 4.03.004 мы добавляли прямо на интерфейсе ссылку "Отправить отзыв", вверху посередине. Письмо уходит в ящик, который мы периодически мониторим. Правда, пока почти никто не пишет.
OTRS добавит нам вторую точку входа инцидентов. Это будет уже довольно тяжело обрабатывать. Сейчас инциденты регистрируются через внедренцев и техподдержку, они многие вещи решают сами, что сильно разгружает нас для продуктовой разработки. Тут такое противоречие: если мы будем напрямую работать со всеми конечными пользователями, то коробка перестанет развиваться. А если совсем не будем контактировать, то коробка будет развиваться совсем не туда )) Мы пытаемся балансировать посередине.
Давайте пока что остановимся на ссылке "Отправить отзыв". Я не обещаю, что у нас будут ресурсы для того, чтобы отвечать всем и сразу. Но мы будем стараться. Насчет OTRS пока планов нет, возможно, организуем в будущем.
46. Данила Володькин (skif47) 19.11.15 10:15
(41) sevushka,

4а. Хочется видеть хотя бы примерный список работы на квартал. Т.е. - что именно планируют разработчики выпустить в ближайшее время. Если моя задача входит в этот список - я могу и подождать, если нет - я четко понимаю, что быстрее ее реализовать самостоятельно.
Раз уж у вас все так технологично сделано внутри - ну сделайте что-то полезное для сторонних программистов, которые внедряют ваши системы, тот же форум и вики, хотя бы.


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

Но вы запросто можете спрашивать у внедренцев по конкретным задачам, которые хотели бы видеть. Они обычно в курсе, если что, уточнят у нас.
47. Данила Володькин (skif47) 19.11.15 10:16
(41) sevushka,

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


Обязательно всем передам, спасибо за позитив!
48. Олег Филиппов (comol) 19.11.15 10:38
После декомпиляции удаляем файлы обычных форм, чтобы не засорять коммиты.

Могу отправить на почту то и другое


А можно мне тоже на почту? Я правильно понимаю что вы удаляете весь мусор который генерится для обычных форм и макетов?
49. Данила Володькин (skif47) 19.11.15 11:20
(48) comol, мы на самом деле мы добавили в .git\info\exclude наворотов вроде

*.orig
src/*/Form/*/form
*/managedForm.bareForm.txt
*.txt
src/*/und/*

и на этом успокоились ))

Скиньте почту в ЛС, отправлю сборку целиком.
50. Олег Филиппов (comol) 19.11.15 11:42
(49) skif47, О, может этого и хватит. спс. Но сборку закинь всё равно если не трудно... e-mail кинул в личку...
51. Большаков Павел Сергеевич (bolshakov_2015) 19.11.15 12:09
(42) skif47, (41) sevushka,
Проблема УФ в том, что перестроение списка задач действительно затратная штука. Нам надо ходить за результатами анализа сделок на сервер, т.к. оперативные изменения происходят там. При этом на сервере мы толком ничего не можем закэшировать - это все протухнет сразу как только мы уйдем оттуда (это модуль внешней обработки все-же, можно пытаться кэш как-то сохранять, мы думали над этим, но оказалось что он постоянно норовит испортиться + часто обновить кэш ненамного дольше чем достать его откуда-то ). Выливается это в действительно неприятное подлагивание у пользователя даже на не очень больших периодах анализа задач.
Мы над этим подумали и уже зарелизили крутую штуку: мы не будем обновлять весь список при воздействии только на 1 строку - мы её "зачеркиваем" - вот прямо берем и перечеркиваем пользователю, а список задач заново не обновляем. Его можно обновить 1 раз после обработки всех задач. Кажется что это как раз то, что Вам нужно! Механизм появился в окончательном виде с релиза 4.03.004.01.
Мы даже подумали что можно засекать время обновления и если оно слишком велико - сразу пользователю предлагать попробовать новый режим. Так и сделали. Таймер там сейчас стоит на 10 секунд, но видимо это много, верное значение должно быть в районе 2 сек.
В любом случае, вы можете зайти в настройки и воткнуть этот флажок самостоятельно. называется он "Обновлять данные в списке документов полностью".

С этой штукой даже у клиентов с очень большим количеством заказов (500+ в день) удалось достичь вполне приятного быстродействия.
52. Артур Аюханов (artbear) 19.11.15 13:45
(22)
Матвей Новоселов, он сказал, что вряд ли его кто знает )

Спасибо!
а какой ник он использует?
53. Артур Аюханов (artbear) 19.11.15 13:56
(49) skif47,
comol, мы на самом деле мы добавили в .git\info\exclude наворотов вроде

*.orig
src/*/Form/*/form
*/managedForm.bareForm.txt
*.txt
src/*/und/*

и на этом успокоились ))

это тот же файл, что я на почту получил, верно?
54. m n (nma) 19.11.15 15:03
(52) artbear, Здесь я под этим ником.
55. Данила Володькин (skif47) 19.11.15 15:05
57. m n (nma) 20.11.15 05:46
(56) lustin, Там я не зарегистрирован.
58. Всеволод Соковиков (sevushka) 20.11.15 05:51
(42) skif47, отвечаю в ЛС, здесь это вряд ли интересно, оффтопик же.
59. Иван Титов (Ibrogim) 26.11.15 13:08
(0)
сделайте переменную ИмяРеквизитаОрганизация глобальной, чтобы можно было в одном месте в ПМ ее инициализировать под себя
60. Данила Володькин (skif47) 26.11.15 14:26
(59) Ibrogim, мы сейчас разрабатываем инструмент для маппинга полей документов и сообщений. Эта переменная, скорей всего, будет уже не нужна. Потерпите немного.
Можете пока выслать получившийся у вас модуль (вместе с подключаемым) и файл конфигурации?
61. Иван Титов (Ibrogim) 26.11.15 15:09
(60) skif47,
Можете пока выслать получившийся у вас модуль (вместе с подключаемым) и файл конфигурации?

т.к. через подключаемый модуль у нас не получилось везде заменить организацию на наш реквизит (судя по всему это не возможно) то мы внесли изменения в сам модуль, а ПМ не пользовались. Конфигурация типовая УТ11.1.2, изменения затрагивающие взаимодействия с вашим модулем это только свой реквизит "организация" в документах.

Исправленный под нас ваш модуль можем выслать. Куда?
62. Данила Володькин (skif47) 26.11.15 15:36
63. Иван Петров (dgolovanov) 06.12.15 14:24
А к разработе Диадок вы имеете отношение?
64. Данила Володькин (skif47) 07.12.15 08:41
(63) dgolovanov, отвечал в (38). Разработкой модуля Диадок занимается другая команда. В данный момент мы подумываем объединить с ними усилия.
65. Иван Петров (dgolovanov) 07.12.15 14:50
(64) skif47, понял, спасибо. В Диадоке код совсем не дружелюбный для допиливания. Странно, что два продукта настолько отличаются.
66. Данила Володькин (skif47) 08.12.15 11:22
(65) Буквально сегодня, кстати, в модуль Диадока протянули подключаемый модуль, примерно как в EDI.
Можете написать в ЛС свое видение идеального модуля Диадока? Чего не хватает, что удобно/неудобно, что хотелось бы видеть, любые соображения.
Лично я этими вопросами не заведую, но ребятам передам, они как раз плотно взялись за облегчение кастомизации модуля.
67. Евгений Зикунов (jerokee) 14.01.16 11:51
Добротно все расписано. С 2010 года занимаемся чем-то подобным - сбор заявок от торговых с КПК и передача их дистрибьюторам для осуществления продаж на торговые точки. Но кроме этого в нашей системе учитываются еще складские остатки и движения "нашей" продукции и ДЗ точек перед дистрибьюторами. Также используем разделение интерфейса (как мы его называем) на шаблонную часть и специфику дистрибьютора. Но есть несколько отличий:
1. Поддерживаем платформы 7.7 (хватает еще и таких дистров) и 8.Х. Причем для 8-ки используется единая обработка для ОФ и УФ - при переходе на УФ нам ничего менять не надо (в теории, но могут быть нюансы)
2. Специфика дистрибьютора вынесена в отдельные обработки - слишком много "хотелок" по формам формирования документов + не всегда тривиальное получение данных о продажах, движениях, ценообразовани и ДЗ.
3. с 2015 года для обмена используется SFTP-сервер(а) - пришлось "наваять" свой транспорт на Питоне.
И также столкнулись со многими проблемами, как в предметной области, так и в вопросе реализации (а теперь и развитии) самого интерфейса.
68. Vasya M (zumik3) 15.01.16 09:43
(41) sevushka,
полностью согласен с 1 пунктом.

У нас ~600 заказов в день только от 1 сети. Обрабатывать их этой обработкой просто невозможно.
Решение: выпилил куски общего модуля, где идет обмен через API. Спокойно забираю заказы, сам распознаю, сам пишу в базу. также с DESADV'ами и RECADV'ами.
Работает быстро.
69. Данила Володькин (skif47) 15.01.16 13:37
(67) jerokee, у нас 7.7 тоже есть, ОФ и УФ тоже в уже едином модуле. Транспорт там же, на 1С (нам проще, поскольку нет необходимости лезть на SFTP). Вот вставок на Питоне нет ))
А как ваш проект называется? Интересно было бы посмотреть.
70. Данила Володькин (skif47) 15.01.16 13:46
(68) zumik3, в (41) с sevushka вопрос порешали. У нас есть клиенты с таким количеством заявок - и работают нормально. Есть торговые сети, у которых документов еще больше.
Хотелось бы подробностей от вас: в чем заключается невозможность? Скорость работы, недостаточно данных в интерфейсе, еще что-то?
Контакты есть в моем профиле, пишите, отвечу.
71. Евгений Зикунов (jerokee) 15.01.16 14:22
(69) skif47, Здесь долго и много описывать... В ЛС почему-то ничего написать Вам не могу.
72. Sergey A (serg_info) 18.05.16 20:46
А как же поставщики Алкоголя, не побоюсь этого слова с большой буквы :-)
К ним требования по EDI еще более изощренные, причем у каждой сети свои.
А в их конфигурациях информация по алкореквизиитам и лицензиям у кого где хранится.....
Что-то ничего не написано про DESADVALCO - он у вас не поддерживается?

Я так понял у Вас все в одной системе и EDI и ЮЗД (юридически значимые документы) документы?
Подписывается сразу все из обработки для ЮЗД документов или лезть на сайт?
Обработка квитанций по счетам-фактурам, аннулирование, счета-фактуры исправительные, корректировочные?

Все что отправляется через обработку видно и в WEB интерфейсе?
Единый сайт для ЕДИ и ЭДО (ЮЭД) документов?

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

Я даже раза 3 писал Вашим продажникам запросы со списком сетей и документов - говорил, что если обеспечите обмен по всем - то мы рассмотрим более удобный для нас вариант централизации с более грамотным интеграционным решением.
Но они игнорят такие письма - ответов нет, только рекламные звонки (и это не смотря на платежи от 70+ тыс. руб в месяц за трафик).

73. Данила Володькин (skif47) 19.05.16 09:20
(72) serg_info, DESADVALCO у нас называется ALCRPT. Есть еще вариант, когда прямо в DESADV сведения о лицензиях/производителях добавляются, это, наколько помню, от требований сетей зависит.
Для алко есть конфигурации УТАП и КАТАП, и я пока еще ни разу не видел, чтоб клиенты пользовались им прямо так, как задумывали разработчики. Самописок еще больше, все верно.
Поэтому алкопоставщикам в подключаемом модуле настраиваем заполнение всех этих полей в отдельном событии. Есть несколько шаблонов кода, которые наши внедренцы потом уже допиливают до конкретной конфигурации.
EDI и ЮЗДО - пока отдельные проекты. Но в EDI есть немного ЮЗДО: когда цепочка EDI сообщений доходит до INVOIC, из него автоматом генерируется ЭСФ (+ТОРГ12, если надо), и модуль EDI после этого выдает "а давай-ка теперь все это сертификатом подпишем". Ctrl+A, "Подписать", выбрать сертификат и подождать.
ИСФ/КСФ где-то в начале года прикрутили, дальше была долгая тишина и отсутствие обратной связи. Недавно тема наконец начала набирать обороты, появились некоторые замечания/хотелки, обрабатываем. А так клиенты пользуются.
Аннулирование ЭСФ: в самом модуле EDI не предусмотрено, возможно только посмотреть статусы по отправленным ЭСФ: какие подписаны, по каким сеть отказала/запросила аннулирование. Полный набор плюшек на эту тему - в Диадоке.
Сайты Диадока и EDI - разные. Но туда можно попадать по одной учетной записи, если привязать ее к обоим сервисам. Собственно, и сервисы то разные изначально.
То, что отправлено интеграцией, в мониторинге отражается точно. Попадает ли в веб-интерфейс поставщика, точно не скажу, поскольку не пользуюсь им.
Странно, что продажники не ответили. Переправьте письмо на мой ящик, попингую их, они тут через стенку сидят ))
В самом начале статьи есть ссылка на демо базу с модулем. С тех пор многое поменялось/добавилось/отрефакторилось, но для общего представления должно хватить.
Для написания сообщения необходимо авторизоваться
Прикрепить файл
Дополнительные параметры ответа