Display the data of dropdownlist selected item in gridview

Question

0.00/5 (No votes)

See more:

Hi,

I am trying this code to display the data of dropdownlist selected item in gridview it's working but only "if" condition is working and "else" condition is not. Any help would be appreciated.

protected void BtnSubmit_Click(object sender, EventArgs e)
    {
        SqlConnection con = new SqlConnection(ConfigurationManager.ConnectionStrings["MyConsString"].ConnectionString);

        //string sql = "Select * from ManstaSalary";
        //SqlCommand cmd = new SqlCommand(sql, con);


   if (DDlDepartment.SelectedValue != null)
    {
      con.Open();
      string str = DDlDepartment.SelectedValue;
            //string sql = "Select * from ManstaSalary where Department= '" + str+"'" ;

      SqlDataAdapter ad = new SqlDataAdapter("Select * from ManstaSalary where Department= '" + str + "'", con);
      DataSet ds = new DataSet();
      ad.Fill(ds);
      GridView1.DataSource = ds;
      GridView1.DataBind();
      con.Close();
        }

      else if (DDlDepartment.SelectedValue != null && DDLStaffid.SelectedValue != null)
        {
      con.Open();
      string str1 = DDlDepartment.SelectedValue;
      string str2 = DDLStaffid.SelectedValue;
            //string sql1 = "Select * from ManstaSalary where Department='" + str1 + "'and StaffID='" + str2+ "'";


      SqlDataAdapter ad = new SqlDataAdapter("Select * from ManstaSalary where  Department= '" + str1 + "' and StaffID = '" + str2 + "'", con);
      DataSet ds = new DataSet();
      ad.Fill(ds);

      GridView1.DataSource = ds;
      GridView1.DataBind();
      con.Close();
        }
    }

Posted 19-Jul-10 23:08pm

anuradha_d

Updated 19-Jul-10 23:36pm

LittleYellowBird

v4

Add a Solution

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)

Sandeep Mewara · Answer 1 · 2010-07-19T23:37:00

anuradha_d wrote:
if (DDlDepartment.SelectedValue != null)

anuradha_d wrote:
else if (DDlDepartment.SelectedValue != null && DDLStaffid.SelectedValue != null)

If you notice the difference between two If statements are of DDLStaffid value. If you read about If[^] statement execution then the first condition that is satisfied is executed.

Based on the current implementation, your code would either go into first if or just end of if. Else-If is never going to be executed.
If you really want that condition, then just reverse it. Use the two conditioned if first and then one conditioned.

Hope you got it.

UPDATE:
You definitely need to read on If statements if that was your requirement from start.

Ok. Now, if you want both of them to execute then it can be done in multiple ways. For example
OPTION 1: just separate them!

C#

If(DDlDepartment.SelectedValue != null)
{
  // Do something 1
}

//followed by

if (DDlDepartment.SelectedValue != null && DDLStaffid.SelectedValue != null)
{
  // Do something 2
}

OPTION 2: Cascade the if's

C#

if (DDlDepartment.SelectedValue != null)
{
  // Do soemthing 1

  if (DDLStaffid.SelectedValue != null)
  {
    // Do something 2
  }
}

There can be other ways to handle using, other statements. For now, this should do for you.