String to String is unnecessary
Loose coupling
unnecessary local before return
final 
Unused fields and imports
if, else, do, while are clearer when have the braces
Redundant 'public' modifier in interface