BryanWilhite / SonghayCore

core reusable, opinionated concerns for *all* 🧐 of my C# projects
http://songhayblog.azurewebsites.net/
MIT License
1 stars 0 forks source link

review the usage of `CultureInfo.CurrentCulture` in `String.Format` #136

Closed BryanWilhite closed 2 years ago

BryanWilhite commented 2 years ago

You will want to use CurrentCulture for strings that are to be presented in the UI to the current user (forms, reports).

https://stackoverflow.com/a/475912/22944

image

💡 error/logging messaging should not be confused with normal UI communication

BryanWilhite commented 2 years ago

get rid of it; prefer string interpolation; any future discovery of internationalization issues would be a wonderful privilege 💡

BryanWilhite commented 2 years ago

CultureInfo.CurrentCulture is only left in place for dates/times and numerics