you should use the constant names rather than the values to have a readable message