Ryosuke-Androiddev / CodeCheck

Apache License 2.0
0 stars 0 forks source link

ソースコードの安全性の向上 #2 / ソースコードの可読性の向上 #1 #12

Closed Ryosuke-Androiddev closed 2 years ago

Ryosuke-Androiddev commented 2 years ago

Improved source code readability like below.

ソースコードの安全性の向上 #2 

149815f72af9818eefdc8147c9ee7ab6528646a6‥remove lateinit key word from MainActivity replace it LocalDate.now() 26fd67f307becd0e614ec47dbbc04409d1bceb68‥create bind fun at ViewHolder class with arguments for binding it with view. delete forced downcast operator and replace it requireContext(). 34c309bb02a5f8d6d2f1ecc919e147901a0e2bc9‥add null and empty check line to pass non-null string value to ViewModel. 2a904a9232c8a4abdca5540c55c5a3ed2f5c36a4‥change api response format and create convert fun for converting into List. 535c293bf28d9cf2ac103f8c76367e4293677f03‥modify unexpected null error using Flow at SearchViewModel to catch exception.

Change variable name, class name, arguments name, add comments add new lines and solve nest logic.

ソースコードの可読性の向上 #1

9e370d3faeccf652bdfdb07fc52d30d23126a03f‥Change variable name, add comments and add new lines. 980a0ca514cb52420c9ec99018c0339b1fde145e‥Change class name and add comments. ea0c2345295c53b19884e838e80ecd342ae2a6b5‥Change class name. 6ba8f85a6c9ba885a66a4b2aa7e1399dc502a696‥Change arguments name and solve nest logic.

追加説明

・Activityのlateinit 修飾詞が不要だと判断し、 LocalDate.now()を渡すことで検索実行日時をログに与えました。

・requireContext()を渡すことで、非 null 表明演算子を用いてcontextを取得していた部分の改善を行いました。

・Adapter class内での、強制ダウンキャストをViewHolderに生成されたviewとbindする、bind fun を用いることで強制ダウンキャストの記述を改善しました。

・適宜、クラス名の変更、改行、引数名の変更、ネストロジックの改善を行いました