Form1 should use proper data binding

[bastte]

I haven't done windows forms in a long time so maybe this comment is wrong, but it feels that data binding should be used (for example to extract values from the UI https://github.com/ericgu/AnExerciseInCouplingDesRefWG/blob/master/StockTracker/Form1.cs#L76)

[ericgu]

Can you give us more information? What is the problem that you are trying to solve with data binding, and how does it relate to single responsibility principle?

[bastte]

In my opinion it's not the responsibility of the code-behind (Form1.cs) to know what UI objects are actually in the view.
For example, Form1.cs (which will eventually be renamed for clarity once we identify what it does exactly) shouldn't know that the view contains an element called _textBoxTicker. It's the view's responsibility to data bind to an object and hide it's view elements from the code-behind / view models.

[wli3]

One interesting side, since React is trendy these days, data binding could be further chop down to 2 concerns: Write the data and read the data

Write date is concerning about concurrency and restriction. For example, if we just fetch the data, we don't have to write it again. A more obvious one is when 2 people are clicking the buy button at the same time, who wins

Read date has concern about format the data to ideal structure(how to join tables) and caching