プロを目指すRailsエンジニアのための公開コードレビュー / #railsdm

Rails Developers Meetup #4の発表資料です。

全員分のコードレビュー動画はこちらにあります。

【前編】全部見ます！プロを目指すRailsエンジニアのための公開コードレビュー #railsdm
https://youtu.be/xZk4f4PPUhU

【後編】全部見ます！プロを目指すRailsエンジニアのための公開コードレビュー #railsdm
https://youtu.be/yDxg0sXHXOo

勉強会の告知ページはこちらです。

Rails Developers Meetup #4 リモート会場 - connpass
https://rails-developers-meetup.connpass.com/event/63374/

Transcript

1. ϓϩΛ໨ࢦ͢3BJMTΤϯδχΞͷͨΊͷ
   ެ։ίʔυϨϏϡʔ
   גࣜձࣾιχοΫΨʔσϯ
   ҏ౻
   ३Ұ 
   3BJMT
   %FWFMPQFST
   .FFUVQ
   
2. ABOUT ME ҏ౻
   ३Ұ
   ➤ େࡕ෎๛தࢢग़਎
   ➤ ฌݿݝ੢࿬ࢢࡏॅ
   ➤ גࣜձࣾιχοΫΨʔσϯ
   ➤ 3BJMTϓϩάϥϚ
   ➤ ϦϞʔτϫʔΧʔ
   ➤ Ϊλʔ޷͖
   ➤ ࠓ೔.BUFVT
   "TBUP؍͖ͯͨʂ
3. ͜Μͳͱ͜ΖʹॅΜͰ·͢
4. Έͳ͞Μ͸ࠓͲ͜ʹʁ #railsdm ͰπΠʔτʂ
5. ͪ͜Βͷ࢓ࣄ෦԰͔Β์ૹதͰ͢
6. πΠʔτΛνΣοΫʂ IUUQTUXJUUFSDPNIBTIUBHSBJMTEN TSDIBTI
7. ޷ධൃചதʂ &WFSZEBZ
   3BJMT
   
 34QFDʹΑΔ3BJMTςετೖ໳
   ➤ 3BJMTͰςετΛॻ͘ͳΒ͜Εʂ
   ➤ ߋ৽͕͋Ε͹ແྉΞοϓσʔτ
   ➤ 3BJMT
   ൛΋຋༁͢Δ༧ఆ
   ➤ &16#ɺ.PCJɺ1%'ʹରԠ
   ➤ -FBOQVCͷ8FCαΠτͰൢചத
8. ٕज़ॻ͕ग़·͢ʂ ʮϓϩΛ໨ࢦ͢ਓͷͨΊͷ3VCZೖ໳ʯ
   ➤ ҏ౻
   ३Ұ
   ஶ
   ➤ ೥݄ൃച
   ➤ #มܕ൑
   ϖʔδ
   ➤ ग़൛ࣾɿٕज़ධ࿦ࣾ
   ˞্ه಺༰͸ݱ࣌఺ͷ༧ఆͰ͢ɻ
   ͲΜͳຊͳͷʁ
   ➤ 2JJUB౳Ͱఆධͷ͋Δஸೡͳղઆ
   ➤ 5%%ˍνϡʔτϦΞϧܗࣜͷྫ୊
   ➤ ։ൃͷݱ৔Ͱ໾ཱͭ஌͕ࣝຬࡌ ϓϩΛ໨ࢦ͢ਓͷͨΊͷ 3VCZೖ໳ ҏ౻
   ३Ұ
   ஶ ٕज़ධ࿦ࣾ
9. ࠓճͷ͓୊͸͜ͷຊͷྫ୊ΛϞσΟϑΝΠ ʮୈ̓ষ
   Ϋϥεͷ࡞੒Λཧղ͢ΔʯΑΓൈਮ
10. ຊ೔ͷΞδΣϯμ
11. ຊ೔ͷΞδΣϯμ ➤ ެ։ίʔυϨϏϡʔɾͦͷ̍
    ➤ ެ։ίʔυϨϏϡʔɾͦͷ̎
    ➤ ग़୊ऀͷղ౴ྫ
    ➤ ؾʹͳͬͨϙΠϯτ͋Ε͜Ε
    ➤ ·ͱΊ
    ➤ ࣭ٙԠ౴ʢ͕࣌ؒ͋Ε͹ʣ
    ➤ ࣭໰΍͝ҙݟ͸
    SBJMTEN
    Λ෇͚ͯπΠʔτ
    ➤ ΈΜͳͰʮྑ͍ίʔυʯʹ͍ͭͯҰॹʹߟ͑ͯΈ·͠ΐ͏ʂ
12. ίʔυϨϏϡʔຊ໨
13. ̍ຊ໨ͷίʔυϨϏϡʔ͸ͪ͜Βʂ IUUQTHJUIVCDPN+VOJDIJ*UPUSBJOUJDLFUSBJMTQVMM
14. ίʔυϨϏϡʔຊ໨
15. ̎ຊ໨ͷίʔυϨϏϡʔ͸ͪ͜Βʂ IUUQTHJUIVCDPN+VOJDIJ*UPUSBJOUJDLFUSBJMTQVMM
16. ग़୊ऀͷղ౴ྫ
17. ๻ͷղ౴ྫͰ͢ IUUQTHJUIVCDPN+VOJDIJ*UPUSBJOUJDLFUSBJMTDPNQBSFNBTUFSBOTXFS
18. ؾʹͳͬͨ఺͋Ε͜Ε
19. ࢖༻ࡁΈͷ੾ූʁ
20. ࢖༻ࡁΈͷ੾ූΛද͢ίʔυ 1. OJMͰͳ͚Ε͹ਅ
    ➤ redirect_to ʙ if @ticket.exited_gate 2. QSFTFOU Ͱʮʙ͕ଘࡏ͍ͯ͠Ε͹ʯΛ໌ࣔ
    ➤ redirect_to ʙ if @ticket.exited_gate.present? 3. ΠϯελϯεϝιουΛ࡞ͬͯந৅Խ
    ➤ redirect_to ʙ if @ticket.used? ➤ ύλʔϯ͕̏ಡΈखʹҰ൪΍͍͞͠
21. ʁͰऴΘΔϝιουͷ੹຿
22. ʁͰऴΘΔϝιουͷ੹຿Λߟ͑Δ ➤ before_action :ticket_used? ➤ validate :can_exit? ➤ ྫɿIUUQTHJUIVCDPN+VOJDIJ*UPUSBJOUJDLFUSBJMTQVMM
    ➤ ಛఆͷ৚݅ͰϦμΠϨΫτ͢Δʢ͍͍ͷʁʣ
    ➤ ಛఆͷ৚݅ͰΤϥʔϝοηʔδΛ௥Ճ͢Δʢ͍͍ͷʁʣ
    ➤ ʁͰऴΘΔϝιου͸ਅِ஋Λฦ͚ͩ͢ʹ͢Δ
    ➤ ͦΕҎ֎ͷ෭࡞༻͸࣋ͨͤͳ͍
    ➤ ྫ֎ɿ"DUJWF3FDPSEͷWBMJE ϝιου
23. redirect_to + returnʁ
24. redirect_to + return ͷ return ͍ͬͯΔʁ ➤ ͦͷSFUVSOɺ͍Γ·͢ʁʢ͍Γ·ͤΜʂʣ def abort_used_ticket if @ticket.used? rediect_to root_path return end end
25. redirect_to + return ͷ return ͍ͬͯΔʁ ➤ ͨͩ͠ɺ࣍ͷΑ͏ͳίʔυͳΒSFUVSO͸ඞཁ def abort_used_ticket if @ticket.used? rediect_to root_path # Avoid DoubleRenderError return end redirect_to foo_path end SFEJSFDU@UP΍SFOEFSΛෳ਺ճݺͿͱ%PVCMF3FOEFS&SSPS
26. ·ͱΊ
27. ·ͱΊ ➤ ϝιουͷ੹຿Λߟ͑Α͏
    ➤ ੹຿΍෭࡞༻͸Ͱ͖Δ͚ͩগͳ͘
    ➤ ͍͍ײ͡ʹந৅Խ͠Α͏
    ➤ ʮҙਤΘ͔ΔϩδοΫʯΑΓ΋ʮҙਤ͕෼͔Δ໊લʯΛ
    ➤ ෆཁͳίϯςΩετΛͳͦ͘͏
    ➤ ࡞ͬͨਓ͚͕ͩΘ͔ΔʮӅΕͨҙຯʯ͸ഉআ͢Δ
    ➤ 3BJMTͷػೳΛ࢖͍͜ͳͦ͏
    ➤ γϯϓϧͰෆ۩߹ͷͳ͍ίʔυΛ໨ࢦͦ͏
    ➤ ͔ͩΒɺ͓ޓ͍ʹίʔυϨϏϡʔ͓͋͠͏ʂʂ
28. THANK YOU FOR ALL! ➤ ղ౴Λૹͬͯ͘ΕͨΈͳ͞Μɺ͋Γ͕ͱ͏͍͟͝·ͨ͠ʂ IUUQTHJUIVCDPN+VOJDIJ*UPUSBJOUJDLFUSBJMTQVMMT
29. ͔ͤͬ͘ͳͷͰɺ
30. શһʹίϝϯτ͠·͢ʂ
31. શһͷղ౴ʹίϝϯτ͠·͢ ➤ ͨͩ͠ςΩετͩͱେมͳͷͰɺޱ಄Ͱίϝϯτ͠·͢
    ➤ ίʔυΛݟͳ͕Β๻͕ίϝϯτΛೖΕΔಈըΛ࡞Γ·͢
    ➤ ಈը͕Ͱ͖ͨΒൃදࢿྉͷҰ෦ͱͯ͠ެ։ˍ௨஌͠·͢
    ➤ ର৅͸ݱ࣌఺ͰૹΒΕ͖͍ͯͯΔશϓϧϦΫΤετ ָ͓͠Έʹʂʂ
32. ͝੩ௌ͋Γ͕ͱ͏͍͟͝·ͨ͠
    Twitter: !KODIJUP Blog: IUUQCMPHKOJUPDPN Qiita: IUUQRJJUBDPNKODIJUP Follow me!