第５回課題（大倉）

[ookura-mf]

No description provided.

[mf-sakura]

@ookura-mf
概ね良さそうに思いました:+1:
エラーハンドリング周りで1点だけコメントしました。

[mf-sakura]

Goだと以下の様にする事が多いです。
1行になってシンプルになったり、変数のスコープがif文に狭まるのでメモリ効率が良いです。

Suggested change

	err := userController.Update(*id, *firstName, *lastName)
	if err != nil {
	if err := userController.Update(*id, *firstName, *lastName); err != nil {

[ookura-mf]

なるほど 👀
rubyだとifで代入するのは可読性が悪いので設定によってはlinterに怒られたりしますが、ここではerrに代入するだけなのでこの方が良さそうですね

[mf-sakura]

これも一つ目の戻り値を使わないのであれば、以下の方が良いと思います。

Suggested change

	_, err := tx.Exec("UPDATE users SET first_name = ?, last_name = ? WHERE id = ?", u.FirstName, u.LastName, u.ID)
	if err != nil {
	if _, err := tx.Exec("UPDATE users SET first_name = ?, last_name = ? WHERE id = ?", u.FirstName, u.LastName, u.ID); err !=nil{}