How to exchange the items from one list box to other when both list box are binded to the data source?

Question

5.00/5 (1 vote)

See more:

C#

private void buttonSelectAllAvailableStops_Click(object sender, EventArgs e)
{
   // this.stopBindingSource.DataSource = null;
   // this.routeBindingSource1.DataSource = null;

    foreach (string item in listBoxAvailableStops.Items)
    {
        if (listBoxSelectedStops.Items.Contains(item) == false)
        {
            listBoxSelectedStops.Items.Add(item);
        }
        else
        {
            MessageBox.Show("These Stops are already selected!");
        }
    }
}

if use above code it will show error when binding the list box we cant add the item to the list box.

[edit]OP response identified code fragment; removed rest of code dump for legibility - OriginalGriff[/edit]

Posted 9-Mar-11 19:36pm

sairam.bhat

Updated 9-Mar-11 20:06pm

OriginalGriff

v2

Add a Solution

Comments

m@dhu 10-Mar-11 1:45am

No one will be interested in seeing your entire code please be specific with the code you are having the problem.

sairam.bhat 10-Mar-11 1:54am

k sir private void buttonSelectAllAvailableStops_Click(object sender, EventArgs e)
{

// this.stopBindingSource.DataSource = null;
// this.routeBindingSource1.DataSource = null;

foreach (string item in listBoxAvailableStops.Items)
{
if (listBoxSelectedStops.Items.Contains(item) == false)
{
listBoxSelectedStops.Items.Add(item);
}
else
{
MessageBox.Show("These Stops are already selected!");
}
}
}

sairam.bhat 10-Mar-11 1:56am

if use above code it will show error when binding the list box we cant add the item to the list box.

Sandeep Mewara 10-Mar-11 1:47am

1. Why full code? You should only paste relevant code. No one will look at such long codebase.
2. Why code? Whats the issue/error?

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)

Ryan Zahra · Accepted Answer · 2011-03-09T20:59:00

Solution 2

C#

bool found = false;
foreach (DataRowView drv1 in listBox1.Items)
{
    foreach (DataRowView drv2 in listBox2.Items)
    {
        if (drv1.Row.ItemArray[0].ToString().Equals(drv2.Row.ItemArray[0].ToString()))
        {
        found = true;
        break;
        }
    }
    if (!found)
    {
        DataTable dt = (DataTable)listBox2.DataSource;
        dt.Rows.Add(drv1.Row.ItemArray);
        listBox2.DataSource = dt;
    }
    found = false;
}

Posted 9-Mar-11 20:59pm

Ryan Zahra

Comments

sairam.bhat 10-Mar-11 3:16am

sir i got error in this line - DataTable dt = (DataTable)listBoxSelectedStops.DataSource;
Error- Unable to cast object of type 'System.Windows.Forms.BindingSource' to type 'System.Data.DataTable'.

Ryan Zahra 10-Mar-11 3:22am

What is the data source that you're using to bind with the list box?

sairam.bhat 10-Mar-11 3:26am

MS access

Ryan Zahra 10-Mar-11 3:27am

Ok, but how are you doing the binding?

sairam.bhat 10-Mar-11 3:33am

i will set the data source property of the listBox as stopBindingsource and display member as field name.

sairam.bhat 10-Mar-11 3:46am

help me sir

Ryan Zahra 10-Mar-11 3:48am

You must help me to understand your code in order for me to help you. How are you binding the data source? What is the data type of your data source?

sairam.bhat 10-Mar-11 3:56am

sir
step 1:I created one binding source-stopBindingsource ;
step 2:set the data source property of the stopBindingsource as admeDataset(Dataset) and Data member as Stop(DataTable).
Step 3:set the data source property of the listBox as stopBindingsource and display member as field name.

Ryan Zahra 10-Mar-11 4:01am

If I understood you correctly, do following modifications:

if (!found)
{
DataSet ds = (DataSet)listBox2.DataSource;
ds.Tables[0].Rows.Add((drv1.Row.ItemArray);
listBox2.DataSource = ds;
}

sairam.bhat 10-Mar-11 4:03am

Thank you sir

sairam.bhat 10-Mar-11 4:06am

Unable to cast object of type 'System.Windows.Forms.BindingSource' to type 'System.Data.DataSet'.

sairam.bhat 10-Mar-11 4:09am

i got this error again Unable to cast object of type 'System.Windows.Forms.BindingSource' to type 'System.Data.DataSet'.

Ryan Zahra 10-Mar-11 4:11am

Please show me the code that you're using where this error is showing

sairam.bhat 10-Mar-11 4:13am

private void buttonSelectAllAvailableStops_Click(object sender, EventArgs e)
{

bool found = false;
foreach (DataRowView drv1 in listBoxAvailableStops.Items)
{
foreach (DataRowView drv2 in listBoxSelectedStops.Items)
{
if (drv1.Row.ItemArray[0].ToString().Equals(drv2.Row.ItemArray[0].ToString()))
{
found = true;
break;
}
}
if (!found)
{
DataSet ds = (DataSet)listBoxSelectedStops.DataSource;//i am getting error here
ds.Tables[0].Rows.Add((drv1.Row.ItemArray));
listBoxSelectedStops.DataSource = ds;
}
found = false;
}

sairam.bhat 10-Mar-11 4:13am

i am getting error here
DataSet ds = (DataSet)listBoxSelectedStops.DataSource;

Ryan Zahra 10-Mar-11 4:20am

Replace that line of code with this one:
DataSet ds = ((System.Data.DataSet)(((System.Windows.Forms.BindingSource)listBox2.DataSource)).DataSource));

sairam.bhat 10-Mar-11 4:25am

k sir that is working nw i got error in next line
ds.Tables[0].Rows.Add((drv1.Row.ItemArray));
error
Input string was not in a correct format.Couldn't store <sadsad> in nFontID Column. Expected type is Int16.

Ryan Zahra 10-Mar-11 4:29am

Is the binding for id and value of both listboxes the same?

sairam.bhat 10-Mar-11 4:40am

no sir different one list box i have binded with stop binding Source another list box with route binding source.

Ryan Zahra 10-Mar-11 4:43am

What are the columns and data types of the stopBindingSource and RouteBindingSource?

sairam.bhat 10-Mar-11 4:53am

Stop binding source
nStopID -Number
cStopID -Number
cName-Text
cLang1-text
cLang2-number
nLatitude-number
cStopAudiofile-text

RouteBindingSource
nID-Number
nSelectedStopsID-number(Foreign key)
cSelectedStopsID-text

sairam.bhat 10-Mar-11 4:56am

I binded cName-Text field of the Stop binding source for listbox1 and nSelectedStopsID for listbox2

Ryan Zahra 10-Mar-11 5:00am

Use the below code with caution! Make sure that the columns are the same when filling in the newRow object!

if (!found)
{
DataSet ds = ((System.Data.DataSet)(((System.Windows.Forms.BindingSource)listBox2.DataSource)).DataSource));
object[] oldRow = drv1.Row.ItemArray;
object[] newRow = new object[3];
newRow[0] = oldRow[0];
newRow[1] = oldRow[0];
newRow[2] = oldRow[1];
dt.Rows.Add(newRow);
listBox2.DataSource = ds;
}

sairam.bhat 10-Mar-11 5:00am

I will never forget your help sir

sairam.bhat 10-Mar-11 5:11am

k thanks sir but it will show error in other tables.why?

Ryan Zahra 10-Mar-11 5:15am

What do you mean by "show error in other tables"?

sairam.bhat 10-Mar-11 5:18am

it will show error nConfigID should not be null but that field is in Config DataTable.

Ryan Zahra 10-Mar-11 5:20am

I'm sorry but I can't help you more than I've already did. I mean I don't have all your tables and all your source code in order to keep helping you. I'm sorry...

sairam.bhat 10-Mar-11 5:21am

dt.Tables[0].Rows.Add(newRow); error in this line

Error message:Column 'nConfigID' does not allow nulls. (Null value exception)that is different table.

Ryan Zahra 10-Mar-11 5:25am

It means that there is a null value in oldRow which is being assigned to the newRow. This means that in your database you have a null value.

sairam.bhat 10-Mar-11 5:28am

k sir thanks a lot. how to avoid that error?

Ryan Zahra 10-Mar-11 5:35am

Make sure that the database cannot accept null values. Also, before adding the row to the dataset, make sure the newRow does not contain null values.

sairam.bhat 10-Mar-11 5:37am

k sir thanks

m@dhu · Accepted Answer · 2011-03-09T20:42:00

You are trying to make modification to the collection being iterated within the ForEach block. This gives you error.
You can use a while loop or for loop to iterate through the list items.
Check the sample code
Add the listitems out of the foreach block.

C#

foreach ( string item in listBox1.Items)
            {
                if (listBox2.Items.Contains(item) == false)
                { str += item+"|"; }
                else
                { MessageBox.Show("These Stops are already selected!"); }
            }
            foreach (string s in str.Split('|'))
            {
                listBox1.Items.Add(s);
            }

How to exchange the items from one list box to other when both list box are binded to the data source?

2 solutions

Solution 2

Solution 1

Add your solution here

Preview 0

Existing Members

...or Join us