To insert more than one combobox values into access database using C# windows application.

Question

0.00/5 (No votes)

See more:

I want to insert combobox values into access database using C# windows application. It allows only one combobox to insert value into the database. How can I insert more than one combobox value into the database.

What I have tried:

C#

OleDbConnection con = new OleDbConnection(@"Provider=Microsoft.ACE.Oledb.12.0; Data Source= school.accdb");
                string query = "Insert into stu_registration (CLASS, SECTION) values ('" + comboBox101.Text + "', '"+comboBox102.Text+"')";
                
                
                OleDbCommand cmd = new OleDbCommand(query, con);
                


                con.Open();
                cmd.ExecuteNonQuery();
                con.Close();
            }

Posted 29-Mar-20 18:56pm

vermaji_vermaji

Updated 29-Mar-20 21:11pm

Maciej Los

v2

Add a Solution

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Maciej Los · Answer 1 · 2020-03-29T19:55:00

Two things:

1) Your code is sql injection[^] vulnerable!
You should avoid of using concatenated string as a query. Use parameterized queries instead.

C#

string sConn = @"Provider=Microsoft.ACE.OLEDB.12.0;Data Source=D:\school.accdb;Persist Security Info =False;";
DataTable dt = new DataTable();

//student id to find
int stuIdToFind = 111; 
//select statement
string sComm = "SELECT Stu.* FROM Students AS Stu WHERE Stu.StuId=?;";

using (OleDbConnection oConn = new OleDbConnection(sConn))
	{
		oConn.Open();
		using (OleDbCommand oComm  = new OleDbCommand(sComm, oConn))
		{
			//add parameter with value
			oComm.Parameters.Add(new OleDbParameter(){Value=stuIdToFind});
			using (OleDbDataReader oRdr = oComm.ExecuteReader())
			{
				dt.Load(oRdr);
			}
		}
	}

2) You're using SECTION as a field name but(!) this word is one of reserved words in MS Access.
See: List of reserved words in Access - Office | Microsoft Docs[^]
I'd suggest to change a field name or use square brackets around a field name:

SQL

SELECT [SECTION] FROM YourTableName;

OriginalGriff · Answer 2 · 2020-03-29T21:12:00

In addition to the very accurate comments by Maciej, You can insert multiple rows (if that's what you are trying to do in several ways:
1) Use a multiple INSERT

C#

INSERT INTO MyTable (Col1, Col2) VALUES (1, 2), (3, 4), 5, 6)

But in your case that's still goingh to be SQL Injection vulnerable.
2) Do your INSERT operation in a loop.
3) Use a DataTable and INSERT via a DataAdapter: c# - Insert dataset records in database - Stack Overflow[^]

To insert more than one combobox values into access database using C# windows application.

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0