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

Transcript

1. ۦ͚ग़͠ΤϯδχΞͷίʔυΛ ϨϏϡʔͯ͠Έͨ 2019/05/16 Laravel Meetup Miyazaki #1 େ௩ਅݴ
2. ຊ೔ͷྲྀΕ • ࣗݾ঺հ • ۦ͚ग़͠ΤϯδχΞͱ͸ • ݚम಺༰ʹ͍ͭͯ • ϨϏϡʔͯ͠Έͨ
3. ࣗݾ঺հ
4. ࣗݾ঺հ ɾ໊લɹɿେ௩ ਅݴʢ͠Μ͛Μʣ ɾTwitterɿ@mametra ɾॴଐɹɿ߹ಉձࣾϊϚυϦ ୅ද ɾεΩϧɿLaravel, Vue.js, Firebase ɾझຯɹɿΩϟϯϓ, ϑΣε ɾ࢓ࣄɹɿLaravelͰͷWebγεςϜ։ൃ HTML5ϋΠϒϦουΞϓϦ։ൃ
6. ۦ͚ग़͠ΤϯδχΞͱ͸
7. ۦ͚ग़͠ΤϯδχΞͱ͸ • ϓϩάϥϛϯάΛษڧ࢝͠Ίͯ1೥ະຬ • ৽ଔ΋͘͠͸࣮຿ܦݧ͕ઙ͍ • TwitterͰ XXX@ۦ͚ग़͠ΤϯδχΞ ͱ໊৐ͬ ͍ͯΔ
8. ݚम಺༰ʹ͍ͭͯ
9. Πϯλʔϯੜͷ঺հ • ٶ࿬͘Μ • ٶ࡚ެཱେֶ4೥ • ϓϩάϥϛϯάྺ1೥൒ • Laravel,PHP͸ະܦݧ • ٶຊ͘Μ • ٶ࡚େֶ2೥ • ϓϩάϥϛϯάྺ1೥ • Laravel,PHP͸ະܦݧ
10. ݚम಺༰ • Նࠒ·Ͱʹ࣮຿ՄೳͳεΩϧΛ͚ͭΔ • Laravel੨ຊɺެࣜνϡʔτϦΞϧΛ׬ྃ • 3݄͔Β࣮ફԋशΛ։࢝
11. ࣮ફԋश • ๭ITษڧձαʔϏεΛύΫΔࢀߟ • ཁ݅ఆٛʙઃܭʙ࣮૷ʙςετ • ୲౰ػೳΛׂΓৼΓɺϓϧϦΫϕʔεͰ࣮૷
16. ϨϏϡʔͯ͠Έͨ
17. ϨϏϡʔͱ͸ • ιϑτ΢ΣΞ඼࣭ͷ޲্͓Αͼଐਓੑͷഉআ • εΩϧͷ޲্ٴͼφϨοδͷڞ༗ • ϨϏϡʔ͸ίʔυʹରͯ͠ߦ͏
18. ϨϏϡʔର৅ • EventController • લఏ৚݅ • ϧʔςΟϯάఆٛࡁΈ • Model͸ϦϨʔγϣϯͷΈ ΞΫγϣϯ ػೳ JOEFY Ұཡදࣔ DSFBUF ৽ن࡞੒ TUPSF ొ࿥ TIPX ৄࡉදࣔ FEJU ฤू VQEBUF ߋ৽ EFMFUF ࡟আ
23. ϨϏϡʔํ਑ • 200ߦͷ Fat Controller ղফΛ࠷༏ઌ • Request૚ɺService૚Λ௥Ճ • Fat Model ͸໨ΛͭΉΔ
24. ୈ1஄
26. ୈ1஄ • FormRequestΫϥεͷಋೖ • όϦσʔγϣϯͱલॲཧʢ೔෇Ճ޻ʣΛ࣮૷ • store, updateϝιουͷҾ਺ʹ FormRequestΫϥεΛλΠϓώϯτͰࢦఆ • ϝιου͕ݺͼग़͞ΕΔલʹόϦσʔγϣϯ
29. 200 → 150
30. ୈ2஄
32. ୈ2஄ • ΞΫηαͱappendsͷ࢖༻ • appendsΛ࢖͏͜ͱͰɺςʔϒϧʹͳ͍ΧϥϜ ΛEventΦϒδΣΫτͷϓϩύςΟʹ௥ՃͰ͖Δ • ΞΫηαʹऔΓ͏Δ஋Λఆٛ͢Δ • EventΦϒδΣΫτ͔Β௥Ճͨ͠ϓϩύςΟΛऔ ಘͰ͖Δ
36. 150 → 130
37. ୈ3஄
39. ୈ3஄ • ServiceΫϥεͷಋೖ • ϏδωεϩδοΫ͸Controllerʹॻ͔ͳ͍ • EventServiceΫϥε͸ίϯετϥΫλʔΠϯ δΣΫγϣϯʹΑΓEventController಺ͰΞΫ ηεՄೳʹͳΔ
42. 130 → 85
45. ·ͱΊ
46. ·ͱΊ • Fat Controller͸ݏΘΕΔͷͰεϦϜΛҙࣝ͢Δ • Request૚΍Service૚ͳͲಛఆͷ੹຿Λ࣋ͬͨ ΫϥεʹΘ͚Δͱશମͱͯ͠ͷݟ௨͕͠޲্͢Δ • Service૚ͱModel૚ͷؒʹRepository૚ΛೖΕ ΔϨΠΞʔυΞʔΩςΫνϟ΋͋ͬͨΓɺ Laravel͸ࣗ༝౓͕ߴ͘ɺۦ͚ग़͠ΤϯδχΞΛ ໎ΘͤΔ͜ͱ͕ଟ͍
47. ͝ਗ਼ௌ͋Γ͕ͱ͏͍͟͝·ͨ͠ʂ