On Friday, 13 December 2013 at 13:34:07 UTC, Daniel Murphy wrote:
"Rikki Cattermole" <[email protected]> wrote in message
news:[email protected]...
You have a lot of global variables of same type. With very
similar default values.
Example windX and runTmr. They are both doubles.
Perhaps an alternative way to write it is like this:
double
windX = 0,
runTmr = 0;
You also have a lot of enums that like WIDTH and HEIGHT that
could be transformed into a single enum.
e.g.
enum int WIDTH = 800;
enum int HEIGHT = 600;
Would become:
enum : int {
WIDTH = 800,
HEIGHT = 600
}
Adds a couple extra lines but hey when you got 20 odd values
and repeating the type it kinda looks ugly.
I would not consider either of those an improvement.
Depends on style I guess. To me its less to read so = good.
