>Ok, don't worry, at least I'll run some test before commit, and if in >trouble, i'll ask.
No. Do not commit the code if it works that way. At all. >> I'm not suggesting that you eliminate the method - just asking you to change >> its return type to Query.Method: >> >> public abstract Method getMethod(); >> >Ok, more generic. It's not more generic, it's more specific. A String return value could be anything. An enum is more restrictive and gives the caller a better idea of what to expect.
