Se me multiplica la consulta de 2 tablas

[unasemana]

Hola que tal,

tengo un pequenio problema al hacer una consulta a 2 tablas de mi base de datos.

Al realizar la consulta se me repiten los campos 2 veces, es ddecir :

Código:

35 	Y2K 	Batman 	1.00 	1000.00 	1120.00 	aprobado
34 	h1n1 	y2k 	1.00 	1000.00 	1120.00 	APROBADO
35 	Y2K 	Batman 	1.00 	1000.00 	1120.00 	aprobado
34 	h1n1 	y2k 	1.00 	1000.00 	1120.00 	APROBADO

el codigo que utilizo es este:

Código:

<?php 
// conectar al servidor

$connect = mysql_connect ("localhost","nimopart_invoice","invoices");

//conectar bd

mysql_select_db ("nimopart_invoices");

// buscar en bd
$query = mysql_query ( "SELECT * FROM mcb_invoice_items, mcb_invoice_item_amounts WHERE invoice_item_custom_1 = 'aprobado' ");

//atrapar resultados y mostrar

 WHILE ($columnas = mysql_fetch_array($query)):

$invoice_id = $columnas ['invoice_id'];
$item_name = $columnas ['item_name'];
$item_description = $columnas ['item_description'];
$item_qty = $columnas ['item_qty'];
$item_price = $columnas ['item_price'];
$item_total = $columnas ['item_total'];
$invoice_item_custom_1 = $columnas ['invoice_item_custom_1'];


echo "  
    <tr>  
      <td width='150'>".$columnas['invoice_id']."</td>  
      <td width='150'>".$columnas['item_name']."</td>  
      <td width='150'>".$columnas['item_description']."</td>
          <td width='150'>".$columnas['item_qty']."</td>
          <td width='150'>".$columnas['item_price']."</td>
          <td width='150'>".$columnas['item_total']."</td>
           <td width='150'>".$columnas['invoice_item_custom_1']."</td>     

    </tr>  
"; 




  endwhile;
?>

que puedo hacer para que no se me multiplique?, los nombres de las tablas son: mcb_invoice_items y mcb_invoice_item_amounts

[gnzsoloyo]

Código MySQL:

SELECT * 
FROM mcb_invoice_items, mcb_invoice_item_amounts 
WHERE invoice_item_custom_1 = 'aprobado' "
 

No estás poniendo las condiciones del JOIN, por lo que se está generando por lo menos un producto cartesiano.

Esto es una aproximación:

Código MySQL:

SELECT * 
FROM mcb_invoice_items I INNER JOIN mcb_invoice_item_amounts A ON I.invoice_id = A.invoice_id 
WHERE invoice_item_custom_1 = 'aprobado'
 

Pero el esquema de relaciones no es claro, porque una factura tiene dos partes (cabecera y detalle), más la relación con los productos, clientes y otros datos. Y el que pones no queda claramente definido ´cómo es.

¿Podrías postear la estructura y relaciones de las tablas involucradas?

[unasemana]

OK, leí que las comas "," hacían el papel de JOIN.

la primera tabla ( mcb_invoice_items)

invoice_item_id , item_name ,item_description, item_qty , item_price , invoice_item_custom_1

la segunda tabla (mcb_invoice_item_amounts) tiene estos campos:
invoice_item_amount_id, invoice_item_id , item_subtotal , item_tax, item_total

l

[leonardo_josue]

Las comas hacen la función del JOIN, pero no si no especificas sobre qué campos es la unión entonces se realiza el producto cartesiano:

Código MySQL:

mysql> SELECT * FROM tablaA;
+------+-------------+
| idA  | descripcion |
+------+-------------+
|    1 | uno         |
|    2 | dos         |
+------+-------------+
2 rows in set (0.00 sec)
 
mysql> SELECT * FROM tablaB;
+------+-------------+
| idB  | descripcion |
+------+-------------+
|    1 | one         |
|    2 | two         |
+------+-------------+
2 rows in set (0.00 sec)
 
mysql> #Producto cartesiano
mysql> SELECT * FROM tablaA, tablaB;
+------+-------------+------+-------------+
| idA  | descripcion | idB  | descripcion |
+------+-------------+------+-------------+
|    1 | uno         |    1 | one         |
|    2 | dos         |    1 | one         |
|    1 | uno         |    2 | two         |
|    2 | dos         |    2 | two         |
+------+-------------+------+-------------+
4 rows in set (0.00 sec)
 
mysql> SELECT * FROM tablaA INNER JOIN tablaB;
+------+-------------+------+-------------+
| idA  | descripcion | idB  | descripcion |
+------+-------------+------+-------------+
|    1 | uno         |    1 | one         |
|    2 | dos         |    1 | one         |
|    1 | uno         |    2 | two         |
|    2 | dos         |    2 | two         |
+------+-------------+------+-------------+
4 rows in set (0.00 sec)
 

En lo particular recomiendo siempre utilizar JOIN's, pero sea que utilices comas o JOINS DEBES ESPECIFICAR EL CRITERIO DE UNION EN CADA CASO.

Código MySQL:

mysql> SELECT * FROM tablaA, tablaB
    -> WHERE tablaA.idA = tablaB.idB;
+------+-------------+------+-------------+
| idA  | descripcion | idB  | descripcion |
+------+-------------+------+-------------+
|    1 | uno         |    1 | one         |
|    2 | dos         |    2 | two         |
+------+-------------+------+-------------+
2 rows in set (0.00 sec)
 
mysql>
mysql> SELECT * FROM tablaA
    -> INNER JOIN tablaB ON tablaA.idA = tablaB.idB;
+------+-------------+------+-------------+
| idA  | descripcion | idB  | descripcion |
+------+-------------+------+-------------+
|    1 | uno         |    1 | one         |
|    2 | dos         |    2 | two         |
+------+-------------+------+-------------+
2 rows in set (0.00 sec)
 

Saludos
Leo.

[unasemana]

Gracias por la ayuda me hiciste ver algo que no veía, lo he logrado cambiando esto:

Código:

$query = mysql_query ( "SELECT * FROM mcb_invoice_items, mcb_invoice_item_amounts WHERE invoice_item_custom_1 = 'aprobado' ");

por esto:

Código:

$query = mysql_query ( "SELECT * FROM mcb_invoice_items, mcb_invoice_item_amounts WHERE mcb_invoice_items.invoice_item_id = mcb_invoice_item_amounts.invoice_item_id and  invoice_item_custom_1 = 'aprobado' ");

espero le sirva a alguien más

[gnzsoloyo]

Borrada la parte que ya te explicó Leo (se me adelantó )
Tengo la impresión de que el diseño tiene algún error, pero para estar seguro: ¿Qué representa la tabla "mcb_invoice_item_amounts"? ¿La cabecera de la factura u otra cosa?

[gnzsoloyo]

Consejo: No uses la coma, a pesar de lo que dicen los manuales. Acostúmbrate a usar INNER JOIN, LEFT JOIN o RIGHT JOIN. De ese modo lograrás mejor control sobre lo que haces.
Y preferentemente no uses el asterisco (*). Especifica qué campos usarás, de ese modo no traerás datos "basura" al proceso.

[unasemana]

El diseño puede que tenga un error pero en este caso por suerte no es importante, solo interesa tener esos datos y ya,no es algo que va a tener acceso el publico. solo alguien interno.


Muchas gracias!