search

HiromuMasuda / category_classifier

1 stars 0 forks source link

view.py の処理 #19

Open y-abe opened 6 years ago

y-abe commented 6 years ago

https://github.com/HiromuMasuda/gunosy_assignment/blob/master/classification_machine/views.py

HiromuMasuda commented 6 years ago

コメントありがとうございます。

https://github.com/HiromuMasuda/gunosy_assignment/pull/25 で修正しました。

HiromuMasuda commented 6 years ago

エラーの内容に応じてHTTPステータスコードを変えてみてください

とありますが、エラーの内容として

などの問題が考えられると思いましたが、それを404や500などで返すのはUXが悪いと考え、全てエラーを吸収し、200で返した上でエラーメッセージを表示するという処理にしていました。

この考え方の改善すべき点、またコメントの意図を教えていただきたいです。

y-abe commented 6 years ago

などの問題が考えられると思いましたが、それを404や500などで返すのはUXが悪いと考え、全てエラーを吸収し、200で返した上でエラーメッセージを表示するという処理にしていました。

確かにそうですね。HTMLを描画する場合はこれで問題ないです！ 失礼しました。

（実際にサービスを作るときはステータスコード別にテンプレートを用意しておく場合が多いかと思います。今回はやらなくて大丈夫ですが）