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

Transcript

1. 1.

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

2. 2.

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

3. 3.

   ࣗݾ঺հ

4. 4.

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

   ϑΣε ɾ࢓ࣄɹɿLaravelͰͷWebγεςϜ։ൃ HTML5ϋΠϒϦουΞϓϦ։ൃ
5. 5.

   None
6. 6.

   ۦ͚ग़͠ΤϯδχΞͱ͸

7. 7.

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

8. 8.

   ݚम಺༰ʹ͍ͭͯ

9. 9.

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

   ٶຊ͘Μ • ٶ࡚େֶ2೥ • ϓϩάϥϛϯάྺ1೥ • Laravel,PHP͸ະܦݧ
10. 10.

    ݚम಺༰ • Նࠒ·Ͱʹ࣮຿ՄೳͳεΩϧΛ͚ͭΔ • Laravel੨ຊɺެࣜνϡʔτϦΞϧΛ׬ྃ • 3݄͔Β࣮ફԋशΛ։࢝

11. 11.

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

12. 12.

    None
13. 13.

    None
14. 14.

    None
15. 15.

    None
16. 16.

    ϨϏϡʔͯ͠Έͨ

17. 17.

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

18. 18.

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

    ػೳ JOEFY Ұཡදࣔ DSFBUF ৽ن࡞੒ TUPSF ొ࿥ TIPX ৄࡉදࣔ FEJU ฤू VQEBUF ߋ৽ EFMFUF ࡟আ
19. 19.

    None
20. 20.

    None
21. 21.

    None
22. 22.

    None
23. 23.

    ϨϏϡʔํ਑ • 200ߦͷ Fat Controller ղফΛ࠷༏ઌ • Request૚ɺService૚Λ௥Ճ • Fat

    Model ͸໨ΛͭΉΔ
24. 24.

    ୈ1஄

25. 25.

    None
26. 26.

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

    ϝιου͕ݺͼग़͞ΕΔલʹόϦσʔγϣϯ
27. 27.

    None
28. 28.

    None
29. 29.

    200 → 150

30. 30.

    ୈ2஄

31. 31.

    None
32. 32.

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

    ಘͰ͖Δ
33. 33.

    None
34. 34.

    None
35. 35.

    None
36. 36.

    150 → 130

37. 37.

    ୈ3஄

38. 38.

    None
39. 39.

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

40. 40.

    None
41. 41.

    None
42. 42.

    130 → 85

43. 43.

    None
44. 44.

    None
45. 45.

    ·ͱΊ

46. 46.

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

    Laravel͸ࣗ༝౓͕ߴ͘ɺۦ͚ग़͠ΤϯδχΞΛ ໎ΘͤΔ͜ͱ͕ଟ͍
47. 47.

    ͝ਗ਼ௌ͋Γ͕ͱ͏͍͟͝·ͨ͠ʂ