Enhance/#45 create seed #53
Conversation
|
IMO 今回、ランダムなダミーデータを生成するfakerというgemが既にインストールされているので １つの使用例としては以下のような形になります ユニークなemailアドレスの生成 ランダムな文字列の生成 以下が公式になります！ |
backend/db/seeds.rb
Outdated
| Memo.create!( | ||
| title: "title_#{n+1}", | ||
| content: "content_#{n+1}" | ||
| ) | ||
|
|
||
| Comment.create!( | ||
| memo_id: n+1, | ||
| content: "content_#{n+1}" | ||
| ) | ||
| end |
There was a problem hiding this comment.
初PRお疲れ様です！僕も初レビュー頑張ります！笑
コメントはメモに紐づけることと、１つのメモに対して複数のコメントをつけたいと思いました。
以下のように今読み出しているmemoを変数に入れて、そのidをcommentのmemo_idに設定して作成すると良いと思いました！
memo = Memo.create!(
title: "title_#{n+1}",
content: "content_#{n+1}"
)
10.times do |m|
Comment.create!(
memo_id: memo.id,
content: "comment_#{m+1} for #{memo.title}"
)
There was a problem hiding this comment.
レビューいただきありがとうございます！
指摘いただいた仕様部分の修正を行いましたので、再度ご確認お願いいたします。
fakerについては、ご指摘いただいた部分に加えて、パスワードもランダム生成できるようでしたので、fakerを使用する形で修正しました！
backend/db/seeds.rb
Outdated
| Comment.create!( | ||
| memo_id: memo.id, | ||
| content: "comment_#{m+1} for #{memo.title}" | ||
| ) |
There was a problem hiding this comment.
せっかくなので
CommentのcontentもFaker使用しても良いかなと思いました！
backend/db/seeds.rb
Outdated
| memo = Memo.create!( | ||
| title: Faker::Lorem.sentence(word_count:5), | ||
| content: Faker::Lorem.sentence(word_count:5) | ||
| ) |
There was a problem hiding this comment.
titleとconentでは生成する文字数変更して実際に想定されるデータに近い形式にすると良いかなと思いました！
理由としてはフロントエンド側(フロントに限らずですが）でこれらのダミーデータを使用して画面の表示確認等をした際に、より本番に近い状態で検証できるからです！
今回のアプリでいうとSlackの「就職関連の共有や相談」チャンネルに投稿された内容が入るのかなと思うので、それぐらいの長さを想定してみてはどうでしょうか！
Fakerにはsentence以外に段落を入れて複数の文を生成するなどできるオプションもあるので確認してもらえたらと思います！
backend/db/seeds.rb
Outdated
|
|
||
| 10.times do |n| | ||
| User.create!( | ||
| # email: "example_#{n+1}@example.com", |
There was a problem hiding this comment.
こちらのコメントは不要かと思うので削除してもらえたらと思います！（何か意図あったらすみません)
|
@rikuya98 |
backend/db/seeds.rb
Outdated
| 10.times do |n| | ||
| User.create!( | ||
| email: Faker::Internet.unique.email, | ||
| password: Faker::Internet.password(min_length: 8) | ||
| ) | ||
|
|
||
| memo = Memo.create!( | ||
| title: Faker::Lorem.sentence(word_count:10), | ||
| content: Faker::Lorem.paragraphs(number: 5) | ||
| ) | ||
| 10.times do |m| | ||
| Comment.create!( | ||
| memo_id: memo.id, | ||
| content: Faker::Lorem.sentence(word_count:5) | ||
| ) | ||
| end | ||
| end |
There was a problem hiding this comment.
Nits/Idiomatic
rubyの場合はインデントは2 spaceなので、揃えていただきたいです〜 🙏
追加で、このコーディング規約をRubocopのルールに含めたいので、issueを切っていただきたいです！
（以下のような記述を.rubocop.ymlに追加すると強制できる。）
Layout/IndentationWidth:
Width: 2参考: https://docs.rubocop.org/rubocop/cops_layout.html#layoutindentationstyle
There was a problem hiding this comment.
レビューありがとうございます！
2 space承知しました、修正します！
また、下段のissueを切る部分について、勉強不足で大変申し訳ないのですが内容がよくわからず、もう少しご説明いただくことは可能でしょうか、、、
This reverts commit 6f4be6d.
対応するissue
対応内容