Problema con select y mostrar resultados

[jesusoful]

Hola. Hace poco publiqué un tema sobre un problema que tenía mostrando resultados... Resuelto ese, tengo otro parecido, pero éste por más que le doy las vueltas, no consigo encontrarle el fallo. El script en cuestión es éste:

Código PHP:

  function muestraAmigos() {
    
    // Instanciamos objeto de conexión con base de datos.
    $c = new bd();
    
    // Buscamos todos los usuarios que tengan el ID de nuestro usuario como amigo.
    $amigos=mysql_query("SELECT amigos.us1,amigos.us2,usuarios_perfil.username,usuarios_perfil.id FROM amigos,usuarios_perfil WHERE usuarios_perfil.id != '".$_SESSION['id']."' && amigos.us1 = '".$_SESSION['id']."' || amigos.us2 = '".$_SESSION['id']."'");
 
    while($us=mysql_fetch_array($amigos)) {
        $this->id_atacado=$us["id"];
        $this->atacado=$us["username"];
        echo  '<option value="'.$this->id_atacado.'">'.$this->atacado.'</option>';
        
    }
} 
   

Luego, dentro de un SELECT instancio un objeto de la clase en la que está contenido éste método, y después llamo (con el objeto instanciado) a éste método. Hasta ahí todo correcto, sin problema. El problema viene a la hora de mostrar los registros, me muestra registros dobles y el propio registro del usuario. Es decir:

• MiUsuario
• Amigo1
• Amigo1
• Amigo2
• Amigo2
• Amigo3
• Amigo3
• Y asi sucesivamente

Varias dudas:
1- No sé por que me muestra mi usuario, si como primera condición le estoy diciendo que NO coja los registros cuya ID sea igual a mi sesión de usuario.
2- No sé por qué me muestra registros dobles, si sólo existe un registro dentro de la base de datos "amigos" que apunte hacia dicha ID.

En fin... Dudas de novato, ya que apenas llevo 1 mes programando en PHP, y algo más de 1 semana con POO en PHP. Agradezco su ayuda de antemano.

Saludos!!

[emiliodeg]

pues hoy estas de buenas, pq solucionarlo es muy facil

sobre el final de tu sql agrega group by amigos.us1,amigos.us2,usuarios_perfil.username,usu arios_perfil.id

o agrupa por las columna q quieras q no se repita

[jesusoful]

EDITO

He probado a agrupar solo por nombre de usuario, y ahora me muestra sólo una vez cada registro. El problema viene ahora en que también me muestra mi usuario, cuando yo le estoy diciendo en el query que si el id es igual a mi sesión, que no me lo coja... Y la session corresponde con mi id (lo he comprobado).

Saludos.

[emiliodeg]

revisa q datos tienes cargados en las tablas, utiliza alguna tabla con menos datos para facilitar la tarea

proba con agregar parentesis, me imagino q lo q queres es algo asi

usuarios_perfil.id != '".$_SESSION['id']."' && (amigos.us1 = '".$_SESSION['id']."' || amigos.us2 = '".$_SESSION['id'].")'

sino pasa de nuevo y lo revisamos

[jesusoful]

Oh!! Fantástico

Justo eso quería... Una cosa más que ya sé: Pueden usarse paréntesis en querys (ni se me había pasado por la cabeza...).

Mil gracias!!.

Saludos.

[emiliodeg]

jejejej de nada