1. C# / Говнокод #13599

    +135

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    private readonly bool[] showDetails = new bool[20]
                                                      {
                                                          false, false, false, false, false, false, false, false, false, false,
                                                          false, false, false, false, false, false, false, false, false, false
                                                      };

    Запостил: taburetka, 12 Августа 2013

    Комментарии (7) RSS

    • private readonly bool[] showDetails = new bool[20] 
                                                   { 
                                                    fail, fail, fail, fail, fail, fail, fail, fail, fail, fail,
                                                    fail, fail, fail, fail, fail, fail, fail, fail, fail, fail 
                                                   };
      Ответить
    • Во-первых, элементы readonly-массива менять можно, а во-вторых, это может быть оптимизацией, чтобы не создавать новый массив параметров при каждой передаче в функцию. Конечно, можно было бы не присваивать элементам false явно, но как есть - понятнее.

      За что этот код тут вообще???
      Ответить
      • Если уж так хочется создать массив из двадцати булевых переменных, то вот этот кусок кода уже лишний:
        {
                                                              false, false, false, false, false, false, false, false, false, false,
                                                              false, false, false, false, false, false, false, false, false, false
                                                          };

        Так как по умолчанию они все инициализируются в false.
        К тому же, если это аргументы для метода, типа:
        public static void Shit(bool firstArg, bool secondArg, bool thirdArg)

        То за такое впринципе руки можно оторвать, так как даже в книге Макконела это описывается, как говнокод.
        Ответить
        • Судя по негроидной инициализации еще как используется

          P.s Всегда умиляли методы SomeShit (AnotherShit.a, AnotherShit.b, AnotherShit.c, AnotherShit.d, AnotherShit.e,....)
          Ответить
          • >>public static void Shit(bool firstArg, bool secondArg, bool thirdArg)
            жжосч, сотона))
            Ответить
    • (from x in Enumerable.Range(0, 4).Select(_ => false)
      from y in Enumerable.Repeat(false, 5)
      select x && y).ToArray();
      Ответить

    Добавить комментарий