session problem

ganesh15 · August 13th, 2003, 02:26 AM

Is there any problem in the code below?

Always printing Counter initialized, please reload this page to see it increment, no matter how many I refresh also.

By right the counter should increase when I refresh the page.

I'm running PHP Version 4.3.1 on linux machine with apache.

Please help.

<?
session_start();
session_register("SESSION");

if (!isset($SESSION)) {
    $SESSION["count"] = 0;
    echo "<li>Counter initialized, please reload this page to see it increment";
} else {
    echo "<li>Waking up session $PHPSESSID";
    $SESSION["count"]++;
}
echo "<li>The counter is now $SESSION[count] ";
?>

Regards,
ganesh

ganesh15 · August 13th, 2003, 03:07 AM

Working when I change codes the way below... but do not know the actual reason or problem.. can anyone explain..

<?
session_start();
session_register('myvar');

if (!isset($HTTP_SESSION_VARS["myvar"])) {
    $HTTP_SESSION_VARS["myvar"] = 0;
    echo "<li>Counter SDS initialized, please reload this page to see it increment";
} else {
    echo "<li>Waking up session PHPSESSID";
    $HTTP_SESSION_VARS["myvar"]++;
}
echo "<li>The counter is now ";

echo $HTTP_SESSION_VARS["myvar"]
?>

Regards,
ganesh

richard.york · August 13th, 2003, 03:52 AM

Hi Ganesh,
You just have a little typo in there...

You should be using $_SESSION instead of $SESSION.

: )
Rich

:::::::::::::::::::::::::::::::::
Smiling Souls
http://www.smilingsouls.net
:::::::::::::::::::::::::::::::::

ganesh15 · August 13th, 2003, 05:04 AM

Thanks. That solved my problem.

nikolai · August 18th, 2003, 01:18 PM

Careful! Look at the original code -- he passed "SESSION" to session_register. You definitely do _NOT_ want to session_register $_SESSION!!

$_SESSION replaces $HTTP_SESSION_VARS and should NOT be used when session_register() is used.

The simplest correct rewriting of your script is as follows:

Code:

<?php
session_start();

if (!isset($_SESSION['count']))
{
    $_SESSION["count"] = 0;
    echo "<li>Counter initialized, "
       . "please reload this page to see it increment.";
}
else
{
    echo "<li>Waking up session " . session_id();
    $_SESSION['count']++;
}

echo "<li>The counter is now $SESSION[count]";
?>

Take care,

Nik
http://www.bigaction.org/