How to add the current items into combobox at runtime

Question

0.00/5 (No votes)

See more:

In my forms I have combobox linked to sql server, the items in combobox is identity, two textbox and datetimepickers and button add ,when I press the button add at runtime i want to see the numbers of current adding information in combobox, please if you have any help answer me.

Thanks a lot.

Posted 22-Mar-15 4:03am

Member 11545313

Updated 22-Mar-15 5:00am

Afzaal Ahmad Zeeshan

v2

Add a Solution

Comments

Kornfeld Eliyahu Peter 22-Mar-15 10:10am

What have you tried?[^]

Member 11545313 22-Mar-15 10:28am

i tried in button added
try
{
cn.Open();
cmd.CommandText = ("insert into film (nom_film,date_sortie,duree)values('" + textBox1.Text+ "','"+this.dateTimePicker1.Text+"','"+textBox3.Text+"')");
cmd.Connection = cn;
cmd.ExecuteNonQuery();

//the problems is inside loop
if (comboBox1.SelectedIndex != 0)
{
this.comboBox1.Items.Add(this.comboBox1.SelectedIndex);
//comboBox1.Refresh();
}

textBox1.Clear();
textBox1.Focus();
textBox3.Clear();
MessageBox.Show("Film Ajouter Avec Succees", "insertion Filme", MessageBoxButtons.OK, MessageBoxIcon.Information);
cn.Close();
}
catch (SqlException ex)
{
MessageBox.Show(ex.Message.ToString());
}

Afzaal Ahmad Zeeshan 22-Mar-15 11:03am

You can add new controls to the Control collection.

Member 11545313 23-Mar-15 17:23pm

Give me an exemple plz i need to handle this

Dave Kreskowiak 22-Mar-15 11:18am

Do not post your question in multiple forums. All that does is make collaboration on an answer very difficult and can get you ignored.

Member 11545313 23-Mar-15 7:24am

i'm post it just in this forum think's for advice

1 solution

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)

manchanx · Answer 1 · 2015-03-22T05:40:00

This:

C#

//the problems is inside loop
if (comboBox1.SelectedIndex != 0)
{
  this.comboBox1.Items.Add(this.comboBox1.SelectedIndex);
}

is not a loop. It's an if-Statement. The code in the block of the if-Statement gets executed once if the predicate of the if-Statement evaluates to true.

I assume you want to insert the newly created record as a combobox-item in any case, right? Then you won't need an if-Statement. Currently you only would insert the new combobox-item if there's any item currently selected. And currently you actually insert the SelectedIndex of the combobox - as you can easily read from your code. That doesn't make much sense. I assume you would want something like this:

C#

//no if-Statement here
comboBox1.Items.Add(String.Concat(textBox1.Text, " ,", dateTimePicker1.Text, " ,", textBox3.Text));

And then there are a couple of other problems with your code:
- Use SQL-Parameters. Simple example here[^]
- If an exception occurs you report it and then keep going as if nothing happened. You should do something to handle the error besides reporting it.
- Your connection won't get closed in case of an exception being thrown.

edits: typos