thats not really efficient, you has to create new string r every your loop iteration. It will be much better to use strings.Builder instead

This is the smartest solution I've seen.

There's a difference between an empty string "" and null and it looks like you're returning null instead of "".