Looks good to me. Though we potentially call `prop.ToString()` twice in this method. It's not a free call so might be worth caching that value in a temporary variable.