Re: session_name("CCLTrolley")

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 2005-10-01 at 23:57, John Taylor-Johnston wrote:
> $TrolleyContents is a string.
> Basically what I want to accomplish here is if $TrolleyContents already 
> exists append $AddToTrolley to $TrolleyContents, if not register 
> $TrolleyContents.
> Am I going about it right?
> John
> 
> <?php
> #printcontents.php
> 
> session_name("CCLTrolley");
> session_start();
> 
> if (isset($HTTP_POST_VARS["AddToTrolley"]))
> {
> $TrolleyContents = $TrolleyContents.",".$HTTP_POST_VARS["AddToTrolley"];
> }else{
> session_register("TrolleyContents");
> }
> echo $TrolleyContents;
> 
> phpinfo();
> ?>

Looks a bit odd to me :) But could be because you're using outdated
semantics. It should be sufficient to do the following:

<?php

session_name( 'CCLTrolley' );
session_start();

//
// Initialize the trolley.
//
if( !isset( $_SESSION['TrolleyContents'] ) )
{
    $_SESSION['TrolleyContents'] = '';
}

//
// Add new entry.
//
if( isset( $_POST['AddToTrolley'] ) )
{
    if( $_SESSION['TrolleyContents'] ) == '' )
    {
        $_SESSION['TrolleyContents'] = $_POST['AddToTrolley'];
    }
    else
    {
        $_SESSION['TrolleyContents'] .= ','.$_POST['AddToTrolley'];
    }
}

echo $_SESSION['TrolleyContents'];

phpinfo();

?>

Is there a reason you're using a comma delimited string? I would
recommend using an array instead:

<?php

session_name( 'CCLTrolley' );
session_start();

//
// Initialize the trolley.
//
if( !isset( $_SESSION['TrolleyContents'] ) )
{
    $_SESSION['TrolleyContents'] = array();
}

//
// Add new entry.
//
if( isset( $_POST['AddToTrolley'] ) )
{
    $_SESSION['TrolleyContents'][$_POST['AddToTrolley']] =
$_POST['AddToTrolley']
}

echo implode( ',', $_SESSION['TrolleyContents'] );

phpinfo();

?>

Cheers,
Rob.
-- 
.------------------------------------------------------------.
| InterJinn Application Framework - http://www.interjinn.com |
:------------------------------------------------------------:
| An application and templating framework for PHP. Boasting  |
| a powerful, scalable system for accessing system services  |
| such as forms, properties, sessions, and caches. InterJinn |
| also provides an extremely flexible architecture for       |
| creating re-usable components quickly and easily.          |
`------------------------------------------------------------'

-- 
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php


[Index of Archives]     [PHP Home]     [Apache Users]     [PHP on Windows]     [Kernel Newbies]     [PHP Install]     [PHP Classes]     [Pear]     [Postgresql]     [Postgresql PHP]     [PHP on Windows]     [PHP Database Programming]     [PHP SOAP]

  Powered by Linux