On Jun 4, Luinrandir Hernson said:

>$homepage = "/home/yourdomain/counter/";
>
>$counterfile = "$homepage counter.txt"; #Full file path of counter.txt
>
>$imagefile{'0'}="$homepage 0.gif"; #Full file path of 0.gif
>$imagefile{'1'}="$homepage 1.gif"; #Full file path of 1.gif
>$imagefile{'2'}="$homepage 2.gif"; #Full file path of 2.gif
>$imagefile{'3'}="$homepage 3.gif"; #Full file path of 3.gif
>$imagefile{'4'}="$homepage 4.gif"; #Full file path of 4.gif
>$imagefile{'5'}="$homepage 5.gif"; #Full file path of 5.gif
>$imagefile{'6'}="$homepage 6.gif"; #Full file path of 6.gif
>$imagefile{'7'}="$homepage 7.gif"; #Full file path of 7.gif
>$imagefile{'8'}="$homepage 8.gif"; #Full file path of 8.gif
>$imagefile{'9'}="$homepage 9.gif"; #Full file path of 9.gif

Why not use an array?  A hash here seems silly.

You have a space in $counterfile -- this is going to cause problems for
you later.  Why not remove the trailing slash from $homepage?

  $homepage = "/home/yourdomain/counter";
  @images = map "$homepage/$_.gif", 0 .. 9;

-- 
Jeff "japhy" Pinyan      [EMAIL PROTECTED]      http://www.pobox.com/~japhy/
Eruséro Marillion -- wielder of Ringril, known as Hesinaur, the Winter-Sun
Are you a Monk?  http://www.perlmonks.com/     http://forums.perlguru.com/
Perl Programmer at RiskMetrics Group, Inc.     http://www.riskmetrics.com/
Acacia Fraternity, Rensselaer Chapter.         Brother #734
**      Manning Publications, Co, is publishing my Perl Regex book      **

Reply via email to