Trigger dla INSERT.. problem składniowy, proszę o sugestie Opcje

[mikajlo]

Witam,
jestem początkującym samoukiem i potrzebuję utworzyć trigger, który po wstawieniu danych do tabeli T1, wstawi jedną z wstawionych wartości do tabeli T2.
Z tym, że jeśli wartość ta jest null albo "" (czyli defacto pusta) ma nie wykonywać inserta..

Akutalnie wygląda to tak:

[SQL] pobierz, plaintext 
 
DELIMITER $$
CREATE TRIGGER insertClub AFTER INSERT ON users
  FOR EACH ROW BEGIN
    SET @var = NEW.clubUser;
    IF (@var IS NOT NULL OR @var <> '') THEN
    INSERT INTO kluby (nameClub) VALUES(@var);
    END IF;
  END$$
DELIMITER ;
[SQL] pobierz, plaintext 

I do tabeli T2 dodawane są nowe wartości.. ale są również dodawane wartości puste ( '' ) które nie mają być dodawane.. (ma po prostu nie robić tego kroku).

Pewnie jest jakiś niewielki błąd.. Średniozaawansowani nie będą pewnie na pierwszy rzut oka zauważą co jest nie tak..

[irmidjusz]

zamień OR na AND

[mikajlo]

Dzięki.. ruszyło..

Jednakże mam inny teraz problem i nie chcąć tworzyć nowego tematu podczepie się tutaj i zapytam: czy jest możliwość wykonania takiego triggera, że..
- wstawiam wpis do tabeli zawody z jakąś datą (date)
- w bazie danych są utworzone tabele na wzór - wyniki_rok, tj. wyniki_2010 ; wyniki_2011 ; wyniki_2012...
- i teraz jak dodawana jest data 2013-10-05 do tabeli zawody, to na tej podstawie ma sprawdzić czy istnieje tabela wyniki_2013.. jesli nie to ma ją utworzyć..


Czy jest taki "myk" w ogóle możliwy? Jeśłi tak to proszę o jakieś sugestie (neuralgiczne miejsca z kodem, który należy tu wykorzystać.. czyli jak sprawdzać czy dana tabela isnieje i jak sam rok można wyciągnąć z daty..)

[mmmmmmm]

Rozwiązanie bezsensowne... Tworzenie tabel dla poszczególnych lat.
Np. policz sumę zakupów jednego klienta za wszystkie lata (IMG:style_emoticons/default/smile.gif) powodzenia.
A tak serio, to w jednej tabeli dodaj sobie indeksowane pole 'rok'. reszta bez zmian.

[mikajlo]

Wiem, że rozwiązanie jest nie najszczęśliwsze ale tak to wlasnie ma wyglądać... dlatego się pytam czy jest to możliwe do zaimplementowania w triggerze..

[irmidjusz]

nie pomagamy tworzyć bezsensownych rozwiązań

[mikajlo]

to nie dziękuje..

[alegorn]

nie lubie kategoryzowania.
stwierdzenie ze podział tabel na lata jest z punktu złe - nie do końca pokrywa się z prawdą.

potrafię sobie wyobrazić kilka istotnych powodów, dla których takie rozwiązanie ma sens << np generowanie raportów.

czy tutaj jest to dobrym rozwiązaniem? nie wiem. nie znam modelu aplikacji.

jedyna rada - to w procedurze która zakłada tabelę 'zawody_rok', dopisz automatycznie procedurę 'wyniki rok' skoro masz zawody - to musisz miec wyniki, generowanie tego w locie jest bez sensu.

j.

[mikajlo]

@alegorn - wreszcie jakiś konkretny głos..

Tylko teraz musze znaleźć rozwiązanie do tego.. (składniowe..)

[sazian]

możesz użyć PREPARE - a przynajmniej tak myślę (IMG:style_emoticons/default/biggrin.gif)

[SQL] pobierz, plaintext 
        SET @query=concat("SELECT ?,?,",@zmienna3,' FROM tabela WHERE id=?');
 
        PREPARE stmt FROM @query;
        EXECUTE stmt USING @zmienna1, @zmienna2,@zmienna4;
        DEALLOCATE PREPARE stmt;
 
[SQL] pobierz, plaintext 

w przykładzie masz pokazane 2 sposoby przekazywania zmiennych, aby zmodyfikować nazwę tabeli chyba będziesz musiał użyć sposobu z concat - zmienna3

[mikajlo]

No właśnie próbuje coś w tym kierunku i ...

tak jak pisałem muszę wyciągnąć rok z daty zamiast całości.. więc znalazłem coś takiego:

[SQL] pobierz, plaintext 
SELECT YEAR(dataZawody) FROM zawodynazwa WHERE dataZawody LIKE '2012%';
[SQL] pobierz, plaintext 

i to działa dając tylko rok.. Tylko teraz ten uzyskany rok chciałbym przekazać dalej... i nie wiem jak (IMG:style_emoticons/default/sad.gif)

Chciałbym uzyskać coś takiego:

[SQL] pobierz, plaintext 
DELIMITER //
CREATE PROCEDURE dodajZiW(IN Zname varchar(50), IN DATA date)
BEGIN
DECLARE zmienna VARCHAR(1000) DEFAULT str; 
SET zmienna = SELECT YEAR(dataZawody) FROM zawodynazwa WHERE dataZawody LIKE '2012%';
(lub coś z CONCAT('SELECT YEAR(dataZawody) FROM zawodynazwa WHERE dataZawody like 'DATA); tylko TO nie chce działać)
I jak mamy wyciągniętą tą date TO możemy zrobić CREATE... (no ale najpierw trzeba przejść pierwszy krok..)
END //
DELIMITER ; 
 
[i]gdzie:[/i]
CALL dodajZiW("Zawody","2012-05-04");
 
[SQL] pobierz, plaintext 

[mmmmmmm]

[SQL] pobierz, plaintext 
SELECT YEAR(dataZawody) INTO zmienna FROM zawodynazwa WHERE dataZawody LIKE '2012%';
[SQL] pobierz, plaintext 

Swoją drogą nie uważasz, że to zapytanie jest trochę bezsensowne? Jaka data zaczyna się na 2012??

[mikajlo]

Mój błąd.. tam ma być przekazana do procedury zmienna Data

Witam po dłuższej przerwie..
prace z triggerem zostały na moment wstrzymane ale teraz trzeba to dokończyc..

Z racji ograniczeń triggera w związku z tworzeniem nowych tabel zastanawiam się czy jest możliwość takiego rozwiązania..
(są to jeszcze niedopracowane składniową rozwiązania..)

EDIT:
Po prawie dwóch godzinach myślenia nad kodem mam taki komunikat:
"Dynamic SQL is not allowed in stored function or trigger"

Co w przygotowanym poniżej kodzie może wywoływać taki komunikat? I jak sobie z tym poradzić?




Tworzę trigger, który wywołuje w sobie procedure, która otrzymuje jako parametr datę z operacji triggera..

[SQL] pobierz, plaintext 
DELIMITER //
CREATE TRIGGER createTableWyniki AFTER INSERT ON zawodynazwa 
  FOR EACH ROW 
  BEGIN
  #SET @data = (SELECT YEAR(dataZawody) FROM zawodynazwa WHERE dataZawody = NEW.dataZawody);
  CALL procCreateTableWyniki(NEW.dataZawody);
END//
DELIMITER ;
[SQL] pobierz, plaintext 

a procedura ma za zadanie utworzyć nową tabele, która w nazwie tabeli ma mieć przekazany wcześniej rok, coś na wzór - wyniki_2013
(tylko tutaj chyba trzeba skorzystać z polecenia PREPARE do utworzenia tej zmiennej tablicowej ? Dokładnie nie wiem jak to może/powinno wyglądać..)

[SQL] pobierz, plaintext 
DELIMITER //
CREATE PROCEDURE procCreateTableWyniki(IN dataORG CHAR(10))
BEGIN
    SET @dataMOD = (SELECT YEAR(dataZawody) FROM zawodynazwa WHERE dataZawody = dataORG);      #tutaj z daty '2013-01-01' chce dalej wyciągnąć tylko rok '2013'
    SET @wynikiTab = CONCAT('wyniki_',@dataMOD); 
    SET @sql = CONCAT('CREATE TABLE ',@wynikiTab,' (idWynik int(50) NOT NULL AUTO_INCREMENT)');
 
    IF NOT EXISTS (SELECT * FROM lata WHERE rok = @dataMOD) THEN
       PREPARE s1 FROM @sql;
       EXECUTE s1;
    END IF;
END//
DELIMITER ;
[SQL] pobierz, plaintext 

Wszelka pomoc/sugestie mile widzane (IMG:style_emoticons/default/smile.gif)
(ps. proszę tylko o nie komentowania tego rozwiązania w stylu - "tak się nie robi, że tabela ma nazwe wyniki_2012, wyniki_2013.." bo ja o tym wiem.. ale to jest jak już wspominałem bardziej udowodnienie tego "że się da".

Ten post edytował mikajlo 15.12.2012, 15:34:03