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: