Ver Mensaje Individual
  #1 (permalink)  
Antiguo 03/04/2013, 19:17
sandymiau
 
Fecha de Ingreso: mayo-2012
Ubicación: En la Tierra
Mensajes: 41
Antigüedad: 11 años, 11 meses
Puntos: 3
seleccionar datos

Hola.

Antes que nadad mi código funciona, solo que quisiera me lo revisen y me digan si es posible compactarlo.

Tengo una base de datos de inmobiliaria: casas, terrenos, ranchos, etc.. y una busqueda con elementos select en HTML:


Código HTML:
Ver original
  1. <form action="http://localhost/buscar.php" method="post">
  2.                       <p class="look" >Busqueda<p/>
  3.                       <p>
  4.                      <label for="Operacion">Operaci&oacute;n:</label>
  5.                      <select name="Operacion" id="Operacion">
  6.                                            <option value="1">Renta</option>
  7.                        <option value="2">Venta</option>
  8.                         </select>
  9.                      </p>
  10.                     <p>
  11.                                             <label for="TipoInmueble" >Tipo de inmueble:</label>
  12.                      <select name="TipoInmueble" id="TipoInmueble">
  13.                                            <option value="0">selecciona</option>  
  14.                        <option value="1">Casa</option>
  15.                                            <option value="2">Departamentos</option>
  16.                        <option value="3">Oficinas</option>
  17.                        <option value="4">Locales</option>
  18.                        <option value="5">Terrenos</option>
  19.                        <option value="6">Ranchos</option>
  20.                        <option value="7">Bodegas</option>
  21.                        <option value="8">Edificios</option>
  22.                      </select>
  23. <input type="submit" name="Submit" value="Buscar" class="buttonSubmit" />
  24.                      </p>
  25. </form


El codigo PHP que realice es el siguiente:


Código PHP:
Ver original
  1. <?php
  2.         // put your code here
  3.         require_once 'midb1.php';
  4. $db_server = mysql_connect($db_hostname, $db_username, $db_password);
  5. if (!$db_server) die("Unable to connect to MySQL: " . mysql_error());
  6. mysql_select_db($db_database)
  7. or die("Unable to select database: " . mysql_error());
  8. $operacion = $_POST['Operacion'];
  9.  
  10. if ($_POST['Operacion']== 1 && $_POST['TipoInmueble'] == 1 && isset ($_POST['colonia']) &&  isset ($_POST['Presupuesto']) && isset ($_POST['Clave'])  )
  11. {    
  12. $tablaciudades = mysql_query("SELECT * FROM casitas WHERE win = 'C' AND operacion = 'R';");
  13.      
  14.      if($tablaciudades === FALSE) {
  15.     die(mysql_error()); // TODO: better error handling
  16.      }    
  17.      while($row = mysql_fetch_array($tablaciudades))
  18. {
  19.      echo '<ul>';
  20.          echo '<li>' . '<img src="'.$row['Imaenes'] . '"/>' . '</li><li class = "price">' .  $row['Precio'] . '</li><li> colonia'. " " .$row['colonia']." " . $row['codigo postal']. " " . $row['Municpio'] . " " . $row['Estado'] . '</li>';
  21.          echo '<li>Baños ' . $row['num_banios'] . '</li><li>Habitaciones ' . $row['num_recamaras'] . '</li><li>clave ' .  $row['clave'] . '</li> . <li class = "price">' .  $row['win'] . '</li>';
  22.          echo '</ul>';
  23. }
  24.  
  25.  }elseif ($_POST['Operacion']== 2 && $_POST['TipoInmueble'] == 1 && isset ($_POST['colonia']) &&  isset ($_POST['Presupuesto']) && isset ($_POST['Clave'])){
  26.      
  27.      $tablaciudades = mysql_query("SELECT * FROM casitas WHERE win = 'C' AND operacion = 'V';");
  28.      
  29.      if($tablaciudades === FALSE) {
  30.     die(mysql_error()); // TODO: better error handling
  31.      }    
  32.      while($row = mysql_fetch_array($tablaciudades))
  33. {
  34.     echo '<ul>';
  35.          echo '<li>' . $row['Imaenes'] . '</li><li class = "price">' .  $row['Precio'] . '</li><li> colonia'. " " .$row['colonia']." " . $row['codigo postal']. " " . $row['Municpio'] . " " . $row['Estado'] . '</li>';
  36.          echo '<li>Baños ' . $row['num_banios'] . '</li><li>Habitaciones ' . $row['num_recamaras'] . '</li><li>clave ' .  $row['clave'] . '</li>';
  37.          echo '</ul>';
  38. }
  39.  }elseif ($_POST['Operacion']== 2 && $_POST['TipoInmueble'] == 2 && isset ($_POST['colonia']) &&  isset ($_POST['Presupuesto']) && isset ($_POST['Clave'])){
  40.      
  41.      $tablaciudades = mysql_query("SELECT * FROM casitas WHERE win = 'C' AND operacion = 'V';");
  42.      
  43.      if($tablaciudades === FALSE) {
  44.     die(mysql_error()); // TODO: better error handling
  45.      }    
  46.      while($row = mysql_fetch_array($tablaciudades))
  47. {
  48.     echo '<ul>';
  49.          echo '<li>' . $row['Imaenes'] . '</li><li class = "price">' .  $row['Precio'] . '</li><li> colonia'. " " .$row['colonia']." " . $row['codigo postal']. " " . $row['Municpio'] . " " . $row['Estado'] . '</li>';
  50.          echo '<li>Baños ' . $row['num_banios'] . '</li><li>Habitaciones ' . $row['num_recamaras'] . '</li><li>clave ' .  $row['clave'] . '</li>';
  51.          echo '</ul>';
  52. }
  53.  }
  54.  
  55.         ?>


El código como veran, tiene muchos if, ¿como puedo mejorarlo?

Salu2.