ポートフォリオスパルタコースでは毎週何かしらのイベントを開催しています。前の内容ですが、今回もまた「Railsポートフォリオレビュー会」です。あ、週に一人くらいまでなら無償でレビューできると思うのでお気軽にお声がけください。TechEssentials雑なメモですが、こんなフィードバックをしました。permalink」からURLをコピって貼り付けていきましょー。:userはなくていい?コミットメッセージ「1」とかはわかりづらすぎるREADMEに何も書かれていないのでどんなアプリなのかわからない。テーブル設計とかインフラ構成とかあってもいいかも。endassociationを使えばメソッドが不要なはず。user&.authenticate(params[:session][:password])かろりーなhttps://github.com/RyotaroNishimura/toikke/blob/master/config/database.ymlmysqlのユーザーとパスワードが直書きになっているので、.gitignoreファイル等で管理に含めないほうがよさそうだと思いました。JUNhttps://github.com/RyotaroNishimura/toikke/blob/c910cb2941370a399ead0591150222815110229e/app/views/layouts/application.html.erb#L3viewportを記載した方がスマホでみやすくなると思いました!てるhttps://github.com/RyotaroNishimura/toikke/blob/c910cb2941370a399ead0591150222815110229e/app/models/user.rb#L92せっかく関連付けしているのでFavoriteでなく、favoritesを使った方が良いかと!(みけたさんがpostクラスに言及していることとほぼ同じです!)https://github.com/RyotaroNishimura/toikke/blob/c910cb2941370a399ead0591150222815110229e/app/models/user.rb#L100存在しているかどうかの判定ならpresent?使った方が明示的でわかりやすいかと!https://github.com/RyotaroNishimura/toikke/blob/c910cb2941370a399ead0591150222815110229e/app/models/post.rb#L8belongs_toで関連付けするとデフォルトで外部キーのnilにvalidateがかかります。guideに乗っております!(https://railsguides.jp/upgrading_ruby_on_rails.html?version=6.1#active-record%E3%81%AEbelongs-to%E3%81%AF%E3%83%87%E3%83%95%E3%82%A9%E3%83%AB%E3%83%88%E3%82%AA%E3%83%97%E3%82%B7%E3%83%A7%E3%83%B3%E3%81%A7%E5%BF%85%E9%A0%88)https://github.com/RyotaroNishimura/toikke/blob/c910cb2941370a399ead0591150222815110229e/app/controllers/users_controller.rb#L2ログインを許可しないアクションの方が少なそうなのでonlyでなくexcept使った方がコード短くて良いかなと!https://github.com/RyotaroNishimura/toikke/blob/c910cb2941370a399ead0591150222815110229e/app/controllers/users_controller.rb#L81登録されたユーザーでないと編集ができないにしているのでしょうか?logged_in_userでログインしているかどうかを制限しているかつさらにeditとupdateに関して現在のユーザーであるかどうか判定する理由が何故なのか知りたいです!!https://github.com/RyotaroNishimura/toikke/blob/c910cb2941370a399ead0591150222815110229e/app/controllers/posts_controller.rb#L35update_attributesを使用したのか気になりました!https://github.com/RyotaroNishimura/toikke/blob/c910cb2941370a399ead0591150222815110229e/app/controllers/posts_controller.rb#L44current_user使った方が安全性増すと思います!!こんな感じで毎週勉強会をやってます。スパルタコースはサポートの質が落ちないように少数のコミュニティとして活動しています。実際に未経験からRailsの業務委託で仕事をしている人もいるくらいなのでちゃんと学習すればスキルはつきます。普段はSlackでやりとりしてますのでリンクを貼っておきますね。Slackコミュニティへの招待リンクではでは。この記事を書いた人に相談してみるだいそん@Railsエンジニアはじめまして、Webエンジニアのだいそんです。エンジニア歴は10年目。Reactを使ったサービス開発を行っています。その傍ら別のWeb系企業のCTOを努めています。これまで200人以上の方にメンタリングを行っており、Web系企業へ転職された方やフリーランスになられた方など様々な方のキャリアアップをサポートしてきました。2023年1月21日だいそん@Railsエンジニアはじめまして、Webエンジニアのだいそんです。エンジニア歴は10年目。Reactを使ったサービス開発を行っています。詳しくはこちらだいそん@Railsエンジニアさんの他の記事Railsポートフォリオレビュー会を開催しましたRailsポートフォリオレビュー会を開催しました。Railsポートフォリオレビュー会を開催しました。Railsポートフォリオレビュー会Markdown記法の書き方Markdownを使った書き方をご紹介します。約3,000名のメンターから選べるプログラミング、デザイン、マーケティング、起業など学びたいスキルでピンポイントに探せます。見つからない場合は、メンターを募集することも可能です。個別に教えてもらえるあなたが悩んでいることをダイレクトに相談して、マンツーマンで解決までたどり着くことができます。リーズナブルスクールなどの企業ではなく、個人のメンターと直契約するため比較的リーズナブルに教えてもらうことができます。