como es la forma correcta de programar?

hoberwilly · #1 (**permalink**) 29/05/2009, 15:56

Buenas amigos, quisiera por favor me podrian ayudar de como es la manera de programar lo siguiente:
1. Tengo una pagina que es el index (usa include)

Código php:

Ver original<?php
session_start();
include("conec.php");
$link = conectarse();
$p = "";
if(isset($_GET['p'])){
    $p = $_GET['p'];
}
?>
<html>
<head>
</head>
<body>
<?php include("include/cabecera.php"); ?>
<table>
<tr>
  <td valign="top"><br>
  <?php
  if($p==1){
    include("include/pagina1.php");
  }elseif($p==2){
    include("include/pagina2.php");
  }else{
    include("include/nosotros.php");
  }
  ?>
  </td>
</tr>
</table>
</body>
</html>

este recoje x la URL la variable p (1=Pagina1 , 2=Pagina2)

2. Este es una hoja que esta incluida en el index (cabecera.php)

Código php:

Ver original<table border="0" cellpadding="0" cellspacing="0" align="center" width="100%">
<tr>
 <td align="center" width="10%"><a href="index.php?p=1">PAGINA 1</a></td>
  <td align="center" width="10%"><a href="index.php?p=2">PAGINA 2</a></td>
 </tr>
</table>

3. Y según el click, si clickeo en PAGINA 1 (de la cabecera) imprime pagina1.php:

Código php:

Ver original<?php
$categoria=$_GET['p'];
$_pagi_sql="SELECT ID,PROD,PREC FROM CATALOGO 
            WHERE EST='1' AND CATEGORIA='DAMA' AND STOCK>0 
            ORDER BY ID ASC";
if(mysql_num_rows($_pagi_result)==0){                                               
    echo "<h3>No hay productos</h3>";
}else{
?>
<table border="0" width="100%" height="100%" cellpadding="0" cellspacing="13" align="center" class="tablita">     
<tr>
  <td height="1">hay mas</td>
</tr>
</table>
<?php
}
?>

y, si clickeo en PAGINA 2 (de la cabecera) imprime pagina2.php:

Código php:

Ver original<?php
$categoria=$_GET['p'];
$_pagi_sql="SELECT ID,PROD,PREC FROM CATALOGO 
            WHERE EST='1' AND CATEG='VARON' AND STOCK>0 
            ORDER BY ID ASC";
if(mysql_num_rows($_pagi_result)==0){                                               
    echo "<h3>No hay productos</h3>";
}else{
?>
<table border="0" width="100%" height="100%" cellpadding="0" cellspacing="13" align="center" class="tablita">     
<tr>
  <td height="1">hay mas</td>
</tr>
</table>
<?php
}
?>

Si se han fijado SOLO HAY UNA DIFERENCIA entre pagina1.php y pagina2.php que esta en la sentencia SQL:
...WHERE EST='1' AND CATEGORIA='DAMA' ...
...WHERE EST='1' AND CATEGORIA='VARON' ...

Explico mi caso debido a que estas mismas paginas van incrementandose (pero solo con la unica diferencia de CATEGORIA) y veo k en el futuro para dar mantenimiento va causar verdaderos problemones.

Existe una manera optima como programarlos???Ayudenme...

Gracias de antemano,

pateketrueke · #2 (**permalink**) 29/05/2009, 17:09

si, la idea es separar mas el código... si ya notaste que solo hay un cambio, concentrate en programar la lógica para que ese cambio sea aplicado...

a eso se le llama redundancia, mientras menos tengas mas fácil te va a ir en el futuro...

pues si $p fuera $categoria ¿¿crees que sería menos código o no???

Código PHP:

  $sql = "SELECT * FROM tabla WHERE columna = '$p'";

o algo así, espero te sirva... suerte!

jaao_death · #3 (**permalink**) 29/05/2009, 17:21

la verdad me parece muy ambiguo eso que dices Hober, especifica más qué es lo que quieres hacer, por otro lado te recomiendo que utilices el api mysqli, hay que modernizarse, y que escapes la variable $p con mysqli::real_escape_string para que no te hagan una sql-inyection.