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

O jeito correto de converter é assim:

private void txtValorFinal_TextChanged(object sender, EventArgs e) {
    if (decimal.TryParse(txtValorFinal.Text, out var valorFinal) && decimal.TryParse(txtValorFornecedor.Text, out var valorFornecedor)) {
        txtLucro.Text = (valorFinal - valorFornecedor).ToString("0.00");
    } else {
        //tratar o erro aqui
    }
}

var insert = new Ado("Data Source=DESKTOP-AJLR3DB\\SQLEXPRESS;Initial Catalog=DBGestor;Integrated Security=True");
insert.InsertProduto(txtNomeProduto.Text,
    //tem que fazer o mesmo do acima,
    cmbTipo.Text,
    //tem que fazer o mesmo do acima,
    //tem que fazer o mesmo do acima,
    //tem que fazer o mesmo do acima);
MessageBox.Show("Produto cadastrado com sucesso!", "Sucesso!", MessageBoxButtons.OK);
    //e se der erro?
this.Close(); //duvido que isto está correto aqui
this.Dispose(); //quase certo que não deveria estar aqui

Assim:

//o mais certo é receber um objeto com estas propriedades e não esse monte de parâmetro
public void InsertProduto(string nomeProduto, int quantidade, string tipo, decimal valorFornecedor, decimal valorFinal, decimal lucro) {
    using (var conn = new SqlConnection(ConectionString)) {
        conn.Open();
        //não sei o nome das colunas no banco de dados chuetei o que deveria ser
        var cmString = "INSERT INTO produtos (Nome, Quantidade, Tipo, ValorFornecedor, ValorFinal, Lucro) VALUES (@Nome, @Quantidade, @Tipo, @ValorFornecedor, @ValorFinal, @Lucro";
        var cmd = new SqlCommand(cmdString, conn);
        cmd.Parameters.AddWithValue("@Nome", nomeProduto);
        cmd.Parameters.AddWithValue("@Quantidade", quantidade);
        cmd.Parameters.AddWithValue("@Tipo", tipo);
        cmd.Parameters.AddWithValue("@ValorFornecedor", valorFornecedor);
        cmd.Parameters.AddWithValue("@ValorFinal", valorFinal);
        cmd.Parameters.AddWithValue("@Lucro", lucro);
        cmd.ExecuteNonQuery();
    }
}

Veja Qual a forma correta de usar os tipos float, double e decimal?.

E procure sobre SQL Injection.

Então há alguns erros no código:

  • Está convertendo de forma que gera erro, tem que verificar se deu certo.
  • Está usando ponto flutuante binário para valor monetário, precisa ser decimal.
  • Está Tratando exceção, e de um jeito bastante errado (tem pleo menos 3 erros só na exceção) quando o correto é usar using.
  • Está permitindo SQL Injection quando deveria parametrizar a query.
  • Está abusando de parâmetros quando deveria usar um objeto
  • Está dispondo de algo em lugar errado.
  • Provavelmente o campo Lucro não deveria existir no banco de dados porque é informação duplicada, mas depende do contexto para ter certeza.

Por sorte deu erro em uma situação. Não pode confiar no que funciona, é preciso estar certo.

Isto funciona:

Fiat 147 todo bagaçado andando pelas ruas

Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil