I personally hate these double tests (first not null and then test)
I guess it is a style thing.
I do agre that isempty is far better than equals("")