wisdom_speaking 2019-02-10 09:07:43

[C#] コードが短い方が綺麗とは言うけれど…… このエントリーをはてなブックマークに追加

投稿者からのアピールポイント

後輩のコード 一瞬何言ってるのかわからなかったけど解説によると うるう年かを判定してそれに色付けするためのコードらしい

とりあえずif文使えと言っておいた

        public string GetColorString()
        {
            var thisYear = DateTime.Today.Year;
            return IsLeapYear(thisYear) ?
                "#FF0000" : IsLeapYear(thisYear + 1) ?
                "#00FF00" : IsLeapYear(thisYear + 2) ?
                "#0000FF" : "#000000";
        }

        public bool IsLeapYear(int startYear)
        {
            return startYear % 400 == 0 || (startYear % 100 != 0 && startYear % 4 == 0);
        }

使い方ヒント: 「これは臭う」という行を見付けたら、各行のsmellをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(1)

#1 rai_suta 2019-02-17 10:56:34  

三項演算子はこう書いたほうが個人的にはまだ読みやすい

return IsLeapYear(thisYear)
  ? "#FF0000"  
  : IsLeapYear(thisYear + 1)
    ? "#00FF00"
    : IsLeapYear(thisYear + 2)
      ? "#0000FF"
      : "#000000";

コメント投稿には、twitter認証が必要です。

Twitter認証

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[C] マジックナンバー回避

このエントリーをはてなブックマークに追加

そもそも使うなって言われてる理由わかって...

#define ZERO 0
#define ONE 1
// (以下略

鑑賞する »

[Java] 変更する時のこと考えてる?

このエントリーをはてなブックマークに追加

その時の仕様では有効日数が3日だったけど...

// ◯◯様の要望により,有効日数を定数化
private static in...

鑑賞する »

[C++] extern "C"を使わず頑張った

このエントリーをはてなブックマークに追加

実際はもっと多くのパラメータがある関数を...

[main.c]
extern int _Z3subii(int, int);...

鑑賞する »