Ryosuke-Androiddev / CodeCheck

Apache License 2.0
0 stars 0 forks source link

ソースコードの可読性 #1 #11

Closed Ryosuke-Androiddev closed 2 years ago

Ryosuke-Androiddev commented 2 years ago

I implemented source code change for reading source code easily. It could be roughly divided into 3 type changes.

Class name Change , Arguments Name Change , Function Name Change, Variable Name Change, Space and Indent

Please check these committed details like below

fe98f21af9a932d9bb69068d9a2eed5f7f360d52 ‥ class name change (layout xml name change) with comments. ab1af0607eef06a8e00561b20e68ed6891e7f91c ‥ variable name change and confirm binding class name because I changed class name. 4c33065b55532851bfd9f0cc87b031477c5dfec0 ‥ function name change with comments and put arguments on separated lines

Modified Nested ViewModel Logic using Retrofit

I Implemented Retrofit Object (Singleton pattern) for searching Github Repository.

Please check these committed details like below

71213e77fa463b677f30fe325c22c86159397b7e ‥ generate model class using plugin Kotlin data class File from Json, also deleted no necessity arguments 0670c91dc155ebc3791f067abbfb21dff55801a4 ‥ modify nested ViewModel logic using Retrofit

追加説明

・命名規約をもとに、クラス名、メソッド名、引数名、xmlで、レイアウトエラーが起きないように参照情報の確認を行いました。

・また、Ktor client から、Retrofitに移行することで処理のネストを浅くし、適切なインデント、スペース、改行を行いソースコードの可読性の向上に努めました。適宜、必要だと感じた部分にコメントを追加しました。