p256 リスト20.40の実装の仕方が本文中では曖昧

[smirror]

大変勉強になる本をありがとうございます。

実装しながら読み勧めているのですが、p256の「リスト20.40の実装すればよい」と書かれていて単純にリスト20.40追加すればよいのかと思っていましたが、実際にはコミットのように書き換えが必要なので、本だけを見ていると気づきにくいように思いました。

-	r := store.Repository{Clocker: clock.RealClocker{}}
+	clocker := clock.RealClocker{}
+	r := store.Repository{Clocker: clocker}
+	rcli, err := store.NewKVS(ctx, cfg)
+	if err != nil {
+		return nil, cleanup, err
+	}
+	jwter, err := auth.NewJWTer(rcli)
+	if err != nil {
+		return nil, cleanup, err
+	}

- 	at := &handler.AddTask{
-		Service:   &service.AddTask{DB: db, Repo: &r},
-		Validator: v,
-	}
-	mux.Post("/tasks", at.ServeHTTP)
-	lt := &handler.ListTask{
-		Service: &service.ListTask{DB: db, Repo: &r},
-	}
-	mux.Get("/tasks", lt.ServeHTTP)
+	at := &handler.AddTask{
+		Service:   &service.AddTask{DB: db, Repo: &r},
+		Validator: v,
+	}
+	lt := &handler.ListTask{
+		Service: &service.ListTask{DB: db, Repo: &r},
+	}
+	mux.Route("/tasks", func(r chi.Router) {
+		r.Use(handler.AuthMiddleware(jwter))
+		r.Post("/", at.ServeHTTP)
+		r.Get("/", lt.ServeHTTP)
+	})

[budougumi0617]

• https://github.com/budougumi0617/go_todo_app/releases/tag/v1.0.11

ご指摘ありがとうございました。たしかにわかりにくいですね 😢 正誤表を更新しました!!

ちなみにこのあたりはリスト20.32の実装ですでに修正・追加されていると思うのですが、いかがでしょうか。

-	r := store.Repository{Clocker: clock.RealClocker{}}
+	clocker := clock.RealClocker{}
+	r := store.Repository{Clocker: clocker}
+	rcli, err := store.NewKVS(ctx, cfg)
+	if err != nil {
+		return nil, cleanup, err
+	}
+	jwter, err := auth.NewJWTer(rcli)
+	if err != nil {
+		return nil, cleanup, err
+	}

[smirror]

ご回答ありがとうございます。

ちなみにこのあたりはリスト20.32の実装ですでに修正・追加されていると思うのですが、いかがでしょうか。

そうなんですね! 手元にあるのが初版なので、探しても無いなと思って読んでいました。
確認していたのですが既に修正済みなのを見落としていてすみませんでした:bow:

[budougumi0617]

おっ jwter, err := auth.NewJWTer(rcli) あたりは初版でも載っているはず（P250です）が、diffになっていないのでわかりにくいかもしれません（違うところの指摘でしたらごめんなさい）。

いえ、他にもなにかわかりにくいところがありましたらいくらでもコメントください！