search

shota-hope / True_Neapolitan_Pizza_Club

7 stars 1 forks source link

BATTLE OF RUNTEQ Vol2レビュー #104

Open hisaju opened 2 years ago

hisaju commented 2 years ago

コードはある程度きれいに書かれていて指摘する点もほとんど見当たらなかったので問題ないかなと思いました。1点あるとすればgonを使うのは印象がよくないので、JSONで取得するようにするか、viewのhiddenタグなどに入れてJSからそのhiddenタグを取得するような形で渡されるといいかなと思います。また機能的に単純過ぎて就活に持っていくにはちょっと弱いので写真をもっと使いつつ食べログの機能を模倣していくといいかなと思いました。

https://github.com/shota-hope/True_Neapolitan_Pizza_Club/blob/1b748f507d9f4218a56cbe8b6499c5b0b0c1253c/app/controllers/prefectures_controller.rb#L18

マジックナンバーになってしまっているので、configに切り出しましょう

shota-hope commented 2 years ago

@hisaju ひさじゅさん、お忙しいところレビューありがとうございます！ gon,configの点、早急に改善します。 そして食べログちゃんと使ってみます。。！