[ocena skryptu] wspólni znajomi Opcje

[sebekzosw]

Witam! Mam pewien skrypt:

[PHP] pobierz, plaintext 
$znajomy = 2;
$ja = 1;
 
$znajomi1 = array();
$znajomi1_query = mysql_query("SELECT `u`.`user_id` FROM  `friends` AS  `f` LEFT JOIN `users` AS  `u` ON  `u`.`user_id` = ( IF(  `f`.`user1` =".$znajomy.",  `f`.`user2` ,  `f`.`user1` ) ) WHERE (`user1` =".$znajomy." OR `user2` =".$znajomy.") AND `status` =1;");
while($znajomi1_row = mysql_fetch_assoc($znajomi1_query)) $znajomi1[] = $znajomi1_row['user_id'];
 
$znajomi2 = array();
$znajomi2_query = mysql_query("SELECT `u`.`user_id` FROM  `friends` AS  `f` LEFT JOIN `users` AS  `u` ON  `u`.`user_id` = ( IF(  `f`.`user1` =".$ja.",  `f`.`user2` ,  `f`.`user1` ) ) WHERE (`user1` =".$ja." OR `user2` =".$ja.") AND `status` =1;");
while($znajomi2_row = mysql_fetch_assoc($znajomi2_query)) $znajomi2[] = $znajomi2_row['user_id'];
 
print_r(array_intersect($znajomi1, $znajomi2));
[PHP] pobierz, plaintext 

Który porównuje mi moich i innej osoby znajomych. Oto wyniki:

Kod

moi znajomi:
Array
(
    [0] => 1996
    [1] => 1
    [2] => 24306
    [3] => 12639
    [4] => 5057
    [5] => 1757
)

znajomi kogoś:
Array
(
    [0] => 2
    [1] => 1757
)

wspólni znajomi:
Array
(
    [5] => 1757
)

Dodatkowo zamieszczam zrzut tabeli friends:

[SQL] pobierz, plaintext 
CREATE TABLE `friends` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `user1` int(11) NOT NULL COMMENT 'id usera 1 - zapraszajacego',
  `user2` int(11) NOT NULL COMMENT 'id usera 2 - zaproszonego',
  `status` int(1) NOT NULL DEFAULT '0' COMMENT '0 - niezaakceptowane, 1 - zaakceptowane',
  `data` int(11) NOT NULL,
  PRIMARY KEY  (`id`),
  KEY `id` (`id`,`user1`,`user2`,`status`,`data`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=2 ;
[SQL] pobierz, plaintext 

proszę o ocenę i poprawki

Ten post edytował sebekzosw 25.07.2010, 17:56:08

[phpion]

Ocena? Pała, siadaj.

Takie rzeczy możesz robić (i powinieneś!) na poziomie samej bazy danych. Co jeśli chciałbym do tego dołożyć np. imiona i nazwiska wspólnych znajomych? Możesz walnąć kolejne zapytanie (czyli mamy już 3). Możesz też w pierwszym lub drugim zapytaniu dołączyć dane użytkowników (ale przy np. 1000 znajomych pierwszego użytkownika i 0 wspólnych pobierasz zbędnie 1000 rekordów i zapisujesz je do tablicy).

Niestety MySQL nie wspiera INTERSECT. Jednak w Google znajdziesz rozwiązania okrężne. Podaję pierwsze z brzegu:

Doing an INTERSECT

An INTERSECT is simply an inner join where we compare the tuples of one table with those of the other, and select those that appear in both while weeding out duplicates. So

[SQL] pobierz, plaintext 
SELECT member_id, name FROM a
INTERSECT
SELECT member_id, name FROM b
[SQL] pobierz, plaintext 

can simply be rewritten to

[SQL] pobierz, plaintext 
SELECT a.member_id, a.name
FROM a INNER JOIN b
USING (member_id, name)
[SQL] pobierz, plaintext 

Źródło: http://www.bitbybit.dk/carsten/blog/?p=71

PS: Twój index też jest z dupy. Po pierwsze: id masz kluczem głównym więc po co pakować go w dodatkowy index (szczególnie na pierwszy miejscu). Po drugie: sprawdzałeś czy taka kombinacja (wszystkich pól!) sprawi, że index zostanie użyty? Nie sprawdzałeś, bo nie zostanie użyty. Poza tym: po co Ci kolumna id? Postaw klucz główny na dwóch kolumnach (user1 i user2). Patrząc dalej na strukturę tabeli: pole status powinno być typu ENUM z określoną listą wartości, polu data lepiej nadać typ DATETIME lub TIMESTAMP ("od tego są one, od tego one są, od tego są, od tego są one").

Ten post edytował phpion 26.07.2010, 08:02:45

[sebekzosw]

nie mam pojęcia jak to zapytanie całe połączyć, może jakaś pomoc?

[Czyluk]

http://funkcje.net/nsc/4/2867/0/38/page.html
Kwerendy są bardzo przydatne, radzę się ich nauczyć

Ten post edytował Czyluk 27.07.2010, 10:58:50

[sebekzosw]

nie potrafię tego wymyślić...

znalazłem takie coś: http://forums.digitalpoint.com/showthread.php?t=590030 ale nie działają te zapytania ... ;/

zna ktoś rozwiązanie?

Kod

SET @user = 2; @znajomy = 921;

SELECT
    IF(`f1`.`user1` = @user, `f1`.`user2`, `f1`.`user1`) AS `user_id`
FROM
    `friends` AS `f1`
WHERE
(`f1`.`user1` = @user OR `f1`.`user2` = @user) AND
`f1`.`status` = 1
AND
    IF(`f1`.`user1` = @user, `f1`.`user2`, `f1`.`user1`)
    IN (
        SELECT
            IF(`f2`.`user1` = @user, `f2`.`user2`, `f2`.`user1`)
        FROM
            `friends` AS `f2`
        WHERE
        (`f2`.`user1` = @znajomy OR `f2`.`user2` = @znajomy) AND
        `f2`.`status` = 1
    )
HAVING
    `user_id` != @znajomy;

co myślicie o tym zapytaniu?