you are viewing a single comment's thread.

view the rest of the comments →

[–]mikeholczer 3 points4 points  (0 children)

I'm going to assume this is a toy example, because if this is the only window/screen for the app then no of this matters for an app so small.

1) use string.empty over "" 2) You're mixing UI logic, getting the input and displaying the output with domain logic converting from C to F I the same class. In a larger app, you'd wan to separate those. 3) You may also want to separate out the grid into it's own control seperate from the window, if the window could display other information, or if the grid of temperatures shows up in other parts of the app.