バグを修正

[kokoichi206]

Issue 番号

Closed #3

対応背景

• バグを修正

やったこと

• メモリリークを削除
  • viewmodel に context を渡す方法から、AndroidViewModel を利用するように変更
  • binding した情報を destroy で手放すよう変更
• クラッシュしないようエラーを補足
  • API 呼び出し時
• （横画面などで）縦幅が小さい時、リポジトリの詳細画面の情報が表示されない問題を解決
  • スクロール可能にすることで対応

やってないこと

UI before / after

Before

After

https://user-images.githubusercontent.com/52474650/205421008-b884f2aa-13b0-4fc5-a9e6-18d42ac279d0.mov

補足

[kokoichi206]

Binding についてコメント。

MainFragment ではメンバー変数を用意せずに onViewCreated の中のみのスコープの変数でバインドしている。

override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
    val binding = FragmentMainBinding.bind(view)
    ...

DetailFragment では、『メンバー変数を用意し、onViewCreated でバインドし、onDestroyView でリリース』している。 特にこちらの Fragment でも onViewCreated 以外で使ってなさそうなので、どちらかに合わせた方が良さそう。。。

→ MainFragment の方法に合わせるか！