実務課題のレビュワーをして学んだこと

はじめに

一回殴ってください💦すみませんでしたー！！！！！！

めちゃかき乱して時間食って結局どれも違ったみたいな結末で、へぇーなことはあったかもですが時間だけかけさせた、私はただの迷惑な人です。

ごめんなさい。。。

学び

formではなくてもreact hook form

まず一番のびっくり仰天な箇所です。

ともえさんの実務課題ではreact hook formの使用が仕様書に書いてある要件にありましたが、今回は書いてなかったかつ、各チェックボックスの値が変わる度にAPI叩いて更新処理を行うとのことだったので、formタグは使用しなかった。なので、react hook formを使うという頭にならなかったのですが、コメントに使って管理とあり・・そうなんーーー？？？？？これはもう仕方ないでした。
私のスキルではわかりようがなかったのでもう仕方ない、何かしらinputタグを使うならhook formなんかなと思いました。(今まで私もレビューしていただいててわからなかったので例外はありそう・・)
要件満たしていて動いてて他と同じ書き方で(ここが結構難しいと思うところ・・)型安全かどうかで見ていたので、疑いもしなかったです。

知らなかった・・formじゃなくても使うんだ・・でした。
formタグのonSubmitありきだと思っていました。思い込みだったようです。

渡した値だけupdate

今回、フロントから3つのうち渡された1つを更新する処理だったのですが、バックエンドに結構コメントを多くしていてそれがもう的外れだったこと！！

貴重な時間返せって私なら思います。殴ってください！！！！！！w
ここで2日は食ったような・・申し訳ないしかないです💦

最初のコードが型安全じゃないなと思って、型の修正(3つ全部がオプショナルだったので、3つのうちいずれかに変更)とバリデーションを追加(更新するカラムのkeyを変数にしていたので、他のkeyでも受け取ったカラムを更新しちゃうと思い・・)するようコメントしました。

でも、prismaのdataで渡した項目はundefinedだったら無視されるんですね。。。
なんとなくfalseになって更新されそうって思ってしまっていました(よく考えると存在しているかもわからないundefinedをfalseで更新することも意味不明w)
普通にundefinedなら更新されないだけだからすべてのカラムを渡せばいいだけでした。

めっちゃややこしくしてしまった。。ごめんなさいい。。。