Any reason not to check the length here?  Seems that we should make sure it isn't an empty string and is truthy before invoking the index operator.