Ret; lad os se, hvad vi har her.
Først skal koden blokeres som følger:
variable declarations
cursor declarations
handler declarations
everything else
Så din DECLARE CURSOR c2
skal vises mellem DECLARE CURSOR c1
og DECLARE CONTINUE HANDLER
. Desuden behøver du kun én CONTINUE HANDLER
fordi den træder i kraft fra erklæringen til afslutningen af proceduren.
Dernæst er erklæringen
INSERT INTO ip_ER_subtotal
SELECT Starting_Pitcher, Game_Date, Game_Number, innings_pitched, 0.0
FROM starting_pitchers_game_log;
De navngivne kolonner i SELECT
klausul er de kolonner, du vælger fra, ikke dem du indsætter i, så de skal være kolonner i tabellen starting_pitchers_game_log
. Også da kolonnerne ikke bliver kopieret fra starting_pitchers_game_log
(det vil sige ip_total
, er_total
og era
) alle har standardværdier, kan du bruge en kolonneliste på INSERT
erklæring, som sådan:
INSERT INTO pitcher_stats_temp
(Starting_Pitcher, Game_Date, Game_Number, innings_pitched, er)
SELECT pitcher_id, game_date, game_seq, innings_pitched, runs
FROM starting_pitchers_game_log;
Dette sparer indtastning, dokumenterer, hvilke kolonner du faktisk indsætter værdier i, og isolerer din INSERT
sætning fra den fysiske rækkefølge af kolonner i kilde- og måltabellerne.
Dernæst, når du er færdig med CURSOR c1
sløjfe, undlad at afkorte tabellen eller du mister alt det arbejde, du lige har udført! TRUNCATE TABLE
sletter alle rækker i tabellen i øjeblikket og bruges her til at rydde resultaterne af den forrige kørsel ud.
Endelig skal de to sløjfer have forskellige etiketter, f.eks. fetch_loop_1
og fetch_loop_2
. Du skal også nulstille accum
og end_of_cursor
før du går ind i den anden sløjfe. Men i dette tilfælde tror jeg, at vi kan gøre alt i én løkke med én markør, hvilket gør koden enklere og dermed nemmere at vedligeholde.
Her er den komplette procedure:
DROP PROCEDURE IF EXISTS pitcher_stats_era;
DELIMITER $$
CREATE PROCEDURE pitcher_stats_era()
BEGIN
DECLARE pit_id CHAR(10);
DECLARE gdate DATE;
DECLARE seq INT;
DECLARE in_pit REAL;
DECLARE er INT;
DECLARE accum_ip REAL;
DECLARE accum_er INT;
DECLARE earned_run_avg REAL;
DECLARE prev_year YEAR(4);
DECLARE end_of_cursor BOOLEAN;
DECLARE no_table CONDITION FOR SQLSTATE '42S02';
DECLARE c1 CURSOR FOR
SELECT pitcher_id, game_date, game_seq, innings_pitched, earned_runs
FROM pitcher_stats_temp
ORDER BY pitcher_id, game_date, game_seq;
DECLARE CONTINUE HANDLER FOR NOT FOUND
SET end_of_cursor := TRUE;
DECLARE EXIT HANDLER FOR no_table
BEGIN
SIGNAL no_table
SET MESSAGE_TEXT = "Work table not initialized. Please call pitcher_stats_reset() before continuing",
MYSQL_ERRNO = 1146;
END;
------------------------------------------------------------------
-- The following steps are now performed by pitcher_stats_reset()
------------------------------------------------------------------
-- TRUNCATE TABLE ip_subtotal; -- Clear our work table for a new run
-- Copy data from main table into work table
-- INSERT INTO ip_subtotal
-- (pitcher_id, game_date, game_seq, innings_pitched, earned_runs)
-- SELECT pitcher_id, game_date, game_seq,
-- IFNULL(innings_pitched, 0), -- replace NULL with 0, if
-- IFNULL(runs, 0) -- column not initialized
-- FROM starting_pitchers_game_log;
---------------------------------------------------------------------
SET end_of_cursor := FALSE; -- reset
SET prev_year := 0; -- reset control-break
OPEN c1;
fetch_loop: LOOP
FETCH c1 INTO pit_id, gdate, seq, in_pit, er;
IF end_of_cursor THEN
LEAVE fetch_loop;
END IF;
-- check control-break conditions
IF YEAR(gdate) != prev_year THEN
SET accum_ip := 0.0;
SET accum_er := 0;
SET prev_year := YEAR(gdate);
END IF;
SET accum_ip := accum_ip + in_pit;
SET accum_er := accum_er + er;
IF accum_er = 0 THEN -- prevent divide-by-zero
SET earned_run_avg := 0;
ELSE
SET earned_run_avg := (accum_ip / accum_er) * 9;
END IF;
UPDATE pitcher_stats_temp
SET ip_total = accum_ip,
er_total = accum_er,
std_era = earned_run_avg
WHERE pitcher_id = pit_id
AND game_date = gdate
AND game_seq = seq;
END LOOP;
CLOSE c1;
END
$$
DELIMITER ;
Det burde gøre arbejdet. Hvis nogen finder en fejl, så påpeg venligst det.
EDIT:Jeg har lige tilføjet noget kode for at illustrere, hvordan man beskytter mod nulværdier, der kommer fra kildetabellen, og hvordan man undgår en divider-med-nul på ERA-beregningen.
EDIT:Jeg har skiftet tilbage til mine oprindelige kolonne- og tabelnavne for at mindske min egen forvirring.
EDIT:Koden er ændret for at være i overensstemmelse med svaret på Hvordan kan jeg tilføje en kolonne til en arbejdstabel ved hjælp af en ny lagret procedure