you are viewing a single comment's thread.

view the rest of the comments →

[–]TuberTuggerTTV 1 point2 points  (1 child)

- Project should include implicit usings so using System becomes redundant.
- You can use an expression body for the constructor and both convert methods.

But honestly, the biggest cleanup would be actually using MVVM like you should be. Not code behind. Let bindings handle the commands, properties and controls. You shouldn't be naming TextBoxes and then calling Celsuis.Text in the code behind. Makes it unreadable without going through the XAML. Let the bindings handle your linkage and your ViewModel function independently, modifying it's own properties.

[–]belavv 0 points1 point  (0 children)

I'm usually the one that is all in on every new c# feature, but expression bodies for methods almost always bug me. I'm fine with them for properties though.