読みやすいコードの書き方 第 8 回 / Code readability: Session 8 (ver. 2, Ja)

第 8 回: レビュー

---

セッションリスト

第 1 回: 導入と原則
- https://speakerdeck.com/munetoshi/code-readability-session-1-ver-2-ja

第 2 回: 命名
- https://speakerdeck.com/munetoshi/code-readability-session-2-ver-2-ja

第 3 回: コメント
- https://speakerdeck.com/munetoshi/code-readability-session-3-ver-2-ja

第 4 回: 状態
- https://speakerdeck.com/munetoshi/code-readability-session-4-ver-2-ja

第 5 回: 関数
- https://speakerdeck.com/munetoshi/code-readability-session-5-ver-2-ja

第 6 回: 依存関係 I
- https://speakerdeck.com/munetoshi/code-readability-session-6-ver-2-ja

第 7 回: 依存関係 II
- https://speakerdeck.com/munetoshi/code-readability-session-7-ver-2-ja

第 8 回: レビュー
- https://speakerdeck.com/munetoshi/code-readability-session-8-ver-2-ja

---

関連書籍 「読みやすいコードのガイドライン - 持続可能なソフトウェア開発のために」
- https://gihyo.jp/book/2022/978-4-297-13036-7

英語版のリスト: https://gist.github.com/munetoshi/65a1b563fb2c271f328c121a4ac63571#file-code-readability-links-md

以前のバージョン (英語): https://speakerdeck.com/munetoshi/code-readability

© 2019-2023 Munetoshi Ishikawa, supported by LINE corporation

Transcript

1. ಡΈ΍͍͢-ίʔυͷॻ͖ํ / ୈ-8-ճ ϨϏϡʔ #8
2. લճɾલʑճͷ͓͞Β͍: ґଘؔ܎ ݁߹౓: ಺༰ɾ֎෦ɾڞ௨ɾ੍ޚ݁߹Λ؇࿨͢Δ ํ޲: Ұํ޲ʹ͢Δɾ८ճΛখ͍͞ൣғʹ੍ݶ͢Δ ॏෳ: ਺चͭͳ͗ͷґଘ΍ηοτͷॏෳΛආ͚Δ ໌ࣔੑ: ΫϥεͷఆٛͰཧղͰ͖Δґଘؔ܎Λ࢖͏
3. ߨٛͷߏ੒ - ಋೖͱݪଇ - ࣗવݴޠ: ໋໊, ίϝϯτ - ܕͷߏ଄: ঢ়ଶ, ؔ਺ - ܕؒͷߏ଄: ґଘؔ܎I, ґଘؔ܎II - ϨϏϡʔ ίʔυϨϏϡʔ > ಋೖ
4. ίʔυϨϏϡʔ͕ඞཁͳཧ༝ ಡΈ΍͢͞ͷվળ - ଞͷਓͷࢹ఺Ͱ֬ೝ͢Δ ϩδοΫͷਖ਼͠͞ͷ֬ೝ - ΤοδέʔεɾΤϥʔ/ྫ֎ॲཧɾڝ߹ঢ়ଶ... - جຊతʹ͸ϓϧϦΫΤετΛ࡞ͬͨਓ͕੹೚Λ࣋ͭ ίʔυϨϏϡʔ > ಋೖ
5. ίʔυϨϏϡʔͷޮ཰ ϨϏϡʔͦͷ΋ͷͷޮ཰΋ॏཁ - ಡΈ΍͢͞͸ੜ࢈ੑͷͨΊ - ϨϏϡʔ͕ඇޮ཰తͩͱຊ຤స౗ ίʔυϨϏϡʔ > ಋೖ
6. ࠓճͷ಺༰ ϨϏϡʔΠʢϨϏϡʔΛड͚Δଆʣ: - ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετͷ࡞Γํ - ίϝϯτͷద༻ͷ࢓ํ ϨϏϡʔΞʢϨϏϡʔΛ͢Δଆʣ: - ݪଇ - ίϝϯτͷ಺༰ ίʔυϨϏϡʔ > ಋೖ
7. ࠓճͷ಺༰ ϨϏϡʔΠʢϨϏϡʔΛड͚Δଆʣ: - ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετͷ࡞Γํ - ίϝϯτͷద༻ͷ࢓ํ ϨϏϡʔΞʢϨϏϡʔΛ͢Δଆʣ: - ݪଇ - ίϝϯτͷ಺༰ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ
8. ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετͷ࡞Γํ - ੹೚ൣғΛ໌ࣔ͢Δ - αΠζΛখ͘͞อͭ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ
9. ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετͷ࡞Γํ - ੹೚ൣғΛ໌ࣔ͢Δ - αΠζΛখ͘͞อͭ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > ੹೚ൣғͷ໌ࣔ
10. ϓϧϦΫΤετͷ੹೚ൣғͷ໌ࣔ ϓϧϦΫΤετͦͷ΋ͷͷઆ໌Λॻ͘ - ओͳ໨తɾୡ੒͍ͨ͜͠ͱ ϓϧϦΫΤετͷ੹೚ͷڥքΛ໌֬ʹ͢Δ - ߦΘͳ͍͜ͱ (out-of-scope) - ࠓޙͷܭը ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > ੹೚ൣғͷ໌ࣔ
11. ੹೚ൣғͷ໌ࣔͷॏཁੑ 1/2 ϨϏϡʔΞͷ੹೚: - վળՄೳͳ఺͕͋Ε͹มߋͷཁٻ/ఏҊΛ͢Δ ࡍݶͷͳ͍มߋͷཁٻ͕ߦΘΕΔՄೳੑ͕͋Δ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > ੹೚ൣғͷ໌ࣔ
12. ੹೚ൣғͷ໌ࣔͷॏཁੑ 2/2 ߹ҙΛऔΓ΍͘͢͢Δ - վળΛͲͷϓϧϦΫΤετͰߦ͏͔ - ݱࡏͷมߋ͸࠷ऴతͳઃܭʹ߹͍ͬͯΔ͔ σβΠϯυΩϡϝϯτ౳΋ซ༻͢Δ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > ੹೚ൣғͷ໌ࣔ
13. ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετͷ࡞Γํ - ໨తΛ໌ࣔ͢Δ - αΠζΛখ͘͞อͭ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ
14. ϓϧϦΫΤετͷαΠζͱϨϏϡʔͷޮ཰ େ͖ͳϓϧϦΫΤετͷϨϏϡʔ͸ඇޮ཰ - ཧղΛ͢Δ·Ͱʹ͕͔͔࣌ؒΔ - ϨϏϡʔͷΠςϨʔγϣϯ͕௕͘ͳΔ Ұ೔ͷλεΫΛෳ਺ͷϓϧϦΫΤετʹ෼ׂ͢Δ ʮࠓिͷϓϧϦΫΤετʯΛ࡞Βͳ͍ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ
15. ϓϧϦΫΤετͷ෼ׂํ๏ - େ͖ͳػೳ࣮૷ʹରͯ͠ - ௥Ճ࡞ۀʹରͯ͠ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ
16. ϓϧϦΫΤετͷ෼ׂํ๏ - େ͖ͳػೳ࣮૷ʹରͯ͠ - ௥Ճ࡞ۀʹରͯ͠ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > େ͖ͳػೳ࣮૷࣌
17. େ͖ͳػೳ࣮૷ʹର͢Δ෼ׂ๏ ػೳͷ։ൃ͸਺ि͔ؒΒ਺ϲ݄͔͔Δ͜ͱ΋ - 1 ͭͷϓϧϦΫΤετʹ͸·ͱΊΒΕͳ͍ - ࡞੒్தͷػೳΛϚʔδ͢Δඞཁ͕͋Δ τοϓμ΢ϯํࣜͱϘτϜΞοϓํࣜ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > େ͖ͳػೳ࣮૷࣌
18. τοϓμ΢ϯํࣜ ߏ଄͚ͩΛ࡞ΓɺޙͰ࣮૷ΛຒΊΔ 1. ߏ଄Λઆ໌͢ΔͨΊͷεέϧτϯΫϥεΛ࡞Δ 2. ࠓޙͷϓϥϯΛ TODO ΍ϓϧϦΫΤετͷίϝϯτͰઆ໌͢Δ class UserProfilePresenter( val userProfileUseCase: UseCase val profileRootView: View ) { fun showProfileImage() = TODO("#12345: ...") ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > େ͖ͳػೳ࣮૷࣌
19. ϘτϜΞοϓํࣜ ෦඼Λઌʹ࡞Δ 1. ʮ࢖͏ଆʯ͕ͳ͍ঢ়ଶͰ୯७ͳΫϥεɾؔ਺Λ࣮૷͢Δ 2. কདྷͲ͏࢖͏͔Λ TODO ΍ϓϧϦΫΤετͷίϝϯτͰઆ໌͢Δ data class UserProfileData(val ..., val ...) // TODO(#12346): ... object UserNameStringUtils { @Suppress("unused") // TODO(#12347): ... fun normalizeEmoji(userName: String): String = ... ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > େ͖ͳػೳ࣮૷࣌
20. ϓϧϦΫΤετͷ෼ׂํ๏ - େ͖ͳػೳ࣮૷ʹରͯ͠ - ௥Ճ࡞ۀʹରͯ͠ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
21. ௥Ճ࡞ۀͷൃੜྫ 1/3 લఏ 1: ࣌ࠁදࣔͷػೳΛ࣮૷ class CurrentClockIndicator(...) { fun showCurrentTime() { val clockText = toClockText(...) // ... } companion object { private fun toClockText(...: Long): String = ... } } ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
22. ௥Ճ࡞ۀͷൃੜྫ 2/3 ࣮૷࣌ͷίϛοτϦετ Commit 1: `CurrentTimeIndicator` ͷεέϧτϯΫϥε࡞੒ Commit 2: ϓϥΠϕʔτϢʔςΟϦςΟؔ਺ `toClockText` ࣮૷ Commit 3: `CurrentTimeIndicator` ͷ `showCurrentTime` ࣮૷ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
23. ௥Ճ࡞ۀͷൃੜྫ 3/3 લఏ 2: toClockText ͷίϐʔ͕طଘΫϥεʹ͋Δ͜ͱ͕ൃ֮ DateTimeTextFormatter ͱͯ͠ڞ௨ίʔυΛநग़͍ͨ͠ object DateTimeTextFormatter { fun toClockText(...: Long): String = ... } class CurrentClockIndicator(...) { fun showCurrentTime() { val clockText = DateTimeTextFormatter.toClockText(...) // ... ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
24. ௥Ճ࡞ۀͷϓϧϦΫΤετͷऔΓѻ͍ ໰୊: ͲͷΑ͏ͳϓϧϦΫΤετΛ࡞੒͢Ε͹Α͍͔ - Ξϯνύλʔϯ 1: 1ͭͷϓϧϦΫΤετʹ·ͱΊΔ - Ξϯνύλʔϯ 2: ࣌ܥྻͰϓϧϦΫΤετΛ෼ׂ - ղܾҊ 1: ௥Ճ࡞ۀΛઌʹ׬ྃͤ͞Δ - ղܾҊ 2: ίϛοτΛ࠶ߏ੒͢Δ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
25. ௥Ճ࡞ۀͷϓϧϦΫΤετͷऔΓѻ͍ ໰୊: ͲͷΑ͏ͳϓϧϦΫΤετΛ࡞੒͢Ε͹Α͍͔ - Ξϯνύλʔϯ 1: 1ͭͷϓϧϦΫΤετʹ·ͱΊΔ - Ξϯνύλʔϯ 2: ࣌ܥྻͰϓϧϦΫΤετΛ෼ׂ - ղܾҊ 1: ௥Ճ࡞ۀΛઌʹ׬ྃͤ͞Δ - ղܾҊ 2: ίϛοτΛ࠶ߏ੒͢Δ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
26. Ξϯνύλʔϯ 1: 1ͭͷϓϧϦΫΤετʹ·ͱΊΔ ϓϧϦΫΤετͷίϛοτ Commit 1: `CurrentTimeIndicator` ͷεέϧτϯΫϥε࡞੒ Commit 2: ϓϥΠϕʔτϢʔςΟϦςΟؔ਺ `toClockText` ࣮૷ Commit 3: `CurrentTimeIndicator` ͷ `showCurrentTime` ࣮૷ Commit 4: `toClockText` Λ `DateTimeTextFormatter` ͱͯ͠நग़ Commit 4 ͷ໾ׂ͕ҟͳΔ && มߋ͕ڊେʹͳΔ ʮநग़࿙Ε͕ͳ͍͔ʯͳͲͷ֬ೝࣄ߲͕૿͑Δ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
27. ௥Ճ࡞ۀͷϓϧϦΫΤετͷऔΓѻ͍ ໰୊: ͲͷΑ͏ͳϓϧϦΫΤετΛ࡞੒͢Ε͹Α͍͔ - Ξϯνύλʔϯ 1: 1ͭͷϓϧϦΫΤετʹ·ͱΊΔ - Ξϯνύλʔϯ 2: ࣌ܥྻͰϓϧϦΫΤετΛ෼ׂ͢Δ - ղܾҊ 1: ௥Ճ࡞ۀΛઌʹ׬ྃͤ͞Δ - ղܾҊ 2: ίϛοτΛ෼ׂɾฒ΂ସ͑ɾ౷߹͢Δ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
28. Ξϯνύλʔϯ 2: ࣌ܥྻͰ෼ׂ͢Δ 1/2 ϓϧϦΫΤετ A ͷίϛοτ Commit 1: `CurrentTimeIndicator` ͷεέϧτϯΫϥε࡞੒ Commit 2: ϓϥΠϕʔτϢʔςΟϦςΟؔ਺ `toClockText` ࣮૷ Commit 3: `CurrentTimeIndicator` ͷ `showCurrentTime` ࣮૷ ϓϧϦΫΤετ B ͷίϛοτ Commit 4: `toClockText` Λ `DateTimeTextFormatter` ͱͯ͠நग़ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
29. Ξϯνύλʔϯ 2: ࣌ܥྻͰ෼ׂ͢Δ 2/2 ϓϧϦΫΤετ A ͷϚʔδޙʹ ϓϧϦΫΤετ B ΛϚʔδ ٕज़తෛ࠴Λট͘ - Ұ࣌తʹίʔυͷίϐʔ͕ߦΘΕΔ - ࡞ۀͷׂΓࠐΈ౳Ͱ ϓϧϦΫΤετ B ͕์ஔ͞Εͯ͠·͏ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
30. ௥Ճ࡞ۀͷϓϧϦΫΤετͷऔΓѻ͍ ໰୊: ͲͷΑ͏ͳϓϧϦΫΤετΛ࡞੒͢Ε͹Α͍͔ - Ξϯνύλʔϯ 1: 1ͭͷϓϧϦΫΤετʹ·ͱΊΔ - Ξϯνύλʔϯ 2: ࣌ܥྻͰϓϧϦΫΤετΛ෼ׂ͢Δ - ղܾҊ 1: ௥Ճ࡞ۀΛઌʹ׬ྃͤ͞Δ - ղܾҊ 2: ίϛοτΛ࠶ߏ੒͢Δ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
31. ղܾҊ 1: ௥Ճ࡞ۀΛઌʹ׬ྃͤ͞Δ 1/2 ฒߦͯ͠ DateTimeTextFormatter ͷϓϧϦΫΤετΛ࡞Δ ద༻৚݅: toClockText ࣮૷લʹίʔυͷॏෳʹؾ͕͍ͭ৔߹ ϓϧϦΫΤετ A ͷίϛοτ Commit 1: `CurrentTimeIndicator` ͷεέϧτϯΫϥε࡞੒ ϓϧϦΫΤετ B ͷίϛοτ Commit 4: `toClockText` Λ `DateTimeTextFormatter` ͱͯ͠நग़ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
32. ղܾҊ 1: ௥Ճ࡞ۀΛઌʹ׬ྃͤ͞Δ 2/2 ϓϧϦΫΤετ B ͷϚʔδޙ ϓϧϦΫΤετ A ΛϦϕʔε நग़ޙͷ toClockText Λ࢖ͬͯϓϧϦΫΤετ A Λ࡞੒Մೳ ϓϧϦΫΤετ A ͷίϛοτ Commit 1: `CurrentTimeIndicator` ͷεέϧτϯΫϥε࡞੒ Commit 3: `CurrentTimeIndicator` ͷ `showCurrentTime` ࣮૷ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
33. ௥Ճ࡞ۀͷϓϧϦΫΤετͷऔΓѻ͍ ໰୊: ͲͷΑ͏ͳϓϧϦΫΤετΛ࡞੒͢Ε͹Α͍͔ - Ξϯνύλʔϯ 1: 1ͭͷϓϧϦΫΤετʹ·ͱΊΔ - Ξϯνύλʔϯ 2: ࣌ܥྻͰϓϧϦΫΤετΛ෼ׂ͢Δ - ղܾҊ 1: ௥Ճ࡞ۀΛઌʹ׬ྃͤ͞Δ - ղܾҊ 2: ίϛοτΛ࠶ߏ੒͢Δ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
34. ղܾҊ 2: ίϛοτΛ࠶ߏ੒͢Δ ίϛοτΛ෼ׂɾฒ΂ସ͑ɾ౷߹͢Δ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
35. ղܾҊ 2: ίϛοτΛ࠶ߏ੒͢Δ 1/4 Step 1: Commit 4 Λ෼ׂ Commit 1: `CurrentTimeIndicator` ͷεέϧτϯΫϥε࡞੒ Commit 2: ϓϥΠϕʔτϢʔςΟϦςΟؔ਺ `toClockText` ࣮૷ Commit 3: `CurrentTimeIndicator` ͷ `showCurrentTime` ࣮૷ Commit 4_EXISTING: طଘίʔυͷ `toClockText` Λநग़ Commit 4_NEW: `CurrentTimeIndicator` ͷ `toClockText` Λநग़ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
36. ղܾҊ 2: ίϛοτΛ࠶ߏ੒͢Δ 1/4 Step 2: ίϛοτΛฒ΂ସ͑Δ Commit 4_EXISTING: طଘίʔυͷ `toClockText` Λநग़ Commit 1: `CurrentTimeIndicator` ͷεέϧτϯΫϥε࡞੒ Commit 2: ϓϥΠϕʔτϢʔςΟϦςΟؔ਺ `toClockText` ࣮૷ Commit 4_NEW: `CurrentTimeIndicator` ͷ `toClockText` Λநग़ Commit 3: `CurrentTimeIndicator` ͷ `showCurrentTime` ࣮૷ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
37. ղܾҊ 2: ίϛοτΛ࠶ߏ੒͢Δ 1/4 Step 3: த 3 ͭͷίϛοτΛ౷߹͢Δ Commit 4_EXISTING: طଘίʔυͷ `toClockText` Λநग़ Commit 1: `CurrentTimeIndicator` ͷεέϧτϯΫϥε࡞੒ --squashed--ɹ --squashed--ɹ Commit 3: `CurrentTimeIndicator` ͷ `showCurrentTime` ࣮૷ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
38. ղܾҊ 2: ίϛοτΛ࠶ߏ੒͢Δ 1/4 Step 4: 2 ͭͷϓϧϦΫΤετΛ࡞੒͢Δ ϓϧϦΫΤετ B Commit 4_EXISTING: طଘίʔυͷ `toClockText` Λநग़ ϓϧϦΫΤετ A Commit 1: `CurrentTimeIndicator` ͷεέϧτϯΫϥε࡞੒ Commit 3: `CurrentTimeIndicator` ͷ `showCurrentTime` ࣮૷ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ > ௥Ճ࡞ۀʹର͢ΔରԠ
39. ϓϧϦΫΤετΛখ͘͞อͭͨΊʹ: ิ଍ ࣮૷લʹܭըʹ͍ͭͯ߹ҙΛͱΔ - Ͳͷ࡞ۀΛ͍ͭɺͲͷϓϧϦΫΤετͰߦ͏͔ - Ұ࣌తͳෛ࠴Λڐ͢ͷ͔ɺͲ͏ղফ͢Δͷ͔ มߋ͕ൃੜͨ͠ΒૣΊʹٞ࿦͢Δ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > খ͍͞ϓϧϦΫΤετ
40. ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ: ·ͱΊ ϓϧϦΫΤετͷ੹೚ൣғΛ໌ࣔ͢Δ - ओͳ໨తɾୡ੒͍ͨ͜͠ͱ - ߦΘͳ͍͜ͱɾࠓޙͷܭը ϓϧϦΫΤετΛখ͘͢͞Δ - େ͖ͳػೳ࣮૷࣌: τοϓμ΢ϯɾϘτϜΞοϓ - ௥Ճ࡞ۀ࣌: ฒྻ࡞ۀɾίϛοτͷ࠶ߏ੒ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > ·ͱΊ
41. ༨ஊ: ίϛοτ΁ͷద༻ ಉ͡ߟ͑ํΛίϛοτʹ΋ద༻͢Δ - ίϛοτͷ੹೚ൣғ - ίϛοτͷ෼ׂͷ࢓ํ ίʔυϨϏϡʔ > ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετ > ·ͱΊ
42. ࠓճͷ಺༰ ϨϏϡʔΠʢϨϏϡʔΛड͚Δଆʣ: - ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετͷ࡞Γํ - ίϝϯτͷద༻ͷ࢓ํ ϨϏϡʔΞʢϨϏϡʔΛ͢Δଆʣ: - ݪଇ - ίϝϯτͷ಺༰ ίʔυϨϏϡʔ > ίϝϯτͷద༻
43. ίϝϯτΛద༻͢Δ্Ͱॏཁͳ఺ ίϝϯτΛػցతʹద༻͠ͳ͍ ίʔυϨϏϡʔ > ίϝϯτͷద༻
44. ίϝϯτΛػցతʹద༻͠ͳ͍ - ίϝϯτͷҙਤΛཧղ͢Δ - ͳ࣭ͥ໰͞Ε͔ͨɾޡղ͞Ε͔ͨΛߟ͑Δ - ଞͷ৔ॴʹ΋ಉ͡ίϝϯτΛద༻͢Δ ίʔυϨϏϡʔ > ίϝϯτͷద༻
45. ίϝϯτΛػցతʹద༻͠ͳ͍ - ίϝϯτͷҙਤΛཧղ͢Δ - ͳ࣭ͥ໰͞Ε͔ͨɾޡղ͞Ε͔ͨΛߟ͑Δ - ଞͷ৔ॴʹ΋ಉ͡ίϝϯτΛద༻͢Δ ίʔυϨϏϡʔ > ίϝϯτͷద༻
46. ίϝϯτͷҙਤΛཧղ͢Δ ఏҊ͞ΕͨίʔυΛͦͷ··ϖʔετ͠ͳ͍ - ओʹͲͷ෦෼ͷվળͳͷ͔Λཧղ͢Δ - ߋͳΔվળΛࢼΈΔ - ʮਖ਼͠͞ʯΛݕূ͢Δ: Τοδέʔεɾྫ֎ɾڝ߹ঢ়ଶ... ίʔυϨϏϡʔ > ίϝϯτͷద༻
47. ίϝϯτΛػցతʹద༻͠ͳ͍ - ίϝϯτͷҙਤΛཧղ͢Δ - ͳ࣭ͥ໰͞Ε͔ͨɾޡղ͞Ε͔ͨΛߟ͑Δ - ଞͷ৔ॴʹ΋ಉ͡ίϝϯτΛద༻͢Δ ίʔυϨϏϡʔ > ίϝϯτͷద༻
48. ͳ࣭ͥ໰͞Ε͔ͨɾޡղ͞Ε͔ͨΛߟ͑Δ ࣭໰ɾޡղ͸ίʔυͷಡΈʹ͘͞ʹΑͬͯҾ͖ى͜͞ΕΔ ίϝϯτͷྫ: "͜͜Ͱ null νΣοΫ͍ͯ͠Δͷ͸ͳͥͰ͔͢ʁ" - νΣοΫ͕ෆཁͳΒऔΓআ͘ - ίʔυͷίϝϯτͰཧ༝Λઆ໌͢Δ - ཧ༝͕Θ͔Γ΍͘͢ͳΔΑ͏ʹม਺Λ࢖͏ (isUserExpired ౳) ίʔυϨϏϡʔ > ίϝϯτͷద༻
49. ίϝϯτΛػցతʹద༻͠ͳ͍ - ίϝϯτͷҙਤΛཧղ͢Δ - ͳ࣭ͥ໰͞Ε͔ͨɾޡղ͞Ε͔ͨΛߟ͑Δ - ଞͷ৔ॴʹ΋ಉ͡ίϝϯτΛద༻͢Δ ίʔυϨϏϡʔ > ίϝϯτͷద༻
50. ଞͷ৔ॴʹ΋ಉ͡ίϝϯτΛద༻͢Δ ྨࣅͨ͠৔ॴ͕ͳ͍͔Λ֬ೝ͢Δ ίϝϯτͷྫ: "@Nullable Λ͜ͷҾ਺ʹ͚͍ͭͯͩ͘͞ɻ": - ଞͷҾ਺ɾ໭Γ஋ʹ΋ඞཁͰͳ͍͔Λߟ͑Δ - มߋ͞Εͨଞͷؔ਺ʹ΋ඞཁͰͳ͍͔Λߟ͑Δ - ʮࠓޙͷϓϧϦΫΤετʯʹ΋൓ө͢Δ ίʔυϨϏϡʔ > ίϝϯτͷద༻
51. ϨϏϡʔίϝϯτͷద༻: ·ͱΊ ϨϏϡʔίϝϯτͷҙਤΛཧղ͢Δ - ͦͷίϝϯτ͕ॻ͔Εͨཧ༝ɾഎܠΛߟ͑Δ - ߋͳΔվળΛࢼΈΔ - ଞͷίʔυɾϓϧϦΫΤετʹ΋ద༻͢Δ ίʔυϨϏϡʔ > ίϝϯτͷద༻ > ·ͱΊ
52. ࠓճͷ಺༰ ϨϏϡʔΠʢϨϏϡʔΛड͚Δଆʣ: - ϨϏϡʔ͠΍͍͢ϓϧϦΫΤετͷ࡞Γํ - ίϝϯτͷద༻ͷ࢓ํ ϨϏϡʔΞʢϨϏϡʔΛ͢Δଆʣ: - ݪଇ - ίϝϯτͷ಺༰ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ
53. ϨϏϡʔΞͱͯ͠ͷݪଇ ϨϏϡʔΠΛϦεϖΫτ͢Δ - ઈରʹ๫ݴΛు͔ͳ͍ɺਓ֨൱ఆΛ͠ͳ͍ - ϨϏϡʔͷର৅: ίʔυɾ࢓༷ɾϓϩηεͳͲ (!= ϨϏϡʔΠ) ޮ཰తͳϨϏϡʔΛߦ͏ - ϦιʔεΛແݶʹ࢖ͬͯ͸ͳΒͳ͍ - ඞཁͳ࣌ؒΛ΋࡟Βͳ͍Α͏ʹ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ
54. ϦεϖΫτͱޮ཰ͷͨΊͷݪଇ - ϨϏϡʔϦΫΤετΛແࢹ͠ͳ͍ - ໰୊ͷ͋ΔϓϧϦΫΤετΛڋ൱͢Δ - కΊ੾ΓΛҙࣝ͗͢͠ͳ͍ - ʮఏҊʯҎ֎ͷํ๏Λߟྀ͢Δ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ
55. ϦεϖΫτͱޮ཰ͷͨΊͷݪଇ - ϨϏϡʔϦΫΤετΛແࢹ͠ͳ͍ - ໰୊ͷ͋ΔϓϧϦΫΤετΛڋ൱͢Δ - కΊ੾ΓΛҙࣝ͗͢͠ͳ͍ - ʮఏҊʯҎ֎ͷํ๏Λߟྀ͢Δ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ > ແࢹ͠ͳ͍
56. ϨϏϡʔϦΫΤετΛແࢹ͠ͳ͍ ʮ͍ͭ·Ͱʹฦ৴Λ͢Δ͔ʯϧʔϧΛܾΊΔ - ྫ: ϦΫΤετ͔Βۈ຿೔׵ࢉͰ 24 ࣌ؒҎ಺ ໰୊: ϨϏϡʔͷظݶʹ͍ͭͯ࠷΋ʮΑ͘ͳ͍ʯߦಈ͸ʁ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ > ແࢹ͠ͳ͍
57. ϨϏϡʔϦΫΤετ΁ͷରԠ Α͍: - ظݶ಺ʹϨϏϡʔ͢Δ - ଞͷਓʹϦμΠϨΫτ͢Δ - ʮϨϏϡʔ͕೉͍͠ʯ ͱฦ৴͢Δ ͋·ΓΑ͘ͳ͍: ϨϏϡʔϦΫΤετΛແࢹ͢Δ ͔ͳΓΑ͘ͳ͍: ʮϨϏϡʔ͢Δʯͱݴ࣮ͬͯࡍʹ͸ϨϏϡʔ͠ͳ͍ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ > ແࢹ͠ͳ͍
58. ϨϏϡʔ͕೉͍͠ͱ͖ ๩͍͠ͳΒɺʮࠓ๩͍͠ʯͱฦ৴͢Δ ϨϏϡʔΠͷऔΕΔߦಈ: - ଞͷλεΫΛϒϩοΫ͠ͳ͍ͳΒɺ୯ʹϨϏϡʔΛ଴ͯΔ - ۓٸੑ͕ߴ͍ͳΒɺଞͷϨϏϡʔΞΛ୳ͤΔ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ > ແࢹ͠ͳ͍
59. ϦεϖΫτͱޮ཰ͷͨΊͷݪଇ - ϨϏϡʔϦΫΤετΛແࢹ͠ͳ͍ - ໰୊ͷ͋ΔϓϧϦΫΤετΛڋ൱͢Δ - కΊ੾ΓΛҙࣝ͗͢͠ͳ͍ - ʮఏҊʯҎ֎ͷํ๏Λߟྀ͢Δ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ > ໰୊ͷ͋ΔϓϧϦΫΤετͷڋ൱
60. ໰୊ͷ͋ΔϓϧϦΫΤετΛڋ൱͢Δ 1/2 ໰୊ͷଟ͍ϓϧϦΫΤετʹ࣌ؒΛ͔͚͗͢ͳ͍ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ > ໰୊ͷ͋ΔϓϧϦΫΤετͷڋ൱
61. ໰୊ͷ͋ΔϓϧϦΫΤετΛڋ൱͢Δ 2/2 ৄࡉΛಡΉΑΓ΋ɺϓϧϦΫΤετΛ࡞Γ௚ͯ͠΋Β͏ େ͖͗͢ΔϓϧϦΫΤετ: - খ͍͞ϓϧϦΫΤετʹ෼ׂͯ͠΋Β͏ ໨తɾલఏɾߏ଄ʹॏେͳؒҧ͍͕͋ΔϓϧϦΫΤετ: - εέϧτϯΫϥεΛ࡞ͬͯ΋ΒͬͨΓɺܰٞ͘࿦Λͯ͠ΈΔ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ > ໰୊ͷ͋ΔϓϧϦΫΤετͷڋ൱
62. ϦεϖΫτͱޮ཰ͷͨΊͷݪଇ - ϨϏϡʔϦΫΤετΛແࢹ͠ͳ͍ - ໰୊ͷ͋ΔϓϧϦΫΤετΛڋ൱͢Δ - కΊ੾ΓΛҙࣝ͗͢͠ͳ͍ - ʮఏҊʯҎ֎ͷํ๏Λߟྀ͢Δ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ > కΊ੾ΓΛҙࣝ͗͢͠ͳ͍
63. కΊ੾ΓΛҙࣝ͗͢͠ͳ͍ క੾͕͍ۙͱ͍͏ཧ༝ͰϨϏϡʔͷج४ΛԼ͛ͳ͍ - ٸ͍Ͱ͍Δ৔߹͸ͦͷཧ༝Λઆ໌ͯ͠΋Β͏ - ୅ସҊΛ୳͢ʢ࢓༷ͷ؆ུԽɾϦεέδϡʔϧʣ ྫ֎: ܦӦઓུͰॏཁͳػೳɾଟେͳӨڹͷ͋Δόά - ςετɾࠓޙͷվળํ๏ͷ߹ҙͱλεΫνέοτ͕ඞਢ - ϨϏϡʔΞ͸ϨϏϡʔΠΑΓৗʹྫྷ੩Ͱ͋Ε ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ > కΊ੾ΓΛҙࣝ͗͢͠ͳ͍
64. ϦεϖΫτͱޮ཰ͷͨΊͷݪଇ - ϨϏϡʔϦΫΤετΛແࢹ͠ͳ͍ - ໰୊ͷ͋ΔϓϧϦΫΤετΛڋ൱͢Δ - కΊ੾ΓΛߟྀ͠ͳ͍ - ʮఏҊʯҎ֎ͷํ๏Λߟྀ͢Δ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ > ఏҊҎ֎ͷίϝϯτ
65. ఏҊҎ֎ͷํ๏ ίϝϯτͷ಺༰͸ඞͣ͠΋ʮఏҊʯͰ͋Δඞཁ͸ͳ͍ ෛՙͷόϥϯεɾϨϏϡʔΠͷ੒௕: - Θ͔Βͳ͍৔߹ɺղಡʹ࣌ؒΛ͔͚ΔΑΓ΋࣭໰͢Δ - ໰୊ͷࢦఠʹͱͲΊɺݪҼ΍ৄࡉΛௐ΂ͯ΋Β͏ - ୯ͳΔʮ஌ࣝڞ༗ʯͷίϝϯτΛॻ͍ͯΈΔ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ > ఏҊҎ֎ͷίϝϯτ
66. ϨϏϡʔΞͱͯ͠ͷݪଇ: ·ͱΊ ϦεϖΫτΛͭͭ͠ɺޮ཰ͷΑ͍ϨϏϡʔΛ͢Δ - ϨϏϡʔ͕Ͱ͖ͳ͍৔߹͸ͦ͏ฦ৴͢Δ - ໰୊ͷ͋ΔϓϧϦΫΤετʹ࣌ؒΛ͔͚͗͢ͳ͍ - ޮ཰తͰޮՌతͳίϝϯτͱ͸Կ͔Λߟ͑Δ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ > ·ͱΊ
67. ࠓճͷ಺༰ ϨϏϡʔΠʢϨϏϡʔΛड͚Δଆʣ: - ϓϧϦΫΤετͷ࡞Γํ - ίϝϯτͷద༻ͷ࢓ํ ϨϏϡʔΞʢϨϏϡʔΛ͢Δଆʣ: - ݪଇ - ίϝϯτͷ಺༰ ίʔυϨϏϡʔ > ϨϏϡʔΞͷݪଇ
68. ίϝϯτͷ಺༰ 1/3 ؾ͍ͮͨ͜ͱͳΒԿͰ΋ʢղੳπʔϧ͕͋ΔͳΒͦΕʹ೚ͤΔʣ - ίʔσΟϯάελΠϧɾن໿ - ݴޠ΍ϓϥοτϑΥʔϜݻ༗ͷΠσΟΦϜ - ςετίʔυɾಈ࡞֬ೝͷ಺༰ - ϓϧϦΫΤετ΍ίϛοτͷେ͖͞ɾߏ଄ ίʔυϨϏϡʔ > ίϝϯτͷ಺༰
69. ίϝϯτͷ಺༰ 2/3 ؾ͍ͮͨ͜ͱͳΒԿͰ΋ - ϓϩδΣΫτ΍λεΫͷ໨తɾൣғ - ίʔυͷෳࡶੑͱୡ੒͍ͨ͜͠ͱͷόϥϯε - όά΍ηΩϡϦςΟ্ͷܽؕ - ύϑΥʔϚϯε΍ϑοτϓϦϯτͷมԽ ίʔυϨϏϡʔ > ίϝϯτͷ಺༰
70. ίϝϯτͷ಺༰ 3/3 ͜ͷϓϨθϯςʔγϣϯͰղઆͨ͠಺༰ ݪଇɾ໋໊ɾίϝϯτɾঢ়ଶɾؔ਺ɾґଘؔ܎ ίʔυϨϏϡʔ > ίϝϯτͷ಺༰
71. ίϝϯτͷέʔεελσΟ ؔ਺ʹԾҾ਺Λ௥Ճ͢Δྫ fun showPhotoView( photoData: PhotoData ) { if (!photoData.isValid) { return } ... ίʔυϨϏϡʔ > ίϝϯτͷ಺༰ > έʔεελσΟ
72. ίϝϯτͷέʔεελσΟ ؔ਺ʹԾҾ਺Λ௥Ճ͢Δྫ fun showPhotoView( photoData: PhotoData, isInEditScreen: Boolean ) { if (!photoData.isValid) { if (isInEditScreen) { showDialog() } return } ... ίʔυϨϏϡʔ > ίϝϯτͷ಺༰ > έʔεελσΟ
73. ίϝϯτͷέʔεελσΟ ໋໊: ʮͲ͜Ͱʯݺ͹ΕΔ͔ΑΓ΋ʮԿΛ͢Δ͔ʯΛઆ໌͢Δ isInEditScreen → showsDialogOnError ίʔυϨϏϡʔ > ίϝϯτͷ಺༰ > έʔεελσΟ
74. ίϝϯτͷέʔεελσΟ ໋໊: ʮͲ͜Ͱʯݺ͹ΕΔ͔ΑΓ΋ʮԿΛ͢Δ͔ʯΛઆ໌͢Δ fun showPhotoView( photoData: PhotoData, isInEditScreen: Boolean ) { if (!photoData.isValid) { if (isInEditScreen) { showDialog() } return } ... ίʔυϨϏϡʔ > ίϝϯτͷ಺༰ > έʔεελσΟ
75. ίϝϯτͷέʔεελσΟ ໋໊: ʮͲ͜Ͱʯݺ͹ΕΔ͔ΑΓ΋ʮԿΛ͢Δ͔ʯΛઆ໌͢Δ fun showPhotoView( photoData: PhotoData, showsDialogOnError: Boolean ) { if (!photoData.isValid) { if (showsDialogOnError) { showDialog() } return } ... ίʔυϨϏϡʔ > ίϝϯτͷ಺༰ > έʔεελσΟ
76. ίϝϯτͷέʔεελσΟ ґଘؔ܎: ਅِ஋ʹΑΔ੍ޚ݁߹Λղফ͢Δ - showDialog ͷݺͼग़͠Λநग़͢Δ - showPhotoView ͸੒൱Λਅِ஋Ͱฦ͢ ίʔυϨϏϡʔ > ίϝϯτͷ಺༰ > έʔεελσΟ
77. ίϝϯτͷέʔεελσΟ ґଘؔ܎: ਅِ஋ʹΑΔ੍ޚ݁߹Λղফ͢Δ fun showPhotoView( photoData: PhotoData, showsDialogOnError: Boolean ) { if (!photoData.isValid) { if (showsDialogOnError) { showDialog() } return } ... ίʔυϨϏϡʔ > ίϝϯτͷ಺༰ > έʔεελσΟ
78. ίϝϯτͷέʔεελσΟ ґଘؔ܎: ਅِ஋ʹΑΔ੍ޚ݁߹Λղফ͢Δ fun showPhotoView( photoData: PhotoData ): Boolean { if (!photoData.isValid) { return false } ... ίʔυϨϏϡʔ > ίϝϯτͷ಺༰ > έʔεελσΟ
79. ίϝϯτͷέʔεελσΟ ґଘؔ܎: ਅِ஋ʹΑΔ੍ޚ݁߹Λղফ͢Δ fun showPhotoView(photoData: PhotoData): Boolean { if (!photoData.isValid) { return false } ... return true ίʔυϨϏϡʔ > ίϝϯτͷ಺༰ > έʔεελσΟ
80. ίϝϯτͷέʔεελσΟ ґଘؔ܎: ਅِ஋ʹΑΔ੍ޚ݁߹Λղফ͢Δ ฤूը໘: val isViewShown = showPhotoView(...) if (!isViewShown) { showErrorDialog(...) } ͦΕҎ֎ͷը໘: showPhotoView(...) ίʔυϨϏϡʔ > ίϝϯτͷ಺༰ > έʔεελσΟ
81. ίϝϯτͷέʔεελσΟ ίϝϯτ: ؔ਺໊͕໭Γ஋Λઆ໌͍ͯ͠ͳ͍৔߹ɺίϝϯτΛॻ͘ /** * Shows a photo at the center if the given photo data is ..., * and returns true. * Otherwise, this returns false without showing the view. */ fun showPhotoView(photoData: PhotoData): Boolean { if (!photoData.isValid) { return false } ίʔυϨϏϡʔ > ίϝϯτͷ಺༰ > έʔεελσΟ
82. ίϝϯτͷ಺༰: ·ͱΊ ؾ͍ͮͨ͜ͱ͸ԿͰ΋ॻ͘ - ελΠϧ͔ΒϓϩδΣΫτͷ໨త·Ͱ - ҰճͷࢦఠͰ͸ؾ͔ͮͳ͍͜ͱ΋͋Δ ʮಡΈ΍͍͢ίʔυͷॻ͖ํʯΛಡΈฦͯ͠ΈΔ ίʔυϨϏϡʔ > ίϝϯτͷ಺༰ > ·ͱΊ
83. ·ͱΊ ಡΈ΍͢͞ͷվળʹϨϏϡʔ͸ॏཁ ϨϏϡʔΠ: - ϓϧϦΫΤετ͸খ͘͞ߏ଄తʹ࡞Δ - ϨϏϡʔίϝϯτͷҙਤΛཧղ͢Δ ϨϏϡʔΞ: - ϦεϖΫτͱޮ཰Λཱ྆͢Δ ίʔυϨϏϡʔ > ·ͱΊ