From 23e0437747220c29f6020d417db5885fca5a66ea Mon Sep 17 00:00:00 2001
From: ken7253 <kentaro.tachikawa@nijibox.co.jp>
Date: Fri, 17 May 2024 14:20:02 +0900
Subject: [PATCH] writing

---
 hooks-testing/slides.md | 151 +++++++++++++++++++++++++++++++++++++++-
 reuse/styles/base.css   |   6 ++
 2 files changed, 155 insertions(+), 2 deletions(-)

diff --git a/hooks-testing/slides.md b/hooks-testing/slides.md
index 68c88f2..409d0d3 100644
--- a/hooks-testing/slides.md
+++ b/hooks-testing/slides.md
@@ -79,7 +79,7 @@ export const sum = (array: number[]): number => {
 - NaNがあった場合`0`として扱う（無視する）
 - `Infinity`が含まれていた場合は常に`Infinity`を返す
 
-この関数に対してのテストを書く
+この関数に対してのテストを書く想定
 
 ---
 
@@ -238,13 +238,160 @@ export const sum = (array: number[], randomize: number) => {
 
 ## 単体テストの考え方をHooksにも適用する
 
+```ts
+import { useAtom } from "jotai";
+import { useRouter } from "next/router";
+import { someAtom } from "@/store/someAtom";
+
+export const useFizzBuzz = () => {
+  const { pathname } = useRouter();
+  const [fizzBuzz, setFizzBuzz] = useAtom(someAtom);
+  const result =
+    parseInt(pathname, 10) % 15 === 0
+      ? "FizzBuzz"
+      : parseInt(pathname, 10) % 5 === 0
+      ? "Fizz"
+      : parseInt(pathname, 10) % 3 === 0
+      ? "Buzz"
+      : parseInt(pathname, 10);
+
+  setFizzBuzz([...fizzBuzz, result]);
+  return fizzBuzz;
+};
+```
+
+`pathname`を参照元にFizzBuzzをしてその履歴をstoreに格納するHooks
+
 ---
 
 ### 依存を整理する
 
+このHooksをテストしやすいように修正してみる。
+
+````md magic-move
+
+```ts
+import { useAtom } from "jotai";
+import { useRouter } from "next/router";
+import { someAtom } from "@/store/someAtom";
+
+export const useFizzBuzz = () => {
+  const { pathname } = useRouter();
+  const [fizzBuzz, setFizzBuzz] = useAtom(someAtom);
+  const result =
+    parseInt(pathname, 10) % 15 === 0
+      ? "FizzBuzz"
+      : parseInt(pathname, 10) % 5 === 0
+      ? "Fizz"
+      : parseInt(pathname, 10) % 3 === 0
+      ? "Buzz"
+      : parseInt(pathname, 10);
+
+  setFizzBuzz([...fizzBuzz, result]);
+  return fizzBuzz;
+};
+```
+
+```ts
+import { useAtom } from "jotai";
+import { someAtom } from "@/store/someAtom";
+
+export const useFizzBuzz = (pathname: string) => { // pathnameは引数として取るように
+  const [fizzBuzz, setFizzBuzz] = useAtom(someAtom);
+  const result =
+    parseInt(pathname, 10) % 15 === 0
+      ? "FizzBuzz"
+      : parseInt(pathname, 10) % 5 === 0
+      ? "Fizz"
+      : parseInt(pathname, 10) % 3 === 0
+      ? "Buzz"
+      : parseInt(pathname, 10);
+
+  setFizzBuzz([...fizzBuzz, result]);
+  return fizzBuzz;
+};
+```
+
+```ts
+import { useAtom, type Atom } from "jotai";
+
+export const useFizzBuzz = <T>(pathname: string, atom: Atom<T>) => { // atomも引数として渡す
+  const [fizzBuzz, setFizzBuzz] = useAtom(atom);
+  const result =
+    parseInt(pathname, 10) % 15 === 0
+      ? "FizzBuzz"
+      : parseInt(pathname, 10) % 5 === 0
+      ? "Fizz"
+      : parseInt(pathname, 10) % 3 === 0
+      ? "Buzz"
+      : parseInt(pathname, 10);
+
+  setFizzBuzz([...fizzBuzz, result]);
+  return fizzBuzz;
+};
+```
+
+````
+
+---
+layout: center
+---
+
+## なにか見覚えのあるやり方だ…
+
 ---
+layout: two-cols
+---
+
+### Hooks
+
+```ts
+import { useAtom, type Atom } from "jotai";
+
+export const useFizzBuzz = <T>(
+    pathname: string, atom: Atom<T>
+  ) => {
+  const [fizzBuzz, setFizzBuzz] = useAtom(atom);
+  const result =
+    parseInt(pathname, 10) % 15 === 0
+      ? "FizzBuzz"
+      : parseInt(pathname, 10) % 5 === 0
+      ? "Fizz"
+      : parseInt(pathname, 10) % 3 === 0
+      ? "Buzz"
+      : parseInt(pathname, 10);
+
+  setFizzBuzz([...fizzBuzz, result]);
+  return fizzBuzz;
+};
+```
+
+::right::
+
+### Function
+
+```ts
+export const sum = (array: number[], randomize: number) => {
+  if (array.some(v => v === Infinity || v === -Infinity)) {
+    return Infinity;
+  }
+
+  const sumAll = array.reduce(
+    (a, c) => a + (Number.isNaN(c) ? c : 0), 0
+  );
+
+  return sumAll * randomize;
+}
+```
+
+- 副作用を引数で受け取りテストしやすい
+- シグニチャーの情報が増え分かりやすい
 
-### テストケースのラベルを見直して責務も見直す
+<div style="background-color: var(--c-main); border-radius: 8px; padding: 4px 8px;">
+<p style="font-size: 16px;text-wrap: balance;line-height: 1.6;text-align: center;">
+モジュールが持つ<strong>責務が引数の数に現れる</strong><br>のでそれを基準にコード分割のタイミングを探る
+</p>
+</div>
 
 ---
 
diff --git a/reuse/styles/base.css b/reuse/styles/base.css
index 25a61f4..faf1556 100644
--- a/reuse/styles/base.css
+++ b/reuse/styles/base.css
@@ -89,6 +89,11 @@ deepl-inline-trigger {
   }
 }
 
+/* 2カラムレイアウト */
+#app .slidev-layout.two-columns {
+  gap: 16px;
+}
+
 /* セクション見出し用 */
 #app .slidev-layout.section {
   & h2 {
@@ -140,6 +145,7 @@ deepl-inline-trigger {
 
 /* strong */
 #app strong {
+  margin: 0 4px;
   font-size: 1.25em;
   font-weight: 900;
   color: var(--c-accent);