property.GetValue ignores CultureInfo

Question

It seems that for some reason property.GetValue ignores CultureInfo. Here is what I try to achieve:

public static IEnumerable<string> GetViewModelProperties(this IDocumentViewModel vm) {
     foreach (var property in vm.GetType().GetProperties().Where(p => (p.PropertyType.IsPrimitive ||
                                                                 p.PropertyType.GetInterfaces().Any(t => t.IsGenericType && t.GetGenericTypeDefinition() == typeof(IEnumerable<>))) &&
                                                                  p.GetIndexParameters().Count() == 0))
     {
        yield return property.Name + ":" + property.GetValue(vm, System.Reflection.BindingFlags.GetProperty, null, null, System.Globalization.CultureInfo.InvariantCulture);
     }
}

which I simply save to disk using

System.IO.File.WriteAllText("filename.txt", settings.ToString());

and in the resulting file, for property Frequency of type double with value 50.33 I got

Frequency:50,33

which is CurrentCulture (Polish uses coma as separator), but not

Frequency:50.33

as I'd expect. Any ideas what might be wrong?

score 3 · Accepted Answer · answered Oct 04 '13 at 11:19

The GetValue function of PropertyInfo returns an object, not a string - and here's the misconception in your code. That object gets converted to a string because of the + operator with the first argument there being a string (property.Name), but the CultureInfo is not applied to that string conversion.

The solution is to explicitly use the Convert.ToString(object, IFormatProvider) function, i.e.

yield return property.Name + ":" + Convert.ToString(property.GetValue(vm, System.Reflection.BindingFlags.GetProperty, null, null, System.Globalization.CultureInfo.InvariantCulture), System.Globalization.CultureInfo.InvariantCulture);

property.GetValue ignores CultureInfo

1 Answers1