search

GlobeTownDevs / stopwatch

Stopping your watches
0 stars 1 forks source link

Global variables #2

Closed tbtommyb closed 7 years ago

tbtommyb commented 7 years ago

Your .js file adds lots of global variables (lines 1 -7). Maybe you could move everything to be inside the stopwatch object you make on line 7, or use a module pattern?

marisid commented 7 years ago

Good point for big chunks of code and future instances. Refactoring our code to follow your suggestion would take us rather long now without affecting our functionality. But very good point :smile: