i want to fill textbox using combobox value as where condition

Question

1.00/5 (2 votes)

See more:

what is the wrong ??

C#

try
 {
    SqlConnection con = new SqlConnection(@ "jjj");
    SqlCommand comm = new SqlCommand("select [??????_??????] from client where [???   _??????]= '" + comboBox1.SelectedValue + "' ", con);
   con.Open();
   prevtxt.Text =(string)comm.ExecuteScalar();
   con.Close();
 }
 catch (System.Exception ex)
 {
   MessageBox.Show(ex.Message);
  }

Posted 31-Jan-13 12:14pm

Member 9702410

Updated 31-Jan-13 12:18pm

DinoRondelly

v3

Add a Solution

Comments

DinoRondelly 31-Jan-13 18:18pm

Is this a test? You tell me whats wrong ...

Winston Madiano 1-Feb-13 0:49am

My idea, use sqlparameters for security against SQL injection. (Ex. command.parameters.addWithvalue("@","") )

If the datatype of your (Where [??? _??????]") is string then convert combobox Selected value to string.

Dispose your objects properly.. by using "using", or "dispose" keyword.

Sergey Alexandrovich Kryukov 31-Jan-13 18:21pm

The questions like "what's wrong" cannot be valid if you don't tell us what do you try to achieve. What is wring for one purpose can be right for another...
Of course, if you stuff a query with ???????, you cannot expect anything good...
—SA

Member 9702410 31-Jan-13 18:33pm

NO , i already want to fill textbox using combobox value as where condition an i try this code but there is no text in textbox

DinoRondelly 31-Jan-13 18:36pm

Does the query return anything?

Member 9702410 31-Jan-13 18:57pm

No.

DinoRondelly 31-Jan-13 18:58pm

So the query doesnt return anything and you are wondering why the text box is empty?

Member 9702410 31-Jan-13 19:02pm

yes.

DinoRondelly 31-Jan-13 19:03pm

If the query doesnt return a record then there is nothing to put in the text box and therefor the text box will not have a value, make sense?

Member 9702410 31-Jan-13 19:13pm

you can see the code again

try
{
SqlConnection con = new SqlConnection(@" mm");
SqlCommand comm = new SqlCommand("select name from client where id= '" + comboBox1.SelectedValue + "' ", con);
con.Open();
prevtxt.Text =(string)comm.ExecuteScalar();
con.Close();
}
catch (System.Exception ex)
{
MessageBox.Show(ex.Message);
}

Member 9702410 31-Jan-13 18:34pm

I'm sorry first ???????? represent column name and also second ???????? represent another column name

Member 9702410 31-Jan-13 18:59pm

the code again:

try
{
SqlConnection con = new SqlConnection(@" mm");
SqlCommand comm = new SqlCommand("select name from client where c_name= '" + comboBox1.SelectedValue + "' ", con);
con.Open();
prevtxt.Text =(string)comm.ExecuteScalar();
con.Close();
}
catch (System.Exception ex)
{
MessageBox.Show(ex.Message);
}

Sergey Alexandrovich Kryukov 31-Jan-13 22:50pm

Please, stop writing this sample in comments. Can you see "Improve question" above?
Click it and put everything in one piece, the body of the question, taking account everything, and do it very clearly.
—SA

vinodkumarnie 31-Jan-13 23:54pm

What is the data type of id(column) in your table....?

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)

panchocantu · Answer 1 · 2013-02-01T14:57:00

Solution 2

your textbox need be active multiline and in the moment of asigment use the += and don't use only =

Posted 1-Feb-13 14:57pm

panchocantu