XP祭り2020 #xpjug で発表した内容です。
コードレビューが教育に良い理由と、 更に効果を高めるTipsを紹介しました。
ίʔυϨϏϡʔࠈ͔Βൈ͚ग़ͨ͢ΊͷϖΞϓϩҭ๏ʙֶशՊֶͷࢹ͔Βʙڭҭ৺ཧֶΛֶͿձ ޥ ޫىʢӬγεςϜϚωδϝϯτʣ91ࡇΓ
View Slide
ࣗݾհ•ޥޫى !LBXBHPJL• ΤϯδχΞˠεΫϥϜϚελʔ• ΞδϟΠϧίʔνˍ৫։ൃ• ύʔιφϧίʔνՊֶͷྗΛआΓͯɺָ͘͠ಇ͓͘ख͍͠·͢• मɿίʔυϨϏϡʔΛ༻ֶ͍ͨशࢧԉ• ϖΞϓϩΛҎ্
ΞδΣϯμ•ϖΞϓϩ͕ҭʹྑ͍ཧ༝•ޮՌΛߋʹߴΊΔ5JQT
ίʔυϨϏϡʔͬͯΔਓʂ
ίʔυϨϏϡʔͰҭͯΔग़དྷ্͕ͬͨͷ͕Α͔͔ͬͨɺѱ͔ͬͨͷ͔͔Δಉ࣌ؒ͡ʹಇ͔ͳ͍͍ͯ͘
͜Μͳͷͣʔͬͱଓ͍͍ͯΔͷݟ͔͚ͨ͜ͱ͋Γ·͢ʁ
͍ͭʹͳͬͨΒͰ͖ΔΑ͏ʹͳΔͷʂʁ·ͨɺ΄ͱΜͲશ෦Γͩ͠ʙઌഐʹਵ໎͔͚ͯ͠·ͬͯΔ
ؒҧ͑ͨΓํʹؾ͚ͮແ͍ίʔυϨϏϡʔͰҭͯΔਖ਼͍͠Γํ͕͔Βͳ͍Γ͠ʹͳΔग़དྷ্͕ͬͨͷ͕Α͔͔ͬͨɺѱ͔ͬͨͷ͔͔Δಉ࣌ؒ͡ʹಇ͔ͳ͍͍ͯ͘Ͳ͏ͯ͠ɺͩΊͳϓϩάϥϜ͕Ͱ͖͔͔ͨΒͳ͍͍ͬͯͨ͜ͱͷҰ෦·ͨશ෦͕ϜμʹͳΔϓϩάϥϛϯά͕ݏ͍ʹͳΔϞνϕʔγϣϯԼࣗྗͰղܾͰ͖ͳ͍ग़དྷ্͕Δͷ͕͍Կಉࣦ͡ഊΛ͢Δ
ؒҧ͑ͨΓํʹؾ͚ͮແ͍ίʔυϨϏϡʔͰҭͯΔਖ਼͍͠Γํ͕͔Βͳ͍Γ͠ʹͳΔग़དྷ্͕ͬͨͷ͕Α͔͔ͬͨɺѱ͔ͬͨͷ͔͔Δಉ࣌ؒ͡ʹಇ͔ͳ͍͍ͯ͘Ͳ͏ͯ͠ɺͩΊͳϓϩάϥϜ͕Ͱ͖͔͔ͨΒͳ͍͍ͬͯͨ͜ͱͷҰ෦·ͨશ෦͕ϜμʹͳΔϓϩάϥϛϯά͕ݏ͍ʹͳΔϞνϕʔγϣϯԼࣗྗͰղܾͰ͖ͳ͍ग़དྷ্͕Δͷ͕͍Կಉࣦ͡ഊΛ͢ΔϖΞϓϩͰҭͯΔΓ͠ʹͳΒͳ͍ਖ਼͍͠Γํ͕͔Δؒҧ͑ͨΓํʹؾ͚ͮΔ
গ͚ͩ͠ཧͷ͠
ెఋੑ• ࢣঊͱఋࢠͰֶͿελΠϧ• ํʹఋࢠೖΓͯ͠ɺٕज़Λڭ͑ͯΒ͏• ษڧΛڭ͑Δͱ͖• ՈఉڭࢣελΠϧ• ͜ͷํ๏ߋʹਐԽ͍ͯͯ͠ʮೝతెఋ੍ʯͱݺΕ͍ͯ·͢
ెఋੑ• ࢣঊͱఋࢠͰֶͿελΠϧ• ํʹఋࢠೖΓͯ͠ɺٕज़Λڭ͑ͯΒ͏• ษڧΛڭ͑Δͱ͖• ՈఉڭࢣελΠϧ• ͜ͷํ๏ߋʹਐԽ͍ͯͯ͠ʮೝతెఋ੍ʯͱݺΕ͍ͯ·͢ग़དྷ্͕ͬͨΒݟͤͯʁ൷ධ͢Δ͔Β
ెఋੑ• ࢣঊͱఋࢠͰֶͿελΠϧ• ํʹఋࢠೖΓͯ͠ɺٕज़Λڭ͑ͯΒ͏• ษڧΛڭ͑Δͱ͖• ՈఉڭࢣελΠϧ• ͜ͷํ๏ߋʹਐԽ͍ͯͯ͠ʮೝతెఋ੍ʯͬͯͷ͕͋Γ·͢"MMBO$PMMJOT $PHOJUJWF"QQSFOUJDFTIJQ5FBDIJOHUIF$SBGUPG3FBEJOH 8SJUJOH BOE.BUIFNBUJDT5FDIOJDBM3FQPSU/Pग़དྷ্͕ͬͨΒݟͤͯʁ൷ධ͢Δ͔Β
ଟ͘ͷจݙͰɺհ͞Ε͍ͯ·͢https://booth.pm/ja/items/2368560
ೝతెఋ੍ͷεςοϓͰ͖ͳ͍͜ͱΛͰ͖ΔΑ͏ʹ͢Δɻ՝Λఏڙ͢Δɻ͓खຊΛݟͤΔ؍͠ϑΟʔυόοΫ࣮͠ࢪΛॿ͚Δࣗͷڵຯ͕࣋ͯΔ෦Λ୳ࣗ͢ͱ͓खຊΛൺֱͯ͠ߟ͑ΔࣝࢥߟΛઆ໌Ͱ͖ΔΑ͏ʹ͢Δ
ϖΞϓϩ͕ҭʹྑ͍ཧ༝Ͱ͖ͳ͍͜ͱΛͰ͖ΔΑ͏ʹ͢Δɻ՝Λఏڙ͢Δɻ͓खຊΛݟͤΔ؍͠ϑΟʔυόοΫ࣮͠ࢪΛॿ͚Δࣗͷڵຯ͕࣋ͯΔ෦Λ୳ࣗ͢ͱ͓खຊΛൺֱͯ͠ߟ͑ΔࣝࢥߟΛઆ໌Ͱ͖ΔΑ͏ʹ͢Δ
࠷ॳ͔Βɺεςοϓ͕ຬͨ͞Ε͍ͯΔϖΞϓϩ͕ҭʹྑ͍ཧ༝Ͱ͖ͳ͍͜ͱΛͰ͖ΔΑ͏ʹ͢Δɻ՝Λఏڙ͢Δɻ͓खຊΛݟͤΔ؍͠ϑΟʔυόοΫ࣮͠ࢪΛॿ͚Δࣗͷڵຯ͕࣋ͯΔ෦Λ୳ࣗ͢ͱ͓खຊΛൺֱͯ͠ߟ͑ΔࣝࢥߟΛઆ໌Ͱ͖ΔΑ͏ʹ͢Δ
ޮՌΛߋʹߴΊΔ5JQTͷʹΓ·͠ΐ͏
ޮՌΛߋʹߴΊΔ5JQTͰ͖ͳ͍͜ͱΛͰ͖ΔΑ͏ʹ͢Δɻ՝Λఏڙ͢Δɻ͓खຊΛݟͤΔ؍͠ϑΟʔυόοΫ࣮͠ࢪΛॿ͚Δࣗͷڵຯ͕࣋ͯΔ෦Λ୳ࣗ͢ͱ͓खຊΛൺֱͯ͠ߟ͑ΔࣝࢥߟΛઆ໌Ͱ͖ΔΑ͏ʹ͢Δ
ࣦഊΛ͔ͯ͠Β͓खຊΛݟͤΔ• ؆୯ʹઆ໌͢Δͱʁ• ·ͣɺޙഐʹઌʹͬͯΒ͍·͠ΐ͏• Ͱ͖ͳ͔ͬͨΒɺΘΓʹ͓खຊΛݟͤ·͠ΐ͏• ྫ͑ʁ• ػೳΛͭՃ͢Δͱͯ͠ʮͰ͖Δͱ͜Ζ·ͰͬͯΈͯʯͱ͓ئ͍͢Δ• ʮ͡Ό͋ɺ͓खຊݟͤΔͶʯͱɺ͍ͬͯΔͱ͜ΖΛݟͤΔ• ʮࣦഊͤ͞ΔݏͳઌഐʯͬͯࢥΘΕͨ͘ͳ͍ͷͰ͋Εɺઆ໌Λɻ• ͳͥɺྑ͍ͷʁ• ͕ࣗԿ͕Ͱ͖͍ͯͳ͍͔ཧղͰ͖ΔͷͰɺͲ͜ΛݟͨΒ͍͍͔Θ͔Δ• Ͱ͖ͳ͍ͱ͜Ζɺࣗͱҧ͏ͱ͜ΖʹࢹͰ͖Δ͓खຊΛݟͤΔجૅྗ61
ॿ͚ͨ͜ͱϦετΛ࡞Δ• ؆୯ʹઆ໌͢Δͱʁ• ޙഐ͕Ͱ͖ͳ͔ͬͨΓɺؒҧ͑ͨΒॿ͚ͯ͋͛ͯɺ༰ΛϦετʹՃ• ҰॹʹαϙʔτݮΒ͍͚ͯ͠ΔΑ͏ʹೝࣝ߹ΘͤΛ͢Δ• ྫ͑ʁ• ςΩετϑΝΠϧͰɺॿ͚ͨ͜ͱϦετͷϑΝΠϧΛ࡞Δ• (JUͷૢ࡞Ͱɺ༨ܭͳϑΝΠϧΛՃ͍ͯͨ͠ΒϦετʹೖΕΔ• ϓϩάϥϜͰɺྫ֎ॲཧʹൈ͚͕͋ͬͨΒϦετʹೖΕΔ• ͳͥɺྑ͍ͷʁ• ઌഐɺͰ͖Δ͚ͩॿ͚Δ෦ΛݮΒͯ͠ҰਓલʹͳͬͯΒ͍͍ͨ• ޙഐɺԿ͕Ͱ͖ΔΑ͏ʹͳΕ͍͍͔໌֬ʹͳΔ• ͳ͘ͳΒͳ͍߲͕͋Εɺڭ͑ํΛม͑ΔͳͲɺରࡦΛߟ͑ΒΕΔͰ͖ͳ͍͜ͱΛͰ͖ΔΑ͏ʹ͢Δɻ՝Λఏڙ͢Δɻجૅྗ61
ग़དྷͳ͍͜ͱΛΓӽ͑Δ՝• ؆୯ʹઆ໌͢Δͱʁ• ޙഐ͕Ͱ͖ͳ͍͜ͱ͕͋Εɺूதͯ͠औΓΊΔ՝Λఏڙ͢Δ• ྫ͑ʁ• (JUͷૢ࡞ͰɺϒϥϯνΛͲ͔͜ΒͬͨͷӨڹΛཧղ͍ͯ͠ͳ͔ͬͨ• (JUͷૢ࡞ͰɺϒϥϯνͷΓସ͑࣌ʹૢ࡞͕Θ͔Βͣखؒऔ͍ͬͯͨ• 8FCαʔϏεʰ-FBSO(JU#SBODIJOHʱΛհ͢Δ• ͳͥɺྑ͍ͷʁ• Ͱ͖ͳ͍͜ͱɺࢥ͏ଘࣦഊͰ͖Δڥͷํ্͕ୡ͢Δ• ࣦഊ͠ͳ͍Ͱ͓͜͏ͱߟ͑͗͢Δͱɺ҆શࡦͰίϐϖʹ͢Δ• ྗ͠ͳ͍ͱɺίπΛ͔ͭΉલʹΕͯ͠·͏• ʹճͰ͖Δ͜ͱ• ݄ʹճ͚ͩΔ͜ͱͰ͖ͳ͍͜ͱΛͰ͖ΔΑ͏ʹ͢Δɻ՝Λఏڙ͢Δɻجૅྗ61
ग़དྷͳ͍͜ͱΛΓӽ͑Δ՝Λ• ؆୯ʹઆ໌͢Δͱʁ• ޙഐ͕Ͱ͖ͳ͍͜ͱ͕͋Εɺूதͯ͠औΓΊΔ՝Λఏڙ͢Δ• ྫ͑ʁ• (JUͷૢ࡞ͰɺϒϥϯνΛͲ͔͜ΒͬͨͷӨڹΛཧղ͍ͯ͠ͳ͔ͬͨ• (JUͷૢ࡞ͰɺϒϥϯνͷΓସ͑࣌ʹૢ࡞͕Θ͔Βͣखؒऔ͍ͬͯͨ• 8FCαʔϏεʰ-FBSO(JU#SBODIJOHʱΛհ͢Δ• ͳͥɺྑ͍ͷʁ• Ͱ͖ͳ͍͜ͱɺࢥ͏ଘࣦഊͰ͖Δڥͷํ্͕ୡ͢Δ• ࣦഊ͠ͳ͍Ͱ͓͜͏ͱߟ͑͗͢Δͱɺ҆શࡦͰίϐϖʹ͢Δ• ྗ͠ͳ͍ͱɺίπΛ͔ͭΉલʹΕͯ͠·͏• ʹճͰ͖Δ͜ͱ• ݄ʹճ͚ͩΔ͜ͱͰ͖ͳ͍͜ͱΛͰ͖ΔΑ͏ʹ͢Δɻ՝Λఏڙ͢Δɻ
ग़དྷͳ͍͜ͱΛΓӽ͑Δ՝• ؆୯ʹઆ໌͢Δͱʁ• ޙഐ͕Ͱ͖ͳ͍͜ͱ͕͋Εɺूதͯ͠औΓΊΔ՝Λఏڙ͢Δ• ྫ͑ʁ• (JUͷૢ࡞ͰɺϒϥϯνΛͲ͔͜ΒͬͨͷӨڹΛཧղ͍ͯ͠ͳ͔ͬͨ• (JUͷૢ࡞ͰɺϒϥϯνͷΓସ͑࣌ʹૢ࡞͕Θ͔Βͣखؒऔ͍ͬͯͨ• 8FCαʔϏεʰ-FBSO(JU#SBODIJOHʱΛհ͢Δ• ͳͥɺྑ͍ͷʁ• Ͱ͖ͳ͍͜ͱɺࢥ͏ଘࣦഊͰ͖Δڥͷํ্͕ୡ͢Δ• ࣦഊ͠ͳ͍Ͱ͓͜͏ͱߟ͑͗͢Δͱɺ҆શࡦͰίϐϖʹ͢Δ• ूத͠ͳ͍ͱɺίπΛ͔ͭΉલʹΕͯ͠·͏• ʹճͰ͖Δ͜ͱ• ݄ʹճ͚ͩΔ͜ͱͰ͖ͳ͍͜ͱΛͰ͖ΔΑ͏ʹ͢Δɻ՝Λఏڙ͢Δɻجૅྗ61
બͷཧ༝ΛਘͶΔ• ؆୯ʹઆ໌͢Δͱʁ• ޙഐ͕બΜͩํ๏ʹ͍ͭͯɺͳͥબΜͩͷ͔͏• ଞͷํ๏Ͱμϝͳͷ͔ɺީิΛڍ͛ͯҙݟΛฉ͘• ྫ͑ʁ• ޙഐ͕ɺσʔλͷอଘͷͨΊʹ.Z42-ΛఏҊ͢Δ• ઌഐɺʮͲ͏ͯ͠3FEJTɺ42-JUFͰͳ͍ͷʯͱཧ༝Λฉ͍ͯΈΔ• ઌഐɺ͔͍ͬͯͯҙਤతʹฉ͍ͯΈΔ͜ͱͰޙഐͷཧղΛਂΊΔ• ͳͥɺྑ͍ͷʁ• ࣗͷબΜͩબࢶͷಛΛઆ໌͢Δ͜ͱͰɺཧղ͕ਂ·Δ• ผͷ໘ʹૺ۰ͨ͠ͱ͖ʹɺԠ༻͢Δ͜ͱ͕Ͱ͖ΔࣝࢥߟΛઆ໌Ͱ͖ΔΑ͏ʹ͢ΔԠ༻ྗ61
·ͱΊ• ϖΞϓϩ͕ҭʹྑ͍ཧ༝• ֶशՊֶͰྑ͍ͱ͞ΕΔೝతెఋ੍ͷ࠷ॳͷεςοϓ͕ຬͨ͞Ε͍ͯΔ• ޮՌΛߋʹߴΊΔ5JQT• ࣦഊΛ͔ͯ͠Β͓खຊΛݟͤΔ• ॿ͚ͨ͜ͱϦετΛ࡞ͬͯೝࣝΛ߹ΘͤΔ• ग़དྷͳ͍͜ͱΛΓӽ͑Δ՝Λఏڙ͢Δ• બͷཧ༝ΛਘͶΔ
• ͱɺ͍͔ͭ͘ͷ5JQTΛհ͖ͯ͠·ͨ͠ɻ
•͔͠͠• ϖΞϓϩͨ͘͠ͳ͍• ͜Ε·Ͱͷݫ͍͠ϑΟʔυόοΫͰؔੑ͕ྑ͘ͳ͍• ઌഐ͕ޙഐʹରͯ͠ݫ͍͠• ޙഐͷֶशʹର͢ΔΔؾ͕͍• ϓϩάϥϛϯάҎ֎ͷࣄͰࠔ͍ͬͯΔ• ͳͲͳͲ• ͏·͍͔͘ͳ͍͜ͱଟ͍ͱࢥ͍·͢• ͦΜͳͱ͖͝૬ஊ͍ͩ͘͞• ৬ʹݺΜͰ͍͚ͨͩΕ• ՝Λݟ͚ͭͯվળͷ͓ख͍͠·͢
͏·͍͔͘ͳ͍߹• ͬͱใΛಘ͍ͨ• 5XJUUFSɿIUUQTUXJUUFSDPNLBXBHPJL• 'BDFCPPLɿ IUUQTXXXGBDFCPPLDPNLBXBHPJ• $POOQBTTɿIUUQTDPOOQBTTDPNVTFSLBXBHPJ• ૬ஊ͍ͨ͠• %JTDPSEɿLBXBHPJ• ϝʔϧɿ MFBSOJOHQSPDFTTVQEBUF!HNBJMDPN• ͍߹ΘͤϑΥʔϜɿIUUQTGPSNTHMFR.E'9(S(8$LFNZ5XJUUFS 'BDFCPPL͍߹ΘͤϑΥʔϜ
kawagoi
optimisuke
nayuts
shibuiwilliam
mottox2
onecareer_tech
smt7174
kojira
lwug
aki_iinuma
oracle4engineer
ouchi2501
cimadai
shlominoach
marktimemedia
phodgson
akosma
skipperchong
hursman
notwaldorf
jakevdp
destraynor
geoffreycrofte
reverentgeek
chriscoyier