I actually originally had the code you posted, and asked a friend about it it was good practice to put a possibly nil value into the string like that. He told me he would use a ternary which makes it more declarative, but clearer that it may be nil. I changed it, but I'm not attached to one or the other.