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
        }
    }

    //essa classe Ado parece desnecessária ou mal formulada
    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 https://pt.stackoverflow.com/q/219211/101.

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][1]][1]


  [1]: https://i.sstatic.net/LHKoL.png