a13x Posted November 7, 2020 Report Share Posted November 7, 2020 Только что, DaVinci сказал: Как я понял надо пройтись по всем родительским категориям, выбрать все относящиеся к ним товары и создать для них новые branch записи. Наверное да, теоретически должно работать. Только стоит переживать за запрос формирующий новые привязки Думаю запрос это решаемо, не обязательно делать 1 большой, можно и циклом пройтись, не думаю что очень часто меняют вложенность категории перенося категорию из 1 раздела в другой, а при перемещении в пределах раздела ничего и не нужно делать. Quote Link to post Share on other sites
phukortsin Posted November 7, 2020 Report Share Posted November 7, 2020 1 час назад, a13x сказал: Думаю запрос это решаемо, не обязательно делать 1 большой, можно и циклом пройтись, не думаю что очень часто меняют вложенность категории перенося категорию из 1 раздела в другой, а при перемещении в пределах раздела ничего и не нужно делать. Безусловно, это решаемо. Другой вопрос - какие трудозатраты потребуются, что все это решить. Думаю, что вслед за первым решением, если Вы такое дадите, найдется много замечаний о разных недостатках, и Вы увязнете в многочисленных исправлениях (если захотите довести работу до конца). А когда (если) доведете, выяснится самое интересное - эффекта никакого доработка не даст, и запрос с тем же успехом будет вешать сервер. Потому как Вы, похоже, даже толком не проанализировали, откуда берется тот проблемный запрос... 4 часа назад, a13x сказал: Что значит нерациональность? Если у вас есть идея как исправить это - пишите, но боюсь что таковой нет. Я написал как это можно исправить, но врядли кто-то это будет делать, лучше вон перепишут симплу, только толку от этого как всегда ноль! Нерациональность - неразумное использование чего-либо. В данном случае, применительно к категориям, одна из нерациональностей состоит в том, что ВСЕГДА, при каждом построении каждой страницы, из базы запросом извлекаются ВСЕ категории со всеми полями. А используется на странице из этого объема максимум процентов 10. И этот момент далеко не единственный... Quote Link to post Share on other sites
a13x Posted November 7, 2020 Report Share Posted November 7, 2020 37 минут назад, phukortsin сказал: Безусловно, это решаемо. Другой вопрос - какие трудозатраты потребуются, что все это решить. Думаю, что вслед за первым решением, если Вы такое дадите, найдется много замечаний о разных недостатках, и Вы увязнете в многочисленных исправлениях (если захотите довести работу до конца). А когда (если) доведете, выяснится самое интересное - эффекта никакого доработка не даст, и запрос с тем же успехом будет вешать сервер. Потому как Вы, похоже, даже толком не проанализировали, откуда берется тот проблемный запрос... Жесть, что за чушь тут написана? Я рассказал в чём проблема, дал решение, мне ещё и написать код который это будет устранять? Мне это не надо, у меня всё отлично работает на моей сборке. Quote Link to post Share on other sites
DaVinci Posted November 7, 2020 Author Report Share Posted November 7, 2020 16 минут назад, a13x сказал: Жесть, что за чушь тут написана? Я рассказал в чём проблема, дал решение, мне ещё и написать код который это будет устранять? Мне это не надо, у меня всё отлично работает на моей сборке. в целом тут никому не лень писать много слов но ни строчки кода. в как тяжело дается практика подкрепления своих слов хоть каким то примером. Quote Link to post Share on other sites
phukortsin Posted November 7, 2020 Report Share Posted November 7, 2020 17 минут назад, a13x сказал: Я рассказал в чём проблема, дал решение Про проблему рассказали раньше Вас. Проблема в тяжелом запросе, который показали тоже без Вас. Вы почему-то транслировали проблему в большое число категорий. С чего Вы взяли, что тот вопрос зависит от большого числа категорий? И с чего Вы взяли, что придуманное Вами заполнение таблицы s_products_categories вопрос решит? Про дополнительные неудобства от такого способа я пока не говорю... Quote Link to post Share on other sites
DaVinci Posted November 7, 2020 Author Report Share Posted November 7, 2020 (edited) с деревьями всегда было сложно, есть множество решений. но в общем есть вариант который решает множество проблем baum/baum - скачано 1 042 229 раз a13x, я соглашусь что ваш пример очевидно рабочий, но действительно подводных камней много. важно понимать на сколько хорошо обкатаны частные случаи, в каких "боевых" условиях был применен данный подход, на сколько хорошо вы сами знаете преимущества и недостатки вашего подхода и что немаловажно понимает ли последующий разработчик подход который вы применили, ведь чаще всего обратной связи от разработчиков не дождешься. Мануала у вас нет, примеров нет. Как последующему программисту вникнуть в суть вашей работы? Комментарии? Это не серьезно. Edited November 7, 2020 by DaVinci Quote Link to post Share on other sites
ST_RU Posted November 7, 2020 Report Share Posted November 7, 2020 21 час назад, a13x сказал: лучше вон перепишут симплу Noxter на сантехниках отлично переделал Simpla всё по правилам сделал.. потом я доделывал и в порядок фронт приводил %) ps хватит в ветке спорить / или почистить тему.. Quote Link to post Share on other sites
DaVinci Posted November 7, 2020 Author Report Share Posted November 7, 2020 7 минут назад, phukortsin сказал: Про проблему рассказали раньше Вас. Проблема в тяжелом запросе, который показали тоже без Вас. Вы почему-то транслировали проблему в большое число категорий. С чего Вы взяли, что тот вопрос зависит от большого числа категорий? И с чего Вы взяли, что придуманное Вами заполнение таблицы s_products_categories вопрос решит? Про дополнительные неудобства от такого способа я пока не говорю... соглашусь, утверждать то что описанный a13x подход решает эту проблему как минимум преждевременно. Quote Link to post Share on other sites
Kasha Posted November 7, 2020 Report Share Posted November 7, 2020 3 минуты назад, ST_RU сказал: Noxter на сантехниках отлично переделал Simpla всё по правилам сделал.. потом я доделывал и в порядок фронт приводил %) Он не доделал его очень сильно, о чем сам неоднократно и говорил. Там много болячек было, которые поправлял в том числе и я. Quote Link to post Share on other sites
DaVinci Posted November 7, 2020 Author Report Share Posted November 7, 2020 (edited) 11 минут назад, ST_RU сказал: Noxter на сантехниках отлично переделал Simpla https://uslugi-masterov.ru про это что л речь? или это https://ru-santehnik.ru? Значит я поправлял после всех вас )))) Edited November 7, 2020 by DaVinci Quote Link to post Share on other sites
ST_RU Posted November 7, 2020 Report Share Posted November 7, 2020 В 07.11.2020 в 20:42, DaVinci сказал: после всех вас )))) :))))))) / ЦАРЬ то не настоящий! (с) я ему про тебя говорил. синий сайт переделал .. зелёный сам, по макетам. нормальный же сайт. не отвлекаемся от темы топика Quote Link to post Share on other sites
ABSORBER Posted November 8, 2020 Report Share Posted November 8, 2020 Почистил тему от флуда. Прошу прощения за излишнюю агрессию в адрес @DaVinci @a13x @Noxter Призываю всех писать по делу, не флудить и стараться излагать в сдержанной форме, без попытки унизить или оскорбить собеседника. Quote Link to post Share on other sites
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.