Problema al recoger una variable

[Mugiwarakun]

Hola buenas, explico mi problema.

Yo tenía unos scripts que siempre me funcionaron para añadir campos a las bases de datos de manera simple. Estos scripts consisten en un único archivo php que funciona así:

Código PHP:

  $opcion = $opcion;

switch($opcion) {

case 0: function rellenar_formulario(); break;
case 1: function validar_formulario(); break;
default: loquesea(); break;

}  
   

Lo que sucede es que, al acceder a la página por defecto siempre me coge 0 ya que no existía valor inicial y se me iba a la función para rellenar el formulario y este, una vez rellenado, llamaba a esta misma página con el esquema formulario.php?opcion=1 y se iba a la otra función.

Por alguna razón que desconozco esto ya no me funciona, me habla siempre de "undefined variable", error que he mirado por decenas de webs pero nadie tenía el código al estilo del mío.

¿Alguna solución? gracias.

[oscarbt]

Creo que deberias iniciar opcion:

$opcion=0;

antes del switch

[iviamontes]

$opcion = $opcion; ?????
y donde declaraste inicialmente $opcion ?

[Mugiwarakun]

Mirad un ejemplo de código que funcionaba:

Código PHP:

  $paramj = $valorj; (Aquí no usé el mismo nombre, pero en otro archivo si). 
 
switch ($paramj) 
  { 
      case 0: 
            rellenar_formj(); 
            break; 
    case 1: 
            validar_formj(); 
            break; 
    default: 
            echo "Esta opción no vale."; 
            exit; 
  }  
   

Con esto yo accedía a la página y por defecto me entraba en 0 ya que no existe una declaración previa de $valorj.

[iviamontes]

lo que pasa es que si pones a $opcion = 0 siempre va a entrar al case 0

y puedes recoger el valor de opcion por get
aunq creo q mejor pones el codigo de la pagina que te envia allí

[Mugiwarakun]

Ya lo he solucionado, menuda tontería, el problema era que tenía "register_globals" en off en el php.ini, ya funciona como debería.

[iviamontes]

ufff no me parece bueno ponerlo en on.. !!!

[Mugiwarakun]

Es sólo a nivel local para hacer estos scripts simples para añadir campos a bases de datos.

[Nano_]

Saludos

Estoy totalmente de acuerdo con iviamontes, desde la version 4.3 de php (sino estoy mal) fue deshabilitada para tener un mejor seguridad (ya que es un hueco de seguridad desde que no se sepa controlar as variables que se declaran)

[iviamontes]

Iniciado por Mugiwarakun

Es sólo a nivel local para hacer estos scripts simples para añadir campos a bases de datos.

amigo si te acostumbras a trabajar mal en local, que puede asegurarte que después trabajes bien ???
mejor haz tus códigos siempre buenos y ya harás de eso una practica sana y lo harás sin darte cuenta, es mi consejo...

[Mugiwarakun]

No había visto esta respuesta.

Yo sé que hay que controlar bien las variables, sencillos controles para la variable, tipo de si el campo está vacío, caracteres introducidos, etc, pero bueno.

Me da curiosidad el tema del get, me gustaría saber poder hacer esto sin register globals.

El código es así:

Código PHP:

  //error_reporting(0);
    
    //First we create the database connection that we are going to work with.
    
    $connection = @mysql_connect("localhost", "user", "password");
    mysql_select_db("food", $connection);

    //Second we take a variable to choose if wi8e fill out the formularie or if we are adding the data.

    $val = $opc;
    
    switch($val)
    {
            case 0:
                fillouttype();
                break;
            case 1:
                addtype();
                break;
            default:
                echo("Error.");
                exit;    
    }
    
    //Function to fill the formularie out.
    
    function fillouttype()
    {
            global $connection;
            
               echo '<form action="add-type-of-food.php?opc=1" method="post" name="typeoffood">
                           Description Spanish: <input type="text" name="description_es" maxlength="20" /><br />
                        Description English: <input type="text" name="description_en" maxlength="20" /><br />
                        Description Japanese: <input type="text" name="description_jp" maxlength="20" />&nbsp;
                        Romaji: <input type="text" name="romaji" maxlength="20" /><br /><br />
                        Previously Added: <select name="previouslyadded"><option selected>Verify</option>';

                $addedtypes = mysql_query("SELECT description FROM type_es ORDER BY description", $connection);
                
                for($x = 0; $x < mysql_num_rows($addedtypes); $x++)
                {
                        echo '<option>'.mysql_result($addedtypes, $x, "description").'</option>';
                }

            echo '</select><br /><br /><input type="submit" value="Submit!" name="submit" /></form>';
            
            mysql_close($connection);
    }
    
    //Function to add the data to the database.
    
    function addtype()
    {
            global $connection, $description_es, $description_en, $description_jp, $romaji;
            
            if($description_es == '' || $description_en == '' || $description_jp == '' || $romaji == '')
            {
                        echo 'You must fill all the fields out. Please come back and do it.<br />
                              <input type="button" value="Come Back!" onclick="javascript:history.go(-1)" />';
                        exit;
            }
            
            $tables = array("type_es", "type_en", "type_jp");
            $description_value = array($description_es, $description_en, $description_jp);
            
            foreach($tables as $value => $content)
            {
                        if($content == "type_jp")
                        {
                                mysql_query("INSERT INTO ".$content." VALUES ('', '".$description_value[$value]."', '".$romaji."')", $connection);        
                        }
                        else
                        {
                                mysql_query("INSERT INTO ".$content." VALUES ('', '".$description_value[$value]."')", $connection);    
                        }    
            }
            
            echo "<input type=\"button\" value=\"Add other!\" onclick=\"javascript:location.href='add-type-of-food.php'\" />";
            
            mysql_close($connection);