黒歴史リポジトリを使ったリファクタリング勉強法のススメ

Transcript

1. 1.

   ࠇྺ࢙ϦϙδτϦΛ ࢖ͬͨϦϑΝΫλϦϯά ษڧ๏ͷεεϝ ɹɹɹɹɹɹɹɹ fuqda(@fuqda90) 2019.08.14(Wed) Otemachi.rb#20

2. 2.

   ࣗݾ঺հ 
 Tama.rb Organizor
 Work @ STORES.jp, inc.
 RubyͰ!Λ৯΂ΔΑ͏ʹͳΓ2೥໨Ͱ͢ Twitter

   : fuqda(@fuqda90)
 Github : Shigeyuki-fukuda
3. 3.

   ࠓ೔ͷ࿩

4. 4.

   ௚࣮ۙફͨ݁͠Ռ
 ޮՌ͕͋ͬͨΦεεϝ ษڧ๏ͷ࿩"

5. 5.

   ʲ࣭໰ʳ
 ٕज़ॻಡΜͩޙ ࣮ફͯ͠·͔͢ʁ

6. 6.

   ʲ࣭໰2ʳ
 ಡΜ͚ͩͩͩͱ਎ʹ෇͍͔ͨෆ҆… ͱ͍͏͔…๨Ε·ͤΜ͔? (ͪͳΈʹ๻͸ͦ͏Ͱ͢#)

7. 7.

   ؾܰʹຊͰֶΜͩ಺༰Λ
 ࣮ફ͍ͨ͠

8. 8.

   (࠷ѱյΕͯ΋ྑ͍ΞϓϦͰ$)

9. 9.

   ͦ͜ͰΦεεϝ͍ͨ͠ʂ

10. 10.

    ۦ͚ग़͠ͷࠒʹॻ͍ͨίʔυ
 (ࠇྺ࢙ϦϙδτϦ)ͷ ϦϑΝΫλϦϯά %✨

11. 11.

    ཧ༝

12. 12.

    ୯७ʹϦϑΝΫλϦϯάग़དྷΔ༨஍͕ଟ͍ →௚͢༨஍͕গͳ͍ͱ࡞ۀͯͯͭ͠·Βͳ͍ͷͰ ɹۃ୺ʹ伱ͩΒ͚ͷίʔυΛϘίϘίʹ௚͘͢Β͍͕ ɹ୊ࡐͱͯ͠ྑ͍ ษڧΛͨ͠ݱ࣌఺ͱ͢Δલͷࠩ෼͕໌֬ →Ͳͷ͘Β͍਎ʹ෇͍͍ͯΔ͔ʁΛ஌͕ࣝͳ͔ͬͨࠒ ɹͱൺ΂Δ͜ͱͰݱঢ়ͷཧղ౓ͷ೺Ѳʹ໾ཱͪͦ͏

13. 13.

    ΋ͬͱݴ͏ͱ ϖΞϓϩͰ΍ΔͷΛ Φεεϝ͍ͨ͠✋

14. 14.

    ϖΞϓϩͰ΍Δํ͕͍͍ཧ༝ ࢥߟͷ੔ཧ͕ग़དྷΔ →ҰਓͩͱԿͱͳͬͪ͘͜ͷίʔυͷํ͕ྑ͍͔Βͱ ɹᐆດʹ͢Δͱ͜ΖΛ૬ख͕͍ΔͱԿނͦͷॻ͖ํ͕ ɹྑ͍ͷ͔Λઆ໌͢Δඞཁ͕ग़ͯ͘Δʂ ɹͦͷͨΊઆ໌ͷաఔͰᐆດͳཧղʹؾ෇͚ͯྑ͍ →૬खͱࣗ෼ͰΞϓϩʔν͕ҧ͏৔໘͕͋Δ͸ͣͳͷͰ ɹͦΕ͕ͦͷ··ֶͼʹͳΔ

15. 15.

    ࣮ࡍʹ΍͍ͬͯͬͨ͜ͱ

16. 16.

    Tama.rbͷ˞ࣗݾ঺հ༻ΨνϟΞϓϦͷ
 ϦϑΝΫλϦϯά ※Tama.rbؔ܎ऀͷ๭ࢯ͕ۦ͚ग़͠ͷࠒʹ ࡞ͬͨΞϓϦΛڐՄΛ΋Βͬͯ௚͢͜ͱʹ… ϝϯόʔ @fuqda @ken3ypa

17. 17.

    ʲΞϓϦ֓ཁʳ ࣗݾ঺հΨνϟ ɾQuestion(࣭໰) ɾConjunction(઀ଓࢺ) ɾTheme(ςʔϚ) 
 ͜ΕΒͷΧςΰϦʔʹ֘౰͢Δొ࿥ϫʔυΛ
 ΨνϟͷΑ͏ʹҾ͍ͯ ϥϯμϜͰը໘ʹදࣔ͠ ͦΕΛݩʹࣗݾ঺հͯ͠΋Β͏ΞϓϦ

18. 18.

    ϦϑΝΫλϦϯάͷΰʔϧ ៉ྷͳίʔυʹ௚͢(ͦΕ͚ͩ͡Όͳ͍) ⭕ ۦ͚ग़͔͠Βࠓ·ͰֶΜͩΦϒδΣΫτ ࢦ޲ͷΞϨίϨ΍֤छॻ੶ͷֶश಺༰Λ Ξ΢τϓοτग़དྷΔ͔ΛϦϑΝΫλϦϯά ͷதͰ֬ೝ͢Δ͜ͱ

19. 19.

    ϦϑΝΫλϦϯά͢Δ ্ͰνΣοΫ߲ͨ͠໨

20. 20.

    ɾ໋໊ͷଥ౰ੑνΣοΫ ɹ (Ϋϥε໊ɾϝιου໊ɾΧϥϜ໊etc…) ͦͷ໊લͰ఻ΘΔʁ ex)ͱ͋ΔϕϯνϟʔاۀͰϗϯτʹ͋ͬͨා໋໊͍ class EmailMailer →Կʹର͢Δϝʔϧ͕͍ͨ͠Ϋϥεͳͷ͔શ͘఻ΘΒΜ ޿͗͢Δ໋໊͡Όͳ͍ʁ ex)

    Ϋϥε໊ɿGroup, Operation →޿໊͍લʹ͸prefix͕ඞཁͦ͏ γεςϜ্Ϳ͔ͭΔ໊લ͡Όͳ͍ʁ 
 ex) ΧϥϜ໊ɿsize, count →ͦͷੲ๭ϑΝογϣϯܥαʔϏεͰ෰ͷαΠζΛsizeͱ͍͏ ΧϥϜͰ؅ཧͨ͠ͱ͜ΖORϚούʔͷ಺෦ͷϝιουͱ Ϳ͔ͭͬͯҙਤ͠ͳ͍ڍಈΛͨ͜͠ͱ͕͋ͬͨͦ͏ͳ
21. 21.

    ɾΫΤϦͷଥ౰ੑνΣοΫ

22. 22.

    ɾϞσϦϯάͷଥ౰ੑνΣοΫ ຊ౰ʹ͜ͷϞσϧඞཁʁ →QuestionϞσϧҰͭʹ
 ɹ ActiveRecord::EnumΛੜ΍ͯ͠؅ཧ͢Ε͹ ɹ Conjunction(઀ଓࢺ)ͱ ɹ ThemeϞσϧෆཁͳͷͰ͸ʁ ※Πϝʔδ

    enum category: { conjunction: 0, theme: 1 }
23. 23.

    ɾRailsత/OOPత͔νΣοΫ 
 ੹຿͕ద੾ʹ෼ׂ͞ΕͯΔʁ →AΫϥε͕BΫϥεͷதͷॲཧΛ஌Γա͗ͯͳ͍͔ →ԿͰ΋԰ͷαʔϏεΫϥεͱ͔࡞ͬͯͨΒຣࡴ(ࠓճ͸ແ͠) ͜ͷ࣮૷ͳ͍ͱμϝʁ →ෳ਺ͷϞσϧͰڞ௨ͨ͠ॲཧ͕ ɹActiveSupport::Concernʹ·ͱΊΒΕ͍͕ͯͨ
 ͦ΋ͦ΋ແବͳϞσϧΛ࡟আͨ͠Β ActiveSupport::Concern͍Βͳ͘ͳ͍ʁ^_^;

    ɹ➡ ຣࡴ
24. 24.

    ૯࿦

25. 25.

    ɾϦϑΝΫλϦϯάͰग़ͯ͘ΔΞΠσΞ
 ɹͦͷ΋ͷ͕ࠓͷࣗ෼ͷ࣮ྗ ɾۀ຿ͩͱେਓͷࣄ৘ͰϦϑΝΫλϦϯάʹ
 ɹͳ͔ͳ͔खΛ෇͚ΒΕͳ͍͔΋ͳͷͰ… झຯͷίʔυͰͿͬյ͠ͳ͕ΒֶΜͰ ɹۀ຿ίʔυʹ΋൓өग़དྷͨΒྑͦ͞͏✨ ɾϖΞϓϩɾϞϒϓϩͰࠓճͷΑ͏ͳϦϑΝΫλϦϯάΛ ͢Δͱ୯७ʹָ͍͠ & ษڧʹͳΔͷͰ

    ೔ʑͷֶशϚϯωϦଧഁʹ͓ࢼ͋͠Ε
26. 26.

    ͝੩ௌ͋Γ͕ͱ͏͍͟͝·ͨ͠3