Re: Code not showing properly

i cannot get any meaning out of the shuffle over the tied array. in the while, you put everything in three arrays, so the shuffle is pointless.

HOOPS, overlooked a bogus statement as well: ($File[$i++], $Name[$i++], $Keywords[$i++]) = split(/:/); first, the closing parenthesis is missing. second, you increase $i by every loop by 3! instead, move the incrementing out of it:

($File[$i], $Name[$i], $Keywords[$i]) = split(/:/);
$i ++;
[download]

you hardcoded that the arrays will have 24 elemenets. i do not think that is what you wanted. plus, arrays begin counting by zero. consider:

for ($z=0; $z<scalar (@File); $z++) { ... }
[download]

you can easily separate html markup from perl code with HTML::Template.

HTH

Comment on Re: Code not showing properly Select or Download Code

Replies are listed 'Best First'.
Re: Re: Code not showing properly by JimJx (Beadle) on Jun 13, 2003 at 06:13 UTC
Hi Kabel, Thanks for the reply! i cannot get any meaning out of the shuffle over the tied array. in the while, you put everything in three arrays, so the shuffle is pointless. Actually, not pointless. I shuffled the whole file before reading the information into the arrays, that is all. I do not want each field shuffled individually.... Unless I completely misunderstood you which is possible. :-) first, the closing parenthesis is missing. second, you increase $i by every loop by 3! instead, move the incrementing out of it I caught both of those right after I made the post..... Don't know what I was thinking at the time, but both are fixed. you hardcoded that the arrays will have 24 elemenets. i do not think that is what you wanted. plus, arrays begin counting by zero In the first iteration of this, yes the arrays will have 24 elements.... After this page executes, they get cut down to between 1 and 9, depending on choices made. As far as the arrays starting with 0, for some reason I have always used 1 for the beginning. But that doesn't change anything except the number assigned. It doesn't change the fact that I still wind up with empty IDs in the code when the page renders. Jim	[reply]

Replies are listed 'Best First'.

Re: Re: Code not showing properly
by JimJx (Beadle) on Jun 13, 2003 at 06:13 UTC

Thanks for the reply!
i cannot get any meaning out of the shuffle over the tied array. in the while, you put everything in three arrays, so the shuffle is pointless.
Actually, not pointless. I shuffled the whole file before reading the information into the arrays, that is all. I do not want each field shuffled individually.... Unless I completely misunderstood you which is possible. :-)

first, the closing parenthesis is missing. second, you increase $i by every loop by 3! instead, move the incrementing out of it
I caught both of those right after I made the post..... Don't know what I was thinking at the time, but both are fixed.

you hardcoded that the arrays will have 24 elemenets. i do not think that is what you wanted. plus, arrays begin counting by zero
In the first iteration of this, yes the arrays will have 24 elements.... After this page executes, they get cut down to between 1 and 9, depending on choices made. As far as the arrays starting with 0, for some reason I have always used 1 for the beginning. But that doesn't change anything except the number assigned. It doesn't change the fact that I still wind up with empty IDs in the code when the page renders.

Jim

[reply]