# 8/11 ポートフォリオレビュー ### githubのリポジトリ https://github.com/hata-junnosuke/runboost オンラインエディタで確認できるやつ https://github1s.com/hata-junnosuke/runboost ### サイトのURL https://runboost.herokuapp.com/users/sign_in ## サービス面 - Bootstrap臭が強く就活のためのポートフォリオ感がある - いろいろ機能があってすごい反面導線が分かりづらい気がした - デザインを変えるだけで見違えるようになるはず - NikeRunClubなどのアプリとの差別化が難しそう ## 技術面 - README.mdがデフォのままになってるので環境構築方法やサービス概要を書いた方が良い ( https://github.com/ryota1116/zero_calorie ) - 不要なファイルが残ってる。(例 https://github.com/hata-junnosuke/runboost/blob/master/quit ) - 他人のプロフィールを編集できてしまう https://runboost.herokuapp.com/users/4/edit - `/profile/edit` - コミットログが適当(5.13 とか) - テスト書いてない - 外部キー制約がついていない https://github.com/hata-junnosuke/runboost/blob/master/db/schema.rb - references型でuser_idなどの外部キーを作ってる https://github.com/hata-junnosuke/runboost/blob/master/db/migrate/20210515102312_create_blogs.rb - `rails g model blog title:string body:text user:references` - NOT NULL制約が全然ついてない - `dependent: :destroy`などのオプションがついてないhttps://github.com/hata-junnosuke/runboost/blob/08f8c76312567e8f66b353364d344f4b9dc02211/app/models/user.rb#L8 - rubocopで指摘されるはずなのになおってないということはrubocopを走らせてなさそう - モデルのバリデーションが甘い。usernameのpresence: trueとかがない - rubyのバージョンが古い https://github.com/hata-junnosuke/runboost/blob/master/.ruby-version - 他人のblogを削除できてしまう https://github.com/hata-junnosuke/runboost/blob/08f8c76312567e8f66b353364d344f4b9dc02211/app/controllers/blogs_controller.rb#L34 `current_user.blogs.find(params[:id])` # その他 ## RSpec [使えるRSpec入門・その1「RSpecの基本的な構文や便利な機能を理解する」 \- Qiita](https://qiita.com/jnchito/items/42193d066bd61c740612) [使えるRSpec入門・その2「使用頻度の高いマッチャを使いこなす」 \- Qiita](https://qiita.com/jnchito/items/2e79a1abe7cd8214caa5) [使えるRSpec入門・その3「ゼロからわかるモック(mock)を使ったテストの書き方」 \- Qiita](https://qiita.com/jnchito/items/640f17e124ab263a54dd) [使えるRSpec入門・その4「どんなブラウザ操作も自由自在!逆引きCapybara大辞典」 \- Qiita](https://qiita.com/jnchito/items/607f956263c38a5fec24) [今日から使える!RSpec 3で追加された8つの新機能 \- Qiita](https://qiita.com/jnchito/items/a4a51852c2c678b57868) [Capybaraチートシート \- Qiita](https://qiita.com/morrr/items/0e24251c049180218db4) [RSpec スタイルガイド](https://github.com/willnet/rspec-style-guide) [FactoryBot the Right Way / toshimaru](https://www.youtube.com/watch?v=n0epZM-lZvw&feature=youtu.be) [Clean Test Code Revised \- Speaker Deck](https://speakerdeck.com/willnet/clean-test-code-revised) [rspecを読みやすくメンテしやすく書くために](https://zenn.dev/yuji_developer/articles/52cc0e356b3748) [Everyday Rails… Aaron Sumner 著 et al\. \[Leanpub PDF/iPad/Kindle\]](https://leanpub.com/everydayrailsrspec-jp)