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

Otemachi.go #1 CodeReviewComments

Otemachi.go #1 CodeReviewComments

大手町.go #1 にてCodeReviewCommentsについて発表した際のスライドです。
Goのコードレビュー時によくみかけられるコメントがリスト化されたCodeReviewCommentsについて、いくつかピックアップして説明しています。

Otemachi.go #1
https://otemachi-go.connpass.com/event/122418/

CodeReviewComments
https://github.com/golang/go/wiki/CodeReviewComments

Ryo Nonoyama

March 20, 2019
Tweet

More Decks by Ryo Nonoyama

Other Decks in Technology

Transcript

  1.   • 3ZP/POPZBNB • -JRVJE *OD JOେखொϏϧ • 3VCZ3BJMT

    d • 5ZQF4DSJQU7VFKT d • (PMBOH d
  2.       • "5PVSPG(P • IUUQTUPVSHPMBOHPSH

    • )PXUP8SJUF(P$PEF • IUUQTHPMBOHPSHEPDDPEFIUNM • &GGFDUJWF(P • IUUQTHPMBOHPSHEPDFGGFDUJWF@HPIUNM • ͳͲͳͲͦͷଞଟ਺
  3.       • (PMBOHͷίʔυϨϏϡʔͰΑ ͘ݟ͔͚ΒΕΔίϝϯτू •

    &GGFDUJWF(PΛิ͏Α͏ͳυ Ωϡϝϯτͱͯ͠ଘࡏ͢Δ • ن໿͚ͩͰͳ͘ɺϦʔμϒϧ ίʔυʹ௨͡ΔΑ͏ͳ߲໨΋ ͋Δ IUUQTHJUIVCDPNHPMBOHHPXJLJ$PEF3FWJFX$PNNFOUT
  4.       • શ෦Ͱ໿߲໨ • ͦͷத͔Β͍͔ͭ͘ϐοΫΞοϓͯ͠ಡΜͰΈ·ͨ͠

    • *OJUJBMJTNT • )BOEMF&SSPST • %FDMBSJOH&NQUZ4MJDFT • 3FDFJWFS5ZQF
  5. • ม਺໊ͷதʹ͋Δ಄ࣈޠ΍ུޠʢz63-z l*%zͳͲʣͷදه ʹ͸ɺҰ؏ੑΛ΋ͨͤΔ େจࣈখจࣈ͍ͣΕ͔Ͱ౷Ұ • º *E VTFS*E •

    ˓ *% VTFS*% • HPMJOU͕ࢦఠͯ͘͠ΕΔ಄ࣈޠུޠ͸ޠऑ  • ϓϩδΣΫτ಺Ͱɺ಄ࣈޠུޠΛ࢖͍ͬͯΔ৔߹ʹ͸͜ͷ ϧʔϧͰ౷ҰΛ͢Δ • ࠓͷϓϩδΣΫτ಺ͩͱl,:$zLOPXZPVSDVTUPNFSͱ͔͋Γ·͢ IUUQTHJUIVCDPNHPMBOHMJOUCMPCNBTUFSMJOUHP-
  6. . 1

  7. 1-. 2 9 4 • ϙΠϯλϨγʔόΛ࢖͏ • ஋Ϩγʔόͷ৔߹͸ɺϝιουΛݺͼग़͢ͱ͖ʹม਺ͷ஋Λί ϐʔͯ͠ɺ৽ͨͳ஋Λϝιουʹ౉͢ •

    ৽ͨͳ஋͕ϝιου಺Ͱมߋ͞Εͨͱͯ͠΋ɺݩͱͳͬͨ஋͸ มߋ͞Εͣ໨తΛՌͨͤͳ͍ͨΊɺ͜ͷέʔεͰ͸Ϩγʔό͸ ϙΠϯλͱ͢Δɻ
  8. 0-3. 42 / • ஋ϨγʔόΛ࢖͏ • NBQ͸࿈૝഑ྻɺGVOD͸ؔ਺஋ GVODUJPOWBMVF ɺνϟω ϧ͸ΰϧʔνϯؒͰϝοηʔδΛૹΓ߹͏ࡍʹ࢖ΘΕΔ

    • ͜ΕΒ͸͢΂ͯࢀরܕͰ͋ΔͨΊɺ࣮ମ΁ͷϙΠϯλΛ࣋ͬ ͍ͯΔͱߟ͑ͯྑ͍ • ͦͷͨΊɺ͜ΕΒʹϙΠϯλϨγʔόΛ࢖͏ͱɺϙΠϯλ΁ ͷϙΠϯλΛ౉͍ͯ͠ΔΑ͏ͳঢ়ଶͱͳΔͨΊϙΠϯλࢀর ͷෛՙ͕͔͔ͬͯ͠·͏ɻ • ΑͬͯɺϨγʔό͕ࢀরܕͰ͋Δ৔߹͸ϙΠϯλΛ࢖͏΂͖ Ͱ͸ͳ͍
  9.      • $PEF3FWJFX$PNNFOUT • IUUQTHJUIVCDPNHPMBOHHPXJLJ$PEF3FWJFX$PNNFOUT •

    $PEF3FWJFX$PNNFOUT ೔ຊޠ༁ • IUUQTRJJUBDPNLOTIJUFNTCCDED • 2JJUB(PͰYYYͷϙΠϯλΛऔ͍ͬͯΔϓϩάϥϜ͸͍͍ͩͨશ෦ؒҧ͍ͬͯΔ • IUUQTRJJUBDPNSVJVJUFNTFBBFGGEDDE • ܕͱϙΠϯλΛ෼͔Γ΍͘͢ղઆɻ஋ܕͱࢀরܕʹ͍ͭͯ΋ղઆ͍ͯ͠·͢ɻ • IUUQOBTVTUIBUFOBCMPHDPNFOUSZ • 2JJUBۭͷ4MJDFͷએݴํ๏ͷҧ͍ • IUUQTRJJUBDPNCJPIVOTJUFNTEBB • 2JJUB(P ύϑΥʔϚϯενϡʔχϯά • IUUQTRJJUBDPNOBPJOBJUFNTEEEGBCGCG ͋Γ͕ͱ͏͍͟͝·ͨ͠ʂ