search

lcl-saito / intern-training

0 stars 0 forks source link

メールアドレスがドメイン別にいくつ存在するかを表示する #3

Open lcl-yamashita opened 6 years ago

lcl-yamashita commented 6 years ago

問題

複数のメールアドレスを引数として取り、ドメイン別にメールアドレスが何個あるかを表示するメソッドを作成してください。

以下の条件を満たすこと。

手順

1.メソッドを定義する。 2.引数を設定する。（配列として受け取れるように、引数の前にアスタリスクを書く） 3.受け取った引数をgroup_byメソッドを使用して、ハッシュに変換し、ドメイン毎にキーで分ける。 4.each文を使い、キー名（ドメイン名）: 値数　となるよう出力する。

エビデンス

saito@saito-no-MacBook-Pro ~/p/trainnig> ruby training3.rb
{"gmail.com"=>3, "yahoo.jp"=>2, "ezweb.ne.jp"=>1, "softbank.ne.jp"=>1}
lcl-saito commented 6 years ago

@lcl-yamashita ドメインとありますが、引数としてもらうメールアドレスとは別に、ドメインを設定する（変数）必要があるという事でしょうか？

lcl-yamashita commented 6 years ago

@lcl-saito

いえ、単純に振り分けてカウントすればよいです

lcl-saito commented 6 years ago

@lcl-yamashita ではメールアドレスと同時にドメインも取得する必要があるという事ですね？

lcl-yamashita commented 6 years ago

@lcl-saito いえ、配列で複数のメールアドレスを受け取り、その中の同じのをカウントするだけです

lcl-saito commented 6 years ago

@lcl-yamashita 手順考えました。 確認をお願いします！

手順 1.メソッドを定義する。 2.引数を設定する（配列として受け取れるように、引数の前にアスタリスクを書く） 3.受け取った引数をドメイン毎に違う配列に代入する。（ここをどう処理するか悩み中） 4.それぞれの配列をハッシュに変換する。 5. ドメイン別に、ハッシュの要素と要素数を出力する。

lcl-yamashita commented 6 years ago

カウントをして表示する手順が抜けてますね

lcl-saito commented 6 years ago

ドメインを表示するのではなく、ドメイン数のみを表示すれば良いでしょうか？

lcl-yamashita commented 6 years ago

以下のようにドメインと数を表示してください aaa@sample.com: 4

lcl-saito commented 6 years ago

承知しました。 手順を修正しましたので、レビューをお願いします。

lcl-yamashita commented 6 years ago

@lcl-saito

ちなみに最終的な値は以下のようになることをイメージできていますか？（中身の値はなんでもいいです）

{
  aaa@sample.com: 4,
  aaa@test.com: 10,
  aaa@staging.com: 2
}
lcl-yamashita commented 6 years ago

手順の項目の粒度は問題ないので実装を始めてください。 手順は説明欄へ追記お願いします。

メモややりとりのコメントを残すのは大事なので随時コメントしていいですが、 最終的にレビューをして貰う際は説明欄のみ見れば全て把握できるようにまとめます。 その為、段階的にまとめておくとよいです。

lcl-saito commented 6 years ago

どのように出力するかはイメージしています。

puts "#{hash}: #{hash.length} "

上記の様にしようと思っています。

lcl-saito commented 6 years ago

image 承知しました。 説明欄を編集して、レビューをお願いする様にします。

また、説明欄とは上記の項目でよろしいでしょうか？

lcl-yamashita commented 6 years ago

はい、そこです

lcl-saito commented 6 years ago

承知しました。

lcl-saito commented 6 years ago

@lcl-yamashita

完成したので、レビューをお願いします。

プルリクエスト https://github.com/lcl-saito/intern-training/pull/5/files

lcl-saito commented 6 years ago

@lcl-yamashita 引数をメールアドレスからドメインに変更しました。 レビューをお願いします。

lcl-saito commented 6 years ago

@lcl-yamashita 出力をドメイン毎にしました。 レビューをお願いします。