#13 のタスク

#2 のタスク

• 非null表明演算子「!!」が無いかチェックし、あれば修正する

!!があったコード

• GitHubSearchViewModel
• RepositoryDetailFragment
• SearchFragment

参考になるサイトは以下です。
Kotlin で null 可能性を使用する

【Null安全】Kotlin Java比較メモ 正しいNull安全の使い方

Null safety

[Kotlin]null安全とは？ null許容型との付き合い方

• 適切にREADME.mdを書く
• コードのwarningの修正　出来ず
• ライブラリのチェック
• Gitの設定チェック
• 気になる点がコードにないかチェック
• マージをmainにする
• コメントを修正

#1 のタスク

• Android Studioで保存すると自動でコード整形したいをもとに、Save Actionsを導入する
• Macroを追加した
  Save Actionsを導入したが、上手く出来ているか自信がなかったため追加。
  これは、Gitで追跡されていないっぽかった？
  Android Studioで自動整形を設定する方法

#3 のタスク

• 検索ボックスに何も入れていない時、検索を実行すると、アプリがクラッシュするので、その解決
  #59 で解決

#2 のタスク

• 不必要なlateinit修飾子を削除する

• SearchActivity.ktにlateinit修飾子があったので、修正する（以下の部分）

class SearchActivity : AppCompatActivity(R.layout.activity_search) {
    companion object {
        lateinit var lastSearchDate: Date
    }
}

公式ドキュメント
properties

本アプリの UI は極めてシンプルです。これでも特に問題はありませんが、やはりいまいち映えないです。使いやすさが損なわれない前提で、UI をブラッシュアップしてください。方向性（エレガント、ゴージャス、モダン等）は問いません。また Jetpack Compose の導入も OK です。

#2 のタスク

• 強制ダウンキャストを行っている箇所を探して修正する

※目印はasやisです。

• SearchFragment に asがあった

参考になるサイトは以下です。

【Kotlin】ダウンキャストが必要な時はいつ？

Type checks and casts

#3 のタスク

• GitHubSearchViewModel
  ライブラリの問題はなさそう
• Item
  build.gradleのバージョンは問題なさそう
• RepositoryDetailFragment
  ライブラリの問題はなさそう
• SearchActivity 問題はなさそう
• SearchFragment
  問題はなさそう

現在のプログラムではテストがありません。UnitTest や UITest を追加してください。

#3 のタスク
例外処理の仕方をチェックし、きちんと例外が処理されているか確認する

• GitHubSearchViewModel
  以下の可能性があるため、try-catchを用いて、修正した。
• client?.get の呼び出し時にネットワーク接続エラーが発生する可能性
• response?.receive() の呼び出し時にレスポンスの取得に失敗する可能性
• jsonBody.optJSONArray("items") の呼び出し時にJSON形式が不正である場合に JSONException が発生する可能性
• jsonItem?.optString や jsonItem?.optLong の呼び出し時にJSONのキーが存在しない場合に JSONException が発生する可能性
• Item は例外処理の必要なし
• RepositoryDetailFragment
  try-catch文もあると◎
• SearchActivity 例外処理の必要なし
• SearchFragment
  try-catch文を使用して、エラーメッセージを表示し、再度入力を促す

#1 のタスク

• #18
• ローカル変数
• ソースファイル名
• プロパティ
• パッケージ
• クラス・オブジェクト
• 関数

本プロジェクトには安全性の低いがコードたくさんあります。下記のリストを参考に、安全性の低いコードを撲滅し、安全性を高めましょう。

• #34
• #35
• #36
• #37

本プロジェクトは様々な原則に違反しています。下記のリストを参考にプログラムをリファクタリングしましょう。

• DRY 原則
• CQS 原則
• 単一責任原則
• インターフェイス分離の原則
• 驚き最小の原則

本プロジェクトは Fragment が必要以上の責務を抱えており、いわゆる Fat Fragment 状態です。最低限の責務の切り出しをしてあげましょう。

※本課題は #6 の下位互換です。

• #50

#1 のタスク

• Android Studioで保存すると自動でコード整形したいをもとに、Save Actionsを導入する
• Macroを追加した
  Save Actionsを導入したが、上手く出来ているか自信がなかったため追加。
  これは、Gitで追跡されていないっぽかった？
  Android Studioで自動整形を設定する方法

#4 のタスク
コンポーネント指向でAndroidアプリを作ろう
https://qiita.com/YusukeIwaki/items/37a3148ee2287ef2db4e
下の3つをシンプルにした。

• SearchFragment
• GitHubSearchViewModel
• RepositoryDetailFragment

#2 のタスク

• GitHubSearchViewModel

• return@runBlocking GlobalScope.async がきになるので、それも必要かも。
  #57 で修正

• Item 想定外のnullの握り潰しなし

• RepositoryDetailFragment
  SearchActivityで、lastSearchDateはnullを許容しているので、それを直す

• SearchActivity 想定外のnullの握り潰しなし

• SearchFragment

 binding.searchInputText.setOnEditorActionListener { editText, action, _ ->
            if (action == EditorInfo.IME_ACTION_SEARCH) {
                editText.text.toString().let {
                    viewModel.searchGithubRepositories(it).apply {
                        adapter.submitList(this)
                    }
                }
                return@setOnEditorActionListener true
            }
            return@setOnEditorActionListener false
        }

#1 のタスク
Kotlinのコード

• GitHubSearchViewModel
• Item.kt
• RepositoryDeatalFragment
• SearchActivity
• SearchFragment

以下を確認する

• ネストされたクラス

xmlのコード

• activity_search
• fragment_repostiroy_detail
• fragment_search
• layout_item

以下を元にチェック

ConstraintLayoutをネストすると激重になる

ゆめみの Android の採用コーディング試験を公開しました ネストを浅くしているか

ConstraintLayout でレスポンシブ UI を作成する

#3 のタスク

• activity_search.xml
  ※unknown fragmentというエラー
  「Pick Layout…」というドロップダウンメニューから、プレビューするレイアウトを選択して解決
• fragment_repository_detail
  ※Warning Hardcoded string "文字列"
  AndroidStudioのtextで文字列を打ち込むとWarningが出る件
• nav_graph
  特にエラーなし？
• layout_item
  特にエラーなし？

• ブランチの運用方法についてREADME.mdに追加

本アプリの機能は極めて単純です。使いやすさが損なわれない前提で、様々な新機能を追加してあげてください。

本プロジェクトのソースコードはとても読みにくいです。下記のリストを参考に読みやすいように修正してください。

• #13
• #14
• #15
• #16
• #17
• その他

参考：https://kotlinlang.org/docs/coding-conventions.html

最近巷では MVC や MVVM などの GUI アーキテクチャから、Redux や VIPER などのアプリアーキテクチャまで様々なアーキテクチャが話題になっています。どれでもいいので本プロジェクトを自分の慣れたアーキテクチャに修正してください。

※本課題は #4 の上位互換です。

本プロジェクトには、至る所にバグが潜んでいます。下記のリストを参考にバグを修正してください。

• #45
• #46
• #47
• #48
• #49

#3 のタスク
メモリリークについて
https://kotlinlang.org/docs/native-memory-manager.html
https://medium.com/@fauzisho/avoiding-memory-leaks-in-kotlin-and-jetpack-android-development-tips-and-examples-86a17bab47a7
https://ponsanlabo.com/kotlin-java-memory-leak-investigate/

• GitHubSearchViewModel
  HttpClient のインスタンスを明示的にclose()メソッドで閉じるようにした。
• Item 関係なし
• RepositoryDetailFragment
  ViewBindingをonViewCreated以外でも使う時とonViewCreatedでしか使わない時
• SearchActivity 問題はなさそう
• SearchFragment
  viewとかbindingの再利用関連を修正すると◎かも
  CustomAdapterクラスのところを修正すると◎？
  https://qiita.com/yfujiki/items/7a01039e0c7b0d25c31e

#1 のタスク
#29 で、ある程度やったので、とりあえず完了