Re: Could you look over my completed (but not working) short .cgi script?

[GrandFather]

Using strictures (use strict; use warnings;) is excellent and something you should always do. Batch declaring variables negates the most valuable diagnostic that strict provides. In fact what you wreck is exactly what your comment describes as the reason for using strict! Consider this fragment of your code:

#prevent Perl from creating undeclared variables
use strict;

#declare variables and assign variables

my ($item, $C_basket, @basket);

#make cookies
$C_basket = cookie(
    -name  => "Basket",
    -value => "@basket",
    -path  => "../../cgi-bin/chap11"
);

Do you see anything wrong there? How about if we rewrite it this way and run it:

use strict;

#make cookies
my $C_basket = cookie(
    -name  => "Basket",
    -value => "@basket",
    -path  => "../../cgi-bin/chap11"
);

Now you get an error: Global symbol "@basket" requires explicit package name which is strict's way of saying "I haven't seen the variable @basket before". In other words, @basket is undefined.

So, what do you expect the cookie value to be? What it will be is an empty string, but if you intended that you should write that. If you didn't intend that, batch declaring variables has masked exactly the sort of bug strict should alert you to.