Un poquito de ayuda por favor

urgido · #31 (**permalink**) 07/12/2009, 11:54

dale, reemplaza lo que te puse por:

<?
$conexion=mysql_connect("localhost","root","juni0r h4ncco") or die("Problemas en la conexion");
mysql_select_db("atlass",$conexion) or die("Problemas en la seleccion de la base de datos");

if(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());
}

// normal

$c=mysql_query("SELECT * FROM paises ORDER BY nombre_pais");
echo '<form id="submitform" name="submitform" action="'.$_SERVER['PHP_SELF'].'" method="post">';
echo '<select name="Pais">';
while($r=mysql_fetch_array($c)){
echo '<option value="'.$r['cod_pais'].'">'.$r['nombre_pais'].'<br></option>';
}
echo '</select><br><input type="Submit" name="eliminar" value="Eliminar Pais" onClick="return confirmSubmit()"></form>';
?>

shadowjhac · #32 (**permalink**) 07/12/2009, 12:13

es hora de irme a la preparatoria, lo revisare ahy, pero de seguro esta super bien , como siempre se espera de ti.

muchas gracias querido maestro y disculpa por quitarte tu preciado tiempo...

saludos..

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.

urgido · #34 (**permalink**) 07/12/2009, 12:24

bien @masterojitos te ganaste un lugar en el post, lo único que puedes!

Saludos ;)

masterojitos · #35 (**permalink**) 07/12/2009, 12:29

Cita:

Iniciado por urgido

lo único que puedes!

jajaja...... sono como que te sientes ofendido ?? o creido por que segun tu ya lo habias hecho muy bien ????

yo solo trate de ayudar a mi amigo a tener una programacion mas ordenada y pulida, ya que la tuya obviamente tiene cosas fatales..... lo cual no es un buen ejemplo de programacion.

si te sientes ofendido por que alguien mejora tu codigo.... ps lo veo mal..

shadowjhac · #36 (**permalink**) 07/12/2009, 13:08

No hay necesidad de pelear.. antes mencione q era recontra novato en php y el unico q m ofrecio su ayuda fue Urgido.

me ayudaste con algunas lineas de codigo pero no lo entendi, Urgido generosamente me ayudo de forma casi completa con mi problema.

Ademas como lo hizo Urgido esta bien, es el codigo q necesito.

Si no es molestia preferiria que el me siga dando clases sobre php

saludos..!!

Pd. Master Urgido. gracias por el codigo que m dist, m sacast de dudas y funciona de maravilla

PD2. ya te mando un privado xD

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

Bueno si lo crees que es bueno empezar a aprender php, aprendiendo de una forma fatal como la que posteo urgido, es tu problema..... aca estamos para brindar ayuda no para dar peleas...
pero si esta en nuestro alcanze mejorar codigos para la mejora de los programadores, bien no?.

shadowjhac · #38 (**permalink**) 07/12/2009, 13:21

Cita:

Iniciado por masterojitos

Bueno si lo crees que es bueno empezar a aprender php, aprendiendo de una forma fatal como la que posteo urgido, es tu problema..... aca estamos para brindar ayuda no para dar peleas...
pero si esta en nuestro alcanze mejorar codigos para la mejora de los programadores, bien no?.

bueno de ah pokito se aprende

PD. ademas es el unico que me brindo su ayuda de manera desinteresada

saludos...!!