Upgrade to Pro — share decks privately, control downloads, hide ads and more …

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

662c3351c86054ffce430e9e2a3699aa?s=47 nozayasu
August 23, 2017

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

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

662c3351c86054ffce430e9e2a3699aa?s=128

nozayasu

August 23, 2017
Tweet

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