Query works in the database but not via code

Question

Query works in the database but not via code

Asked 9 years, 9 months ago

Viewed 4,343 times

2

using (var cnx = new OleDbConnection(new AdministradorDAO().conexao))
            {
                var sql =
            @"insert into usuarios(matricula, nome, senha, nivel, maleta, email) 
            values (@matricula, @nome, @senha, @nivel, @maleta, @email)";


                using (var cmd = new OleDbCommand(sql, cnx))
                {
                    cnx.Open();

                    cmd.Parameters.AddWithValue("@matricula", txt_matricula.Text);
                    cmd.Parameters.AddWithValue("@nome", txt_nome.Text);
                    cmd.Parameters.AddWithValue("@senha", txt_senha.Text);
                    cmd.Parameters.AddWithValue("@nivel", DropDownList_nivel.Text);
                    cmd.Parameters.AddWithValue("@maleta", txt_maleta.Text);
                    cmd.Parameters.AddWithValue("@email", txt_email.Text);

                    try { cmd.ExecuteNonQuery(); }
                    catch { }
                    finally { if (cnx.State == ConnectionState.Open) cnx.Close(); }
                }
            }

When making the Insert directly on the seat, it rotates perfectly

insert into actweb.usuarios (matricula, nome, senha, nivel, maleta, email) values ('TESTE', 'Frederico', 'TESTE', 1, 7000, '[email protected]');

I running directly, without using the right Parameters.

var sql = @"Insert into usuarios(id,name) values (ID_USUARIOS.nextval, 'It will work')";

Insert into actweb.usuarios (matricula, nome, password, nivel, maleta, email) values ('TESTE', 'Frederico', 'TESTE', 1, 7000, '[email protected]');

– user2254936

2014/09/16 at 20:43
You should [Edit] your question to add this information. You know this try catch finnaly does not have any utility in your code? It is even causing problem. The using is solving connection closure.

– Maniero

2014/09/16 at 20:44
1

The catch is swallowing the error. If you take it out, you will have some useful information to figure out where the error is. Then you can solve or post here what the error is, so it is easier to help you.

– Maniero

2014/09/16 at 20:53
I took the catch, now you are showing this error One or more errors occurred while processing the command. ORA-00936: expression not found

– user2254936

2014/09/16 at 20:54
You need to [edit] your question to make it complete with this information and make it easy for anyone who knows Oracle errors and can help you.

– Maniero

2014/09/16 at 20:57

1 answer

Browser other questions tagged c# oracle query

You are not signed in. Login or sign up in order to post.

by Leonel Sanches da Silva • **88,623** points · Answer 1 · 2014-09-17T01:23:39+00:00

There are two things wrong with your code:

try with catch emptiness;
As I recall, the syntax for insertion in Oracle does not start with @, and yes by two-points;

The right thing would be something like this:

using (var cnx = new OleDbConnection(new AdministradorDAO().conexao))
{
    var sql =
        @"insert into usuarios(matricula, nome, senha, nivel, maleta, email) 
        values (:matricula, :nome, :senha, :nivel, :maleta, :email)";

    using (var cmd = new OleDbCommand(sql, cnx))
    {
        cnx.Open();

        cmd.Parameters.AddWithValue("matricula", txt_matricula.Text != "" ? txt_matricula.Text : DbValue.Null);
        cmd.Parameters.AddWithValue("nome", txt_nome.Text);
        cmd.Parameters.AddWithValue("senha", txt_senha.Text != "" ? txt_senha.Text : DbValue.Null);
        cmd.Parameters.AddWithValue("nivel", DropDownList_nivel.Text != "" ? DropDownList_nivel.Text : DbValue.Null);
        cmd.Parameters.AddWithValue("maleta", txt_maleta.Text != "" txt_maleta.Text : DbValue.Null);
        cmd.Parameters.AddWithValue("email", txt_email.Text != "" ? txt_email.Text : DbValue.Null);

        try 
        { 
            cmd.ExecuteNonQuery(); 
        }
        catch (Exception e) 
        {
            throw; 
        }
        finally 
        { 
            if (cnx.State == ConnectionState.Open) cnx.Close(); 
        }
    }
}