Писал импорт каталога для одного сайта. Начал тестировать и удивился сильным тормозам: скорость примерно в 1 элемент инфоблока в секунду. Начал было думать, что хостинг никудышный, но нет. При ковырянии кода напоролся на такое вот "замечательное" решение из маркетплейса, именуемое "Список 2.0". http://marketplace.1c-bitrix.ru/solut...ipol.auen/ Убивает обмен моментально ))
Сортировка вариантов списка происходит на КАЖДУЮ установку значения элемента инфоблока. В результате этого "гениального" решения, при N вариантах значений списка и M товаров, количество запросов на обновление одного такого свойства у всех товаров будет не M, а M*N! Точнее даже не M*N, а M*N*2, т.к. еще есть дополнительный запрос сброса кэша по тэгу. А если таких свойств несколько (у клиента их было с десяток), то будет полный бздец. В общем, ребята, не надо так делать. При разработке свойств учитывайте, что есть не только интерфейсная часть свойства в админке, но и разного рода взаимодействия через API. Всем доброго утра понедельника
Покровский Дмитрий, зачем оправдываться я не понимаю. Если специфика рынка говорит вам - "Вы либо делаете гавнокод но продаёте, либо не делает гавнокод и в убытке" - и вы сознательно выбираете первое - к чему оправдываться? Передо мной? обществом? вендором? Зачем - делайте своё дело - вы же уже выбрали.
Покровский Дмитрий, Если вам станет легче - у меня на маркетплейсе есть тоже модули и там тоже есть гавнокод))) А есть решения, которые я не пропускаю без насильной проверки и теста. Как пример один импорт БД с удалйнного сервера со своей спецификой работы, я потратил в 2 раза больше времени на разработки, человек заплатил больше, зато уже пол года ни проблем ни ошибок и расширяемость проста...А маркетплейс - там есть и хорошие решения и есть плохие. Каждый сам выбирает что он будет делать))))
Пилецкий Антон написал: Как уже сказал Дмитрий, проводить полноценное тестирование накладно для небольшой развивающейся студии или отдельному разработчику. Можно несколько лет не вылезать из отладки,
Есть BDD и TDD. Такой способ есть разработки. Пишется функционал вместе с тестами, а не сначала весь функционал а потом кучу тестов писать, да и к тому же юнит тесты не всегда нужны особенно в сфере модулей, больше скорее функциональные тесты
Бедросова Юлия, "... Правдой по закону считается только то, что доказано в суд. Это базовый принцип презумпции невиновности. ..." Презумпция невиновности действует в уголовном праве, в гражданском — все по-умолчанию виновны. И то, что ты не "верблюд" приходится доказывать именно ответчику. Так, что все – что сказано все правда, пока не доказано обратное. А если доказано обратное – это повод обозвать "верблюдом" в ответ. Сорри. Офтоп
Иванов Иван написал: в гражданском — все по-умолчанию виновны. И то, что ты не "верблюд" приходится доказывать именно ответчику
В суде — да. А в социуме и в том числе на форумах и блогах подобных этому, презумпция виновности или невиновности — на личный выбор человека. Если человек все принимает на свой счет и считает, что нужно оправдываться перед каждым воинствующим троллем неудачником и завистником — это его выбор. Если не считает нужным этой ерундой заниматься и верит в то, что подавляющее большинство пользователей умные и адекватные люди, которые разберутся кто старается пользу обществу приносить, а кто "бедный зато не ворую" мутит воду на ровном месте, умничает и повышает ЧСВ на критике более успешных товарищей — это тоже его выбор.
kopoBko написал: т.е. по вашему пост об ошибке это проявление неадекватности
Я думаю, пост об ошибке может быть как адекватным, так и неадекватным. Когда упомянутые выше itconstruct делают в своем блоге обзоры готовых решений, они делают это адекватно, грамотно и корректно, что только добавляет им чести. Когда же кто-то ругает решение и его разработчиков излишне эмоционально, на мой взгляд, он лишь делает негативный пиар себе и себя же унижает. А чтобы видеть разницу между адекватным и неадекватным обзором решения - нужно, наверное, просто повзрослеть - подрасти морально и эмоционально.
Я физфак закончил. Мы привыкли мыслить физическими величинами. Может потому и не понимаю юмора Но так врезалось в память изречение одного из "лириков" в одном из толчков нашего ВУЗа
Писать на стенах туалетов увы, мой друг, не мудрено. Среди говна мы все поэты. Среди поэтов мы - ...
Коваленко Алексей, а, представьте, ТС стоило лишь перефразировать свой отзыв, написав, что-то вроде: "Ребята, обратите внимание - при импорте каталога из различных источников ваше решение существенно замедляет этот процесс. При N вариантов значений...". И его бы зауважали, а сейчас я, например, зауважала Дмитрия Покровского - ему удалось сохранить достоинство и выдержку в такой некрасивой ситуации, думаю, такого адекватного человека ждет успех, а что до багов - и на солнце бывают пятна.
Юлия Бедросова, +100500 ! Можно же без иронии, без сарказма, а нормально по человечески указать автору на ошибку. Не ошибается только тот, кто ничего не делает. Спутники и те падают... Все тут люди, Богов нет. Мы же наблюдаем такой тренд — целенаправленного выискивания "косяков" чтобы тут плясать с ними публично...
Забродин Роман написал: целенаправленного выискивания "косяков"
Я не искал косяки целенаправленно. Как я уже написал выше, я лишь разгребал говно на сайте вот этого гневного человека http://dev.1c-bitrix.ru/community/for...age369786/ И это говно натворили "добрые и адекватные" разработчики. Баг в рассматриваемом модуле - это лишь малая часть в зловонной куче. Я захотел изменить взгляд этого конкретного человека на продукт под названием "1С-Битрикс". И знаете, я думаю, что мне это удалось. В результате этот человек будет работать с Битриксом и посоветует уже посоветовал использовать данный продукт другим своим знакомым предпринимателям. А это уже мои (и ваши) потенциальные клиенты.
Пилецкий Антон, вы сделали благое дело и вы молодец! (говорю это серьезно, без иронии) Но, зачем нужно было потом транслировать гнев этого человека возникший в следствии его собственных странных ожиданий, как например:
"коробочного решения" -- которое, как известно, рассчитано на ламеров
на других разработчиков и авторов??? Мы так например, всех наших покупателей сразу честно предупреждаем, что им обязательно потребуется грамотный специалист для внедрения и дальнейшего администрирования сайта. И даже завели свой реестр таких опытных и ответственных партнеров. А то, что "CMS 1С-Битрикс с Типовым решением может управлять любая домохозяйка" — это люди видимо сами додумывают, не понятно вообще на чем основываясь, и соответственно очень сильно переоценивая свои силы... У нас в компании даже стажеры уже имеющие высшее техническое образование сначала целый месяц проходят и сдают все он-лайн курсы 1С-Битрикс и только потом мы допускаем их к администрированию (даже не программированию) клиентских сайтов. На наш взгляд, предприниматели, которые сразу с наскока пытаются все установить и настроить самостоятельно — поступают по меньшей мере, самонадеянно...
Забродин Роман, а мне кажется, Антон Пилецкий не погасил гнев клиента, а просто перенаправил его гнев на разработчиков модуля, которые, если разобраться - вообще не при чем: как выяснилось в ходе обсуждения, замедляющая процесс импорта фича модуля была опциональна и отключалась галочкой, чего Антон Пилецкий не разглядел, ковыряя в два часа ночи код модуля, а разглядеть был должен так как клиент обратился к его услугам именно за тем, чтобы Антон разобрался в работе установленных на сайте решений.
Вот если бы Антон нашел галочку, разобрался, для чего она и ткнул в нее клиента, показав, что не Битрикс плохой и не решение плохое, а в настройках нужно разбираться и руководства читать, вот тогда он был бы молодец.
Вообще, я думаю, что всплывший факт, о том, что автосортировка значений Списка 2.0 была опциональна - это серьезный повод для того, чтобы Антон Пилецкий принес разработчикам решения публичные извинения. Однако на такой шаг способны только зрелые личности.
Без поиска этой галочки я и по двум указанным строчкам кода видел, что есть возможность отключить сортировку. Но решение уже себя скомпрометировало. Ковыряться в нем, выискивая другие баги, которые потенциально могут положить обмен, я не стал. На добровольческих началах делать работу за тех, кто за это получил деньги у меня нет желания. Модуль ждет обновления.
Бедросова Юлия написал: Вообще, я думаю, что всплывший факт, о том, что автосортировка значений Списка 2.0 была опциональна - это серьезный повод для того, чтобы Антон Пилецкий принес разработчикам решения публичные извинения. Однако на такой шаг способны только зрелые личности.
А я считаю, что даже если фича опциональна и отключается простой галочкой, то она ни в коем разе не должна сказываться отрицательным образом на работе сайта. Можете считать меня незрелой личностью, ваше право.
Забродин Роман написал: гнев этого человека возникший в следствии его собственных странных ожиданий
У меня такое подозрение, что человеку попросту впарили коробочное решение, не объяснив толком что к чему. Отсюда и возникли эти странные ожидания. Кстати разработчик этого коробочного решения уже канул в лету и передает привет из Шри-Ланки! Называть его не буду, а то опять скажете, что я злой и несправедливый))
Лично я бы Антону спасибо сказал, если бы он в моем модуле нашел подобный изъян, без разницы, где об этом написали, в блоге, в отзывах или лично, т.к. это + к качеству моего продукта, экономия моего времени, ну и опыт другого разработчика будет полезен каждому. Все правильно подмечено, кто-то пишет модуль, тестирует его как надо, заботится о его качестве, а кто-то пишет ..овно ради наживы, впарить и забить на человека с его проблемами, а это очень-очень неуважительно к клиенту и просто некрасиво. Мне если пишут о баге в модуле, я открываю PHPStorm, ищу и исправляю ошибку и искренне благодарю человека за сообщение об ошибке и помощь, накасячил, исправляю за свой счет, очень полезно, мотивирует, а не хочешь исправлять, вот и получай...
Группы на сайте создаются не только сотрудниками «1С-Битрикс», но и партнерами компании. Поэтому мнения участников групп могут не совпадать с позицией компании «1С-Битрикс».