Please check the code session storing only last value

Question

1.00/5 (1 vote)

See more:

C#

for (int i = 0; i < GridView1.Rows.Count; i++)
            {

                Label ID = (Label)GridView1.Rows[i].Cells[0].FindControl("ID");
                int lID = Convert.ToInt32(ID.Text);
                con.Open();
                using (SqlCommand command = new SqlCommand("select Vendor_Name from VandorDetail where ID=@ID", con))
                { // 2. define parameters used in command object 
                    SqlParameter param = new SqlParameter();
                    param.ParameterName = "@ID"; param.Value = lID; // 3. add new parameter to command object 
                    command.Parameters.Add(param); // // Read in the SELECT results. // 
                    SqlDataReader reader = command.ExecuteReader();
                    while (reader.Read())
                    { Session["Vendor_Name"] = Convert.ToString(reader["Vendor_Name"]); }
                    con.Close();
                }
            }

Posted 2-Apr-14 0:57am

Member 10578683

Updated 2-Apr-14 0:58am

King Fisher

v2

Add a Solution

Comments

ArunRajendra 2-Apr-14 7:00am

What's the issue?

Member 10578683 2-Apr-14 7:01am

Session shows the last value

Member 10578683 2-Apr-14 7:03am

When I am clicking the button of 1st row in my gridview in the seesion last row's vendor_Name is Appearing

King Fisher 2-Apr-14 7:07am

if your query returns more than one value ,the session value will be overwrite by last value

Member 10578683 2-Apr-14 7:11am

Query I have Written to get One value

King Fisher 2-Apr-14 7:30am

take out the query from for loop

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)

OriginalGriff · Answer 1 · 2014-04-02T01:02:00

Solution 1

Well, yes...it would.
What you have done is the equivalent of:

C#

string myString = "";
for (int i = 0; i < 10; i++)
   {
   myString = i.ToString();
   }

And you are surprised that myString always equals "9"...
When you assign to the Session, it is the same as assigning to any other collections: you overwrite the previous content.

Posted 2-Apr-14 1:02am

OriginalGriff

Comments

Member 10578683 2-Apr-14 7:07am

But whast will i write Yaar to get exact value

OriginalGriff 2-Apr-14 7:12am

How would I know? :laugh:
I don't know what you are trying to do - or what the rest of your code expects to find in the session variable!
If I tell you "use a list" and add each item to it, then set the Session to the whole list then I could break other code which depends on that variable being a single string!
Sorry, but you will have to either give more details, or work it out for yourself...