2012-09-27 33 views
0

所以这里是我的问题,我写了一个存储过程来完成以下任务。在表events中,对于不再存在的场所可能存在事件。并非所有事件都与场地相关联,但是场馆ID字段中有整数值,否则它是NULL(或可能为零,但这是占的)。定期地,场地会从我们的系统中删除,当发生这种情况时,无法在该确切时间删除与该场地相关的所有事件。相反,稍后会定期运行一个任务,删除场地标识不再引用场馆表中现有记录的每个事件。我写了一个存储过程,它似乎工作。MySQL - 优化孤儿记录疏导

这是存储过程:

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中的全部功能。谢谢你们。

+0

此问题更适合[DBA](http://dba.stackexchange.com)。投票结束并迁移到那里。 –

+0

我在那里没有账户,这将如何影响我获得答案的能力? – usumoio

+0

与新用户在此发布问题时的方式相同 - 完全没有。这里的新用户可以获得与长期用户所做的相同的工作(有时甚至更多):-)您甚至可以合并您的帐户信息,以便此处的帐户与其中的帐户连接,并显示在用户个人资料中。只需使用您在此使用的相同凭据登录即可。 –

回答

2

Everithing简单得多:

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是一种说明性语言,专门用于说什么要做。

+0

很好的答案。是的,我完全忘记了一个子查询可以解决这个问题。这是我今天的教训。谢谢。 – usumoio