Forum PHP.pl > [ROZWIĄZANY]Problem z pobieraniem danych + funkcja

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [ROZWIĄZANY]Problem z pobieraniem danych + funkcja

Forum PHP.pl > Forum > Bazy danych > MySQL

krun

17.06.2008, 00:31:15

Witam.

Mam następujący problem.
Napisałem sobie funkcje, która z kilku tabel wyciąga jedną wartość i to działa ładnie, zapytanie zwraca taki wynik:

Kod

+---+---+---+
| a | b | c |
+---+---+---+
| 1 | 1 | 3 |
+---+---+---+
| 4 | 1 | 1 |
+---+---+---+
| 5 | 1 | 1 |
+---+---+---+
| 1 | 2 | 3 |
+---+---+---+
| 4 | 2 | 1 |
+---+---+---+
| 5 | 2 | 1 |
+---+---+---+
| 1 | 3 | 3 |
+---+---+---+
| 4 | 3 | 1 |
+---+---+---+
| 5 | 3 | 1 |
+---+---+---+

a i b to wartości pobierane bezpośrednio z bazy danych, c to wartość pobierana z funkcji, i teraz moje pytanie - dlaczego jeśli przyrównam b z c(z funkcją to zwracane są tylko 2 wartości? 4-1-1 i 5-1-1, powinna zostać zwrócona jeszcze wartość 1-3-3 w końcu tu też b-c!

Ps.
Przepraszam za mało precyzyjny temat, ale nie wiedziałem jak to zatytułować.

nevt

17.06.2008, 08:10:24

pokaż to porównanie - pokaż tą funkcję - mamy wrózyć z fusów czy z tej tabelki co pokazałeś (to się chyba nazywa numerologia czy jakoś tak...)

krun

21.06.2008, 23:16:06

Funkcja wygląda tak:

[SQL] pobierz, plaintext 
CREATE FUNCTION `znajdz_id_do_wyslania`(`inpt_email_id` INT, `inpt_data` TIMESTAMP) RETURNS int(11) UNSIGNED
BEGIN DECLARE `w` INT(11);
  DECLARE `p` INT(11);
  DECLARE `i` INT(11);
 
  SET `w` = (SELECT `wyslane`.`id` FROM `wyslane` WHERE `wyslane`.`id_emaila`=`inpt_email_id` 
    ORDER BY `wyslane`.`data` DESC LIMIT 1   );
 
  IF (SELECT `wyslane`.`data` FROM `wyslane` WHERE `wyslane`.`id`=`w`) <= `inpt_data` THEN
    SET `p` = (SELECT `listy`.`priorytet`  FROM `listy`, `wyslane` 
      WHERE `wyslane`.`data` <= `inpt_data` AND `wyslane`.`id_listu`=`listy`.`id` AND `wyslane`.`id_emaila`=`inpt_email_id` 
      ORDER BY `wyslane`.`id` DESC LIMIT 1   );
  ELSEIF `w` IS NULL THEN
    SET `p`=0;
  ELSE
    RETURN NULL;
  END IF;
 
  SET `p` = (`p` + 1);
 
  SET `i` = (SELECT `listy`.`id` FROM `listy` WHERE `listy`.`priorytet`=`p`);
 
  RETURN `p`;
 
END
[SQL] pobierz, plaintext

Zapytanie tak

[SQL] pobierz, plaintext 
SELECT `emaile`.`id`, `listy`.`id`, znajdz_id_wyslanego(`emaile`.`id`, 'jakas-data') 
    FROM `listy`, `emaile` WHERE `listy`.`id`=znajdz_id_wyslanego(`emaile`.`id`, 'data-ta-sama-co wczesniej')
[SQL] pobierz, plaintext

A tabele tak:

[SQL] pobierz, plaintext 
CREATE TABLE `emaile` (
  `id` int(11) UNSIGNED NOT NULL AUTO_INCREMENT,
  `data` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP,
  `email` varchar(256) NOT NULL,
  `klucz` varchar(20) NOT NULL,
  PRIMARY KEY  (`id`),
  UNIQUE KEY `unq` (`email`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ;
 
CREATE TABLE `listy` (
  `id` int(10) UNSIGNED NOT NULL AUTO_INCREMENT,
  `temat` varchar(500) NOT NULL,
  `tresc` longtext NOT NULL,
  `priorytet` int(11) NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ;
 
CREATE TABLE `wyslane` (
  `id` int(10) UNSIGNED NOT NULL AUTO_INCREMENT,
  `id_emaila` int(10) UNSIGNED NOT NULL,
  `id_listu` int(10) UNSIGNED NOT NULL,
  `data` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP,
  PRIMARY KEY  (`id`),
  UNIQUE KEY `unq` (`id_emaila`,`id_listu`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ;
[SQL] pobierz, plaintext

Ale dziwne dla mnie jest to, że 3 != 3!

Przepraszam za natarczywość, ale bardzo zależy mi na odpowiedzi. Ma ktoś pojęcie czym mój problem może być spowodowany?

nevt

21.06.2008, 23:38:48

szczerze mówiąc wszystko wydaje sie być ok. ... dziwne...

co prawda twoja funkcja zwraca liczbę typu int(11) którą porównujesz z polem id typu int(10) unsigned ... ale zasadniczo nie powinno to mieć znaczenia dla wyniku porównania, na wszelki wypadek jednak spróbuj zmienić funkcję tak, żeby też zwracała int(10) unsigned ...

a drugie podejrzenie, to dwukrotne wywołanie twojej funkcji w każdym wierszu zapytania ... teoretycznie nie powinno mieć to wpływu na rezultat ale na pewno nie jest wydajne... spróbuj zmienić zapytanie na coś w rodzaju:

[SQL] pobierz, plaintext 
SELECT `emaile`.`id`, `listy`.`id`, znajdz_id_wyslanego(`emaile`.`id`, 'jakas-data') AS`znalezione_id` FROM `listy`, `emaile` WHERE `listy`.`id`=`znalezione_id`;
[SQL] pobierz, plaintext

krun

24.06.2008, 22:01:19

Sprawdziłem, nie działa. Ale najdziwniejsze jest to, że jak porównuje to z liczbami wprowadzonymi ręcznie czyli znajdz_id_wyslanego()=2 to też jest to falsz. działa tylko w przypadku liczby 1. Ktoś zna możliwy powód?

Druga sprawa

Cytat(nevt @ 21.06.2008, 22:38:48 )

drugie podejrzenie, to dwukrotne wywołanie twojej funkcji w każdym wierszu zapytania ... teoretycznie nie powinno mieć to wpływu na rezultat ale na pewno nie jest wydajne... spróbuj zmienić zapytanie na coś w rodzaju:

[SQL] pobierz, plaintext 
SELECT `emaile`.`id`, `listy`.`id`, znajdz_id_wyslanego(`emaile`.`id`, 'jakas-data') AS`znalezione_id` FROM `listy`, `emaile` WHERE `listy`.`id`=`znalezione_id`;
[SQL] pobierz, plaintext

To nie działa,

Cytat

Nie znaleziono kolumny 'znalezione_id' w where clause.

Ale przy pojedynczym wywołaniu funkcji też nie zwraca tych wyników które powinien, więc na pewno nie jest to wina podwójnego wywołania a mój błąd gdzieś w funkcji.. Tylko kto mi powie jaki?

osiris

25.06.2008, 23:49:07

Czy w definicji funkcji `znajdz_id_wysylanego` nie powinenes przypadkiem uzywac zmiennych o identyfikatorach poprzedzonych znakiem @?

Tj.

[SQL] pobierz, plaintext 
DECLARE @w INT;
 
SET @w = (SELECT ....);
...
[SQL] pobierz, plaintext

Pozdrawiam

krun

30.06.2008, 00:47:26

Cytat(osiris @ 25.06.2008, 22:49:07 )

Czy w definicji funkcji `znajdz_id_wysylanego` nie powinenes przypadkiem uzywac zmiennych o identyfikatorach poprzedzonych znakiem @?

Tj.

[SQL] pobierz, plaintext 
DECLARE @w INT;
 
SET @w = (SELECT ....);
...
[SQL] pobierz, plaintext

Pozdrawiam

Nie wydaje mi się. Dlaczego miałbym ich używać?

Problem rozwiązał się właściwie sam... Na innym systemie próbowałem wrzucić tą funkcje i zostałem poproszony o dodanie do niej dwóch linijek:

[SQL] pobierz, plaintext 
READS SQL DATA
DETERMINISTIC
[SQL] pobierz, plaintext

Sama funkcja powinna wyglądać tak:

[SQL] pobierz, plaintext 
CREATE FUNCTION `znajdz_id_do_wyslania`(`inpt_email_id` INT, `inpt_data` TIMESTAMP) RETURNS int(11) UNSIGNED
    READS SQL DATA
    DETERMINISTIC
BEGIN DECLARE `w` INT(11);
  DECLARE `p` INT(11);
  DECLARE `i` INT(11);
 
  SET `w` = (SELECT `wyslane`.`id` FROM `wyslane` WHERE `wyslane`.`id_emaila`=`inpt_email_id`
    ORDER BY `wyslane`.`data` DESC LIMIT 1  );
 
  IF (SELECT `wyslane`.`data` FROM `wyslane` WHERE `wyslane`.`id`=`w`) <= `inpt_data` THEN
    SET `p` = (SELECT `listy`.`priorytet` FROM `listy`, `wyslane` 
      WHERE `wyslane`.`data` <= `inpt_data` AND `wyslane`.`id_listu`=`listy`.`id` AND `wyslane`.`id_emaila`=`inpt_email_id` 
      ORDER BY `wyslane`.`id` DESC LIMIT 1  );
  ELSEIF `w` IS NULL THEN
    SET `p`=0;
  ELSE
    RETURN NULL;
  END IF;
 
  SET `p` = (`p` + 1);
 
  SET `i` = (SELECT `listy`.`id` FROM `listy` WHERE `listy`.`priorytet`=`p`);
 
  RETURN `p`;
 
END
[SQL] pobierz, plaintext

Dziękuje wszystkim którzy starali się mi pomóc.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.