星座を返すアプリケーションを作成したい

Question

前提・実現したいこと

ユーザーから生年月日を受け取って星座を返すプログラムを作成したい。

要件

• 生年月日の標準入力を受け取る
• 当てはまる星座を返却する

参考

• 生年月日入力フォーマット: {yyyy-mm-dd}
• 出力星座フォーマット: あなたの星座は{ 当てはまる星座名 }です！

該当のソースコード

puts "生年月日を入力してください"
puts "{yyyy-mm-dd}"
input = gets.chomp
yyyy, mm, dd = input.split("-").map(&:to_i)

if mm == 3
  if 21 <= dd && dd <= 31
    puts "あなたの星座は牡羊座です！"
  elsif 1 <= dd && dd <= 20
    puts "あなたの星座は魚座です！"
  else
    puts ""
  end
end
  if mm == 4
    if 1 <= dd && dd <= 19
      puts "あなたの星座は牡羊座です！"
    elsif 20 <= dd && dd <= 30
      puts "あなたの星座は牡牛座です！"
    else
      puts ""
  end
end
if mm == 5
  if 1 <= dd && dd <= 20
    puts "あなたの星座は牡牛座です！"
  elsif 21 <= dd && dd <= 31
    puts "あなたの星座は双子座です！"
  else
    puts ""
  end
end
if mm == 6
  if 1 <= dd && dd <= 21
    puts "あなたの星座は双子座です！"
  elsif 22 <= dd && dd <= 30
    puts "あなたの星座はかに座です！"
  else
    puts ""
  end
end
if mm == 7
  if 1 <= dd && dd <= 22
    puts "あなたの星座はかに座です！"
  elsif 23 <= dd && dd <= 31
    puts "あなたの星座は獅子座です！"
  else
    puts ""
  end
end
if mm == 8
  if 1 <= dd && dd <= 22
    puts "あなたの星座は獅子座です！"
  elsif 23 <= dd && dd <= 31
    puts "あなたの星座は乙女座です！"
  else
    puts ""
  end
end
if mm == 9
  if 1 <= dd && dd <= 22
    puts "あなたの星座は乙女座です！"
  elsif 23 <= dd && dd <= 30
    puts "あなたの星座は天秤座です！"
  else
    puts ""
  end
end
if mm == 10
  if 1 <= dd && dd <= 23
    puts "あなたの星座は天秤座です！"
  elsif 24 <= dd && dd <= 31
    puts "あなたの星座は蠍座です！"
  else
    puts ""
  end
end
if mm == 11
  if 1 <= dd && dd <= 22
    puts "あなたの星座は蠍座です！"
  elsif 23 <= dd && dd <= 30
    puts "あなたの星座は射手座です！"
  else
    puts ""
  end
end
if mm == 12
  if 1 <= dd && dd <= 21
  	puts "あなたの星座は射手座です！"
  elsif 22 <= dd && dd <= 31
    puts "あなたの星座は山羊座です！"
  else
    puts ""
  end
end
if mm == 1
  if 1 <= dd && dd <= 19
    puts "あなたの星座は山羊座です！"
  elsif 20 <= dd && dd <= 31
    puts "あなたの星座は水瓶座です！"
  else
    puts ""
  end
end
if mm == 2
  if 1 <= dd && dd <= 18
    puts "あなたの星座は水瓶座です！"
  elsif 19 <= dd && dd <= 29
    puts "あなたの星座は魚座です！"
  else
    puts ""
  end
end

どこがおかしいか教えていただきたいです。

Comments

[dodox86]

現在のコードで、どういう問題があるのですか？

[stdio]

PHPの仕様は詳しくありませんが、文字列と数字を比較しようとして違うとか言われているとかですかね。
それなら一度mmとddの型変更をしてみてはいかがでしょうか？

[dodox86]

＞stdioさん
PHPではなく、Rubyですが。。。(タグにも付いています）また、オブジェクトのクラス自体はInteger(整数)になっているようです。

irb(main):006:0> yyyy,mm,dd = gets.chomp.split("-").map(&:to_i)
2020-06-18
=> [2020, 6, 18]
irb(main):007:0> p yyyy.class
Integer
=> Integer
irb(main):008:0> p mm.class
Integer
=> Integer
irb(main):009:0> p dd.class
Integer
=> Integer

[stdio]

Rubyでしたか、これは失礼しました。
それなら、どんな問題があるのでしょうかね。
プログラムを見ている限り、特にエラー等もなさそうですが...

[winterboum]

ざっと見ではおかしくないですが、期待どおりの動きでないとすると、どこでしょう？
それをいただかないときついです

[winterboum]

んでもって、星座の変わる日が年に依ってすこしずつ動くのご存知ですか？

Answer 1

勝手に推定するに「一応動くものはできたのだが、『より良いプログラム』にするという視点でおかしな所はないか？」という問いであろうと考えます。
良い の判断にはどういう物差しを使うのか、がありここは好みとか思想とか哲学が入ってくるので色々ですが

1. 短い
2. 場合分けやloopのネストが浅い
3. 似たような処理の繰り返しが(すく）ない

辺りはまぁ共通だろうと思います。
その視点でみると、２はまぁ良いとして 1,3に引っかかります。

例えば

[["牡羊",[3,21..31],[4,1..19],
 ["牡牛",[4,20..30],[5,1..20],
   :
   :
]

と言うような配列を用意して、loopしてif,elsif を一組で済ませるというのを考えてみてください。

個人的な好みとしては
yday = Time.new(*(input.split("-").map(&:to_i))).ydayして1/1からの経過日数を得て、["牡羊",81..110],,,, にしたいところです。
これだとloopも無くすことができるかも。
なのですがこれだとうるう年かどうかで二組用意してどちらを使うかの判定が必要になるからちとやだな、、との思いもあります。