Populating a textbox based on a combo box selection

Question

0.00/5 (No votes)

See more:

Good Day

I am trying to populate a text box based on a combo box selection,and all I am getting in the text box is "System.Data.DataRowview". below is my code.

-this is the code that populates the combo boxes

C#

private void frmEmployeesJobs_Load(object sender, EventArgs e)
 {
     string empSql = string.Format("Select  (CAST(employee_id AS VARCHAR(MAX))+ ' - '+employee_name) as employee FROM employee");

     DataTable empTable = FillTable(empSql);
     cmbEmp.DataSource = empTable;
     cmbEmp.DisplayMember = "employee";

     string jobSql = string.Format("Select  (CAST(job_id AS VARCHAR(MAX))+ ' - '+job_name) as job, job_price  FROM job");

     DataTable jobTable = FillTable(jobSql);
     cmbJob.DataSource = jobTable;
     cmbJob.DisplayMember = "job";

 }

this is the code that attempts to fill the textbox.

C#

private void cmbJob_SelectedIndexChanged(object sender, EventArgs e)
        {
            if (cmbJob.SelectedIndex == -1)
            {
                txtPrice.Text = string.Empty;
            }
            else
            {
              
                txtPrice.Text = cmbJob.SelectedItem.ToString();
              
            }

basically what i want to do is fill the price textbox when ever a job is selected
        }

Thanks in advance

Posted 12-Feb-13 0:20am

Georges23

Add a Solution

2 solutions

Solution 1

Hi,

just try this.

C#

txtPrice.Text = cmbJob.SelectedText.ToString(); //if selected Text to be displayed

txtPrice.Text = cmbJob.SelectedValue.ToString(); // if selected value to be displayed

hope it helps.

Posted 12-Feb-13 0:23am

Karthik Harve

Updated 12-Feb-13 0:34am

v2

Comments

Georges23 12-Feb-13 6:28am

Karthik Harve there is no such thing in c sharp thanks

Karthik Harve 12-Feb-13 6:35am

Does updated answer works ?

Jibesh 12-Feb-13 6:53am

No it wont work. You need to set the DisplayMember and ValueMember to get the data. See my solution.

Karthik Harve 12-Feb-13 6:58am

yes. i did't saw the code lines, which are binding the combobox. you are correct. thanks.

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)

Jibesh · Accepted Answer · 2013-02-12T00:32:00

Solution 2

If you are already bind a datasource to combo you can use the combo.SelectedValue property instead of SelectedItem.
i.e

C#

txtPrice.Text = cmbJob.SelectedValue.ToString();

also the set the cmbJob.ValueMember = "job"; do the same for other combobox.

Posted 12-Feb-13 0:32am

Jibesh

Updated 12-Feb-13 0:50am

v2

Comments

Georges23 12-Feb-13 6:36am

Jibesh it gives me this value in the textbox when i do what you said. "System.Data.DataRowview"

Georges23 12-Feb-13 6:37am

no nothing works

Jibesh 12-Feb-13 6:38am

moment.. i re-call something.. you need to set ValueMember also.. let me check it.. do read about.

http://msdn.microsoft.com/en-us/library/microsoft.dynamics.mobile.framework.controls.combobox.valuemember(v=mag.11).aspx

Georges23 12-Feb-13 6:41am

how do i do that ?

Jibesh 12-Feb-13 6:51am

just like the way you use DisplayMember, set the ValueMember property. see my updated solution.

The reason why you need to set both DisplayMemeber and ValueMemeber is, it is possible to use different values for display and for internal processing.

Georges23 12-Feb-13 7:10am

still does not work

Jibesh 12-Feb-13 7:23am

does it work now?

Georges23 12-Feb-13 7:11am

here is my updated solution.
"private void frmEmployeesJobs_Load(object sender, EventArgs e)
{
string empSql = string.Format("Select (CAST(employee_id AS VARCHAR(MAX))+ ' - '+employee_name) as employee FROM employee");

DataTable empTable = FillTable(empSql);
cmbEmp.DataSource = empTable;
cmbEmp.DisplayMember = "employee";
cmbEmp.ValueMember = "employee";

string jobSql = string.Format("Select (CAST(job_id AS VARCHAR(MAX))+ ' - '+job_name) as job, job_price FROM job");

DataTable jobTable = FillTable(jobSql);
cmbJob.DataSource = jobTable;
cmbJob.DisplayMember = "job";
cmbJob.ValueMember = "job";

}

private void cmbJob_SelectedIndexChanged(object sender, EventArgs e)
{
if (cmbJob.SelectedIndex == -1)
{
txtPrice.Text = string.Empty;
}
else
{
//string[] parts = cmbJob.Items[cmbJob.SelectedIndex].ToString().Split(' ');
txtPrice.Text = cmbJob.SelectedValue.ToString();
//txtPrice.Text = parts[0];
}
}"

Jibesh 12-Feb-13 7:18am

hummm.. I guess I read your mind and you tried put a break point in the selecedIndexchanged method and it displays datagridviewrow right.

here is another tricky way - just reorder the statments use it like this

cmbJob.DisplayMember = "job";
cmbJob.ValueMember = "job";
cmbJob.DataSource = jobTable;

set the datasource, after setting the Value and Display member.! gime me my $

Georges23 12-Feb-13 8:12am

it worked thank you very much jibesh

Jibesh 12-Feb-13 13:38pm

you welcome.
if the above solution solved your problem, you can mark the solution as accepted.

Georges23 12-Feb-13 8:12am

it worked thank you very much jibesh