[SOLUCIONADO] Duda puntual: Cadenas de fechas

[agusax]

Hola chic@s
Tengo un problemilla y es que no me viene a la mente como solucionar el siguiente problema:

Tengo este programa de test:

Código C++:

FCT_TEST_BGN(Fecha - Comparacion: desigualdad) {
      const Fecha f(1, 2, 2000);
      const Fecha g(f);
      const Fecha h(2, 3, 2001);
      fct_chk(!(f != g));
      fct_chk(f != h);
      fct_chk(f != "00002/003/02001"); // Problema en esta linea
      fct_chk("1/1/2001" != g);
}
 

Mi constructor a partir de una cadena de la siguiente manera:

Código C++:

Fecha::Fecha (const char *fecha){
    int iCheck;
 
    iCheck = sscanf(fecha, "%02d/%02d/%04d", &iDia_, &iMes_, &iAno_);
    if(iCheck == 3){
        fecha_actual();
        if(!es_valida(iDia_, iMes_, iAno_))
            throw Invalida("Error: No ha introducido una fecha correcta");  
    }
    else{
        throw Invalida("Error: Formato de fecha no valido. Formato valido: dd/mm/aaaa");
    }
}
 

El caso es que me da el siguiente problema:

Código:

terminate called after throwing an instance of 'Fecha::Invalida'
Fecha - Comparacion: desigualdad ..................................Abortado

La clase Invalida está dentro de la clase Fecha:

Código C++:

class Invalida {
    public:
        Invalida(const char* cError) : cError_(cError) {}
        const char* por_que() const noexcept;
 
    private:
        const char* cError_;
};
 

No sabría como solucionar esa devolución del error.
Muchas gracias por su tiempo

PD: no sé si debería controlar la cantidad de ceros a la izquierda, ya que el otro método de test de fecha incorrecta funciona correctamente

[eferion]

Está claro que si tu clase Fecha tiene que entender que 00004 puede ser un día válido no tiene sentido limitar la entrada de sscanf a 2 caracteres.

Código C++:

iCheck = sscanf(fecha, "%d/%d/%d", &iDia_, &iMes_, &iAno_);
 

Esta versión no te debería dar esos problemas.

Otra cosa es que esa fecha se tenga que interpretar como no válida... en cuyo caso lo que no tiene sentido es que devuelvas una excepción en el constructor de la clase Fecha. En este caso tendrías que tener un booleano interno en Fecha que indicase si la fecha almacenada es válida. Al realizar las comparaciones, si una fecha es válida y la otra no automáticamente ambas fechas van a ser diferentes. O eso o haces una sobrecarga del operador de comparación específica:

Código C++:

class Fecha
{
  // versión 1
  bool operator!=(const char* fecha) const;
 
  // versión 2
  friend bool operator!=(const Fecha& fecha1, const char* fecha2);
};
 

Un saludo.

[agusax]

Iniciado por eferion

Está claro que si tu clase Fecha tiene que entender que 00004 puede ser un día válido no tiene sentido limitar la entrada de sscanf a 2 caracteres.

Código C++:

iCheck = sscanf(fecha, "%d/%d/%d", &iDia_, &iMes_, &iAno_);
 

Esta versión no te debería dar esos problemas.

Otra cosa es que esa fecha se tenga que interpretar como no válida... en cuyo caso lo que no tiene sentido es que devuelvas una excepción en el constructor de la clase Fecha. En este caso tendrías que tener un booleano interno en Fecha que indicase si la fecha almacenada es válida. Al realizar las comparaciones, si una fecha es válida y la otra no automáticamente ambas fechas van a ser diferentes. O eso o haces una sobrecarga del operador de comparación específica:

Código C++:

class Fecha
{
  // versión 1
  bool operator!=(const char* fecha) const;
 
  // versión 2
  friend bool operator!=(const Fecha& fecha1, const char* fecha2);
};
 

Un saludo.

Ya probé a poner sscanf(fecha, "%d/%d/%d", &iDia_, &iMes_, &iAno_), pero entonces no me pasa después el test de desbordamiento, debería ser una fecha inválida.
Mi operador de comparación es de este modo:

Código C++:

bool operator == (const Fecha& f1, const Fecha& f2){
    return ((f1.dia() == f2.dia()) && (f1.mes() == f2.mes()) && (f1.anno() == f2.anno()));
}
 
bool operator != (const Fecha& f1, const Fecha& f2){
    return !(f1 == f2);
}
 

Se supone que el constructor, lo uso para almacenar los valores de la cadena en iDia, iMes, iAno

Muchas gracias por responder

[eferion]

Iniciado por agusax

Ya probé a poner sscanf(fecha, "%d/%d/%d", &iDia_, &iMes_, &iAno_), pero entonces no me pasa después el test de desbordamiento, debería ser una fecha inválida.

Entonces sscanf no es una opción para tí.

Puedes optar por varias posibilidades:

• Si estás compilando bajo el estándar C++11 (2011) o posterior puedes usar las expresiones regulares para recuperar los elementos de la fecha.
• Puedes usar strtok para trocear la cadena por los separadores y después comparar cada elemento (día,mes,año) por separado: verificar que todos los dígitos son numéricos, verificar que es un número válido y que está dentro del rango de valores correcto).
• Puedes hacer el proceso descrito en el paso anterior pero todo a mano.

Iniciado por agusax

Mi operador de comparación es de este modo:

Puedes optar por tener tantos operadores como necesites... que tengas ya uno implementado no quiere decir que no puedas tener más si así lo requiere el diseño.

Un saludo.

[agusax]

Este es mi test del desbordamiento

Código C++:

FCT_TEST_BGN(Fecha - Extraccion: desbordamiento) {
      istringstream is("007/007/1997");
      Fecha f;
      fct_chk_ex(Fecha::Invalida, is >> f);
    }
 

Esto es lo que me comenta el profesor:

La Fecha "00002/003/02001" es válida porque es "2/3/2001": 2 de marzo de 2001. La Fecha "007/007/1997" también: "7/7/1997", 7 de julio de 1997; pero en este 2.º caso se está leyendo de un flujo de entrada; la lectura en este caso debería limitarse a 2+1+2+1+4=10 caracteres como mucho: el desbordamiento es en el flujo de entrada, no en la Fecha.

Pista: width o setw: http://en.cppreference.com/w/cpp/io/manip/setw

Por eso con el sscanf me da error la prueba de desbordamiento.
El problema es que no sé como utilizar eso.

[eferion]

Iniciado por agusax

Esto es lo que me comenta el profesor:

En ese caso tienes que tener un constructor específico para istream (buffers de entrada) y otro para char*, de tal forma que el constructor de istream realice un chequeo de desbordamiento mientras que el de char* no debe realizar dicho chequeo.

Código C++:

Fecha::Fecha(const char* cad); // Sin desbordamiento
Fecha::Fecha(istream& stream); // Con desbordamiento
 

Un saludo.

[agusax]

Añado mi flujo de entrada:

Código C++:

istream& operator >>(istream& flujo, Fecha& fecha){
    char* cadAuxiliar = new char [flujo.width()];
 
    flujo >> cadAuxiliar;
    fecha = cadAuxiliar;
    delete [] cadAuxiliar;
 
    return flujo;
}
 

[agusax]

ACABO DE SOLUCIONARLO:

He actualizado el flujo de entrada de los datos y ya me pasa todos los test correctamente, mirando el desbordamiento también.

Código C++:

istream& operator >>(istream& flujo, Fecha& fecha){
 
    char* entrada = new char[11];
    flujo >> setw(11) >> entrada;
    fecha = Fecha(entrada);
    return flujo;
}
 

Muchas gracias por tu tiempo :D

[eferion]

Vale, lo que te falta ahora es llamar al método que comprueba el desbordamiento:

Código C++:

istream& operator >>(istream& flujo, Fecha& fecha){
    char* cadAuxiliar = new char [flujo.width()];
 
    flujo >> cadAuxiliar;
    fecha.ComprobarDesbordamiento(cadAuxiliar); // O como se use
    fecha = cadAuxiliar;
    delete [] cadAuxiliar;
 
    return flujo;
}
 

En cualquier caso... ¿por qué usas char* en vez de la clase std::string? Si es una exigencia de tu profesor vale... en caso contrario deberías cambiar esos usos:

Código C++:

istream& operator >>(istream& flujo, Fecha& fecha){
    std::string cadAuxiliar;
 
    flujo >> cadAuxiliar;
    fecha = cadAuxiliar; // Si puedes usar std::string
    fecha = cadAuxiliar.c_str(); // Si quieres seguir usando char*
 
    return flujo;
}
 

Un saludo.

[agusax]

Era una exigencia del profesor.
Muchas gracias por su tiempo y ayudarme, voy a marcar el post como SOLUCIONADO.