Dónde está el error?

[carlosuc99]

Buenas,

Tengo este código:

Código PHP:

  $query = mysql_query ("SELECT * FROM Pilots");
$filas = mysql_num_rows($query);
 
for($i = 0; $i < $filas; $i++){
    $horas = mysql_result($query, 0, 'Hours');
    
        if($horas < 20){
        mysql_query ("UPDATE Pilots SET Rango = 'IngenierodeVuelo'");
        }
 
        if($horas >= 20){
        mysql_query ("UPDATE Pilots SET Rango = 'PrimerOficial'");
        }
 
        if($horas >= 75){
        mysql_query ("UPDATE Pilots SET Rango = 'Comandante'");
        }
 
        if($horas >= 150){
        mysql_query ("UPDATE Pilots SET Rango = 'ComandanteHelitt'");
        }
        
    
} 
   

Lo que pretendo es crear unas comprobaciones para todas las entradas de la tabla Pilots.

Pero por algún motivo asigna mal el valor $horas. Le pone 99899899899899899899899899899899899899899899899899 8998998998. Este valor debería cambiar dependiendo del piloto.

[eits]

amigo con la info que das es como decir "tengo esta bolsa de agujas rojas, necesito encontrar una aguja azul" ahora, haciéndole al adivino intenta con esto

Código PHP:

$horas = mysql_result($query, $i, 'Hours');
 

cambia el 0 por $i

saludos.

[pateketrueke]

¿Y cómo sabes que esos valores obtienes?

¿En tu base de datos están así?

Deberías imprimir dicho valor para ver que resulta.

Código PHP:

  echo "|$horas|"; 
   

[masterojitos]

Hola, como ya te respondieron... tu error es que en todas las iteraciones colocas "0", mientras que debería ser "$i"... así:

Código PHP:

  $horas = mysql_result($query, $i, 'Hours'); 
   

Aunque, te diría que en vez de usar un mysql_num_rows, un for y un mysql_result....
sería mucho mejor y mas practico, que solo uses un mysql_fetch_array.

Suerte ^^

[carlosuc99]

Iniciado por eits

amigo con la info que das es como decir "tengo esta bolsa de agujas rojas, necesito encontrar una aguja azul" ahora, haciéndole al adivino intenta con esto

Código PHP:

$horas = mysql_result($query, $i, 'Hours');
 

cambia el 0 por $i

saludos.

Ahora el problema es que le asigna a todos los pilotos el rango IngenierodeVuelo, independientemente de las horas. Supongo que será menor que 20, por el if.

[eits]

en primer lugar te sugiero que ya no uses mysql mejor usa el mysqli o pdo_mysql, dos te sugiero que hagas lo que dice pateketrueke con algo como esto

Código PHP:

$query = mysql_query ("SELECT Hours FROM Pilots");
$filas = mysql_num_rows($query);
if($query){
    while($row=mysql_fetch_assoc($query)){
        echo $row['Hours'].PHP_EOL;
    }
}else{
    echo 'Error de consulta';
}
 

[GatorV]

Iniciado por carlosuc99

Ahora el problema es que le asigna a todos los pilotos el rango IngenierodeVuelo, independientemente de las horas. Supongo que será menor que 20, por el if.

Eso es porque tus UPDATEs no tienen la clausula WHERE que es la que indica que row actualizar, sin el WHERE actualiza todas las filas...

[eits]

Iniciado por GatorV

Eso es porque tus UPDATEs no tienen la clausula WHERE que es la que indica que row actualizar, sin el WHERE actualiza todas las filas...

no se como se me pudo pasar algo tan importante, pero en fin ya tienes la solucion.
saludos.

[carlosuc99]

Pues gracias a todos. He ido a la piscina y dándole vueltas me he dado cuenta de lo del Where.

Oye pero para lo de mysqli es solo poner mysqli en vez de mysql?

[GatorV]

No, son algunos cambios sobre todo en el orden de los parámetros entre mysql y mysqli...