O código todo me parece estranho e parece haver problemas de design nele. Pode ser uma questão de estilo e gosto, mas eu faria todo o método SalvarDados()
em apenas 2 linhas. Não vou tentar ensinar dessa forma porque pode confundir quem não tem uma boa noção geral. Eu acho estranho criar uma instância de Crud
só para isso, mas sei que tem quem goste disso. Oque certamente é um um erro é ter esse try-catch
. Ele não está resolvendo problema algum. Na verdade está dificultando identificar o erro. Mesmo que tivesse que capturar uma exceção, não deveria ser Exception
.
Você não deve converter dados que você não tem controle. Deve usar TryParse()
ou algo parecido, se falhar tem como se recuperar no fluxo normal da aplicação. Isso pode ser visto nessa pergunta e nessa também. Veja Diferenças entre Parse() vs TryParse() e Qual a principal diferença entre int.Parse() e Convert.ToInt32()?. Se falhar você deve tomar alguma atitude, mas não pode salvar os dados. O problema pode estar aí.
Mas nem adianta tentar converter algo que não pode ser convertido. Oque talvez deva fazer é:
if (int.TryParse(SalvarCodigo.Text, var out codigo)) { //<=======note o .Text aqui
var d = new Crud();
d.Atualizar(new Contato(codigo, SalvarNome.Text, SalvarFone.Text, SalvarEmail.Text));
Message.Text = "Cadastro alterado com sucesso";
} else {
//aqui trata se o código é inválido
}
Coloquei no GitHub para referência futuraColoquei no GitHub para referência futura.
Tenho dúvidas e não tem outros problemas.