add item to shopping cart problem

[Josef Blösl]

hi folks!

i have a cart link in all of my listed products on the site. when i
press this link i will generate a url like this

http://www.mysite.com/index.htm?loca...rt&productid=3

after i press this link the site cart.php will be included and add the
wanted product to the cart, and show the cart.

my problem now is after the cart is showed with the new product in it,
when i press the F5 key in the browser to refresh the site, the same
product will added to the cart once more, because the previous generated
url is unchanged in the browser.

have anyone a idea how to fix this problem?

thanx...

[Colin McKinnon]

Josef Blösl wrote:

hi folks!

i have a cart link in all of my listed products on the site. when i
press this link i will generate a url like this

http://www.mysite.com/index.htm?loca...rt&productid=3
after i press this link the site cart.php will be included and add the
wanted product to the cart, and show the cart.

my problem now is after the cart is showed with the new product in it,
when i press the F5 key in the browser to refresh the site, the same
product will added to the cart once more, because the previous generated
url is unchanged in the browser.

have anyone a idea how to fix this problem?

thanx...

Yes - attach a random cookie (not a web cookie) to each request and ignore
requests where the cookie is already in the basket.

yawn.

C.

[Josef Blösl]

Colin McKinnon schrieb:

Josef Blösl wrote:

hi folks!

i have a cart link in all of my listed products on the site. when i
press this link i will generate a url like this

http://www.mysite.com/index.htm?loca...rt&productid=3

after i press this link the site cart.php will be included and add the
wanted product to the cart, and show the cart.

my problem now is after the cart is showed with the new product in it,
when i press the F5 key in the browser to refresh the site, the same
product will added to the cart once more, because the previous generated
url is unchanged in the browser.

have anyone a idea how to fix this problem?

thanx...

Yes - attach a random cookie (not a web cookie) to each request and ignore
requests where the cookie is already in the basket.

yawn.

C.

thanks a lot... good idea

[dracolytch]

Colin,
If you're so bored, then why not give an answer that might help him
improve his code, instead of just dodge a bug.

Josef,
Ultimately, your underlying problem is that you have an "action"
variable. You need to be separating out your action code from your
display code.

Think of each page in PHP as a method in an object. Instead of having
one page, and choosing the behavior based on a switch statement, take
the time to separate out actions to dedicated pages.

In this case, create a cart-additem.php page and a cart-display.php
page. Have cart-additem.php do variable/access validation, and then add
the item to the cart. Then, relocate to the display page.

You will find that this code will be easier to maintain, and will have
few/no state-related bugs.

~D

[dracolytch]

Colin,
If you're so bored, then why not give an answer that might help him
improve his code, instead of just dodge a bug.

Josef,
Ultimately, your underlying problem is that you have an "action"
variable. You need to be separating out your action code from your
display code.

Think of each page in PHP as a method in an object. Instead of having
one page, and choosing the behavior based on a switch statement, take
the time to separate out actions to dedicated pages.

In this case, create a cart-additem.php page and a cart-display.php
page. Have cart-additem.php do variable/access validation, and then add
the item to the cart. Then, relocate to the display page.

You will find that this code will be easier to maintain, and will have
few/no state-related bugs.

~D

[dracolytch]

You need to separate your action code from your display code. You
should never have an action variable or mode variable. Ever.

Each page in PHP is like a function or method in other languages. Each
one should have a unique, specific purpose.

Create a link that goes to the cart-additem.php page.
Have cart-additem.php validate your variables, add the item to the
cart, and then relocate to a cart-display.php page.

Writing your own shopping cart can be a real dangerous thing to do.
Crackers often see custom carts as ripe targets for attack. Harden,
harden, harden your code.

~D

[Josef Blösl]

dracolytch schrieb:

You need to separate your action code from your display code. You
should never have an action variable or mode variable. Ever.

Each page in PHP is like a function or method in other languages. Each
one should have a unique, specific purpose.

Create a link that goes to the cart-additem.php page.
Have cart-additem.php validate your variables, add the item to the
cart, and then relocate to a cart-display.php page.

Writing your own shopping cart can be a real dangerous thing to do.
Crackers often see custom carts as ripe targets for attack. Harden,
harden, harden your code.

~D

first thank you for your post!

i understand what you mean, but how can i say (inside my addtocart.php
script) to the browser he should automatically switch to the next page
(showcart.php)

[dracolytch]

Sorry about the multiple posts... My 'net connection at work keeps
timing out, and leads me to believe responses didn't post. Ugh.

~D

[dracolytch]

Oh, here's the redirect code you're looking for:

header('locatio n: showcart.php');
die();

The die(); at the end ensures termination of the current page. This is
especially useful if you're doing validation checking, and found a
problem. This will redirect you to a new page, and prevent anything
else in the current page from being executed.

~D

[JDS]

On Tue, 19 Jul 2005 14:28:35 +0200, Josef Blösl wrote:

have anyone a idea how to fix this problem?

There are several ways.

One is to include some sort of tracking variable and only do the update if
the tracking variable is flagged correctly.

Another, and the way I would do it, is to use a different PHP script
entirely to do the actual form processing, and send the browser to the
correct "results" page using header("Locatio n: blah.php");

Thus you would have an invisible processing-only PHP script that redirects
the user on completion to the proper results page.

I hope this makes sense.

--
JDS | je*****@example .invalid
| http://www.newtnotes.com
DJMBS | http://newtnotes.com/doctor-jeff-master-brainsurgeon/