質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.47%

  • Ruby

    7938questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby 問題

解決済

回答 3

投稿 ・編集

  • 評価
  • クリップ 0
  • VIEW 1,466

K_T_T_K

score 304

Ruby初心者です。
まだRubyをはじめて本日で3日目の者です。
今までPHPで開発してきたのですが、Rubyを勉強することにしてみました。
FizzBuzzなどその他の問題をやりながら勉強しています。
今回は、Rubyで閏年問題プログラムを作ってみようと思い書いてみました。
以下コードです。
def days(year, month)
  if (year % 4 == 0 && year % 400 == 0 || (year % 100 != 0))
    if (month == 1 || 3 || 5 || 7 || 8 || 10 || 12)
      return 31
    elsif (month == 4 || 6 || 9 || 11)
      return 30
    elsif (month == 2)
      return 28
    end
  else
    if (month == 1 || 3 || 5 || 7 || 8 || 10 || 12)
      return 31
    elsif (month == 4 || 6 || 9 || 11)
      return 30
    elsif (month == 2)
      return 29
    end 
  end
end
puts "年を入力してください"
year = gets.to_i
puts "月を入力してください"
month = gets.to_i
days = days(year, month)
puts "#{year}年の#{month}月は#{days}日です。"
かなり冗長なコードで良くないアルゴリズムですが、これは正しいプログラムなのでしょうか?
間違っていたら添削としてアドバイスを頂たいと思っています。
また、アンサーの方が今回の閏年問題プログラムを作成した場合、どのように
コードを書いたか別解も載せて頂ければかと思います。
宜しくお願いします。
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+3

年月と日数の関係は正しいものとして、話を進めていきます

書き換えたあとのコードをぱっと出すよりは過程も載せたほうが良いとおもって、かなり長くなっています
しかし、参考になれば幸いです


* まず初めにRuby っぽい書き方に換えていきます。うるう年判定もライブラリのメソッドを利用します
※ 関数のなかでの return は明示的に値を返さなければならない場合を除いて 省略することが出来ます
require 'date'
def days(year, month)
  # if (year % 4 == 0 && year % 400 == 0 || (year % 100 != 0))
  if Date.leap?(year)
    # if (month == 1 || 3 || 5 || 7 || 8 || 10 || 12)
    [1, 3, 5, 7, 8, 10, 12].include?(month)
      31
    # elsif (month == 4 || 6 || 9 || 11)
    elsif [4, 6, 9, 11].include?(month)
      30
    # elsif (month == 2)
    elsif month.equal(2)
      28
    end
  else # ....
  # ....
end

* 判定内容に名前を付けてあげます(この段の命名はもうすこし考えたほうがいいとは思います...)
def days31?(month)
  [1, 3, 5, 7, 8, 10, 12].include?(month)
end

def days30?(month)
  [4, 6, 9, 11].include?(month)
end

def days28?(month)
  month.equal?(2)
end

def days29?(month)
  days28?(month)
end

def days(year, month)
  if Date.leap?(year)
    if day31?(month)
      31
    elsif days30?(month)
      30
    elsif days28?(month)
      28
    end
  else
    if days31?(month)
      31
    elsif days30?(month)
      30
    elsif days29?(month)
      29
    end
  end
end

* ひとつ前の書き換えで、フクザツに見えていた重複点が見易くなりました。ここでさらに、case when に書き換えてみます
require 'date'

def days(year, month)
  case month
  when 1, 3, 5, 7, 8, 10, 12
    30
  when 4, 6, 9, 11
    31
  when 2
    Date.leap?(year) ? 28 : 29
  end
end

def display_day_of_month
  puts '年を入力してください'
  year = gets.to_i
  puts '月を入力してください'
  month = gets.to_i
  days = days(year, month)
  puts "#{year}年の#{month}月は#{days}日です。"
end

以上で、コードがだいぶスッキリした見た目になったと思います

長くなりましたが:
* フクザツな処理(ここでの、うるう年判定)はライブラリにお任せする
* 変数のみならず、様々なものに名前を付けてあげる(あるいは解説コメントを付ける)
* 関数は10行前後に留める

という点を意識して、ここまで書き換えてみました

## Links
オブジェクト指向スクリプト言語 Ruby リファレンスマニュアル
Ruby初学者のためのオンラインドキュメントまとめ - Qiita

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2014/11/25 09:18

    回答ありがとうございます。
    大変丁寧わかりやすい説明ありがとうございました。
    参考になります。
    ベストアンサーにさせていただきます。

    キャンセル

+3

ruby の学習という点では、自分でいろいろな条件判定を行うコードを書くのがよいのですが、
アプリケーションの中でつかうなら、既存ライブラリーをつかうほうが、バグがすくなくなります。

私なら次のようにしてしまいます。
# coding: utf-8
require 'date'

def days(year, month)
  (Date.new(year, month, -1) - Date.new(year, month, 1)).to_i + 1
end

(2000..2001).each do |y|
  (1..12).each do |m|
    puts "#{y}年の #{m}月は #{days(y, m)} 日です。"
  end
end
実行結果は次のようになります。
2000年の 1月は 31 日です。
2000年の 2月は 29 日です。
2000年の 3月は 31 日です。
2000年の 4月は 30 日です。
2000年の 5月は 31 日です。
2000年の 6月は 30 日です。
2000年の 7月は 31 日です。
2000年の 8月は 31 日です。
2000年の 9月は 30 日です。
2000年の 10月は 31 日です。
2000年の 11月は 30 日です。
2000年の 12月は 31 日です。
2001年の 1月は 31 日です。
2001年の 2月は 28 日です。
2001年の 3月は 31 日です。
2001年の 4月は 30 日です。
2001年の 5月は 31 日です。
2001年の 6月は 30 日です。
2001年の 7月は 31 日です。
2001年の 8月は 31 日です。
2001年の 9月は 30 日です。
2001年の 10月は 31 日です。
2001年の 11月は 30 日です。
2001年の 12月は 31 日です
参考
- 月末の日付を得る http://rubytips86.hatenablog.com/entry/2014/03/29/082341

ActiveSupport というライブラリーを使うなら、days_in_month というそのものズバリのメソッドがあります。
参考:
- days_in_month http://apidock.com/rails/ActiveSupport/CoreExtensions/Time/Calculations/ClassMethods/days_in_month
ページ下部にある show source をクリックするとソースコードをみることもできます。


投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+1

... ruby の学習という点では、自分でいろいろな条件判定を行うコードを書くのがよいのですが ...

ライブラリーに頼らず、自分で条件判定をするようなコードを書いてみました。
utils.rb と 動作の簡易チェックのための utils_test.rb の2つを以下に示します。
(test も rspec を使わずに書いてみました。
 yard などで API ドキュメントを生成した時に、適切なドキュメントが生成されうようなコメントを
いれることもしなければいけまsんが、今回はそこはサボりました)

$ rubocop utils.rb でのコードスタイルのチェックでも 変な警告は出ないようにしてあります。
(gem install rubocop すると rubocop コマンドが使えるようになります)

utils.rb
# coding: utf-8

# 便利メソッドを定義する。
module Utils
  # 閏年でないときの各月の日数
  NORMAL_DAYS = {
    1 => 31, 2 => 28, 3 => 31,  4 => 30,  5 => 31,  6 => 30,
    7 => 31, 8 => 31, 9 => 30, 10 => 31, 11 => 30, 12 => 31
  }.freeze

  # year 年 month 月の日数を求める
  def days(year, month)
    check_year_month(year, month) # year, month の値範囲をチェックする。

    # 閏年の2月は 29 日まである。
    if month == 2 && leap?(year)
      29
    else
      NORMAL_DAYS[month]
    end
  end

  # year が閏年かを調べる。
  def leap?(year)
    (year % 4 == 0) && ((year % 400 == 0) || (year % 100 != 0))
  end

  # year, month の値範囲をチェックする。
  def check_year_month(year, month)
    if (year < 1900) || (2038 < year) || (month < 1) || (12 < month)
      throw ArgumentError.new("#{year}, #{month}")
    end
  end
end
# --- End of File ---

utils_test.rb
# coding: utf-8

require File.join(File.expand_path(File.dirname(__FILE__)), 'utils.rb')

#--------- test ------
include Utils

TESTS = [
  [2000, 1, 31],
  [2000, 2, 29],
  [2000, 3, 31],
  [2000, 4, 30],
  [2000, 5, 31],
  [2000, 6, 30],
  [2000, 7, 31],
  [2000, 8, 31],
  [2000, 9, 30],
  [2000, 10, 31],
  [2000, 11, 30],
  [2000, 12, 31],

  [2001, 1,  31],
  [2001, 2,  28],
  [2001, 3,  31],
  [2001, 4,  30],
  [2001, 5,  31],
  [2001, 6,  30],
  [2001, 7,  31],
  [2001, 8,  31],
  [2001, 9,  30],
  [2001, 10, 31],
  [2001, 11, 30],
  [2001, 12, 31]
]

TESTS.each do |t|
  ans = Utils.days t[0], t[1]
  fail "#{t.join(' ')} != #{t[0]} #{t[1]} #{ans}" unless ans == t[2]
end

ERR_TESTS = [
  [nil, nil,   "undefined method `<' for nil:NilClass"],
  [2000, nil,  "undefined method `<' for nil:NilClass"],
  [nil, 1,     "undefined method `<' for nil:NilClass"],

  [1899, 12,   'uncaught throw #<ArgumentError: 1899, 12>'],
  [2039, 1,    'uncaught throw #<ArgumentError: 2039, 1>'],

  [2000, 0,    'uncaught throw #<ArgumentError: 2000, 0>'],
  [2000, 13,   'uncaught throw #<ArgumentError: 2000, 13>']

]
ERR_TESTS.each do |t|
  begin
    Utils.days t[0], t[1]
    fail 'エラーが発生しませんでした。'
  rescue => ex
    raise ex.to_s + t.join(' ') if ex.to_s != t[2]
  end
end

# $ rubocop days.rb
# $ ruby days_test.rb

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2014/11/26 01:41

    Katoyさん
    いつも詳しい回答ありがとうございます!
    大変参考になります!
    また、Rubocopを教えて頂ありがとうございます!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.47%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Ruby

    7938questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。