you have at least one error on line 235: the $i variable hasn't been declared.

Otherwise, I would suggest that you remove most of these silly vertical spaces, this makes the code harder to read. In general, I want to maximize the quantity of code lines that I can see on a screen page, because I need less navigating through the screen pages and it is easier to see what's going on. I usually put a blank line between two subroutine definitions or, within one sub, between blocks or pieces of code doing different things.

I would add that that you also need to be much more consistent with your code indentation: that also helps a lot understanding what the code is doing.