チーム開発とコードレビュー

東渋谷ITランチ勉強会
コードレビューについて考えたこと

Transcript

1. ώЄϭ樄咲; πЄϖϹϠϲЄ 2017.08.16 䩚亻ᨕIT϶Ѐώۣ䔶տ 1
2. πЄϖϹϠϲЄ ͭͼΔͯ͡Ҙ 2
3. πЄϖߝ搡΄ݻӤ ϗϹϐυ΄وํ ՛䯭ӧ㯪΄咲憎 ӧٍݳ΄咲憎 πЄϖϹϠϲЄ΄౮ຎᇔ 3
4. ͩ΄Ξ͜΀౮ຎᇔ Ψ஑Ρͩ;΅ ᛔړ΁;͹ͼ΅๕ Δ͚ͭ;᥺͞Δͯ 4
5. ͭͭ͡ 5
6. ౮ຎᇔ=ፓጱ ͽ΅΀͚ͽͯ 6
7. XP΄Ϥ϶μϓΰφ΁ͼ ᐏͫ΢ͼ͚Ρ πЄϖ΄وݶಅํ䰱΄ 䋚ఽ;ᆧ౮͢ፓጱ πЄϖϹϠϲЄ΄ፓጱ 7
8. πЄϖΨ᭗ͭ͵ ϤϺύμϕ揙ሠ 8
9. Ӟᛱጱ΁䮣㵗;ͭͼ πЄϖΨ䨗ͥ䁰ݳ πЄϖ΄姹֛ͽ͘Ρ ϤϺύμϕΎ΄揙ሠ͢ ࿢ΗΟ΢Δͯ 9
10. πЄϖΨ᭗ͭ͵ ϤϺύμϕ揙ሠΨ ਧ௔ጱɾ ጱ΄ 2數΁ΥͧΡ;Ҙ ਧᰁ 10
11. ਧ௔ጱ揙ሠ ϤϺύμϕπЄϖΎ΄揣ձఽ̵ߝ搡Ύ ΄റ࿢ஞ̵etc… πЄϖΨ᭗ͭ͵揙ሠ ጱ揙ሠ ϤϺύμϕπЄϖ΄ቘᥴ塅㾨̵托奞ቘ ᥴ΄Ⴎ̵ͫetc… ਧᰁ 11
12. ͫΟ΁揙ሠଶݳ ΁Ξ͹ͼώЄϭ ٖᒈͷ֖ᗝΨࢥ ړ气ͽᤒͯ;Ҙ 12
13. ਧ௔ɾਧᰁ΁ΞΡړ气 ਧ௔ጱ揙ሠ: ϤϺύμϕπЄϖΎ΄揣ձఽ̵ߝ搡Ύ΄റ࿢ஞ̵etc… ጱ揙ሠ: ϤϺύμϕπЄϖ΄ቘᥴ塅㾨̵托奞ቘᥴ΄Ⴎ̵ͫetc… ਧᰁ 13
14. ֢䮣ᘏ;ׁਂᘏͽ䯤౮ͫ΢͵ώЄϭ ਧ௔ጱ揙ሠΨ᯿憙ͯΡϮЀϝЄ͢੝΀͚ͩ;ͽ ώЄϭ;ͭͼᛔ૩όφμ塅㾨क़΁岉ޱ͢ᡀͥ ͵;͞ᵑਸ਼䦒ͽ͘͹ͼΘ֢䮣塅㾨͢ก嘦΁ړ櫝 ͫ΢ͼ͚Ρͩ;͢ग़͚ ͤͥͭΙͥͭ͵ώЄϭ ਍嬹ᘏ;ׁਂᘏͽ䯤౮ͫ΢͵ώЄϭ ጱ揙ሠΨ᯿憙ͯΡϮЀϝЄ͢੝΀͚ͩ;ͽ ώЄϭ;ͭͼቘమ؟΁ݻ͜͡䋚ᤈێ͢୧ͥ ΚΠͣ΢Ͱ΁ፓ΄ڹ΄՛Ԫ΁᭄Υ΢Ρͩ;͢ग़͚ ਧᰁ 14
15. ᛔᒈᘏ͢Ԇ;΀Π䯤౮ͫ΢͵ώЄϭ ώЄϭ;ͭͼ΄ቘమΨ̵㮆Ո΄䋚ᤈێͽඪ͞ ݳ͹ͼ͠Π̵ώЄϭ;ͭͼ΄֟ᤶΨ֢Πڊ̵ͭ ϤϺύμϕ㭅㮔റ࿢΁䦒樌Ψ;΢ͼ͚Ρͩ;͢ ग़͚ ΀ΗΟ͡΀ώЄϭ 15
16. ͤͥͭΙͥ; ΀ΗΟ͡΄ 晅͚;΅Ҙ 16
17. 䄜۸Ύ΄ 䌏䖕ᚆێ 17
18. ϤϺύμϕΨ 晁አͭͼ͚ͥ ; 18
19. ᵑਸ਼ ϹιτЄ۸ ֛ګ䄜ๅ ϡϩϐϕ etc... ፗᶎͯΡ䄜۸ 19
20. ؋ق΀ ϤϺύμϕ 晁㻑΄͵Η 20
21. ΀ΗΟ͡΀ ώЄϭ΄஠ ᥝ௔; 21
22. ᛔᒈᘏΨ͚͡΁ ᙙͼΡ͡Ҙ΄抓 氂͢͜Δ΢͵ 22
23. ᛔᒈᘏ ; πЄϖಅํఽ 23
24. ਧ௔ጱ揙ሠ; ጱ揙ሠ΄ 㪒ᒈΨͭͼ͚ΡϮЀϝЄΨ ᛔᒈᘏ;ͭͼړ气ͭΔͭ͵ ਧᰁ ਧ௔ጱ揙ሠ: ϤϺύμϕπЄϖΎ΄揣ձఽ̵ߝ搡Ύ΄റ࿢ஞ̵etc… ጱ揙ሠ: ϤϺύμϕπЄϖ΄ቘᥴ塅㾨̵托奞ቘᥴ΄Ⴎ̵ͫetc… ਧᰁ 24
25. 㪒ᒈͯΡ ͩ;΅ 25
26. ϤϺύμϕ΁樛昧ͯΡ πЄϖ;͚͜Ӟᅩ͢伋 ͵ͫ΢Ρ΄ͽ͘΢Ά ͳ΢Ψᛔړ΄Θ΄;ͭ ͼݻͣݳͩ͜; 26
27. ᛔᒈᘏ΁ᬪ ՞ͥͩ;΅ 27
28. ϤϺύμϕ΁ 樛昧ͯΡπЄϖ΄ ᛔ૩ಅํఽΨ 䝭य़ͯΡͩ; 28
29. ΀ΟΆώЄϭ ;ͭͼݐΠ奲 ΖΏͣ΅ 29
30. πЄϖ΄ ᛔ૩ಅํఽΨ ๋य़۸ͭ 30
31. ᛔᒈᘏͽ͘Π 姆ͧΡ͵Η΄ 厏हෆ㯪 31
32. 厏हෆ㯪΄1ͺ͢ πЄϖϹϠϲЄ 32
33. πЄϖ΄وݶಅํ䰱 33
34. ώЄϭ΄抑Θ͢πЄϖΨද࠺ͯΡ͵Η΁䄜 ๅΨے͞Ρ䰱ᴴΨ೮ͺ஠ᥝ͘͢ΠΔ̶ͯق 㹓ͯ͢Ώͼ΄πЄϖΨಅํͯΡ̵ͯ΀Υͷ ق㹓͢πЄϖ΁䌏ͭ揣ձΨ೮ͺͩ;΁΀Π Δ̶ͯͩ΄ದဩͽ΅̵樄咲ᘏ΅̵㮆Ո΄ πЄϖಅํᘏΨ䶲΁ͯΡͩ;΀ͥπЄϖΨ 䄜ๅͯΡͩ;͢ͽͣΔ̶ͯق㹓͢揣ձΨ೮ ͺ;͚͜Ԫ䋚΁ΞΠ̵πЄϖ͢抑΁Θಅํ ͫ΢Ͱ΁ႰԤᇫ䙪᩸ͩ͢Ρͩ;Θ͘ΠΔͱ Ω̶ ΞΠ୚አ https://www.ibm.com/developerworks/jp/java/library/j-xp/index.html XP ΄፥淑 34
35. ΀ΡΑͿ ړ͡ΠΚ͚ͯ 35
36. πЄϖ΁ϮЀϝЄ ق㹓͢䰱ᴴ;揣ձ Ψ೮ͺͩ;;ቘᥴ ͭΔͭ͵ 36
37. πЄϖϹϠϲЄ 37
38. πЄϖ΄وݶಅ ํ䰱Ψፓጱ;ͭ ͵䁰ݳ΄πЄϖ ϹϠϲЄ;΅Ҙ 38
39. ದ悬ϹϦϸΚϤϺύμϕቘᥴଶ΄૧吖 㮆Ո樌΄πЄϖಅํఽ΁晅͚΅͘΢Ϳ ݻͣݳ͜πЄϖ΅ݶͮΘ΄ͽ͘Ρ وݶಅํΨڹ൉;ͯΡ; ՜ᘏ֢͢౮ͭ͵πЄϖΨᛔړΘ晁አͯ Ρͩ;ͽ͘Ρ 晁አͯΡ;͚ͩ͜;΅̵ϮЀϓϗЀφɾ ᵑਸ਼䌏䖕ɾ༷ᥝ抍ก揣ձΨᔉଶ͘΢Ϳ೅ ͩ͜;ͽ͘Ρ 39
40. وݶಅํΨڹ൉΁ ͯΡ;͚ͩ͜;΅ 匍ࣁ΄ᇫ丆Ψᘍ䚅 ͭͺͺΘݶͮࢿ㭳 ΁͘͢Ρͩ; 40
41. ݶͮࢿ㭳΁͘͢Ρͩ; ͽஎ̸΁㮆Ո樌΄૧吖 ͢ईΗΟ΢ ᛔ૩΄πЄϖಅํఽ΅ 䓈͢͹ͼ͚ͥ 41
42. ;΅͚͞ 42
43. எ̸΁Ψ战ͫ΢Ρ΄ͽ͘΢ ΆΞ͚ͽͯ͢ ώЄϭΚտᐒ͡Ο΅ΞΠ෱ ͚揙ሠ͢࿢ΗΟ΢Ρͩ;Θ ग़͚;ఽͮͼ͚Δͯ 43
44. ٚଶπЄϖ΄وݶ ಅํ䰱Ψፓጱ;ͭ ͵䁰ݳ΄πЄϖϹ ϠϲЄ;΅Ҙ 44
45. ᛔړͩ͢΄πЄϖΨ晁 አͯΡ΀ΟͿͯ͜Ρ ͡ҘΨᘍ͞Ρ͵Η΄䱛 տ൉׀;姅姆௔ ΄ςϪЄϕ 45
46. ΀΄ͽ匍ࣁ΄ᇫ䙪ҁದ悬Ϲ ϦϸΚϤϺύμϕቘᥴଶ҂ ͽ̵ᛔړ͢晁አͯΡ΄΀ ΟҘ;͚͜憶ᅩͽझ̸;ወ 㺔Ψಭͨͧ͡͵Π̵ቘᥴͯ Ρ͵Η΄ᤈ傶Ψവ䅏ͭ͵͚ 46
47. Δ;Η 47
48. 1. ϤϺύμϕ晁አ΅䄜۸͢֎̵͜ͳ΢΁͜Δͥ䌏䖕ͯΡ ΄΅΀ΗΟ͡΀ώЄϭ 2. ΀ΗΟ͡΀ώЄϭͽ͘Ρ͵Η΁̵ᛔᒈᘏ΁͚͡΁ᬪ՞ ͥ΄͢͡抓氂;΀Ρ 3. ᛔᒈᘏ΁ᬪ՞̵ͣ姘೮ͯΡ͵Η΁πЄϖ΄ᛔ૩ಅํఽ Ψ䝭य़ͭͼ͚ͣ͵͚ 4. πЄϖ΄ᛔ૩ಅํఽ΄䝭य़ςϪЄϕΨ̵՛奲Ε΁ͭ͵ Θ΄͢πЄϖϹϠϲЄ 5. ΀΄ͽπЄϖϹϠϲЄ΄ፓጱ΅̵πЄϖ΄وݶಅํ䰱 ΄䋚ఽ;ᆧ౮;഍͞͵ πЄϖϹϠϲЄ΄ᘍ੊奾ຎ 48
49. ͭ͠Δ͚ 49