I just failed a code review without any reason given due to my writing, for a function where `s` (of type String) is passed as a parameter

```
if (s == null){
    s = new String();
}

```

Said I should use `s = "";` instead. Why?