Skip to main content
adicionou 4 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

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.

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 futura.

Tenho dúvidas e não tem outros problemas.

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 futura.

Tenho dúvidas e não tem outros problemas.

adicionou 121 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

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 é:

int codigo;
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 futura.

Tenho dúvidas e não tem outros problemas.

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 é:

int codigo;
if (int.TryParse(SalvarCodigo.Text, 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
}

Tenho dúvidas e não tem outros problemas.

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 futura.

Tenho dúvidas e não tem outros problemas.

replaced http://pt.stackoverflow.com/ with https://pt.stackoverflow.com/
Fonte Link

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 Exceptionnã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 perguntanessa pergunta e nessa tambémnessa também. Veja Diferenças entre Parse() vs TryParse()Diferenças entre Parse() vs TryParse() e Qual a principal diferença entre int.Parse() e Convert.ToInt32()?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 é:

int codigo;
if (int.TryParse(SalvarCodigo.Text, 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
}

Tenho dúvidas e não tem outros problemas.

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 é:

int codigo;
if (int.TryParse(SalvarCodigo.Text, 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
}

Tenho dúvidas e não tem outros problemas.

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 é:

int codigo;
if (int.TryParse(SalvarCodigo.Text, 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
}

Tenho dúvidas e não tem outros problemas.

Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando