Достается мне тут общаться по аудиту переписанной насмерть УТ11.3 на предмет быстродействия, оптимизации и прочего. Обнаружил просто прелестное... https://yadi.sk/i/iJx_Jm7y3N7FiE
Вот теперь хочу оторвать руки тому, кто ЭТО сделал...

прикольно же

(1) +1

креативно )

(0) А что, не загружает?

(0) человек старался , делал.
Проверял чтоб форма была красивой.
А ты ругаешь ...

(3) Вот мне тоже интересно.

а чо, выглядит здорово)

(0) Это загрузка прайса из экселя!
Где брать приз?

(0) ;-) Представляю:

А давайте сделаем загрузку заказа поставщику "Самсон" и "ОфисПремьер", вот вам формат...  не, зуб даем что нужно только по нему и других не будет... не надо выбора из справочника, пусть будет две кнопки...

Прошло пару месяцев... тут мы подумали и решили добавить еще и "Артекс"... не, что вы больше не будет...

Был случай, автоматизировали армянскую розничную точку. Дык там спиок поставщиков был: Самвел, Ашот, Айваз, Арик и прочие.

(9) Ты на Ашота батон не кроши)))

Ну что, банить ТС или нет?

(11) Оставь...

(0) Прикольно. :), За что руки то отрывать? Не работает что ли?!?!7
(8) А еще через пару месяцев уже и выпадающий список появится, ибо экрана уже не будет хватать.

"УТ11.3 на предмет быстродействия" причем тут эта форма? форма бантики, ты про код расскажи

(13) А еще через полгода - справочник поставщиков, настройки и т.д. Так родился мегапрайс :))))

(15) тот самый?

Что не так? И как надо было?

(0) сделал как смог, чего вы привязались?

Ну да... Я разрабу предложил перенести это все в РС, а в ответ услышал классическое: "А что это"? И подобных перлов по интерфейсу - мне места на ЯД не хватит... И рука устанет скриншотить.
Но оно работает! Правда, тестовая загрузка из 2500 позиций длится полчаса (внутри еще не рылся, боюсь, если честно).

(16) Это видимо зародыш :))))

тестовая загрузка из 2500 позиций? колонок скока? как к екселю обращается по адо запросом(хотя о чем я)? вагную запросы в цикле при чтении...

(19) Загрузку предложил в РС перенести?

(19) "Я разрабу предложил перенести это все в РС" - накуа?

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

И чем РС будет лучше?

(20) зародыш, зародыша, зародыша, зародыша, зародыша

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

(24) Это ладно. А вот если у каждого поставщика свой порядок и набор колонок - тогда будет песня.

(26) что просили то и получили.

(24) Настройки загрузки вынести в РС. Справочник поставщиков и так уже есть. Перелопаченный чуть менее, чем полностью.

(27) Ну как бы о том и речь. Когда ты знаешь че это прайс, тогда хоть в коде можно определить как грузить.

(29) не взлетит

Что толку от настройки, если ты не знаешь чей этот файл

(29) Работает - не трожь. А то каждый норовит художника обидеть, а он так видит :)

Научного интереса тема на представляет. Глумиться над прошлым разрабом тут никто не собирается. Цели темы исчерпаны.

(32) автор?

(29) И как это повлияет на быстродействие?

35 постов без мегапрайса!

(0) "на предмет быстродействия, оптимизации и прочего" - а расскажите мне пожалуйста, что вы используете для анализа?

Задача была "срочно сделать загрузку по конкретному одному юр.лицу". Сделали обработку. Спустя полгода добавилось ещё одно. Что быстрее - менять всё и делать р/с или добавить копипэстом?
Это сейчас уже удобно всё оценить и переписать.

(36) Осторожно! После третьего упоминания сюда придет Маньяк :)))

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

У меня в 7-ке настроена загрузка через универсальный обработку-загрузчик. Настройка колонок вынесено в справочник (ну типа какая колонка за какой реквизит отвечает, вплоть до пути к обработки по загрузки)

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

(38) "Это сейчас уже удобно всё оценить и переписать" - А вот и не факт, что переписать будет удобнее.

(41) Да, тут у всех не менее 22 см.

(31) Узнать, чей это файл, можно по папке, в которой он лежит.

(42) +100500
Работает - не трожь.
Кроме этого со стороны автора этой обработки это возможно такой хитрый маркетинговый ход - появился новый поставщик, надо делать доработку, а за доработку можно взять денежку.

(41) Здесь просто куча кнопок. На каждую кнопку прописано свое конкретное действие. Колонки и строки прописаны в коде жестко. Поставщик меняет колонки местами - вэллкам править конфиг.

У меня была задача загрузки прайсов. Я делал справочник настроек. Бывало, что поставщик менял формат прайса. Так проще же поменять настройки в справочнике, чем каждый раз править код. А если это еще и распределенка?

(33) Да никто и не глумится. Лично я просто ужасаюсь.

(45) "это возможно такой хитрый маркетинговый ход - появился новый поставщик, надо делать доработку, а за доработку можно взять денежку."

Бинго!

(0) есть анекдот про еврейского врача который пришел в папе-врачу и сказал что вылечил пациента который папа 30 лет лечил.

В РС можно поместить правила загрузки

(44) С чего это вдруг. Они кучи у менеджера лежат в папке "Прайсы от поставщиков"

(48) А чем РС поможет? Все равно под нового поставщика писать загрузчик

(44) Так может так и сделано в обработке? Выбираешь поставщика  он сам знает где прайс лежит (на сайте или в папке) и жмешь кнопку загрузить

Ну вот откуда сразу критика.

А может было жесткое требование бизнес-пользователя - чтобы была кнопка на каждого поставщика. Мы же этого не знаем!!

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

(51) Да, надо. Но в итоге обработка будет ровно одна. Сейчас их - под конкретного поставщика. При этом строки файла обрабатываются в цикле, внутри цикла зовется процедура, которая на каждый вызов получает объект документа, дописывает в ТЧ документа переданную строку и записывает документ.

(55) Моя практика показывает - что не взлетает

(54) >>накликать описание формата
ага, и покормить единорога не забыть

(54) Ну может быть требование было "без изменения конфигурации"

(0) Оригинально. Стильно, модно, молодёжно ))

(57) А что тут такого... если только супер-извращенные форматы.
(58) может

(58) + куча других причин из-за которых сложилось как (0)

(56) Моя практика показывает обратное. С очень редкими исключениями, когда прайс сильно извращенный. Но на это можно и внешнюю обормотку сделать.

(41) У меня в семерка написана грузилка из екселя безо всяких настроек. Пользователю надо только положить файл в папку, остальное обработка делает сама (вообще все). Мечтаю переписать её под восьмерку и не хватает духу - уж больно долго писал. А контора собирается перейти с ТиС на УТ и узнав, что моя грузилка там не работает - пользователи возмутились.

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

(60) Каждый 3-тий поставщик имеет прайс в извращенном формате.
К примеру какой нибудь поставщик АКБ/Шин вполне может сделать бренд - наименование группы. Отдельная колонка с размерами и отдельные колонки с доп.параметрами. И сидишь и склеиваешь наименование

У другого остаток в 5 колонках (типа остаток на складе1, склад 2...) и нужно получить общий остаток.

У третьего цена типовая и тебе нужно самому расчитать "твою" цену. При этом процент наценки/скидки зависит от производителя

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

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

(65) Да я не критикую (0), как я уже писал причин такого может быть много.

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

(65) Да не, тут-то как раз все просто... Файл читается построчно, циклом. Внутри цикла дергается процедура, которая эту строку пихает в ТЧ документа и записывает сам документ. Вот выжимка кода:
Пока НЕ    СокрЛП(ExcelЛист.Cells(ТекущаяСтрока,2).Text)= "" цикл //RowCount цикл
...
ИзФайлаНаСервере(СсылкаДока,Индекс,КодТ,НТС,ЦенаРП,ФактК,СуммаРП,ПоставщикНом);

и дальше:
Процедура ИзФайлаНаСервере(СсылкаДока,Индекс,КодТ,НТС,ЦенаРП,ФактК,СуммаРП,ПоставщикНом)
    
    Объект = СсылкаДока.ПолучитьОбъект();
...
Объект.Записать();

(65) Так что вангую . что  процентов 80 легко уложится в стандартную схему и не нужно будет программировать..

у меня так, по моему просто:
https://content.screencast.com/users/mikrotik1200/folders/Jing/media/49d52866-4da5-4fc0-a8c4-5e27212e2c30/2017-09-21_1534.png
Номенклатура сопоставляется по типовому справочнику "Номенклатура поставщика" по артикулу.

(0) А я чет не вгвоздил, а к чему претензии? К виду формы? Аудитор, призываю тебя к ответу.

(19) Т.е. ты предлагаешь создать целую таблицу, вместо элемента формы??? Ты точно аудитор? или банальный обсиратор и ямогутор?

(70) Как я писал выше из 42 типовой загрузчик используется в 25 случаях, т.е. 59,5%

(73) А ты точно берега не попутал? Не? Учитывая, что ЭТО встроено в конфиг, а не сделано внешней обработкой.

(75) Парень, ты за регистр сначала ответь. Потом мы с тобой поговорим про место размещения обработок.

(74) Это уже есть повод стандартизировать загрузку, а не размазывать каждого поставщика в свои процедуры загрузки.

(75) Ну если конфига на обычных формах (Типа УТ10), то встроенная в конфигу - это единственное место где обработка на УФ может работать

(77) Ага, стандартизатор. Твоя стандартизация заключается в чем? В незримом глазу пользователя размазывании обработки загрузки?

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

(78) Конфига - переписанная насмерть 11.3

(71) а что круто, хотя и непонятна сцепка с регистратором если честно)
(73) он оптимизатор труда прогов (снижатор трудозатратор)..
Нехороший человек)
(74) в смысле после допила стандатных особенностей без особого изврата как суммирование колонок - до 80 довести реально

(80) Ты про условие необходимости и достаточности слыхал?
Так вот при использовании регистра достаточность излишняя.

(83) Если у тебя поставщиков полторы штуки. А если их сотня?

(82) он снижатор трудозатрат прогов и добиватор производительности и логической архитектуры УТ

(82) не понял как ты собрался стандартизировать изврат?

(86) Изврат и я стандартизировать не собираюсь. Я для этого обработку отдельную сделаю. Но вот скажи мне: есть плоский прайс, из которого берется наименование, артикул, цена, описание ну и еще два-три поля. Просто они у каждого поставщика в разных колонках. Шапка по количеству строк отличается. И таких прайсов - много. Есть ли смысл под каждый из них писать собственный загрузчик, если можно из справочника вынуть настройки и передать их в универсальный загрузчик?

(87) хм, уж не об этом ли я писал выше?

У меня так и реализовано. И привел цифры из реальной своей базы. Т.е. типовой загрузчик используется в - 59,5% случаях

(85) Насчет производительности - аргумент, да.. Навскидку не скажу..Померять надо) Может на 0,1% будет медленнее или на 1)

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

(89) то что там наломано никого не ипет. Если пришел оптимизировать - оптимизируй. Доламывать этого оптимизатора никто не звал.

Если обработка работает, и работает правильно, не вижу смысла что-то менять. К тому же на УФ для 2500 тысяч позиций скорость вполне приемлемая. В любом случае загрузка - это одноразовая операция и 2500 позиций минут за пять вполне нормально. Встречал как-то вариант, когда пару-тройку документов с общим количеством позиций не более сотни грузились по часу, поглядел на код и ужаснулся - после добавления каждой новой позиции в документ, этот самый документ не только записывался по новой, но еще и проводился :-))))

(91) прогер не знал про попытки и транзакции походу.

(91) Если бы одноразовая. Заказов поставщикам в день генерируется несколько десятков, поставщиков много, да и фирма немаленькая. И документ, хавала Аллаху, не проводится каждый раз. Но загрузка 2500 позиций (а по этим поставщикам из формы меньше редко бывает) идет от получаса. Что неприемлемо совершенно.

(93) длительность загрузки зависит от алгоритмов, а за архитектруру в виде переключателя ты так и не ответил. Какой же ты оптимизатор?

(94) Еще раз медленно объясняю. Это фигня, когда поставщиков полторы штуки. Можно и кнопками обойтись. В контексте текущей задачи: добавляется еще три поставщика сегодня, завтра - еще десять. Но прайсы у них идентичные. Ты будешь плодить кнопки? И процедуры загрузки? Что ж, твое право. Но потом не обижайся, если пользователи оторвут тебе хорошо, если только руки.

(95) я буду плодить процедуры загрузки прайсов, в зависимости от их типа.

еще раз внятно скажи, что ты собрался в РС засовывать.

(96) Я тебе уже отвечал в (80). Повторяться не собираюсь. Если ты не умеешь читать или у тебя трудности с пониманием прочитанного, то это не ко мне.

(95) Так вроде автор обработки не претендует на господство. Он не собирается захватывать мир. И даже если завтра добавиться от силы 1,5 калеки - ну дорисует еще 2 кнопки. Какие проблемы?

К примеру я у себя тоже закладывал что больше 50 поставщиков не будет . Поэтому мое архитектурном решении не позволит завтра завести 3 сотни новых поставщиков (максимум можно хранить 99 поставщиков). Но это не значит что это решения не правильно и надо мыслить глобально на уровни газпрома