Apart from the fact that that code defines a global variable when there's no need (or recommendation) for it.