[MYSQL/PHP] Nie wydajne zapytanie ? Opcje

[b4x]

[PHP] pobierz, plaintext 
<?php
	function check_uonline($id) {
 
		$p = mysql_fetch_array(mysql_query('SELECT `online` FROM `users` WHERE `id` = "'.$id.'"'));
 
			return $p['online'];
 
}
 
function friends_online($user) {
 
		$u = mysql_query('SELECT `friend` FROM `friends` WHERE `user_id` = "'.$user.'" AND `status` = "1"');
 
		 while($row = mysql_fetch_assoc($u)) {
 
			if($this->check_uonline($row['friend']) >= 1) {
 
				$i++;
 
			}
 
	  }
 
			return $i;
 
	}
 
	function count_stats_users($user) {
 
	$online = mysql_num_rows(mysql_query('SELECT `id` FROM `users` WHERE `online` > "0"'));
	$total = mysql_num_rows(mysql_query('SELECT `id` FROM `users`'));
	$friends_to_accept = mysql_num_rows(mysql_query('SELECT * FROM `friends` WHERE `user_id` = "'.$user.'" AND `inviter` = "0" AND `status` = "0"'));
	$messages = mysql_num_rows(mysql_query('SELECT `id` FROM `messages` WHERE `to` = "'.$user.'"'));
	$new_messages = $this->msgs_count($user);
 
 
		$array = array('online' => $online, 'total' => $total, 'friends_accept' => $friends_to_accept, 'friends_online' => $this->friends_online($user), 'messages' => $messages, 'new_messages' => $new_messages);
 
		return $array;
 
	}
?>
[PHP] pobierz, plaintext 

Nie wiem, jak zrobić, by było to bardziej wydajniejsze.. Ktoś pomoże ?

Ten post edytował b4x 25.07.2008, 10:23:57

[Kicok]

Pierwsze dwie funkcje: mysql.com -> documentation -> LEFT JOIN
Ostatnia funkcja: mysql.com -> documentation -> COUNT

[b4x]

[PHP] pobierz, plaintext 
<?php
$q = mysql_query('SELECT login FROM friends LEFT JOIN users ON (users.online = 1) WHERE friends.friend = '.$user.'') or die(mysql_error());
?>
[PHP] pobierz, plaintext 

Prawie dobrze... tylko że zwraca mi podwójnie wyniki

Tzn

Kod

Array
(
    [0] => login1
)
Array
(
    [0] => login2
)
Array
(
    [0] => login1
)
Array
(
    [0] => login2
)

Edit

[PHP] pobierz, plaintext 
<?php
$q = mysql_query('SELECT distinct login FROM friends LEFT JOIN users ON (users.online = 1) WHERE friends.friend = '.$user.'') or die(mysql_error());
?>
[PHP] pobierz, plaintext 

- tak działa poprawnie... ale czy zapytanie jest poprawne ?

Ten post edytował b4x 25.07.2008, 15:02:42

[Kicok]

Myślałem raczej o czymś takim:

[SQL] pobierz, plaintext 
SELECT f.friend, u.login FROM friends f
JOIN users u ON ( u.id = f.friend )
WHERE f.user_id = $user AND f.STATUS = 1 AND u.online = 1
[SQL] pobierz, plaintext 

Zwróć szczególną uwagę na linijkę w JOIN - ty w swoim zapytaniu w ogóle nie określiłeś jaka jest relacja między tymi tabelami, więc nie wiem czy zwracało poprawne wyniki.