Again ugly, you're basically calling userInfo.indexOf(":") three times. It could be called once and the result reused.