Foros del Web - Ver Mensaje Individual

masterojitos · #33 (**permalink**) 07/12/2009, 12:14

este veo que no tomaste mis opiniones, pero bueno, solo quiero decir algo....

encima lo arreglas.. jaja......... osea por que hacen el codigo asi.......

Código php:

Ver originalif(isset($_POST['eliminar'])){
 $q=mysql_query("SELECT * FROM ciudades WHERE cod_pais='".$_POST['Pais']."'") or die(mysql_error());
 $count=mysql_num_rows($q) or die(mysql_error());
 if($count>0){
  for($i=0;$i<$count;$i++){
    $sql = "DELETE FROM ciudades WHERE cod_pais='".$_POST['Pais']."'";
    mysql_query($sql)  or die(mysql_error());
    echo "Se eliminaron ".$count." ciudades";
  }
 }else{
  echo "No hay ciudades a eliminar, procediendo a eliminar el país...";
 }
 $psql = "DELETE FROM paises WHERE cod_pais='".$_POST['Pais']."'";
 mysql_query($psql)  or die(mysql_error());
}

que fatal esa programacion.....

primer error fatal..... es que cuando haga click en eliminar y haga submit.... cuando quieras eliminar no hay conexion.. ya que esta mas abajo....

y ps.... por que recorres las ciudades para eliminar despues todas.... ???
es decir .:

si hay 20 ciudades, haces un bucle (for) de esas 20 y eliminas de porraso las 20, y luego a la siguiente iteracion imprimes que ya no hay mas que eliminar... jaja..... logico no ????

creo que mas sencillo es defrente eliminar todas no ???? hahaha

creo que tu idea era recorrer las ciudades y luego eliminar una por una..... hehe

cierto, utiliza <?php ya que esta mal el usar solo <?
Bueno, segun yo... el codigo seria asi:

Código html:

Ver original<html>
<head>
<title>Atlas - Paises - A&ntilde;adir </title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<script LANGUAGE="JavaScript">
function confirmSubmit(){
return (confirm("Al eliminar el pais todas las ciudades se eliminaran, desea continuar?")) ? true : false ;
}
</script>
</head>
<body>
<table width="760" border="1" cellpadding="5" bordercolor="#000000" bgcolor="#000000">
  <tr>
    <td colspan="5" bordercolor="#000000" bgcolor="#FFFF00"><h1 class="Estilo2">Atlas - Paises </h1></td>
  </tr>
  <tr>
    <td width="137" bordercolor="#000000" bgcolor="#FFFF00"><span class="Estilo2">- <a href="Index.php">INICIO</a> </span></td>
    <td width="137" bordercolor="#000000" bgcolor="#FFFF00"><span class="Estilo2">- <a href="Add_pais.php">A&Ntilde;ADIR</a> </span></td>
    <td width="137" bordercolor="#000000" bgcolor="#FFFF00"><span class="Estilo2">- <a href="Listar_pais.php">LISTAR</a> </span></td>
    <td width="137" bordercolor="#000000" bgcolor="#FFFF00"><span class="Estilo2">- <a href="Borrar_pais.php">BORRAR</a> </span></td>
    <td width="138" bordercolor="#000000" bgcolor="#FFFF00"><span class="Estilo2">- <a href="Borrart_pais.php">BORRAR TODO</a> </span></td>
  </tr>
</table>
 
<?php
$cn = mysql_connect("localhost","root","juni0rh4ncco") or die("Problemas en la conexion");
mysql_select_db("atlass",$cn) or die("Problemas en la seleccion de la base de datos");
 
if(isset($_POST['eliminar'])){
 mysql_query("DELETE FROM ciudades WHERE cod_pais='".$_POST['pais']."'");
 mysql_query("DELETE FROM paises WHERE cod_pais='".$_POST['pais']."'");
}
 
echo '<form id="submitform" name="submitform" method="post">';
echo '<select name="pais">';
$c=mysql_query("SELECT cod_pais, nombre_pais FROM paises ORDER BY nombre_pais");
while($r=mysql_fetch_array($c)) echo "<option value='$r[0]'>$r[1]</option>";
echo '</select><br />';
ech0 '<input type="Submit" name="eliminar" value="Eliminar Pais" onClick="return confirmSubmit()"></form>';
?>
</body>
</html>

espero me haya sabido explicar..... y ps te sirva el nuevo codigo que te pase.......

suerte.