Kotlinミニアンチパターン
•
1 like•6,291 views
R.kt #3の発表資料です。 https://recruit-lifestyle.connpass.com/event/78873/
More Related Content
What's hot
What's hot (20)
More from Recruit Lifestyle Co., Ltd.
More from Recruit Lifestyle Co., Ltd. (20)
Kotlinミニアンチパターン
- 2. 自己紹介 ● Naoto Nakazato ○ Android開発歴 7年くらい ● Recruit Lifestyle ○ 2017年6月〜 ○ HOT PEPPER Beauty ● アカウント ○ Twitter: @oxsoft ○ Facebook: naoto.nakazato ○ GitHub: oxsoft ○ Qiita: oxsoft
- 6. Destructuring Declarations componentN が定義されているPairやdataクラスでは、 メンバ変数を直接代入できる val (name, age) = Pair("Alice", 27) listOf(Pair("Alice", 27)).forEach { (name, age) -> // ... }
- 7. アンチパターン 変数の宣言順が間違っていることに気づかなかった data class User(val id: String, val name: String) fun init() { users.forEach { (name, id) -> // ... } }
- 10. 解決策の例2 Int や String ではなく、専用の Value Object を定義する val balance: Int val balance: Money
- 12. Nullable Boolean Nullable Booleanに対してelvis演算子を使うと警告が出る Equality check should be used instead of elvis for nullable boolean check if (condition ?: false) { // ... } if (condition == true) { // ... }
- 13. アンチパターン 常に警告に従う if (condition ?: true) { // ... } if (condition != false) { // ... }
- 14. 何が良くないか 分かりにくい if (condition != false) { // ... }
- 15. 解決策の例1 状況に応じて null を true として扱ったり false として扱っ たりするようなケースは少なそう → 早い段階で true か false にして、3値論理を避ける
- 17. #03 SAM変換
- 18. SAM変換 JavaのinterfaceがSingle Abstract Methodの場合、 Kotlinからの呼び出しはラムダで書ける public void setOnClickListener(OnClickListener l) { // ... } public interface OnClickListener { void onClick(View v); } view.setOnClickListener { v -> // ... } Java Kotlin
- 19. アンチパターン Kotlinでもinterfaceを引数に使う interface OnSuccessListener { fun onSuccess() } fun setListener(listener: OnSuccessListener) { // ... }
- 20. 何が良くないか KotlinだとSAM変換できない setListener(object : OnSuccessListener { override fun onSuccess() { // ... } })
- 21. 解決策の例 typealiasを使う typealias OnSuccessListener = () -> Unit fun setListener(listener: OnSuccessListener) { // ... }
- 22. #04 sealed class
- 23. sealed class sealed classにすると、同ファイルのみ子クラス宣言可能 → 子クラスが確定し、else句が不要になる（enum風） sealed class Either<out L, out R> data class Left<out T>(val value: T) : Either<T, Nothing>() data class Right<out T>(val value: T) : Either<Nothing, T>() val name = when (either) { is Left -> "" is Right -> either.value }
- 24. アンチパターン 通常のアプリでは子クラスが確定しているため、 共通処理のための interface を sealed class にしたくなる
- 25. 何が良くないか sealed class にすると（当然だが）多重継承できない → 全てを sealed class にすることはできない 果物 スイカ 球体 地球バナナ sealed class sealed class
- 26. interfaceとして切り出しても… interface として切り出して委譲するパターンだと、 2つの sealed class に属しているかのように振る舞えない 果物 スイカ スイカ球体 sealed class sealed class 球体I_球体 interface
- 27. 解決策の例 sealed class にはせず、interface のままにする when文で処理を分けず、各クラスに実装する
- 28. #05 Data Binding と Kotlin ※アンチパターンではなく悩みです
- 30. Data Binding 3.0.0 Data Binding 3.0.0から、 @NonNull / @Nullable を付与してくれるようになった → Kotlinから安心して呼び出せるように
- 31. アンチパターン（ではないけど） <layout> <LinearLayout> <include android:id="@+id/include_button" layout="@layout/layout_button" /> </LinearLayout> </layout> @Nullable public final LayoutButtonBinding includeButton; include が Nullable に
- 32. 何が良くないか safe callや強制アンラップをしなければいけない binding.includeButton?.button binding.includeButton!!.button
- 33. 解決策の例 Data binding 2.3.3を使い、解決される（？）のを待つ （あるいは、Ankoを使う） 理由を知っている人がいたら教えて下さい dependencies { kapt 'com.android.databinding:compiler:2.3.3' }
- 34. まとめ