search

shinonome-inc / mobile_tak848

0 stars 0 forks source link

課題1の閏年判定を実装 #1

Closed TAK848 closed 2 years ago

TAK848 commented 2 years ago

課題1の閏年判定を実装しました。 レビューをお願いします。 動作画像:

Screen Shot 2022-01-29 at 17 31 21
TAK848 commented 2 years ago

今回はifで真面目に書きましたが，これくらいなら

     return (year % 4 == 0 && year % 100 != 0) || year % 400 == 0

とするのもありかなと思いました。

Nanami-Kurata commented 2 years ago

インデントも揃えられていてとてもいいと思います！

今回はifで真面目に書きましたが，これくらいなら

     return (year % 4 == 0 && year % 100 != 0) || year % 400 == 0

とするのもありかなと思いました。

↑まとめてしまう方がコードが少なくなると思うので、是非こっちでやってみてください。

mcz9mm commented 2 years ago

自分もこれでやった記憶が👍 https://github.com/shinonome-inc/mobile_tak848/pull/1#issuecomment-1024857774

TAK848 commented 2 years ago

皆様ありがとうございます！1行に統合しました！

Nanami-Kurata commented 2 years ago

LGTMです！