A couple of suggestions:

• use strict; and use warnings;
• Use globals only where necessary, to prevent scary action-at-a-distance and simplify debugging.