nitpick but could you make these variable names upper case as they are constants ?