asked on

Button is one click behind on updating session variable.

The problem I am having is that I need to track a number on an asp.net website. When a forward and backward button is clicked, the number should increase or decrease by one. The result I am getting is that when the button is clicked the first time nothing happens. Then the second time it is clicked the textbox displaying the current value of the session variable is finally updated. Another thing that happens is if I click the add button twice then click the minus button....the number will continue to increment. Its like the displayed value is one click behind.

What do I need to do to update the text box immediately after the add/minus button is clicked and not be a click behind?

I am using VS 2008.

Thanks for the help.

Partial Class _Default
    Inherits System.Web.UI.Page
 
    
 
    Protected Sub Page_Load(ByVal sender As Object, ByVal e As System.EventArgs) Handles Me.Load
 
        If Not Page.IsPostBack Then
            Session("counter") = 1
        Else
 
        End If
        TextBox1.Text = Session("counter")
 
    End Sub
 
    Protected Sub btnSubtract_Click(ByVal sender As Object, ByVal e As System.EventArgs) Handles btnSubtract.Click
        Session("counter") -= 1
    End Sub
 
    
    Protected Sub btnAdd_Click(ByVal sender As Object, ByVal e As System.EventArgs) Handles btnAdd.Click
        Session("counter") += 1
    End Sub
End Class

Open in new window

Qaiser_Mehmood_Mughal

I think Firstly you have to convert the value to int to do any subtraction/addition then also
TextBox1.Text = Session("counter")
you above line of code in button's events.

victor79

try

Protected Sub Page_Load(ByVal sender As Object, ByVal e As System.EventArgs) Handles Me.Load
If Not Page.IsPostBack Then
Session("counter") = 1
TextBox1.Text = Session("counter")
End If
End Sub

Protected Sub btnSubtract_Click(ByVal sender As Object, ByVal e As System.EventArgs) Handles btnSubtract.Click
TextBox1.Text -=1
End Sub

Protected Sub btnAdd_Click(ByVal sender As Object, ByVal e As System.EventArgs) Handles btnAdd.Click
TextBox1.Text +=1
End Sub

RocketSauce

ASKER

Thanks for the comments so far. Thanks yes it is the way to go to convert strings to integers/etc when doing calculations.
@Victor79 - your solution does sync the value of the text box with the buttons. But... Is there not a way to keep track of this value without storing the value on a control? My plans will be to remove the text box completely once I figure this out...this won't be a number displayed on the page or to the end user.

Using the same logic i have posted...I have also tried cache, and query strings and the same result happens where the value displayed in the textbox is one click behind.

ASKER CERTIFIED SOLUTION

prairiedog

membership

Create a free account to see this answer

Signing up is free and takes 30 seconds. No credit card required.

See answer

RocketSauce

ASKER

Thanks prairiedog. You explained what I was doing wrong very nicely. It looks like you gave me a push so I could reason the rest out on my own. Take a look at the code below and see if that is the right way to do it. If thats what you intended the final code to look like..I'll give you all your points then. And thank you so much.

Also...I was in a rush trying it out....code below doesn't have the buttons and stuff named properly and no ctype() string conversion going on...but you can see the logic and it compiles and works.

Partial Class _Default
    Inherits System.Web.UI.Page
 
    Protected Sub Button1_Click(ByVal sender As Object, ByVal e As System.EventArgs) Handles Button1.Click
        Session("counter") -= 1
        TextBox1.Text = Session("counter")
    End Sub
 
    Protected Sub Page_Load(ByVal sender As Object, ByVal e As System.EventArgs) Handles Me.Load
        If Not Page.IsPostBack Then
            Session("counter") = 1
            TextBox1.Text = Session("counter")
        End If
        
 
    End Sub
 
    Protected Sub Button2_Click(ByVal sender As Object, ByVal e As System.EventArgs) Handles Button2.Click
        Session("counter") += 1
        TextBox1.Text = Session("counter")
    End Sub
End Class

Open in new window

prairiedog

It looks fine to me. As long as you keep it in mind the order a page is processed, you will know when and where to access the session variable, for example, Page_Load is always fired, no matther it is a page load or post back, then the event handler that causes the post back fires afterwards.

RocketSauce

ASKER

Thanks guys. I appreciate all of you posting your help.