web-dev-qa-db-ja.com

無効な列名のSQLエラー

私は私のデータベースにデータを入力しようとしていますが、それは私に次のエラーを与えています。

無効な列名

これが私のコードです

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

常にあなたがあなたに以下のようにコードを並べ替えることができるように、悪意の発生から安全に保つためにパラメータ化SQLクエリを使用しよう：

また、必ずあなたのテーブルを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

これらの文字列フィールドはおそらく引用符で囲む必要がありますが、パラメータ化されたクエリを使用する必要があります！

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;

ちなみに、元のクエリは次のように修正できます（一重引用符に注意してください）。

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

ただし、これによりSQLインジェクション攻撃に対して脆弱になりますユーザーが入力できるため

'; drop table users; --

あなたのテキストボックスの一つに。または、もっと平凡に、貧しいダニエル・オライリーは毎回あなたの質問を壊すでしょう。

30
Adam Rackis

この行を変更します。

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

これに：

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

挿入コマンドにはテキストが必要です。実際の値の間に一重引用符（ '）が必要なので、SQLはそれをテキストとして理解できます。

[〜＃〜] edit [〜＃〜]：この答えに満足していない人のために、SQLに関してこのコードに問題があることを指摘したいと思います注入。私がこの質問に答えたとき、私は彼のコード上に欠けている一重引用符である点だけを考慮し、それを修正する方法を指摘しました。より良い回答がAdamによって投稿され（そして私はそれに投票しました）、そこで彼は注射の問題を説明し、防ぐ方法を示します。今、リラックスして幸せな人になります。

6
MilkyWayJoe

あなたの問題は、あなたの文字列は引用符で囲まれていないということです。これは、彼らが列名として使用するデータベースエンジンによって解釈されていることを意味します。

あなたは、クエリにあなたの価値を渡すためにパラメータを作成する必要があります。

 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

SQLとパラメーターを文字列として連結するコードを記述しないでください。これにより、コードが SQLインジェクション に開かれます。これは本当に深刻なセキュリティ問題です。

Bind paramsを使用します-ニースのハウツーについては here ...をご覧ください.

4
Yahia

コードc＃を使用してAccess Dbにデータを挿入するには

コード：-

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

'"+texbox1.Text+"','"+texbox2.Text+"','"+texbox3.Text+"'を使用する必要があります

"+texbox1.Text+","+texbox2.Text+","+texbox3.Text+"の代わりに

余分な一重引用符に注意してください。

1
user7603356

最初にデータベース名「School」を作成し、次の列を持つテーブル「students」を作成します1. id 2. name 3. address

visual Studioを開き、接続を作成します。

名前空間school 
 {
 public partial class Form1：Form 
 {
 SqlConnection scon; 
 
 
 public Form1 （）
 {
 
 InitializeComponent（）; 
 
 scon = new SqlConnection（ "Data Source = ABC-PC; trusted_connection = yes; Database =学校;接続タイムアウト= 30 "）; 
} 
 
 // create command 
 
 SqlCommand scom = new SqlCommand（" insert into students （id、name、address）values（@ id、@ name、@ address） "、scon）; 
 
 // pass parameters 
 
 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（" address "、SqlDbType .VarChar）; 
 scom.Parameters ["address"]。Value = this.textBox6.Text; 
 
 
 scon.Open（）; 
 scom.ExecuteNonQuery（）; 
 scon.Close（）; 
 reset（）; 
 
}

ここで解決策も確認してください： 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

あなたの問題はNameキーワードのようです。 FullNameまたはfirstNameとlastNameを使用するのではなく、常にCamelCaseも使用するようにしてください。

0
Jonathan Yaniv Ben Avraham