2

Tenho um método que usar o DateTime.TryParse() para converter uma string para o tipo DateTime, mas ultimamente percebi que string com o formato Double também estaria sendo convertido para data.

Existe alguma solução para corrigir isso?

public class SaidaData : SaidaBase
{
    public SaidaData() { }
    public DateTime? Data { get; set; }
}

public class SaidaBase
{
    public SaidaBase() { }

    public Boolean Sucesso { get; set; }
    public String Mensagem { get; set; }
}

public class EntradaBase
{
    public EntradaBase() { }

    public String Conteudo { get; set; }
    
}

public class Entrada : EntradaBase
{
    public Entrada() { }

    public Int32 PosIni { get; set; }
    public Int32 Tamanho { get; set; }
    public String[] Array { get; set; }
}

#region ConverterCampoDateTime
public static SaidaData ConverterCampoDateTime(Entrada entrada)
{
    var saida = new SaidaData();

    DateTime valor;
    var convertido = DateTime.TryParse(entrada.Conteudo, out valor);

    if (convertido)
    {
        //saida.Mensagem = Resources.Mensagens.OK;
        saida.Mensagem = "OK";
        saida.Sucesso = true;
        saida.Data = valor;
    }
    else
    {
        //saida.Mensagem = Resources.Mensagens.Erro;
        saida.Data = null;
        saida.Mensagem = "ERRO";
        saida.Sucesso = false;
    }

    return saida;
}
#endregion

SS SS

6
  • Não entendi qual é o problema.
    – Maniero
    20/06/2016 às 12:56
  • 1
    Quem manda na conversão não é o "formato" da string de entrada e sim a classe. Se você está usando DateTime.TryParse é óbvio que a conversão vai ser feita para uma data.
    – Jéf Bueno
    20/06/2016 às 13:00
  • @bigown, o DateTime.TryParse recebe uma string ("276,11") e converte para uma data. 20/06/2016 às 13:02
  • @jefersonb, não tem nenhuma forma para verificar isso? 20/06/2016 às 13:04
  • 2
    Dá pra verificar na mão. Agora me explique uma coisa, você está passando este conteúdo para um método chamado ConverterParaDateTime e quer que a conversão não seja feita? Por que chamar o método então?
    – Jéf Bueno
    20/06/2016 às 13:07

1 Resposta 1

4

Um método de parse analisa um texto e tenta achar algo que ele reconhece ali. Se ele reconhecer e puder gerar um resultado ele o fará. Ele faz o melhor esforço possível. Se a intenção é só reconhecer um formato específico tem que definir este formato, para isso existe o TryParseExact(). Exemplo (o formato, cultura e estilo pode ter que ser diferente):

DateTime.TryParseExact(entrada.Conteudo, "dd/MM/yyyy",
    CultureInfo.InvariantCulture, DateTimeStyles.None, out var valor);

Coloquei no GitHub para referência futura.

A confusão talvez ocorra porque os dados dessa classe são confusos. Talvez essas classes estejam suportando coisas demais.

Acho estranho ter essa herança, mas sei lá, não conheço o problema. Só estou dando uma dica que o problema pode ser maior e a arquitetura seja problemática, a conversão errada seja só o sintoma.

Se for para fazer uma classe assim, tenho até minhas dúvidas se precisa de propriedades.

Esses construtores são desnecessários.

Não gosto muito do uso dos tipos do .NET no lugar dos tipos do C#. É gosto, pode usar se quiser, mas é um indicativo de invencionice :)

A parte boa é que está usando um método Try.

2
  • Obrigado pelas dicas, sua solução resolveu o problema, essa arquitetura realmente não é das boas, mas acredito que você já sabe como é trabalha em uma empresa com um sistema fortemente acoplado. 20/06/2016 às 13:14
  • Sei sim como é ;)
    – Maniero
    20/06/2016 às 13:16

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .