[SOLUCIONADO] funcion no funciona

[lmcm2008]

Hola, tengo una problema con una funcion en php que no funciona y no veo porque o donde esta el fallo... ( ya estoy "ofuscado" )

Creo que es la linea del select, pero no estoy seguro.. me podeis echar una mano??

Código:

function selecciona_nacionalidad()
	{
	echo "<SELECT NAME=nacion>";

	$query="SELECT * FROM paises";
	$resultado = mysql_query($query);

	while ($row=mysql_fetch_array($resultado))
		{
		echo"<OPTION NAME=nacion VALUE='.$row['ID'].'>'.$row['PAIS'].'</OPTION>'"; 
		}
	echo "</SELECT>";
	}

Creo que el fallo es esta linea, pero no estoy seguro...

Código:

echo"<OPTION NAME=nacion VALUE='.$row['ID'].'>'.$row['PAIS'].'</OPTION>'";

Ya se que es una tonteria, pero no lo veo....

Alguna sugerencia?
Muchisimas gracias!

[tfe]

Hola:

Estas utilizando mysql_fetch_array
http://php.net/manual/en/function.mysql-fetch-array.php

Creo que en realidad quieres utilizad mysql_fetch_assoc, el cual te devuelve un hash, y entonces sí que podrías utilizar $row['ID']

[Triby]

Sugerencias:

1- Todas las etiquetas y sus atributos se deben escribir en minúsculas, de modo que en lugar de <SELECT NAME=... debería ser <select name=...
2- Todos los atributos deben estar entre comillas, simples o dobles, como gustes, ejemplo: name="nacion"
3- Efectivamente el error está en la línea que mencionas, porque no estás concatenando correctamente, fíjate en las comillas dobles de los extremos y las simples con las que intentas concatenar. Para resolverlo lee esto: http://www.forosdelweb.com/f18/como-...s-bien-588701/

Agrego:

tfe, mysql_fetch_array() puede devolverte array asociativo, numérico o ambos, dependiendo de la configuración, generalmente es ambos; por lo que ese no es el problema.

[lmcm2008]

Vale, entonces siguiendo las recomendaciones de Triby, y después de haber leido la referencia que me dice, esto sería lo que funcionaria??

Código:

function selecciona_nacionalidad()
	{
	echo "<select name=nacion>";

	$query="select * from paises";
	$resultado = mysql_query($query);

	while ($row=mysql_fetch_array($resultado))
		{
		echo"<option name=nacion value=".$row['ID'].">".$row['PAIS']."</option>";
		}
	echo "</select>";
	}

Estaria bien construido ahora? y la linea bien concatenada??
Gracias.

[Triby]

Ahora esta mejor, al menos ya tienes etiquetas y atributos en minúsculas y la línea "bien concatenada", el único problema que veo es que no estás usando comillas para delimitar los valores de los atributos, ejemplo: name=nacion debe ser name="nacion" y, en caso de que recibas un error de undefined index, solo pasa a minúsculas ['ID'] y ['PAIS']

[lmcm2008]

Si, pero entonces la concatenación ya no estaría bien hecha, no?

si pasamos de esto:

Código:

echo"<option name=nacion value=".$row['ID'].">".$row['PAIS']."</option>";

a esto otro..

Código:

echo"<option name="nacion" value=".$row['id'].">".$row['pais']."</option>";

Entonces el name="nacion" no quedaria "fuera" de la linea? Es decir, creo que no estaria bien concatenada, no? No seria mejor que fuse nacion='nacion' ? Y por que pasar ahora el id y el pais en minusculas? Si el campo de la tabla está en Mayusculas?

Es para aclararme bien, porque no lo entiendo del todo...
Gracias

Gracias.

[Triby]

Ups, los option no llevan name, solo value y me parece que no leíste el enlace sobre las comillas!

Código PHP:

// Debes escapar las comillas
echo "<select name=\"nacion\">";
 
// O usar comillas sencillas para encerrar la cadena
echo '<select name="nacion">';
 
// Para el option puedes hacer lo mismo
echo '<option value="' . $row['id'] . '">' . $row['pais'] . '</option>';
 
// O encerrar los elementos de array entre llaves y la cadena entre comillas dobles:
echo "<option value=\"{$row['id']}\">{$row['pais']}</option>";
 

Usa la forma que mejor te acomode, pero asegúrate de que, al final, tu HTML esté correcto; esto puedes verificarlo viendo el código fuente generado en el navegador y, opcionalmente, usando algún validador.

[lmcm2008]

Vaya follon...
Bueno, voy a ver si me entero bien, porque aunque me he leido el artículo, sigo sin enterarme porque una cosa u otra...
Lo pruebo y miro a ver que pasa.
gracias

[lmcm2008]

Parece que funciona bien, pero he tenido que pasar los nombres de los campos a Mayusculas, si no, no los recibía. Ni idea de porque, pero así ha sido.

Continuo peleandome con el código que me falta...
Gracias!

[fire123455]

Es fácil la solución solo de agregar las comillas correspondientes con sus correspondientes slash invertidos o en este caso iniciar echo con comillas simples en este caso seria así prueba esto se que funcionara...

Código PHP:

function selecciona_nacionalidad()
    {
    echo '<select name="nacion">';
 
    $query="select * from paises";
    $resultado = mysql_query($query);
 
    while ($row=mysql_fetch_array($resultado))
        {
        echo'<option value="'.$row['ID'].'">'.$row['PAIS'].'</option>';
        }
    echo "</select>";
    }
 

Suerte...