0

Изучаю SQL. У меня есть таблица products_to_colors она связывает 2 таблицы colors и products, связь многие ко многим. Содержит поля id (просто id записи в таблице, хз зачем сделал эту колонку), product_id, color_id

 id | color_id | product_id
----+----------+-----------
1   | 1        | 7
----+----------+-----------
2   | 2        | 7
----+----------+-----------
3   | 3        | 7
----+----------+-----------
4   | 4        | 7

Столкнулся с задачей обновления данных в этой БД. Представим, что пришёл новый запрос и у продукта изменились цвета, например, продукт с id 7, теперь должен иметь цвета [1, 3, 5, 7], было [1, 2, 3, 4]. Не могу написать адекватный SQL запрос для такой задачи. Конечно можно пройтись с помощью 2 циклов и ЯП (языка программирования) (1 цикл удаляет, 2 добавляет), сделав кучу запросов к БД, но это какой-то говнокод и никакой оптимизации, я так делать не хочу.

Скорее всего должно быть 2 запроса к БД, первый удаляет ненужные связи из products_to_colors, второй добавляет необходимые.

С запросом для удаления связей вроде как всё более менее понятно, сделал так (не знаю насколько это правильно):

DELETE FROM products_to_colors
    WHERE color_id = ANY (
            SELECT color_id FROM products_to_colors WHERE product_id=7
        )
        AND color_id != ANY('{1,3,5,7}')
        AND product_id=7;

Остаётся функция добавления связей с цветами, которых до этого не было. Тут я никак не соображу, что делать. Ну я могу пройтись по массиву со списком id цветов с помощью ЯП и проверить, есть ли нужная связь в БД, если нет, то добавлять. Но это опять куча запросов и какой-то говнокод.

Нашёл такой пост но там немного другая ситуация, не знаю как это адаптировать к моей задаче. По SQL (особенно postgres) очень мало информации в интернете, особенно примеров. Не понятно как учиться писать хороший код =)

2
  • 1) Зачем Вы вообще упоминаете таблицы colors и products? они в принципе не нужны для решения задачи. 2) Как выглядят исходные данные ТОЧНО. "какие-то удались, какие-то добавились" - это ни о чём.
    – Akina
    22 окт 2020 в 17:47
  • @Akina ок, щас подредактирую =)
    – Inventor
    22 окт 2020 в 17:49

2 ответа 2

0

Принципиально это выглядит так:

WITH cte AS ( DELETE FROM products_to_colors
              WHERE (product_id, color_id) IN ( {список пар на удаление} )
            )
INSERT INTO products_to_colors VALUES {список пар на вставку} ;

fiddle

10
  • Интересно а если cte не использовать в запросе, то он выполнится? По идее вы только определили его 22 окт 2020 в 18:00
  • А вот как узнать список пар на вставку и удаление? Сначала выполнить SELECT запрос и посмотреть что в БД? Я думал без этого можно обойтись, вроде как с DELETE у меня получилось, INSERT не могу, есть функции unnest и array_fill, нельзя их как-то использовать, чтобы не увеличивать размер кода и количество запросов?
    – Inventor
    22 окт 2020 в 18:03
  • @AzizUmarov а если cte не использовать в запросе, то он выполнится? Вот не понял фразы, от слова "вообще". По идее вы только определили его Его - это, стесняюсь спросить, кого?
    – Akina
    22 окт 2020 в 18:04
  • @Inventor А вот как узнать список пар на вставку и удаление? ??? пришёл новый запрос и у продукта изменились цвета, например, продукт с id 7, теперь должен иметь цвета [1, 3, 5, 7], было [1, 2, 3, 4] - это означает, что удалять надо (7,1),(7,2),(7,3),(7,4), а вставлять соответственно (7,1),(7,3),(7,5),(7,7). А если просто пришёл новый список, то удалять тупо все (7,*).
    – Akina
    22 окт 2020 в 18:06
  • Вот смотрите With cte ... Select .. from cte Эквивалент select ... From (select ...) Если cte не использовать то каков его эквивалент 22 окт 2020 в 18:06
0

В итоге решил задачу так:

DELETE FROM products_to_colors
WHERE color_id IN (
        SELECT color_id FROM products_to_colors WHERE product_id={product_id}
    )
    AND color_id != ALL({new_colors})
    AND product_id={product_id};

INSERT INTO products_to_colors (color_id, product_id)
    SELECT el, {product_id} FROM unnest({new_colors}) AS el WHERE el NOT IN (
        SELECT color_id FROM products_to_colors WHERE product_id={product_id}
    );

Где: {new_colors} - новые цвета, которые должны быть привязаны к продукту, {product_id} - id продукта

7
  • А почему просто не удалить все записи для указанного продукта? на что этот геморрой с дополнительными проверками при удалении?
    – Akina
    23 окт 2020 в 13:15
  • @Akina ну зачем удалять записи, а потом заново их создавать? А если там 10.000 записей и надо только 1 запись добавить. Мне что сначала все удалять, а потом добавлять? Это же не оптимально
    – Inventor
    23 окт 2020 в 13:20
  • А если там 10.000 записей Для одного продукта? надо только 1 запись добавить Добавить? А в тексте вопроса Вы говорите о том, что Вы получаете новое состояние, а не изменение текущего... Вы уж определитесь, что ли...
    – Akina
    23 окт 2020 в 13:22
  • @Akina ну вот были у продукта цвета с id 1, 2, 3 ... 10000, а стали 1,2,3 ... 10001. Получается добавился новый цвет, мой запрос просто добавит цвет с id 10001, а ваш сначала удалит 10000 записей, а потом добавит эти же 10000 записей и ещё одну. Это же лишние действия, время и нагрузка. Кроме нагрузки на БД в вашем варианте сначала нужно эти пары ещё составить с помощью ЯП (языка программирования), это опять лишний код, время и дополнительная нагрузка
    – Inventor
    23 окт 2020 в 13:26
  • ну вот были у продукта цвета с id А вот теперь расскажите, откуда Вам это известно, если, по Вашему же признанию, "Откуда я должен знать, что сейчас находиться в базе?" в вашем варианте сначала нужно эти пары ещё составить с помощью ЯП (языка программирования) Да с какого перепугу-то? я просто удалю все записи для продукта, и мне вовсе не нужно знать, какие они там были, и есть ли какие-то из них в новом наборе. лишний код, время и дополнительная нагрузка Лишний код - это как раз Ваши дополнительные проверки. И, уж поверьте на слово, почти наверняка с ними время и нагрузка будут выше.
    – Akina
    23 окт 2020 в 13:50

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.