以下、コメントに対しての返答。
1点目: 上記のコードに状態管理を更新するメソッドを追加しました。 その際、ボタンタップ時にUIの更新が行われるためview側でもそれが分かるようなメソッド名にした方がいいのかなと感じました。これについてご意見をいただきたいです。
メソッド名とかは、個人の好みで何とかしてください。
またプロジェクト等ではその中で命名規則とかが決まってると思います。
それに従ってください。
2点目: buttonTapActionの中身を全てviewModelで定義する場合だと、 例えばボタンタップ時にshowDialogメソッドを使ってアラート出す処理などviewModelのメソッドの引数にcontextが入ってしまう場合があります。 またシンプルにviewModelでアラートやインディケーターのようなUIが表示できるのもviewModelにviewが一部表示されてしまい不自然に感じてしまいます。 viewModelにcontextが入る場合、viewModelにUIが表示される場合、この2点で問題がないのかもご意見を伺いたいです。
上位側からcontextを受け渡さなくても良いパッケージというのもあります。
私はGetXというパッケージを使ってますが、グローバルで最新のcontextが得られます。まあ使い方を間違えれば、不具合(副作用)があるので注意が必要ですが。
もしくはViewModel側にcontextを引数で渡してしまうという方法もあると思います。
後「viewModelにviewが一部表示されてしまう不自然さ」は気のせいです。そう思うことにするのです。
Viewが表示されるという感覚を持つのではなく、(Viewを表示しその)結果を取得するAPIの呼び出しをするという感じにするのです。そういう意味にするとDBからのデータ取得と何ら変わらない感覚になると思います。
Viewの実装をViewModelにちまちま書くとそういう感覚が薄れるので、Viewの呼び出しとそのデータ取得もクラス化(カプセル化)してしまえばいいのです。
これにより各クラスの単体テストもよりしやすくなります。