case1 and case2 are non self describing. You should use a different name. In addition these values seem to be constants. If so, they should be declared as "private static final" and an all-uppercase constant name.