Removed srand from generateNumber(). Since moving srand to the constructor, srand should only be declared in one place, and the second declaration was affecting results by returning the same numbers after clicking Generate several times.