Upgrade to Pro — share decks privately, control downloads, hide ads and more …

LT&ディスカッション5ラウンド!うひょさん・よしこさんと改めて考えるReactコンポーネント設計

Fumina Chihama
November 06, 2023
2k

 LT&ディスカッション5ラウンド!うひょさん・よしこさんと改めて考えるReactコンポーネント設計

Fumina Chihama

November 06, 2023
Tweet

Transcript

  1. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 2 はじめに 02 コンポーネント設計 LT&ディスカッションテーマ① 「分割粒度」 03 04 05 質疑応答 01 Agenda コンポーネント設計 LT&ディスカッションテーマ② 「構造」 コンポーネント設計 LT&ディスカッションテーマ③ 「スタイリング」 コンポーネント設計 LT&ディスカッションテーマ④ 「テスト」 コンポーネント設計 LT&ディスカッションテーマ⑤ 「パフォーマンス」 06 07
  2. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 3 Moderator 佐藤 歩 株式会社overflow VPoE @ahomu 株式会社 overflow VPoE として開発組織のマネジメントに 携わる。前職の株式会社 CyberAgent では技術⼈事、EM、 新規サービス開発などを歴任。 Web フロントエンド技術が専⾨で、著書に「超速!Web ページ速度改善ガイド (共著)」など。ほか複数社の技術顧 問‧組織アドバイザーも務める。
  3. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 4 よしこ ⽒ 株式会社ナレッジワーク フロントエンドエンジニア @yoshiko_pg 2015年、株式会社グッドパッチ⼊社。 2017年、株式会社FOLIO⼊社。 2020年、株式会社ナレッジワーク共同創業。 現在は同社の専⾨役員Principalとして フロントエンドエンジニア職に従事。 Google Developer Experts。 著書『HTML5/CSS3モダンコーディング』 登壇者
  4. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 5 うひょ ⽒ 株式会社バベル プリンシパルエンジニア @uhyo_ 2019年に新卒にてLINE株式会社に⼊社。 2022年1⽉より株式会社バベルの技術顧問に就任、同年10 ⽉に正社員として⼊社。現在プリンシパルエンジニアとし てアプリケーションの開発‧設計を担当。 著書『プロを⽬指す⼈のためのTypeScript⼊⾨』 登壇者
  5. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 7 LT①「分割粒度」 よしこ⽒ page model ui 何らかのdomain modelに関連 (UserList, GroupPickerなど) domain modelの知識を持つ UIライブラリとして切り出せる 汎⽤Component domain modelの知識はない ページのentrypointファイルと 1対1で対応するページ実体 domain modelの知識を持つ https://zenn.dev/knowledgework/articles/99f8047555f700 \詳しくはこちら/ Componentファイル構成 Button/ ├─ Button.tsx ├─ Button.module.css ├─ Button.stories.tsx └─ index.ts scaffdogでつくるよ! pageとmodelの間に、「domain modelの知識を持ちうる共通Component」の レイヤーが要ると思ってるがまだ作れてない。 例: 共通ページヘッダ、共通サイドバーなど
  6. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 8 LT①「分割粒度」 うひょ⽒ 1ロジック1コンポーネントが良い感じ 複数ロジックが同居すると急に⾒通しが 悪くなるため。 ロジックコンポーネントたち +まとめコンポーネント の構成が基本。 UIコンポーネントは共通化されたデザイン の場合に発⽣する印象。 その場限りの場合はあまり分けない。 UIコンポーネントを作る場合も 1つのロジック=1つの責務を意識。 QRコード表⽰コンポーネント Q&Aボタンコンポーネント (押すと何か起こる想定) サイドバーコンポーネント
  7. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 10 LT②「構造」 よしこ⽒ あまり⼯夫せず、かなり素朴にそのまま書いてます。 分けるとしたら1Component内のコード分割よりも、Componentそのものの分割のほうをマメにやりがちかも ⾃分の傾向として、⼤枠の設計や共通部品に⽐べて、局所的なComponentの書き⽅は改善できる点が多そう やってないこと • Container/Presentationパターン • 単⼀Component内でのhooksの抜き出し(複数から利⽤する際に初めて抜きだす) やっててよかったこと • レイヤー間の依存ルールの整理と強制 • 外部ライブラリに依存する際のhooks化や腐敗防⽌層の作成 やってて学びがあったこと • render propsは不評だった • Componentディレクトリは全部フラットじゃなくて初⼿は⼊れ⼦のほうがスケールしたとき楽 おさらい:Componentファイル構成 Button/ ├─ Button.tsx ├─ Button.module.css ├─ Button.stories.tsx └─ index.ts
  8. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 11 LT②「構造」 うひょ⽒ 1コンポーネント1フックが好き コンポーネントのロジック部分を抜き出して隣に置くことで、 ロジックのアウトプットが明確になる。 ロジックの中間状態を全部JSXから参照されたりすると 将来的に wow なことになりがち。 const QRCode = () => { const { imageUrl } = useQRCodeLogic(); return <img src={imageUrl} … />; }; const useQRCode = () => { const imageUrl = use(...); return { imageUrl, element: <QRCode imageUrl={imageUrl} … /> }; }; コンポーネントとフックの依存関係を 逆にするとrender hooksパターンになる
  9. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 13 LT③「スタイリング」 よしこ⽒ CSS Modules ⼤好き。⼩回りがきくし潰しもきくから(CSS好き奴のポジショントークであることは否めない) やっていてよかったこと • stylelintでプロパティの並び順やコーディングルールをautofix • typed-css-modulesをCI上でだけ回してセレクタ参照チェック • スタイルだけの再利⽤でもCSS単位での再利⽤を許さず、必ずComponent化して再利⽤する ◦ marginだけは外からあてないといけないので ={margin.top8} のように使えるhelperがある • figmaにもComponentsというファイルを作ってもらい、UI Componentと1:1になるよう同期管理 ◦ Custom Propertiesとfigma側の⾊名なども同様 やっておけばよかったこと • レイアウト系のUI Componentをもっと早くから充実させておけばよかった ◦ やっぱり <Flex direction=”column”> みたいになるもの…?
  10. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 14 LT③「スタイリング」 うひょ⽒ 暗黙の依存関係は避けたい 親コンポーネントが display: flex; を持ち ⼦コンポーネントが flex: 1; を持ってるのは避けたい。 セットで書かれるスタイルはなるべく同じコンポーネントに詰め込む。 いわゆるCSS Modulesが好き CSSの構⽂が直接使えるのはやっぱり偉い。 パフォーマンスも基本的には良好。 マージン持ったコンポーネントはあかんで 親コンポーネントのための実装が⼦コンポー ネントに存在してる状態なのでだめ。 const TwoColumn = ({ left, right }) => ( <div style=”display: flex”> <div style=”flex: 0 1 auto”>{left}</div> <div style=”flex: 1 1 auto”>{right}</div> </div> ); こういうイメージ
  11. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 16 LT④「テスト」 よしこ⽒ Testing Trophy うち Static Visual Regression End to End Unit Integration ここ強い⽅! お待ちしてます!! Playwrightで全CRUD 1⽇1回⾃動で回る reg-suitでPRごと カバレッジ⾼ 誤検出低減がんばってる あんまりできてない Componentはどこテストすべき? repository, usecase, selectorあたり やってるが本質的でない気がする 型も含め結構ガチガチ 基本ここのチェック通れば ⼀定の安⼼感
  12. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 17 LT④「テスト」 うひょ⽒ 基本はフックに対するユニットテスト やはりロジックに関するテストの需要が⼤きいため。 DOM (jsdom) を経由させてもあまり価値が上がらない印象。 実際のUIの挙動を検査するなら、ブラウザを動かしちゃった ⽅が価値が出そう。 const QRCode = () => { const imageUrl = useQRCodeLogic(); return <img src={imageUrl} … />; }; これをテストする RSCではもはやフックですらなくなるため、 さらにテストが簡単になるか。
  13. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 19 LT⑤「パフォーマンス」 よしこ⽒ やってないこと • バンドルサイズのカリカリのチューニング ◦ SaaSでツールなので、ここ頑張るより認証解決速度上げたほうが体感ぐんと変わる印象 ◦ たまに @next/bundle-analyzer ⾒て過剰なものを調整する程度 • Component内でのuseMemo/useCallbackの徹底 ◦ これはパフォーマンス以外にもReact wayに乗れることやスコープの明⽰という側⾯あるので その意味でも早めに慣れて最初からやっておいたらよかったなー やっててよかったこと • Next.jsの採⽤(ページごとにバンドルを分割) • CSS Modulesの採⽤(Zero Runtime) • Custom Hooksでの返り値のuseMemo/useCallbackの徹底(これ強制できるlintありますか?) 別観点でやっておくといいこと • リストのAPI/UIには基本ページネーションをいれておく • ⼀覧表⽰される画像の最適化、動画のサムネイル画像化
  14. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 20 LT⑤「パフォーマンス」 うひょ⽒ コンポーネント分割とSuspense コンポーネントを適切に分けて出せるUIを なるはやで出すのが何だかんだで⼀番有効。 バンドルサイズの削減という意味もあるが、 ネットワークの往復数を削減することが⼀番重要。 useMemoはちゃんと使う 重い計算を避けるというより、同じオブジェクトを 同じままにする最適化‧設計意図の明⽰という 意味が⼤きい。 (React Forget的なやつが来たら変わるかも?)
  15. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 21 よしこからのお知らせ: ナレッジワーク、エンジニア積極募集 こんな⽅々におすすめ! • 優秀なPdM/Designerと背中を合わせてプロダクト開発の技術⾯をリードしたい⽅ • シニアな先輩の背中を追いかけながら綺麗なコードを吸収して最速成⻑したい⽅ • 複数プロダクトを⽀える共通基盤や仕組みの開発が好きな⽅ • 技術だけでなく事業や組織の成功に関⼼のある⽅、その成功を⾒てみたい⽅ https://kwork.studio/recruit-engineer もしくは yoshikoのXにDMください!
  16. ご質問はzoomの Q&Aボタンより お送りください 質問募集中 ※ 匿名でのご質問も 可能です 2023.11.06「LT&ディスカッション5ラウンド!うひょさん‧よしこさんと改めて考えるReactコンポーネント設計」 実況はこちら #Offers_Reactコンポーネ

    ント設計で投稿しよう! 22 うひょからのお知らせ:バベルもエンジニア募集中 https://jobs.babel.jp/ セールスイネーブルメント分野のSaaSを提供。 トランスパイルにBabelではなくSWCを使っているというウワサだ。 プロダクトの成⻑を技術で⽀えたい⼈におすすめ。