[Logic Ali]

This code makes little sense. Ball is a constructor, so why call it without new in the onload handler?

FPS is declared in the onload handler but never referenced.

1000/FPS);

Nothing appears to alter FPS, so why use it solely as a divisor for a constant?