Forum PHP.pl > [ocena skryptu] wspólni znajomi

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [ocena skryptu] wspólni znajomi

sebekzosw

25.07.2010, 17:52:34

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

phpion

26.07.2010, 07:56:37

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:

Cytat

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").

sebekzosw

27.07.2010, 02:12:35

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

Czyluk

27.07.2010, 07:58:03

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

sebekzosw

24.08.2010, 17:22:53

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?

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.