1

Desarrollé un programa para ingresar patentes, km y fecha. Quisiera que solo aceptara mayúsculas y números, sin guiones. El problema es que cuando ingreso una patente con minúscula me aparece el aviso de "reintente ingresa patente", pero la información se almacena igualmente, y necesito que no se almacene si esta en minúsculas. Al igual que necesito que no almacene km menores al anterior ingresado.

Si coloco la consulta después de mi primer if tengo error por no tener definidas las otras variables.

Este es el programa que he realizado hasta ahora

CÓDIGO

<?php
if (isset($_POST['register'])) {
    if (strlen($_POST['Patente']) ==6){
        $Patente = trim($_POST['Patente']);
        $Patente = strtoupper($Patente);
        if (preg_match("/^[A-Z0-9]*$/", $Patente)) {
        ?>
            <h3 class="ok">Se ha ingresado la Patente correctamente</h3>
        <?php
        } else {
        ?>
            <h3 class="bad">Reintente ingresar correctamente la Patente</h3>
        <?php
        }
    }
    if (strlen($_POST['Km']) >=1) {
        $Km = trim($_POST['Km']);
        $sentencia = "SELECT*FROM Km WHERE Km <= $Km";
        $query = mysqli_query($conexion, $sentencia);
        $existe = mysqli_num_rows($query);
        if ($existe >= 0) {
        ?>
            <h3 class="bad">El Km ingresado es menor o igual al anterior</h3>
        <?php
        }
        if (preg_match("/^[0-9]*$/", $Km)); {
        ?>
            <h3 class="ok">Se ha ingresado el Km correctamente</h3>
        <?php
        }
    } else {
    ?>
        <h3 class="bad">Reintente ingresar Km</h3>
    <?php
    }

    $Fecha_reg=date("y/m/d");
    $consulta = "INSERT INTO km(Patente, Km, Fecha_reg) VALUES ('$Patente','$Km','$Fecha_reg')";
    $resultado =mysqli_query($conexion,$consulta);

    if ($resultado) {
    ?>
        <h3 class="ok">Información Almacenada</h3>
    <?php
    } else {
    ?>
        <h3 class="bad">Ha ocurrido un error</h3>
    <?php
    }
} else {
?>
    <h3 class="bad">Complete correctamente la información</h3>
<?php
}
}
?>
3
  • 4
    ¿y donde está la consulta que te lo hace? Si no está dentro de ese condicional seguramente la hará igualmente, que es lo que te está pasando ¿no crees?
    – masterguru
    Commented el 22 nov. 2022 a las 12:55
  • No entiendo porque has aceptado una respuesta que no ha tenido en cuenta los números, tal como indicas en el título, y lo único que hace, como todas las respuestas que te han dado, es pasarlo todo a mayúsculas, pero no verifican si tiene caracteres especiales ni nada, por lo tanto te pueden poner guiones, asteriscos, etc. En tu pregunta tu código ya era correcto, y tan solo debías mover la consulta dentro del primer if, pero como no has puesto donde ejecutas la consulta no se te podia responder concretamente y de forma certera.
    – masterguru
    Commented el 23 nov. 2022 a las 23:13
  • Acabo de darte una respuesta completa, mira a ver si te funciona tal cual y comentamos los problemas que te de
    – masterguru
    Commented el 2 dic. 2022 a las 13:41

4 respuestas 4

3

Prueba con esto y más abajo lo explico:

<?php

// inicializamos una variable que nos irá recogiendo todos los mensajes 
$msg = '';

if (isset($_POST['register'])) {
    // validación patente
    if (strlen($_POST['Patente']) == 6){
        $Patente = trim($_POST['Patente']);
        $Patente = strtoupper($Patente);
        if (preg_match("/^[A-Z0-9]*$/", $Patente)) {
            $msg .= '<h3 class="ok">Se ha ingresado la Patente correctamente</h3>';
            $comprobacion_patente = true;
        } else {
            $msg .= '<h3 class="bad">Reintente ingresar correctamente la Patente</h3>';
            $comprobacion_patente = false;
        }
    } else {
        $msg .= '<h3 class="bad">El total de caracteres permitido para una patente son 6. Reintente ingresar correctamente la Patente</h3>';
        $comprobacion_patente = false;
    }

    // validación km
    // primero evaluamos si la patente era buena, sino no hace falta seguir aún
    if ($comprobacion_patente == true) {
        if (strlen($_POST['Km']) >=1) {
            $Km = trim($_POST['Km']);
            $sentencia = "SELECT * FROM Km WHERE Patente = '$Patente' AND Km >= $Km";
            $query = mysqli_query($conexion, $sentencia);
            $existe = mysqli_num_rows($query);
            if ($existe > 0) {
                $msg .= '<h3 class="bad">El Km ingresado es menor o igual al anterior</h3>';
                $comprobacion_km = false;
            } else {
                if (preg_match("/^[0-9]*$/", $Km)) {
                    $msg .= '<h3 class="ok">Se ha ingresado el Km correctamente</h3>';
                    $comprobacion_km = true;
                } else {
                    $msg .= '<h3 class="bad">El km, no es un número válido!</h3>';
                    $comprobacion_km = false;
                }
            }
        } else {
            $msg .= '<h3 class="bad">Reintente ingresar Km</h3>';
            $comprobacion_km = false;
        }
    }

    // evaluamos si debe o no insertarse el registro
    if ($comprobacion_km == true and $comprobacion_km == true) {
        $Fecha_reg = date("y/m/d");
        // comprobamos la fecha
        $sentencia2 = "SELECT * FROM Km WHERE Patente = '$Patente' AND Fecha_reg = '$Fecha_reg'";
        $query2 = mysqli_query($conexion, $sentencia2);
        $existe2 = mysqli_num_rows($query2);

        if ($existe2 == 0) {
            $msg .= '<h3 class="ok">Fecha correcta</h3>';
            $consulta = "INSERT INTO km(`Patente`, `Km`, `Fecha_reg`) VALUES ('$Patente','$Km','$Fecha_reg')";
            $resultado = mysqli_query($conexion,$consulta);
            if ($resultado) {
                $msg .= '<h3 class="ok">Información Almacenada</h3>';
            } else {
                $msg .= '<h3 class="bad">Ha ocurrido un error</h3>';
            }
        } else {
            $msg .= '<h3 class="bad">La fecha no se puede repetir. Ya existe un registro con esta fecha.</h3>';
        }
    }
} else {
    $msg .= '<h3 class="bad">Complete correctamente la información</h3>';
}
// escupimos todos los mensajes acumulados a la salida
echo $msg;

Los cambios realizados han sido muchos, pero a grandes trechos son:

  1. Se ha creado una variable $msg que va a ir almacenando los diversos mensajes de verificación, error o éxito de las distintas comprobaciones e inserciones, en lugar de mezclar constantemente código PHP con la salida, la cual, desde mi punto de vista, siempre debe ser al final.

  2. Se han estructurado mejor los bloques de validaciones y usando nuevas variables denominadas comprobacion_xxxx se han ido guardando sus resultados para su uso posterior en el momento de calcular si hay que realizar o no la inserción de los datos

  3. Se han modificado las consultas SQL para tener en cuenta más valores, incorporando el de Patente en las claúsulas WHERE, pues de otro modo el resultado no se correspondia con lo esperado y planteado en la pregunta y comentarios

  4. Se ha agregado una consulta para comprobar también si la fecha es coincidente con algun registro anterior en la tabla y proceder adecuadamente en cada caso

  5. Todo lo demás han sido errores menores, tanto de comprensión como de código, que se han ido corrigiendo sobre la marcha

23
  • en este caso con las patentes y las fechas no hay problema, pero no se como hacer q no almacene km menor al que ya este ingresado por patente, es decir, que si una patente X ingreso un km el dia de ayer, hoy esa misma patente no pueda ingresar un Km menor al ingresado ayer. Commented el 2 dic. 2022 a las 13:56
  • mmm... perdona mi desconocimiento... ¿pero que es una patente? ¿y un km? para mi una patente es algo que se registra para que los demas no lo puedan usar sin tu permiso.. y un km es un kilometro... pero me cuesta entender que relación tienen y quizás por eso no lo veo claro aún... unas preguntas más... ¿las patentes esas se pueden repetir en esa tabla? ¿que estructura tiene? ¿id (autoincremental), patente, km y fecha? ¿u otra?
    – masterguru
    Commented el 2 dic. 2022 a las 14:01
  • y cuando dices necesito que no almacene km menores al anterior ingresado ¿a que te refieres? ¿a una patente igual anterior o a la ultima patente registrada en la tabla aunque sea distinta? (vale, no hace falta que respondas, me lo has dicho en tu primer comentario de este hilo)
    – masterguru
    Commented el 2 dic. 2022 a las 14:03
  • Gracias por el interes en ayudarme, la patente es el numero asignado a un vehiculo y km son los kilometros recorridos total del vehiculo. estas patentes no pueden ingresar mas de 1 vez al dia la informacion de su km, y el km no puede ser menor al del dia anterior. en la tabla si se pueden repetir estas patentes, pero como te contaba, el usuario solo puede ingresar información 1 vez por patente al dia Commented el 2 dic. 2022 a las 14:04
  • ah, es una matricula entonces (o el número de bastidor o algo que lo identifica de forma única en tu localización), je.. okok... a ver como lo enfoco pues lo de la fecha no se está verificando tampoco en tu código... de momento voy a actualizar mi respuesta para lo de los km
    – masterguru
    Commented el 2 dic. 2022 a las 14:09
1

En lugar de validar si esta en muysculas o minusculas

Podrias convertir todo el campo a mayusculas

strtoupper — Convierte un string a mayúsculas

Ejemplo

<?php
$str = "Mary Had A Little Lamb and She LOVED It So";
$str = strtoupper($str);
echo $str; // muestra: MARY HAD A LITTLE LAMB AND SHE LOVED IT SO
?>
0
1

¿Estás usando un formulario web? Entonces otra opción es el atributo pattern:

<form action="" method="POST">
  <input type="text" name="Patente" pattern="[A-Z0-9]+" placeholder="Patente" required>
  <input type="submit" value="Enviar">
</form>

2
  • Hola @masterguru, replicando un poco a SO Inglés, me gusta dar respuestas alternativas, al problema o a las otras respuestas. Es algo veo recibido positivamente en SO. Por ejemplo, en SO hay preguntas de 2011 que hoy en día siguen recibiendo respuestas a medida que salen nuevos features de los lenguajes o a alguien se le ocurrió una mejor forma. Entonces mi respuesta para este caso: "¿Estas enredado/a con el backend?, que tal si lo resuelves en el frontend con una simple implementación"
    – aeportugal
    Commented el 23 nov. 2022 a las 23:46
  • 2
    Sí, te doy toda la razón, y me parece muy bien la respuesta como alternativa a su problema real. Ya te digo, mi fallo fue pensar que ninguna respuesta de las que vi habia tenido en cuenta mi comentario sobre la consulta con el insert a la base de datos que quedaba fuera de su condicional y no ví más allá en ese momento. Suerte que lo he repasado y he podido enmendar el error. Gracias! :-)
    – masterguru
    Commented el 23 nov. 2022 a las 23:51
0

para comprobar si tiene letras minusculas puedes utilizar la función ctype_upper

$cadenas = array('PATITO', 'PaTITO', 'patito');
foreach ($cadenas as $palabra) {
    if (ctype_upper($palabra)) {
        echo "La palabra '$palabra' es buena.";
        echo "<br>";
    } else {
        echo "La palabra '$palabra' tiene minusculas.";
        echo "<br>";
    }
}

esto re regresaría lo siguiente:

La palabra 'PATITO' es buena.

La palabra 'PaTITO' tiene minusculas.

La palabra 'patito' tiene minusculas.

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.