Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Speaker Deck
PRO
Sign in
Sign up
for free
駆け出しエンジニアのコードをレビューしてみた
大塚真言
May 16, 2019
Programming
4
1k
駆け出しエンジニアのコードをレビューしてみた
大塚真言
May 16, 2019
Tweet
Share
More Decks by 大塚真言
See All by 大塚真言
shingen29
0
51
shingen29
0
110
shingen29
2
520
shingen29
1
130
shingen29
0
13
shingen29
1
200
shingen29
0
130
shingen29
3
140
shingen29
1
23
Other Decks in Programming
See All in Programming
devinjeon
2
820
orgachem
1
200
takuyaa
4
460
yoshiken
1
220
imk2o
0
180
rukiadia
1
790
kalupas226
0
130
minecr
0
110
coe401_
3
160
dnskimo
7
1.3k
gtongy
0
430
hr01
0
1.9k
Featured
See All Featured
jakevdp
776
200k
bermonpainter
343
26k
paulrobertlloyd
71
3.7k
shpigford
370
42k
marktimemedia
7
440
lara
172
9.7k
michaelherold
226
8.6k
aarron
258
36k
wjessup
340
16k
chriscoyier
146
20k
cherdarchuk
71
260k
caitiem20
311
17k
Transcript
ۦ͚ग़͠ΤϯδχΞͷίʔυΛ ϨϏϡʔͯ͠Έͨ 2019/05/16 Laravel Meetup Miyazaki #1 େ௩ਅݴ
ຊͷྲྀΕ • ࣗݾհ • ۦ͚ग़͠ΤϯδχΞͱ • ݚम༰ʹ͍ͭͯ • ϨϏϡʔͯ͠Έͨ
ࣗݾհ
ࣗݾհ ɾ໊લɹɿେ௩ ਅݴʢ͠Μ͛Μʣ ɾTwitterɿ@mametra ɾॴଐɹɿ߹ಉձࣾϊϚυϦ ද ɾεΩϧɿLaravel, Vue.js, Firebase ɾझຯɹɿΩϟϯϓ,
ϑΣε ɾࣄɹɿLaravelͰͷWebγεςϜ։ൃ HTML5ϋΠϒϦουΞϓϦ։ൃ
None
ۦ͚ग़͠ΤϯδχΞͱ
ۦ͚ग़͠ΤϯδχΞͱ • ϓϩάϥϛϯάΛษڧ࢝͠Ίͯ1ະຬ • ৽ଔ࣮͘͠ܦݧ͕ઙ͍ • TwitterͰ XXX@ۦ͚ग़͠ΤϯδχΞ ͱ໊ͬ ͍ͯΔ
ݚम༰ʹ͍ͭͯ
Πϯλʔϯੜͷհ • ٶ͘Μ • ٶ࡚ެཱେֶ4 • ϓϩάϥϛϯάྺ1 • Laravel,PHPະܦݧ •
ٶຊ͘Μ • ٶ࡚େֶ2 • ϓϩάϥϛϯάྺ1 • Laravel,PHPະܦݧ
ݚम༰ • Նࠒ·Ͱʹ࣮ՄೳͳεΩϧΛ͚ͭΔ • Laravel੨ຊɺެࣜνϡʔτϦΞϧΛྃ • 3݄͔Β࣮ફԋशΛ։࢝
࣮ફԋश • ITษڧձαʔϏεΛύΫΔࢀߟ • ཁ݅ఆٛʙઃܭʙ࣮ʙςετ • ୲ػೳΛׂΓৼΓɺϓϧϦΫϕʔεͰ࣮
None
None
None
None
ϨϏϡʔͯ͠Έͨ
ϨϏϡʔͱ • ιϑτΣΞ࣭ͷ্͓Αͼଐਓੑͷഉআ • εΩϧͷ্ٴͼφϨοδͷڞ༗ • ϨϏϡʔίʔυʹରͯ͠ߦ͏
ϨϏϡʔର • EventController • લఏ݅ • ϧʔςΟϯάఆٛࡁΈ • ModelϦϨʔγϣϯͷΈ ΞΫγϣϯ
ػೳ JOEFY Ұཡදࣔ DSFBUF ৽ن࡞ TUPSF ొ TIPX ৄࡉදࣔ FEJU ฤू VQEBUF ߋ৽ EFMFUF আ
None
None
None
None
ϨϏϡʔํ • 200ߦͷ Fat Controller ղফΛ࠷༏ઌ • RequestɺServiceΛՃ • Fat
Model ΛͭΉΔ
ୈ1
None
ୈ1 • FormRequestΫϥεͷಋೖ • όϦσʔγϣϯͱલॲཧʢՃʣΛ࣮ • store, updateϝιουͷҾʹ FormRequestΫϥεΛλΠϓώϯτͰࢦఆ •
ϝιου͕ݺͼग़͞ΕΔલʹόϦσʔγϣϯ
None
None
200 → 150
ୈ2
None
ୈ2 • ΞΫηαͱappendsͷ༻ • appendsΛ͏͜ͱͰɺςʔϒϧʹͳ͍ΧϥϜ ΛEventΦϒδΣΫτͷϓϩύςΟʹՃͰ͖Δ • ΞΫηαʹऔΓ͏ΔΛఆٛ͢Δ • EventΦϒδΣΫτ͔ΒՃͨ͠ϓϩύςΟΛऔ
ಘͰ͖Δ
None
None
None
150 → 130
ୈ3
None
ୈ3 • ServiceΫϥεͷಋೖ • ϏδωεϩδοΫControllerʹॻ͔ͳ͍ • EventServiceΫϥείϯετϥΫλʔΠϯ δΣΫγϣϯʹΑΓEventControllerͰΞΫ ηεՄೳʹͳΔ
None
None
130 → 85
None
None
·ͱΊ
·ͱΊ • Fat ControllerݏΘΕΔͷͰεϦϜΛҙࣝ͢Δ • RequestServiceͳͲಛఆͷΛ࣋ͬͨ ΫϥεʹΘ͚Δͱશମͱͯ͠ͷݟ௨্͕͢͠Δ • ServiceͱModelͷؒʹRepositoryΛೖΕ ΔϨΠΞʔυΞʔΩςΫνϟ͋ͬͨΓɺ
Laravelࣗ༝͕ߴ͘ɺۦ͚ग़͠ΤϯδχΞΛ ໎ΘͤΔ͜ͱ͕ଟ͍
͝ਗ਼ௌ͋Γ͕ͱ͏͍͟͝·ͨ͠ʂ