Esta bien la estructura??

[higochumbo]

Hola, buenas a todos.

Estoy haciendo una funcion en la que necesito datos de la bd y tengo una consulta hecha fuera de la funcion pero si quiero que la funcion recoja datos de la bd tengo que meter la consulta dentro de la funcion, no se si me he explicado.

Esta es lal funcion:

Código:

function valoracion(){
//Como la conexion ya esta hecha creo una consulta y la almaceno
$query_programa="SELECT * FROM programas";
//Guardo en una variable el resultado de la consulta
$result=mysql_query("$query_programa") or die (mysql_error());


	while ($row = mysql_fetch_array($result)){
		if($row['id_programas']==$_GET['id']){
			$nvotos=$row['votos_programas'];
			$tvotos=$row['votost_programas'];
			
			//Si uno de los dos campos tiene 0 como valor imprimo directamente 0
			if($nvotos==0 || $tvotos==0){
			echo 0;
			}else{
			//Si no divido el total de la suma de los votos entre el numero de votos
			$division = $tvotos / $nvotos;
			//Multiplico por 10 pra sacar un decimal
			$multiplicacion= $division * 10;
			//Y quito el resto de los decimales redondenado hacia abajo
			$total=floor($multiplicacion);
			//Imprimo el numero resultante
			echo $total;
			}
		break;
		}
		
	}

}

La duda que me surge es la de si esto esta bien estructurado asi o si hay otra forma mejor. Funcionar funciona pero si saco la consulta y la pongo fuera de la funcion el mysql_fetch_array($result) no mete ningun dato en ningun array.

Si tengo que meter la consulta dentro de la funcion, por que no tengo que meter tambien la conexion?

Bueno si me habeis entendido bien, si no me explico mejor :P

Saludos.

[pateketrueke]

a todo eso que le llamas se le conoce como contexto, o alcance...

es muy simple, ¿como regar el jardín si se está dentro de la casa?

te sugiero leer lo siguiente:
http://www.php.net/manual/en/languag...bles.scope.php

[wiwi74]

Tienes que ver el alcance de la variable:

Mira:

Código PHP:

   
$query_programa="select * from TU_TABLA"; 
$result=mysql_query($query_programa) or die (mysql_error()); 
 
function valoracion(){ 
 
 //se les asigna a las siguientes variables (externas)  
//un alcance global. Ahora podran ser accedidas desde dentro de la funcion. 
global $result; 
 
while ($row = mysql_fetch_array($result)){ 
 
var_dump($row); 
 
} 
} 
 
valoracion();  
   

Y en esta linea podrias quitar las comillas:
$result=mysql_query($query_programa) or die (mysql_error());

Bueno si hay algo mas no lo se...

[higochumbo]

Oooookey muchas gracias a los dos. Creo que se nota que estoy empezando a trastear con esto :P

Entendido y aplicado.