Ver Mensaje Individual
  #3  
Antiguo 27-04-2011
Avatar de duilioisola
[duilioisola] duilioisola is offline
Miembro Premium
 
Registrado: ago 2007
Ubicación: Barcelona, España
Posts: 1.806
Reputación: 22
duilioisola Es un diamante en brutoduilioisola Es un diamante en brutoduilioisola Es un diamante en bruto
La solución es correcta. Lo que quizás no sea tan correcto es el uso de excepciones para tratar un error previsible.

El uso de excepciones carga mucho al programa, destruye el flujo normal del código y puede hacer que el código no sea fácilmente legible.

En el caso que nos ocupa, sería preferible hacer una simple comprobación y con un IF decidir si es correcto o no.

Código Delphi [-]
   // Compruebo que el edit tenga un numero
   if (StrToIntDef(dmGlobal.g_Mes), 0) > 0) then
   begin
      // Primero compruebo que el mes sea correcto
      if (StrToInt(dmGlobal.g_Mes) <= 12) then
      begin
         // Luego compruebo que el día sea menor que la cantidad de días en el mes
         if (DaysInMonth(StrToInt(Copy(dmGlobal.g_Periodo,1,4)), StrToInt(dmGlobal.g_Mes)) >= StrToInt(edtDia.Text)) then
            wf_vch := EncodeDate(StrToInt(Copy(dmGlobal.g_Periodo,1,4)), StrToInt(dmGlobal.g_Mes),  strtoint(edtDia.Text))
         else
            edtDia.SetFocus;
      end;
   end;

Y para ahorrarnos todo este trabajo, existe la función TryEncodeDate que nos devuelve en su último parámetro la fecha y verdadero o false si se pudo convertir correctamente.
Responder Con Cita