1

Trato de validar dos horas introducidas por el usuario para que sea exclusivamente en formato HH:MM.

Este es mi código, pero no funciona:

def validateDate(date1, date2):  
    try:  
        time_re = re.compile(r'^(1?[0-9]|2[0-3]):[0-5][0-9]$')  
        match(time_re, date1)  
        match(time_re, date2)  
    except :  
        print "Datos o formato incorrecto, deberia ser hh:mm \n"

3 respuestas 3

1

No tiene mucho sentido usar una expresión regular para validar horas o fechas. Existen funciones para eso, como se muestra en la respuesta de @Madh.

Sin embargo, sólo para completar la respuesta, marco algunas cosas que había que corregir en el código para que funcione:

  1. La función match() no es global. Se debe usar re.match(time_re, date1) o time_re.match(date1)

  2. try y except se usan para capturar excepciones. Sin embargo, cuando un texto no coincide con una expresión regular, no dispara una excepción, simplemente devuelve false, por lo que no ibas a capturarlo de esa forma.

  3. No es un error, pero probablemente quieras aceptar horas que empiezan con 0, como 01:23. Para eso, hay que incluirlo en el primer caracter:

    ^([01]?[0-9]|2[0-3]):[0-5][0-9]$
    
  4. Sólo como recomendación:

    • match() devuelve las coincidencias sólo cuando el inicio del texto coincide con el regex. En ese caso, es innecesario utilizar ^. Sin embargo, como regla general, te recomiendo usar re.search() cuando quieras el mismo comportamiento que en otros lenguajes.

    • Se debería evitar el uso de grupos cuando no se utilizan. Si sólo se quiere agrupar una parte del patrón, conviene usar (?:).

Código:

import re

def validateDate(date1, date2):  
    time_re = re.compile(r'^(?:[01]?\d|2[0-3]):[0-5]\d$')
    return time_re.search(date1) and time_re.search(date2)


print "Ej 1"
if not validateDate("01:59", "21:08"):
    print "Datos o formato incorrecto, deberia ser hh:mm \n"

print "Ej 2"
if not validateDate("01:60", "21:08"):
    print "Datos o formato incorrecto, deberia ser hh:mm \n"

print "Ej 3"
if not validateDate("01:59", "24:08"):
    print "Datos o formato incorrecto, deberia ser hh:mm \n"

Resultado:

Ej 1
Ej 2
Datos o formato incorrecto, deberia ser hh:mm 

Ej 3
Datos o formato incorrecto, deberia ser hh:mm 

Demo:

rextester.com

3
  • 1
    Excelente explicación, y muchas gracias por la ayuda de ambos. :))
    – Rasimus.84
    Commented el 10 nov. 2016 a las 16:48
  • @Rasimus.84 ^(?:[01]\d|2[0-3]):[0-5]\d$
    – Mariano
    Commented el 10 nov. 2016 a las 19:06
  • Sale, Gracias ;)
    – Rasimus.84
    Commented el 10 nov. 2016 a las 19:24
1

Es posible hacerlo sin utilizar una expresión regular.

import time

def isValidTime(data):
    try:
        time.strptime(data, "%H:%M")
        return True
    except ValueError:
        return False

Quedando en tú código algo así:

if not (isValidTime(date1) and isValidTime(date2)):
    print "Datos o formato incorrecto, deberia ser hh:mm \n"

Saludos.

4
  • Eso también funcionaria introduciendo H:MM & H:MM, no?
    – Rasimus.84
    Commented el 10 nov. 2016 a las 15:10
  • 1
    La condición del if debería ser if not isValidTime(date1) or not isValidTime(date2): (o usar un paréntesis y dejar el not afuera)
    – Mariano
    Commented el 10 nov. 2016 a las 15:14
  • def openAuto(usrHrOp, usrHrCl): usrHrOp = raw_input("Introduzca hora para activar en formato 'hh:mm': \n") usrHrCl = raw_input("Introduzca hora para desactivar en formato 'hh:mm': \n") if not (validateDate(usrHrOp) and validateDate(usrHrCl)): print "Datos o formato incorrecto, deberia ser hh:mm \n" #Abrir! while (usrHrOp != actual): ...... Saben por que marca? print "Datos o formato incorrecto, deberia ser hh:mm \n" ^ IndentationError: expected an indented block
    – Rasimus.84
    Commented el 10 nov. 2016 a las 15:19
  • @Rasimus.84 porque después del if, las instrucciones que van dentro del mismo tienen que tener un nivel más de tabulación
    – Mariano
    Commented el 10 nov. 2016 a las 15:45
0
def validateDate(date1, date2):  
    try:  
        time_re = re.compile(r'^(0[0-9]|1[0-9]|2[0-3]):[0-5][0-9]$')  
        match(time_re, date1)  
        match(time_re, date2)  
    except :  
        print "Datos o formato incorrecto, deberia ser hh:mm \n"

Espero te sirva.

2
  • Esta respuesta ha sido marcada de baja calidad por la falta de una explicación de estos cambios que has realizado. Agrega una descripción de los mismos a la respuesta para que todos lo podamos entender, gracias!
    – masterguru
    Commented el 4 mar. 2022 a las 6:55
  • Tal y como está redactada, tu respuesta no es clara. Por favor, pulsa en editar para añadir detalles adicionales que ayuden a los demás a entender cómo esto responde a la pregunta formulada. Puedes encontrar más información sobre cómo escribir buenas respuestas en el centro de ayuda.
    – Comunidad Bot
    Commented el 4 mar. 2022 a las 9:58

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.