web-dev-qa-db-fra.com

Nom de colonne incorrect Erreur SQL

J'essaie d'entrer des données dans ma base de données, mais cela me donne l'erreur suivante:

Nom de colonne invalide

Voici mon code 

string connectionString = "Persist Security Info=False;User ID=sa;Password=123;Initial Catalog=AddressBook;Server=Bilal-PC";

using (SqlConnection connection = new SqlConnection(connectionString))
{
  SqlCommand cmd = new SqlCommand();

  cmd.CommandText = "INSERT INTO Data (Name,PhoneNo,Address) VALUES (" + txtName.Text + "," + txtPhone.Text + "," + txtAddress.Text + ");";
  cmd.CommandType = CommandType.Text;
  cmd.Connection = connection;

  connection.Open();
  cmd.ExecuteNonQuery();
}
c#sql-serverdatabase
12
Snake

Essayez toujours d'utiliser une requête SQL paramétrée pour vous protéger des occurrences malveillantes afin de pouvoir réorganiser votre code comme suit:

Assurez-vous également que le nom de colonne de votre table correspond à Name, PhoneNo, Address.

using (SqlConnection connection = new SqlConnection(connectionString))
{
    SqlCommand cmd = new SqlCommand("INSERT INTO Data (Name, PhoneNo, Address) VALUES (@Name, @PhoneNo, @Address)");
    cmd.CommandType = CommandType.Text;
    cmd.Connection = connection;
    cmd.Parameters.AddWithValue("@Name", txtName.Text);
    cmd.Parameters.AddWithValue("@PhoneNo", txtPhone.Text);
    cmd.Parameters.AddWithValue("@Address", txtAddress.Text);
    connection.Open();
    cmd.ExecuteNonQuery();
}
27
Elias Hossain

Vous avez probablement besoin de guillemets autour de ces champs de chaîne, mais vous devriez utiliser des requêtes paramétrées!

cmd.CommandText = "INSERT INTO Data ([Name],PhoneNo,Address) VALUES (@name, @phone, @address)";
cmd.CommandType = CommandType.Text;
cmd.Parameters.AddWithValue("@name", txtName.Text);
cmd.Parameters.AddWithValue("@phone", txtPhone.Text);
cmd.Parameters.AddWithValue("@address", txtAddress.Text);
cmd.Connection = connection;

Incidemment, votre requête initiale aurait pu être corrigée comme ceci (notez les guillemets simples):

"VALUES ('" + txtName.Text + "','" + txtPhone.Text + "','" + txtAddress.Text + "');";

mais cela l'aurait rendu vulnérable aux attaques par injection SQL puisqu'un utilisateur pourrait taper 

'; drop table users; --

dans l'un de vos champs de texte. Ou plus simplement, le pauvre Daniel O'Reilly vous répondrait à chaque fois.

30
Adam Rackis

Changer cette ligne:

cmd.CommandText = "INSERT INTO Data (Name,PhoneNo,Address) VALUES (" + txtName.Text + "," + txtPhone.Text + "," + txtAddress.Text + ");";

pour ça:

cmd.CommandText = "INSERT INTO Data (Name,PhoneNo,Address) VALUES ('" + txtName.Text + "','" + txtPhone.Text + "','" + txtAddress.Text + "');";

Votre commande d'insertion attend du texte et vous avez besoin de guillemets simples (') entre la valeur réelle pour que SQL puisse la comprendre en tant que texte.

EDIT: Pour ceux d’entre vous qui ne sont pas satisfaits de cette réponse, je voudrais signaler qu’il ya un problème avec ce code en ce qui concerne l’injection SQL. Lorsque j'ai répondu à cette question, j'ai uniquement considéré la question en question, à savoir la citation simple manquante de son code, et j'ai indiqué comment la corriger. Adam (et j'ai voté pour cette solution) a publié une bien meilleure réponse, dans laquelle il explique les problèmes liés à l'injection et montre un moyen de prévention. Maintenant, détendez-vous et soyez heureux les gars.

6
MilkyWayJoe

Votre problème est que votre chaîne n'est pas citée. Ce qui signifie qu'ils sont interprétés par votre moteur de base de données comme un nom de colonne.

Vous devez créer des paramètres afin de transmettre votre valeur à la requête.

 cmd.CommandText = "INSERT INTO Data (Name, PhoneNo, Address) VALUES (@Name, @PhoneNo, @Address);";
 cmd.Parameters.AddWithValue("@Name", txtName.Text);
 cmd.Parameters.AddWithValue("@PhoneNo", txtPhone.Text);
 cmd.Parameters.AddWithValue("@Address", txtAddress.Text);
5
Pierre-Alain Vigeant

Vous ne devriez jamais écrire du code concaténant SQL et les paramètres sous forme de chaîne - cela ouvre votre code à injection SQL , ce qui pose un problème de sécurité vraiment sérieux.

Utilisez bind params - pour un Nice comment voir ici ...

4
Yahia

Code Pour insérer des données dans Access Db en utilisant c #

Code:- 

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Data.SqlClient;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace access_db_csharp
{
public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
    }
   public SqlConnection con = new SqlConnection(@"Place Your connection string");

           private void Savebutton_Click(object sender, EventArgs e)
    {
         SqlCommand cmd = new SqlCommand("insert into  Data (Name,PhoneNo,Address) values(@parameter1,@parameter2,@parameter3)",con);
                cmd.Parameters.AddWithValue("@parameter1", (textBox1.Text));
                cmd.Parameters.AddWithValue("@parameter2", textBox2.Text);
                cmd.Parameters.AddWithValue("@parameter3", (textBox4.Text));
                cmd.ExecuteNonQuery();

                }

    private void Form1_Load(object sender, EventArgs e)
    {
        con.ConnectionString = connectionstring;
        con.Open();
    }
}

}

2
Heemanshu Bhalla
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;
using System.Data.SqlClient;
using System.Data;

namespace WpfApplication1
{
    /// <summary>
    /// Interaction logic for MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {
        public MainWindow()
        {
            InitializeComponent();
        }

        private void btnAdd_Click(object sender, RoutedEventArgs e)
        {
            SqlConnection conn = new SqlConnection(@"Data Source=WKS09\SQLEXPRESS;Initial Catalog = StudentManagementSystem;Integrated Security=True");
            SqlCommand insert = new SqlCommand("insert into dbo.StudentRegistration(ID, Name,Age,DateOfBirth,Email,Comment) values(@ID, @Name,@Age,@DateOfBirth,@mail,@comment)", conn);
            insert.Parameters.AddWithValue("@ID", textBox1.Text);
            insert.Parameters.AddWithValue("@Name", textBox2.Text);
            insert.Parameters.AddWithValue("@Age", textBox3.Text);
            insert.Parameters.AddWithValue("@DateOfBirth", textBox4.Text);
            insert.Parameters.AddWithValue("@mail", textBox5.Text);
            insert.Parameters.AddWithValue("@comment", textBox6.Text);

            if (textBox1.Text == string.Empty)
            {
                MessageBox.Show("ID Cannot be Null");
                return;
            }
            else if (textBox2.Text == string.Empty)
            {
                MessageBox.Show("Name Cannot be Null");
                return;
            }


            try
            {
                conn.Open();
                insert.ExecuteNonQuery();
                MessageBox.Show("Register done !");
            }
            catch (Exception ex)
            {
                MessageBox.Show("Error" + ex.Message);
                conn.Close();
            }
        }

        private void btnRetrive_Click(object sender, RoutedEventArgs e)
        {
            bool temp = false;
            SqlConnection con = new SqlConnection("server=WKS09\\SQLEXPRESS;database=StudentManagementSystem;Trusted_Connection=True");
            con.Open();
            SqlCommand cmd = new SqlCommand("select * from dbo.StudentRegistration where ID = '" + textBox1.Text.Trim() + "'", con);
            SqlDataReader dr = cmd.ExecuteReader();
            while (dr.Read())
            {
                textBox2.Text = dr.GetString(1);
                textBox3.Text = dr.GetInt32(2).ToString(); 
                textBox4.Text = dr.GetDateTime(3).ToString();
                textBox5.Text = dr.GetString(4);
                textBox6.Text = dr.GetString(5);
                temp = true;
            }
            if (temp == false)
                MessageBox.Show("not found");
            con.Close();
        }

        private void btnClear_Click(object sender, RoutedEventArgs e)
        {
            SqlConnection connection = new SqlConnection("Data Source=WKS09\\SQLEXPRESS;Initial Catalog = StudentManagementSystem;Integrated Security=True");
            string sqlStatement = "DELETE FROM dbo.StudentRegistration WHERE ID = @ID";
            try
            {
                connection.Open();
                SqlCommand cmd = new SqlCommand(sqlStatement, connection);
                cmd.Parameters.AddWithValue("@ID", textBox1.Text);
                cmd.CommandType = CommandType.Text;
                cmd.ExecuteNonQuery();
                MessageBox.Show("Done");
            }
            finally
            {
                Clear();
                connection.Close();
            }
        }

        public void Clear()
        {
            textBox1.Text = "";
            textBox2.Text = "";
            textBox3.Text = "";
            textBox4.Text = "";
        }
    }
}
2
pavithra madhuwanthi

Vous devez utiliser '"+texbox1.Text+"','"+texbox2.Text+"','"+texbox3.Text+"'

Au lieu de "+texbox1.Text+","+texbox2.Text+","+texbox3.Text+"

Notez les guillemets simples supplémentaires.

1
user7603356

commencez par créer le nom de base de données "Ecole" puis créez une table "étudiants" avec les colonnes suivantes 1. id 2. nom 3. adresse

ouvrez maintenant Visual Studio et créez une connexion:

école d'espaces de noms 
 {
 classe partielle publique Form1: Form 
 {
 SqlConnection scon; 


 public Form1 () 
 {

 InitializeComponent (); 

 scon = new SqlConnection ("Source de données = ABC-PC; confidence_connection = yes; Base de données = école; délai de connexion = 30"); 
 } 

 // créer une commande 

 SqlCommand scom = new SqlCommand ("insérer dans les étudiants (id, nom, adresse) valeurs (@ id, @ nom, @ adresse)", scon); 

 // paramètres de passage 

 scom.Parameters.Add ("id", SqlDbType.Int); 
 scom.Parameters ["id"]. Value = textBox1.Text ;

 scom.Parameters.Add ("name", SqlDbType.VarChar); 
 scom.Parameters ["name"]. Value = this.textBox2.Text; 

 scom.Parameters.Add ("adresse", SqlDbType.VarChar); 
 scom.Parameters ["address"]. Value = this.textBox6.Text; 


 scon.Open (); 
 scom.ExecuteNonQuery (); 
 scon.Close (); 
 réinitialiser();

 }

consultez également la solution ici: http://solutions.musanitech.com/?p=6

0
Hani Mehdi
con = new SqlConnection(@"Data Source=.\SQLEXPRESS;AttachDbFilename=C:\Users\Yna Maningding-Dula\Documents\Visual Studio 2010\Projects\LuxuryHotel\LuxuryHotel\ClientsRecords.mdf;Integrated Security=True;User Instance=True");
        con.Open();
        cmd = new SqlCommand("INSERT INTO ClientData ([Last Name], [First Name], [Middle Name], Address, [Email Address], [Contact Number], Nationality, [Arrival Date], [Check-out Date], [Room Type], [Daily Rate], [No of Guests], [No of Rooms]) VALUES (@[Last Name], @[First Name], @[Middle Name], @Address, @[Email Address], @[Contact Number], @Nationality, @[Arrival Date], @[Check-out Date], @[Room Type], @[Daily Rate], @[No of Guests], @[No of Rooms]", con);
        cmd.Parameters.Add("@[Last Name]", txtLName.Text);
        cmd.Parameters.Add("@[First Name]", txtFName.Text);
        cmd.Parameters.Add("@[Middle Name]", txtMName.Text);
        cmd.Parameters.Add("@Address", txtAdd.Text);
        cmd.Parameters.Add("@[Email Address]", txtEmail.Text);
        cmd.Parameters.Add("@[Contact Number]", txtNumber.Text);
        cmd.Parameters.Add("@Nationality", txtNational.Text);
        cmd.Parameters.Add("@[Arrival Date]", txtArrive.Text);
        cmd.Parameters.Add("@[Check-out Date]", txtOut.Text);
        cmd.Parameters.Add("@[Room Type]", txtType.Text);
        cmd.Parameters.Add("@[Daily Rate]", txtRate.Text);
        cmd.Parameters.Add("@[No of Guests]", txtGuest.Text);
        cmd.Parameters.Add("@[No of Rooms]", txtRoom.Text);
        cmd.ExecuteNonQuery();
0
Yna

Votre problème semble être le mot clé Name. Utilisez plutôt FullName ou firstName et lastName, essayez toujours de vous rappeler d'utiliser également CamelCase.

0
Jonathan Yaniv Ben Avraham