Re: building combo box

The bit that's failing is the "for" inside the array ref — that just can't work. Besides, it looks like you might be trying to:

#!/usr/bin/perl

use warnings;
use strict;
use CGI;

my $cgi = CGI->new();

print $cgi->popup_menu(-name=>"Days", -values=>[ 1 .. 31]);
[download]

Although this isn't the correct way of doing it — you should use one of the DateTime modules to get the number of days in the month (How many days are there in February 2000? February 2004? February 2100?)

davis

Kids, you tried your hardest, and you failed miserably. The lesson is: Never try.

Comment on Re: building combo box Download Code

Replies are listed 'Best First'.
Re^2: building combo box by Fletch (Chancellor) on Sep 17, 2006 at 12:11 UTC
A for inside the array ref won't work, but for future reference a map would work fine (but yes it's unneeded in this case since it's easily made with ..-as-list-constructor). `my $array = [ map { transmogrify_foo( $_ ) } 1..31 ];` [download]	[reply] [d/l]
Re^2: building combo box by jdtoronto (Prior) on Sep 17, 2006 at 13:13 UTC
davis has given you an appropriate answer, I would like to add that you should refer to the LATEST documenation for a module. The `Select` method has been deprecated for a long time, as far back as 1998 and at a quick check it doesn't appear in the current CGI documentation. I don't know where you found the information you have based your code on, but it is more than a little out of date. You might like to refer to Ovids CGI course, it will guide you along a well founded path. jdtoronto	[reply] [d/l]
Re^3: building combo box by jhourcle (Prior) on Sep 17, 2006 at 14:10 UTC
The Select method has been deprecated for a long time, as far back as 1998 and at a quick check it doesn't appear in the current CGI documentation It may be deprecated, but it's there. Look in Non-standard HTML Shortcuts.	[reply]
Re^2: building combo box by beginr (Novice) on Sep 17, 2006 at 10:12 UTC
Thank you your code is working fine for me	[reply]