search

yasu575 / zuboramen

0 stars 0 forks source link

README作成 #1

Closed yasu575 closed 1 year ago

yasu575 commented 1 year ago

確認お願いします。

shurijoc commented 1 year ago

提出ありがとうございます！

全体的に良いと思います。カップラーメンのトッピング困るのでぜひ欲しいです！

就活を踏まえて、以下の観点からレビューさせていただきますね。

💡 就活レビュー観点

1. 課題に対してサービス内で適切な解決策となる機能を提供できているか
2. 就活において十分な技術力を示せる機能を実装しているか
3. 実装期間が適切に収まるか

1に関して

サービスを開発する上で「課題に対して、適切な仮説を立てて、その仮説に対して妥当な解決方法を機能として実装しているか」というところが大事になります。

今回はとてもシンプルな課題感と解決策なので良いと思います！

少しだけ、「コメント機能」と「お気に入り機能」については課題の解決としてプラスアルファだと思うのでMVPではなくて良いかなと思います。 ランキングについてはあると探しやすくなりますしとても良いですね！

2に関して

💡 技術的なところでいうと以下のような箇所がポイントです。

- 基本的なRailsのCRUDを使っている
- CRUDにプラスαで独自のロジックを使用している
  - 例: ActionCableやHotwireなどRailsの技術を使用している

今回でいうと基本的なRails CRUD + APIで構成されているのでプラスアルファの技術アピールポイントが必要かと思います。 例えば以下のような機能などがありますので入れてみてはいかがでしょうか？

3に関して

実装期間の目安は次の通りです。

これを超えると開発するもののハードルが高くなってしまう・モチベーションが続かないなどといったことが起こるので一つの目安としてください！

📆 例
フルコミットの場合: 1ヶ月
フルコミットではない場合: 2ヶ月

2と3に付随して、例のように使用する予定の技術も書いていただけますか？

// 例
- Rails7
- Bootstrap
- mysql
- Hotwire
- Heroku

以上がレビューとなります、ご確認お願いいたします！

P.S.1

日々レビュー内容も改善しているため、もともとのポートフォリオボードとは、若干異なる観点のレビュー箇所があります、申し訳ございません！

P.S.2

Githubのコミット頻度も就活で見られることが多いです。実装が始まったら細かい単位で良いのでできればほぼ毎日（1週間で5日間）コミットするようにしましょう。

yasu575 commented 1 year ago

1に関しては、「コメント機能」と「お気に入り機能」をMVPではなくて実装予定に変更しようかと思います。

2に関しての画像生成をしてOGP用画像の作成。twitterでシェアできるようにするのと検索、絞り込みにインクリメンタルサーチを使用する。のはアドバイス通りやってみようと思いました。ありがとうございます。 あと、投稿作成の商品名なのですが、マスタ情報としてメジャーなカップ麺のデータをもたせておいて選択するだけで入力できるようにしたいと考えていたのですが、技術的に可能でしょうか？できれば、商品名を統一できるようにしたいのと商品名をわざわざユーザーに打たせなくても分かっている内容は簡単に登録できるようにしたいのですが、アドバイスをお願いしたいです。

3に関しての使用する予定の技術が今のとこrailsしか確実に使うもののイメージができていないのですが、今の段階で何の技術を使えそうかアドバイスいただけますでしょうか？

shurijoc commented 1 year ago

マスタ情報としてメジャーなカップ麺のデータをもたせておいて選択するだけで入力できるようにしたいと考えていたのですが、技術的に可能でしょうか？できれば、商品名を統一できるようにしたいのと商品名をわざわざユーザーに打たせなくても分かっている内容は簡単に登録できるようにしたいのですが、アドバイスをお願いしたいです

こちら可能です。ユーザーとカップ麺マスタを作成して投稿レコードにそれぞれのデータを紐づけるイメージです。 問題となるのはカップ麺情報をどうやって取得するかですが、100種類ほどだと思うので手動でも良いかなという印象です。

スクレイピングしてマスタデータを貯めるという方法もありますが、量に対して少しやり過ぎな機能になってしまうかと思います。

3に関しての使用する予定の技術が今のとこrailsしか確実に使うもののイメージができていないのですが、今の段階で何の技術を使えそうかアドバイスいただけますでしょうか？

Railsのみで良いと思います！あとCSSに関してはTailwind / Daisyuiは使用ハードルが高いわりに高クオリティなものを作るのが難しいのでbootstrapとbootstrapテンプレートのようなすでに用意されているものを使用するのをオススメいたします！

画面遷移図に関して

こちらは別PRを立てて再度URL登録からしていただければと思います。

質問への答えの中で分かりづらい箇所などあったら教えてください！ よろしくお願いします！

yasu575 commented 1 year ago

こちら可能です。ユーザーとカップ麺マスタを作成して投稿レコードにそれぞれのデータを紐づけるイメージです。 問題となるのはカップ麺情報をどうやって取得するかですが、100種類ほどだと思うので手動でも良いかなという印象です。 スクレイピングしてマスタデータを貯めるという方法もありますが、量に対して少しやり過ぎな機能になってしまうかと思います。

ありがとうございます。手動で進めていく方向で考えていきたいと思いますが、まだまだ未熟でわからないことが多く、説明の中ではどうやってやるのかわからない状態です。技術面談を入れつつ、カップ麺情報をどうやっていれていくか相談して決めていく方が良いですか？

Railsのみで良いと思います！あとCSSに関してはTailwind / Daisyuiは使用ハードルが高いわりに高クオリティなものを作るのが難しいのでbootstrapとbootstrapテンプレートのようなすでに用意されているものを使用するのをオススメいたします！

わかりました。技術選定はRails7とbootstrapで行きたいと思います。 修正したので確認をお願いします。

こちらは別PRを立てて再度URL登録からしていただければと思います。

失礼しました。別PRを立てて再度URL登録し直しておきます。 ありがとうございます。

yasu575 commented 1 year ago

画面遷移図の方がレビュー待ちでURLを登録し直せない状態なのですが、どうしたらいいですか？ ご迷惑をかけて申し訳ないです。

shurijoc commented 1 year ago

技術面談を入れつつ、カップ麺情報をどうやっていれていくか相談して決めていく方が良いですか？

こちらの方向でお願いします！ READMEの大枠は良いと思うので、画面遷移図・ER図も先んじて進めていただければと思います！

技術選定

こちら確認して良いと思いました、ありがとうございます！

画面遷移図の方がレビュー待ちでURLを登録し直せない状態なのですが、どうしたらいいですか？

こちら画面遷移図のレビューを一旦差し戻しさせていただいたので再度提出をお願い致します！

yasu575 commented 1 year ago

ありがとうございます。その方向で進めさせていただきます。

こちら画面遷移図のレビューを一旦差し戻しさせていただいたので再度提出をお願い致します！ ありがとうございます。わかりました。