indexOf() + 1 vs indexOf() != -1
Well, I believe the second one is more obvious what's going on...

if( that.indexOf('hello') != -1 ) {
}


That's all to it however, both expressions are valid and perfectly ok.