Using new String(...) is always a bad idea. It's basically like saying, "Please, make me a new String, even if you don't have to. I really want to use
more resources". Instead of using new String(...) you should use String.valueOf(...).

Once in a while, I'll see people literally do something like this: s = new String("Foo"); I suspect this may be the result of some refactoring oversight or
a novice at the keyboard (at least I hope so). This is pointless because you already have the string, you don't need to create a new one. This should
be changed to s = "Foo";