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

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. ͝ਗ਼ௌ͋Γ͕ͱ͏͍͟͝·ͨ͠ʂ