teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

1

コメント追加

2018/12/15 06:31

投稿

takahashim
takahashim

スコア1877

answer CHANGED
@@ -36,11 +36,13 @@
36
36
 
37
37
  ### Calendarクラスの例
38
38
 
39
+ もう少し手を入れた方が良さそうですが、元のコードとかけ離れてしまっても分かりにくいかと思うので、あまり変えずにクラス化しました。
39
40
 
40
41
  ```ruby
41
42
  require "date"
42
43
 
43
44
  class Calendar
45
+ ## month_tableは共通で使うので定数にしています
44
46
  MONTH_TABLE = [0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31]
45
47
 
46
48
  # コンストラクタ
@@ -50,6 +52,7 @@
50
52
  end
51
53
 
52
54
  #うるう年か?
55
+ ## Rubyでは真偽値を返すメソッドは`is_xxx`ではなく`xxx?`という命名にする場合が多いです
53
56
  def uruu?(year)
54
57
  (year % 4 == 0 and year % 100 != 0) or year % 400 == 0
55
58
  end