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

駆け出しエンジニアのコードをレビューしてみた

 駆け出しエンジニアのコードをレビューしてみた

Ca1357a6e793898d30cec23068c3cb36?s=128

大塚真言

May 16, 2019
Tweet

Transcript

  1. ۦ͚ग़͠ΤϯδχΞͷίʔυΛ ϨϏϡʔͯ͠Έͨ 2019/05/16 Laravel Meetup Miyazaki #1 େ௩ਅݴ

  2. ຊ೔ͷྲྀΕ • ࣗݾ঺հ • ۦ͚ग़͠ΤϯδχΞͱ͸ • ݚम಺༰ʹ͍ͭͯ • ϨϏϡʔͯ͠Έͨ

  3. ࣗݾ঺հ

  4. ࣗݾ঺հ ɾ໊લɹɿେ௩ ਅݴʢ͠Μ͛Μʣ ɾTwitterɿ@mametra ɾॴଐɹɿ߹ಉձࣾϊϚυϦ ୅ද ɾεΩϧɿLaravel, Vue.js, Firebase ɾझຯɹɿΩϟϯϓ,

    ϑΣε ɾ࢓ࣄɹɿLaravelͰͷWebγεςϜ։ൃ HTML5ϋΠϒϦουΞϓϦ։ൃ
  5. None
  6. ۦ͚ग़͠ΤϯδχΞͱ͸

  7. ۦ͚ग़͠ΤϯδχΞͱ͸ • ϓϩάϥϛϯάΛษڧ࢝͠Ίͯ1೥ະຬ • ৽ଔ΋͘͠͸࣮຿ܦݧ͕ઙ͍ • TwitterͰ XXX@ۦ͚ग़͠ΤϯδχΞ ͱ໊৐ͬ ͍ͯΔ

  8. ݚम಺༰ʹ͍ͭͯ

  9. Πϯλʔϯੜͷ঺հ • ٶ࿬͘Μ • ٶ࡚ެཱେֶ4೥ • ϓϩάϥϛϯάྺ1೥൒ • Laravel,PHP͸ະܦݧ •

    ٶຊ͘Μ • ٶ࡚େֶ2೥ • ϓϩάϥϛϯάྺ1೥ • Laravel,PHP͸ະܦݧ
  10. ݚम಺༰ • Նࠒ·Ͱʹ࣮຿ՄೳͳεΩϧΛ͚ͭΔ • Laravel੨ຊɺެࣜνϡʔτϦΞϧΛ׬ྃ • 3݄͔Β࣮ફԋशΛ։࢝

  11. ࣮ફԋश • ๭ITษڧձαʔϏεΛύΫΔࢀߟ • ཁ݅ఆٛʙઃܭʙ࣮૷ʙςετ • ୲౰ػೳΛׂΓৼΓɺϓϧϦΫϕʔεͰ࣮૷

  12. None
  13. None
  14. None
  15. None
  16. ϨϏϡʔͯ͠Έͨ

  17. ϨϏϡʔͱ͸ • ιϑτ΢ΣΞ඼࣭ͷ޲্͓Αͼଐਓੑͷഉআ • εΩϧͷ޲্ٴͼφϨοδͷڞ༗ • ϨϏϡʔ͸ίʔυʹରͯ͠ߦ͏

  18. ϨϏϡʔର৅ • EventController • લఏ৚݅ • ϧʔςΟϯάఆٛࡁΈ • Model͸ϦϨʔγϣϯͷΈ ΞΫγϣϯ

    ػೳ JOEFY Ұཡදࣔ DSFBUF ৽ن࡞੒ TUPSF ొ࿥ TIPX ৄࡉදࣔ FEJU ฤू VQEBUF ߋ৽ EFMFUF ࡟আ
  19. None
  20. None
  21. None
  22. None
  23. ϨϏϡʔํ਑ • 200ߦͷ Fat Controller ղফΛ࠷༏ઌ • Request૚ɺService૚Λ௥Ճ • Fat

    Model ͸໨ΛͭΉΔ
  24. ୈ1஄

  25. None
  26. ୈ1஄ • FormRequestΫϥεͷಋೖ • όϦσʔγϣϯͱલॲཧʢ೔෇Ճ޻ʣΛ࣮૷ • store, updateϝιουͷҾ਺ʹ FormRequestΫϥεΛλΠϓώϯτͰࢦఆ •

    ϝιου͕ݺͼग़͞ΕΔલʹόϦσʔγϣϯ
  27. None
  28. None
  29. 200 → 150

  30. ୈ2஄

  31. None
  32. ୈ2஄ • ΞΫηαͱappendsͷ࢖༻ • appendsΛ࢖͏͜ͱͰɺςʔϒϧʹͳ͍ΧϥϜ ΛEventΦϒδΣΫτͷϓϩύςΟʹ௥ՃͰ͖Δ • ΞΫηαʹऔΓ͏Δ஋Λఆٛ͢Δ • EventΦϒδΣΫτ͔Β௥Ճͨ͠ϓϩύςΟΛऔ

    ಘͰ͖Δ
  33. None
  34. None
  35. None
  36. 150 → 130

  37. ୈ3஄

  38. None
  39. ୈ3஄ • ServiceΫϥεͷಋೖ • ϏδωεϩδοΫ͸Controllerʹॻ͔ͳ͍ • EventServiceΫϥε͸ίϯετϥΫλʔΠϯ δΣΫγϣϯʹΑΓEventController಺ͰΞΫ ηεՄೳʹͳΔ

  40. None
  41. None
  42. 130 → 85

  43. None
  44. None
  45. ·ͱΊ

  46. ·ͱΊ • Fat Controller͸ݏΘΕΔͷͰεϦϜΛҙࣝ͢Δ • Request૚΍Service૚ͳͲಛఆͷ੹຿Λ࣋ͬͨ ΫϥεʹΘ͚Δͱશମͱͯ͠ͷݟ௨͕͠޲্͢Δ • Service૚ͱModel૚ͷؒʹRepository૚ΛೖΕ ΔϨΠΞʔυΞʔΩςΫνϟ΋͋ͬͨΓɺ

    Laravel͸ࣗ༝౓͕ߴ͘ɺۦ͚ग़͠ΤϯδχΞΛ ໎ΘͤΔ͜ͱ͕ଟ͍
  47. ͝ਗ਼ௌ͋Γ͕ͱ͏͍͟͝·ͨ͠ʂ