Add hbday command

[hirata0813]

概要

hbday コマンドの追加

変更点

• お祝いメッセージを作成する処理をコマンドの方に記述した．
• BirthdayMembersクラスをActiveMembersクラスに変更し，在籍メンバを取り出すメソッドのみを定義した．
• active_membersの要素を一つずつ取り出して，誕生日の判定とお祝いメッセージの表示を一つのループ内で行うようにした．

コードレビューで頂いたアドバイスを基に，コードを修正しました．

コマンドの概要

機能

研究室メンバのうち，その日誕生日の人をお祝いする．

使用方法

swimmy hbday と入力する．

特徴

名簿から在籍メンバの誕生日情報を取得し，それをその日の日付と比較する．

[yoshinari-nomura]

1. Member に birthday? メソッドがないように思えます．
2. ActiveMembers クラスは不要だと思いますclass Hbday の直下に private で fetch_active_members を定義するか，active_members = ... に fetch_active_members の中身を直接書き下してもいいと思います．
3. コメントの # の後に半角スペースを1つ入れましょう．できれば英語で書きましょう．

[hirata0813]

member.rbを変更したコミットとhbday.rbを修正したコミットを1つのPRにまとめました．
以下，変更点です．

member.rbの変更点

名簿の「電話番号(phone)」カラムの後ろに「誕生日(birthday)」カラムを追加するため，member.rbに以下の処理を追加しました．

• attr_reader，initializeメソッド，to_sメソッドにbirthdayカラム用の処理を追加

  • birthdayカラムを日付型として使えるように，以下の処理を追加
  • initializeメソッド内でDate.parse()メソッドを使用
  • to_sメソッド内でstrftime()メソッドを使用

• セルが空の場合でも正しく動作するよう以下の処理を追加

  • initializeメソッド内で，Date.parse()がエラーを起こした場合@birthdayにnilを入れる

• 誕生日かどうかを判定する birthday?() メソッドを追加

• birthday?()メソッドが副作用を起こさないよう，引数として日付を渡す

hbday.rbの修正点

指摘していただいた点をもとに，コードを以下のように修正しました．

• ActiveMembersクラスを無くし，10行目にActiveMembersクラス相当の処理内容を記述

• コメントを英語で書き直し，#の後に半角スペースを1つ挿入

[hirata0813]

操作を間違えてしまい，一度PRをcloseしてしまいました．申し訳ございません．

[yoshinari-nomura]

いい感じです．あとは，インデントを 2 にしてもらえますか．

[hirata0813]

インデントを2に修正しました．