Error en strlen

[Andress_mdq92]

Hola. Tengo un pequeño problema que no logro encontrar donde esta.

En el curso de PHP que estoy haciendo empezamos a ver strlen y count.

Nos dio un ejemplo, pero se ve que lo copie mal en algun lado y me da error.

Por ejemplo si en el formulario dejo vacio "titulo" me dice:

Error en el titulo


Notice: Undefined index: nota in C:\xampp\htdocs\Clase11\strlen.php on line 28
ERROR: ¡NO SE GUARDO!

Lo rojo es lo que esta mal. Les dejo el codigo entero y les marco en rojo la linea 28.

<?php
$error=array();
//validacion
if(strlen($_POST['titulo'])==0){ //strlen te dice cuanto mide una variable
$error['titulo']="si";
}else{
$titulo=$_POST['titulo'];
}
if(strlen($_POST['nota'])==0){
$error['nota']="si";
}else{
$nota=$_POST['nota'];
}
//cierre validacion

if(count($error)>0){
if($error['titulo']=="si"){
echo "<h3> Error en el titulo </h3>";
}
if($error['nota']=="si"){
echo "<h3> Error en la nota </h3>";
}
echo "<h3> ERROR: ¡NO SE GUARDO! </h3>";
}else{

if(file_exists("mensajes.txt")){
$db=unserialize(file_get_contents("mensajes.txt")) ;
}else{
$db=array();
}
$fila=array();
$fila['titulo']=$titulo;
$fila['nota']=$nota;
$db[]=$fila;
file_put_contents("mensajes.txt",serialize($db));
echo "<h1>GUARDADO</h1>";
}

?>

[the_web_saint]

Hola,

Creo que es aqui, deberias declarar o asociar el valor a la variable $error['nota']...

Código PHP:

if(strlen($_POST['nota'])==0){
$error['nota']="si";
}else{
$nota=$_POST['nota'];
}
 

Me comentas

[Andress_mdq92]

Disculpame pero como quedaria?

[the_web_saint]

Prueba así,

Iniciado por Andress_mdq92

Disculpame pero como quedaria?

Código PHP:

if(strlen($_POST['nota'])==0){
    $error['nota']="si";
    }else{
    $error['nota']=$_POST['nota'];
    }
 

saludos

[PIRRUMAN]

cuando se cumple la condicion le agregas un valor a $error['titulo'] y a $error['nota']
pero cuando no se cumple ni siquiera lo creas, por eso al compararlo te marca error y entra al else de $error['nota']=="si"...
prueba:

if(strlen($_POST['titulo'])==0){ //strlen te dice cuanto mide una variable
$error['titulo']="si";
}else{
$titulo=$_POST['titulo'];
$error['titulo']="NO";//o como creas necesario
}
if(strlen($_POST['nota'])==0){
$error['nota']="si";
}else{
$nota=$_POST['nota'];
$error['nota']="NO";//a consideracion
}

asi quedaria tus variables $nota y $titulo con los parametros que necesites

[Andress_mdq92]

Hola, probando como dijo PIRRUMAN si funciona esa parte, pero al poner algo en ambos campos me dice Error al guardar , en vez de GUARDADO.

[PIRRUMAN]

cual es el error que te envia?

[Andress_mdq92]

No, no es un error error. Es que cuando pongo ambos datos, en vez de poner GUARDADO como tiene que decir, dice : ERROR: ¡NO SE GUARDO! como si hubiera algun campo vacio.

[PIRRUMAN]

pon el codigo tal cual lo usas para verificarlo

[Andress_mdq92]

Código PHP:

<?php
$error=array();
//validacion
if(strlen($_POST['titulo'])==0){ //strlen te dice cuanto mide una variable
    $error['titulo']="si";
}else{
    $titulo=$_POST['titulo'];
    $error['titulo']="no";
}
if(strlen($_POST['nota'])==0){
    $error['nota']="si";
}else{
    $nota=$_POST['nota'];
    $error['nota']="no";
}
//cierre validacion
 
if(count($error)>0){
    if($error['titulo']=="si"){
        echo "<h3> Error en el titulo </h3>";
    }
        if($error['nota']=="si"){
            echo "<h3> Error en la nota </h3>";
        }
        echo "<h3> ERROR: ¡NO SE GUARDO! </h3>";
    }else{
        
    if(file_exists("mensajes.txt")){
        $db=unserialize(file_get_contents("mensajes.txt"));
    }else{
        $db=array();
    }}
    $fila=array();
    $fila['titulo']=$titulo;
    $fila['nota']=$nota;
    $db[]=$fila;
    file_put_contents("mensajes.txt",serialize($db));
    echo "<h1>GUARDADO</h1>";
    
 
?>
 

[chwc]

haz una cosa, primero que nada no inicializes las variables como dijieron anteriormente, y luego cambia esto :

Código PHP:

if($error['titulo']=="si"){
 

por esto:

Código PHP:

if(isset($error['titulo'])){
 

y ademas esto

Código PHP:

if($error['nota']=="si"){
 

por esto

Código PHP:

if(isset($error['nota'])){
 

quedandote asi:

Código PHP:

<?php
$error=array();
//validacion
if(strlen($_POST['titulo'])==0){ //strlen te dice cuanto mide una variable
    $error['titulo']="si";
}else{
    $titulo=$_POST['titulo'];
}
if(strlen($_POST['nota'])==0){
    $error['nota']="si";
}else{
    $nota=$_POST['nota'];
}
//cierre validacion
 
if(count($error)>0){
    if(isset($error['titulo'])){
        echo "<h3> Error en el titulo </h3>";
    }
        if(isset($error['nota'])){
            echo "<h3> Error en la nota </h3>";
        }
        echo "<h3> ERROR: ¡NO SE GUARDO! </h3>";
    }else{
        
    if(file_exists("mensajes.txt")){
        $db=unserialize(file_get_contents("mensajes.txt"));
    }else{
        $db=array();
    }}
    $fila=array();
    $fila['titulo']=$titulo;
    $fila['nota']=$nota;
    $db[]=$fila;
    file_put_contents("mensajes.txt",serialize($db));
    echo "<h1>GUARDADO</h1>";
    
 
?>
 

saludos y suerte :D

[Andress_mdq92]

Es que isset no vi en el curso que estoy haciendo, y no puedo usar cosas no vistas.

[chwc]

entonces, hace una cosa, digamos que $error=0 significa todo bien, $error=1 significa error en el titulo, y error=2 significa error en la nota, entonces

Código PHP:

<?php
$error=0;
//validacion
if(strlen($_POST['titulo'])==0){ //strlen te dice cuanto mide una variable
* * $error=1;
}else{
* * $titulo=$_POST['titulo'];
}
if(strlen($_POST['nota'])==0){
* * $error=2;
}else{
* * $nota=$_POST['nota'];
}
//cierre validacion
*
if($error=1)
* * * * echo "<h3> Error en el titulo </h3>";
if($error=2)
* * * * * * echo "<h3> Error en la nota </h3>";
if($error=3){
* * * * 
* * if(file_exists("mensajes.txt")){
* * * * $db=unserialize(file_get_contents("mensajes.txt"));
* * }else{
* * * * $db=array();
* * }
* * $fila=array();
* * $fila['titulo']=$titulo;
* * $fila['nota']=$nota;
* * $db[]=$fila;
* * file_put_contents("mensajes.txt",serialize($db));
* * echo "<h1>GUARDADO</h1>";
}
?>
 

asi creo que funcionaria bien :D
denada por hacerte la tarea XD
saludos y suerte

[PIRRUMAN]

tu error es en la logica no en el codigo:

count($error) siempre sera mayor a cero , puesto que tendra el valor si o el valor no

cambialo a

if($error['titulo']=="si"||$error['nota']=="si")
{
}

otra cosa , para que no siemrpe te envie el mensaje de guardado mete la instruccion dentro de las llaves del else

else{

if(file_exists("mensajes.txt")){
$db=unserialize(file_get_contents("mensajes.txt")) ;
}else{
$db=array();
}}//aqui termina y lo demas se imprimira pase lo que pase
$fila=array();
$fila['titulo']=$titulo;
$fila['nota']=$nota;
$db[]=$fila;
file_put_contents("mensajes.txt",serialize($db));
echo "<h1>GUARDADO</h1>";
//aqui debe ir la llave del else

[Andress_mdq92]

Es que el ejercicio lo dio el profesor jaja, eso esta copiado textualmente del pizarron.

error es en la logica no en el codigo:

count($error) siempre sera mayor a cero , puesto que tendra el valor si o el valor no

Pero como puse al principio del todo

Código PHP:

<?php
$error=array();
//validacion
 
if(strlen($_POST['titulo'])==0){ //strlen te dice cuanto mide una variable
    $error['titulo']="si";
}else{
    $titulo=$_POST['titulo'];
}
 
if(strlen($_POST['nota'])==0){
    $error['nota']="si";
}else{
    $nota=$_POST['nota'];
}
//cierre validacion
 
if(count($error)>0){
    if($error['titulo']=="si"){
        echo "<h3> Error en el titulo </h3>";
    }
        if($error['nota']=="si"){
            echo "<h3> Error en la nota </h3>";
        }
        echo "<h3> ERROR: ¡NO SE GUARDO! </h3>";
    }else{
        
    if(file_exists("mensajes.txt")){
        $db=unserialize(file_get_contents("mensajes.txt"));
    }else{
        $db=array();
    }}
    $fila=array();
    $fila['titulo']=$titulo;
    $fila['nota']=$nota;
    $db[]=$fila;
    file_put_contents("mensajes.txt", serialize($db));
    echo "<h1>GUARDADO</h1>";
    
 
?>
 

Sin decirle que

$error['nota']=="no";

El array $error, tendria 0 lugares ocupados y pasaria a la llave

Código PHP:

if(file_exists("mensajes.txt")){
        $db=unserialize(file_get_contents("mensajes.txt"));
    }else{
        $db=array();
 

Y asi me tirar el error:

Notice: Undefined index: titulo in C:\xampp\htdocs\Clase11\strlen.php on line 27
Error en la nota

ERROR: ¡NO SE GUARDO!


Notice: Undefined variable: nota in C:\xampp\htdocs\Clase11\strlen.php on line 43
GUARDADO

Donde linea 27 y 43 son estas:

if($error['nota']=="si"){
$fila['nota']=$nota;

ESTO EN CASO DE NO COMPLETAR NOTA.

Ahora no comprendo algo, yo recuerdo que esto en clases me funciono bien, a mi y a todos mis compañeros, pero aca en casa me tira error.





Leyendo un poco veo que:

count() devolverá 0 si la variable que se intenta contar no está definida pero también devolverá 0 si la variable contiene un array vacío.

Lo que contradice el error que tira. No entiendo nada :D

[PIRRUMAN]

hagamos una prueba.....cambia todos los $_POST x $_GET e ingresa a la pagina con las variables escritas en ella
por ejemplo

C:\xampp\htdocs\Clase11\strlen.php?titulo=mititulo &nota=minota

par sabe que pasa

[Andress_mdq92]

Haciendo asi me puso: GUARDADO y se guardo bien.

Que significa? :P

[PIRRUMAN]

probaste dejando en blanco una de las dos variables, primero titulo luego nota y viceversa y ambas tambien para cehcar la logica?

....strlen.php?titulo=&nota=

el problema seria donde envias las variables podrias poner como las envias o recoges?

podrias poner tu formulario?

pero has antes la pruieba que te dije

[Andress_mdq92]

A ver, dejando uno en blanco me sigue tirando el mismo error de antes.

El formulario es ese.

Código HTML:

<form id="form1" name="form1" method="post" action="strlen.php">
  <p>
    <label for="titulo">Titulo</label>
    <input type="text" name="titulo" id="titulo" />
  </p>
  <p>
    <label for="nota">Nota</label>
    <input type="text" name="nota" id="nota" />
  </p>
  <p>
    <input type="submit" name="button" id="button" value="Guardar" />
  </p>
</form>
 

[gjx2]

Definitivamente sin usar isset no van a poder evitar que php siga imprimiendo la notificacion de que existe una variable indefinida.

Si un campo se envia vacio ese campo simplemente no pasa al script php por tanto no esta definido y no puedes comprobarlo con strlen de manera que te seguira tirando la notificación al menos que ocultes todos los errores y notificaciones.

error_reporting(0);

También pudieras crear una funcion en javascript que siempre defina el campo aun cuando este vacio.

Lo que me parece extraño por que tu profesor te enseno como usar primero strlen antes que isset
Saludos.

[Andress_mdq92]

Lo mas raro de todo que este ejercicio en clase funciono bien. :S

Puede haber algun error en xampp?

[gjx2]

Funciono dejando un campo vació ? y si lo hizo entonces php.ini tenia desactivadas las notificaciones.

[Andress_mdq92]

Si si, funcionaba dejando campos vacios, solo me decia que campo estaba vacio como era la idea. Si hacia algo mal en otro ejercicio me notificaba errores. No entienod nada jaja

[PIRRUMAN]

yo copio tu codigo tal cual y no me resulta ni un error

crea otro archivo y solo coloca el siguiente codigo tal cual

<?php
$error=array();
//validacion

if(strlen($_POST['titulo'])==0){ //strlen te dice cuanto mide una variable
$error['titulo']="si";
}else{
$titulo=$_POST['titulo'];
}

if(strlen($_POST['nota'])==0){
$error['nota']="si";
}else{
$nota=$_POST['nota'];
}
//cierre validacion

if(count($error)>0){
if($error['titulo']=="si"){
echo "<h3> Error en el titulo </h3>";
}
if($error['nota']=="si"){
echo "<h3> Error en la nota </h3>";
}
echo "<h3> ERROR: ¡NO SE GUARDO! </h3>";
}else{

if(file_exists("mensajes.txt")){
$db=unserialize(file_get_contents("mensajes.txt")) ;
}else{
$db=array();
}}
$fila=array();
$fila['titulo']=$titulo;
$fila['nota']=$nota;
$db[]=$fila;
file_put_contents("mensajes.txt", serialize($db));
echo "<h1>GUARDADO</h1>";


?>
<form id="form1" name="form1" method="post" action="#">
<p>
<label for="titulo">Titulo</label>
<input type="text" name="titulo" id="titulo" />
</p>
<p>
<label for="nota">Nota</label>
<input type="text" name="nota" id="nota" />
</p>
<p>
<input type="submit" name="button" id="button" value="Guardar" />
</p>
</form>

[gjx2]

Pirruman, no te muestra error por que tienes desactivadas las notificaciones si pones esto al principio del script te daras cuenta.

ini_set ('error_reporting', E_ALL);

Ahora trata de trabajar sin definir las variables.

Saludos.

[PIRRUMAN]

no pues sigue sin aparecerme :S y es una pagina totalmente independiente

[Andress_mdq92]

El error no me da solo a mi, a mi compañero tambien. Y a ambos en clase nos daba bien.

[PIRRUMAN]

como veras a mi no me sale el error asi que creo hasta ahi llego mi ayuda , disculpas

[Andress_mdq92]

Bueno, escribo ya que se molestaron en buscarme una solucion y lo mas correcto seria decirles como me fue en clase.

Llego a clase abro el archivo que me seguia tirando la notificacion aca en casa y ahi en clase funcionaba lo mas bien. Claro, yo no sabia la diferencia entre una notificacion y un error en serio.

Asi que ese era el tema, alla estan desactivadas las notificaciones.