0

所以这是我的问题，我写了一个存储过程来完成以下任务。在表格中events，有些活动可能存在于不再存在的场所。并非所有事件都与场地相关联，但那些在场地 id 字段中具有整数值的事件，否则为 NULL（或可能为零，但已考虑在内）。场地会定期从我们的系统中删除，当发生这种情况时，不可能在该确切时间删除与该场地相关的所有事件。相反，稍后会定期运行一个任务，该任务会删除具有不再引用场所表中现有记录的场所 id 的每个事件。我为此编写了一个存储过程，它似乎可以工作。

这是存储过程：

DROP PROCEDURE IF EXISTS delete_synced_events_orphans;
DELIMITER $$
CREATE PROCEDURE delete_synced_events_orphans()
BEGIN 

    DECLARE event_count int(11) DEFAULT 0;
    DECLARE active_event_id int(11) DEFAULT 0;
    DECLARE active_venue_id int(11) DEFAULT 0;
    DECLARE event_to_delete_id int(11) DEFAULT NULL;

    CREATE TEMPORARY TABLE IF NOT EXISTS possible_events_to_delete (
        event_id int(11) NOT NULL,
        venue_id_temp int(11) NOT NULL
    ) engine = memory;

    # create an "array" which is a table that holds the events that might need deleting
    INSERT INTO possible_events_to_delete (event_id, venue_id_temp) SELECT `events`.`id`, `events`.`venue_id` FROM `events` WHERE `events`.`venue_id` IS NOT NULL AND `events`.`venue_id` <> 0;
    SELECT COUNT(*) INTO `event_count` FROM `possible_events_to_delete` WHERE 1;

    detector_loop: WHILE `event_count` > 0 DO
        SELECT event_id INTO active_event_id FROM possible_events_to_delete WHERE 1 LIMIT 1;
        SELECT venue_id_temp INTO active_venue_id FROM possible_events_to_delete WHERE 1 LIMIT 1;

        # this figures out if there are events that need to be deleted
        SELECT `events`.`id` INTO event_to_delete_id FROM `events`, `venues` WHERE `events`.`venue_id` <> `venues`.`id` AND `events`.`id` = active_event_id AND `events`.`venue_id` = active_venue_id;

        #if no record meets that query, the active event is safe to delete
        IF (event_to_delete_id <> 0 AND event_to_delete_id IS NOT NULL) THEN
            DELETE FROM `events` WHERE `events`.`id` = event_to_delete_id;
            #INSERT INTO test_table (event_id_test, venue_id_temp_test) SELECT `events`.`id`, `events`.`venue_id` FROM `events` WHERE `events`.`id` =  event_to_delete_id;
        END IF;

        DELETE FROM possible_events_to_delete WHERE `event_id` = active_event_id AND `venue_id_temp` = active_venue_id;
        SET `event_count` = `event_count` - 1;

    END WHILE;

END $$
DELIMITER ;

这是有问题的两个表的表结构：

CREATE TABLE IF NOT EXISTS events (
    id int(11) NOT NULL,
    event_time timestamp NOT NULL,
    venue_id_temp int(11) NOT NULL
);

CREATE TABLE IF NOT EXISTS venues (
    event_id int(11) NOT NULL,
    venue_id_temp int(11) NOT NULL
);

存储过程按编写的方式工作，但我想知道如何使它运行得更好。似乎它做了很多额外的处理来实现它的目标。是否有更好的方法可以查询手头的数据，或者是否有其他更有用的命令和关键字我可以使用但我不知道，这将使我能够更好地完成这项任务（更少的行更少的计算）。我仍在学习如何使用存储过程，所以我尽可能实用地使用它们来完成任务，我想了解如何进行这个特定的查询，以更好地利用 MySQL 中的全部功能来发挥它的优势。谢谢各位。

4

1 回答 1

2

一切都简单得多：

DROP PROCEDURE IF EXISTS delete_synced_events_orphans;
DELIMITER $$
CREATE PROCEDURE delete_synced_events_orphans()
BEGIN 

  DELETE
    FROM `events`
    WHERE `venue_id` IS NOT NULL AND `venue_id` <> 0
      AND `venue_id` NOT IN (SELECT `id` FROM `venues`)
  ;

END $$
DELIMITER ;

而已。:)

你思考命令式的，试图告诉 MySQL如何完成你的任务。但是 SQL 是一种声明性语言，旨在说明要做什么。

于 2012-12-12T16:07:24.723 回答