PHP OO relacion de categoria

[theoneal]

Lo que pasa es que cuando le doy clic a categoría (ESTACIÓN) me muestra todos los productos creados.
Lo que busco es que solo me muestre los productos relacionados a la
categoría (ESTACIÓN)

Tabla "categorias"
id
categoria

Tabla "productos"
id
modelo
precio
imagen
categoria

Este es el codigo que muestra la categoria:

<div>
<table border="1" id="categoria">
<tr>
<td>Categorias</td>
</tr>
<?php

include_once("config.php");

$result = mysql_query("SELECT * FROM categorias ORDER BY id DESC") or die(mysql_error());
if(!mysql_num_rows($result)){
echo "Todavía no ha sido publicada ninguna categoria";
} else {

while($qry = mysql_fetch_array($result)){
echo "
<tr>
<td><a href='categorias.php?id=".$qry['id']."'>".$qry['categoria']."</a></td>
</tr>
";
}
}
?>
</table>
</div>


y este es el que muestra los productos:

<div id="cuadrogrande">
<?php
include_once("config.php");
$col = 0;
$result = mysql_query("SELECT * FROM $tabla ORDER BY id DESC LIMIT 8") or die(mysql_error());
if(!mysql_num_rows($result)){
echo "Todavía no ha sido publicada ninguna Producto";
} else {
echo "<table><tr>";
while($qry = mysql_fetch_array($result)){
$col++;
echo "<td>
<table id=\"cuadro\">
<tr>
<td rowspan=\"4\" class=\"marco\">
<a href='articulo.php?id=".$qry['id']."'>
<img src=\"".$qry['imagen']."\" class=\"foto\" />
</a>
</td>
<td>Modelo</td>
</tr>
<tr>
<td>".$qry['modelo']."</td>
</tr>
<tr>
<td>Precio</td>
</tr>
<tr>
<td>".$qry['precio']."</td>
</tr>
</table>
<br/>
<td>";
if ($col==2)
{
$col=0;
echo "</tr><tr>";
}
}
echo "
</tr>
</table>";

echo "
</td>
</tr>
</table><br /><br />";
}?>
</div>


Saludos y buen fin de semana
Gracias

[NSD]

1) Usa los highlight de codigo, se hace mucho mas facil de leer.
2)

Código PHP:

$result = mysql_query("SELECT * FROM $tabla ORDER BY id DESC LIMIT 8") or die(mysql_error());
 

2.a) $tabla no esta definida, pero supongo que apunta a la tabla de productos.
2.b) para traer los de una categoria te falta el where:

Código PHP:

$result = mysql_query("SELECT * FROM $tabla WHERE categoria = ".$_GET['id']." ORDER BY id DESC LIMIT 8") or die(mysql_error());
 

3) Deberias sanear o filtrar de alguna manera los datos que metes en la consulta.
3.a) Como el id es un entero, una solucion rapida es intval()

Código PHP:

$result = mysql_query("SELECT * FROM $tabla WHERE categoria = ".intval($_GET['id'])." ORDER BY id DESC LIMIT 8") or die(mysql_error());
 

4) la clase mysql_* esta obsoleta y sera eliminada de php en un futuro cercano, lo mejor es que cuanto antes te pases a mysqli_* que es muy similar (de hecho es una extencion de mysql_*) pero que ofrece nuevas funciones y metodos (mas seguros y mas lentos) para trabajar con la bd.

Código PHP:

<?php 
// Esto iria en el config.php
$enlace = mysqli_connect("127.0.0.1", "usuario", "contraseña", "world"); 
 
if (!$enlace)
{
    $error = mysqli_connect_error();
    $errno = mysqli_connect_errno();
    print "$errno: $error\n";
    exit();
}
// Esto en tu archivo:
 
$consulta = "SELECT * FROM $tabla WHERE categoria = ? ORDER BY id DESC LIMIT 8";
 
$sentencia = mysqli_stmt_init($enlace);
 
if(!mysqli_stmt_prepare($sentencia, $consulta))
    die("Falló la preparación de la sentencia\n");
    
mysqli_stmt_bind_param($stmt, "i", $_GET["id"]);
mysqli_stmt_execute($sentencia);
$resultado = mysqli_stmt_get_result($sentencia);
 
while ($qry = mysqli_fetch_array($resultado, MYSQLI_ASSOC))
{
    // Lo musmo que haces ahora
}
 
// Esto al final, es para matar las conexiones.
mysqli_stmt_close($sentencia);
mysqli_close($enlace);
?>
 

a simple vista puede parecerte mucho mas complejo (de hecho, un poco si lo es) pero una vez te acostumbras se hace igual de facil, ademas el hecho de bindear parametros (mysqli_stmt_bind_param($stmt, "i", $_GET["id"]);) te soluciona la mayor parte de los problemas de inyeccion sql, ya no debes preocuparte por que valor viene en $_GET ya que al bindearlo eliminas el risego de un posible ataque. No nesesitas andar saneando o parseando la variable.

saludos