FishMonger
Veteran
Oct 17, 2012, 11:35 AM
Views: 1488
|
|
Re: [gerble1000] problem using array inside for loop inside if statement
|
|
|
Wow, that's a perfect example of bad spaghetti code.
Your code indentation is inconsistent (i.e., horrible).
Remove the commented out use statements
Remove the second use CGI statement.
Remove ALL goto statements.
Why are you initializing the @skip and @result arrays and then immediately overwrite those initializations?
Use var names that describe what the vars hold. The @skip and @result var names are too generic and don't say anything about what they hold. Personally, I'd get rid of the results array and use separate scalars for each of those elements. Or if you wish, you could use a %results hash.
What does your skip1..skip8 params supposed to do/represent?
Don't put multiple statements on a single line.
Remove the depreciated <center></center> tags and instead use css to style the page.
Remove the empty else{} block.
(This post was edited by FishMonger on Oct 17, 2012, 11:37 AM)
|
MAIN
SEARCH
WHO'S 
LOG 
