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

    +136

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    while (f != null && !string.IsNullOrEmpty(f.FileName) && f.ContentLength != 0)
{
   if (f != null && !string.IsNullOrEmpty(f.FileName) && f.ContentLength != 0)
   {
      // ...
   }
}

    Проверка на всякий случай

    Запостил: ReallyBugMeNot, 02 Февраля 2011

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

    • ava Lure Of Chaos 02.02.2011 14:35 # −1

      ну а вдруг после while файл зануллился
      Ответить

    • ava absolut 02.02.2011 14:47 # 0

      В другом потоке изменили
      Ответить

      • ava Lure Of Chaos 02.02.2011 14:51 # 0

        или в другом процессе вообще
        Ответить

      • ava m1el 02.02.2011 14:58 # −3

        Состояние изменилось в промежутке между двумя операциями? ну-ну
        Ответить

        • ava Lure Of Chaos 02.02.2011 15:07 # +5

          вероятность такая есть, поскольку операция не атомарна
          ваш кэп
          Ответить

          • ava guest 02.02.2011 15:38 # 0

            Есть, но это всё не поможет, если f занулится после if.
            Ответить

            • ava Lure Of Chaos 02.02.2011 15:50 # 0

              нужно синхронизировать потоки и блочить файлы
              Ответить

              • ava guest 02.02.2011 15:54 # 0

                >нужно синхронизировать потоки и блочить файлы
                Кто-то сомневался? 5487 - говнокод, тк этого нет.
                Ответить

    • ava guest 02.02.2011 18:42 # −1

      Эту копипасту компилятор вырежет оптимизатором всё равно...
      Ответить

      • ava abatishchev 02.02.2011 21:46 # +1

        не вырежет
        Ответить

        • ava abatishchev 02.02.2011 22:21 # 0

          L_000c: callvirt instance string ConsoleApplication1.Foo::get_FileName()
          L_0011: call bool [mscorlib]System.String::IsNullOrEmpty(string)
          L_0016: brtrue.s L_001f
          L_0018: ldloc.0
          L_0019: callvirt instance int32 ConsoleApplication1.Foo::get_ContentLeng th()
          L_001e: pop
          L_001f: ldloc.0
          L_0020: brfalse.s L_0037
          L_0022: ldloc.0
          L_0023: callvirt instance string ConsoleApplication1.Foo::get_FileName()
          L_0028: call bool [mscorlib]System.String::IsNullOrEmpty(string)
          L_002d: brtrue.s L_0037
          L_002f: ldloc.0
          L_0030: callvirt instance int32 ConsoleApplication1.Foo::get_ContentLeng th()
          L_0035: brtrue.s L_0008
          Ответить

          • ava guest 02.02.2011 23:39 # 0

            Оптимизацию включили?
            Вообще, любой нормальный доделаный современный компилятор умеет делать оптимизацию expression elimination. Неужели майкрософт в сишарпе это не осилила? В С++ ихнем эта оптимизация работает.
            Ответить

            • ava TarasB 02.02.2011 23:51 # 0

              Это если выражение не имеет побочных эффектов. А как компилятор об этом узнает?
              Ответить

              • ava guest 03.02.2011 01:51 # 0

                А это уже называется global optimization.

                А вообще, часть выражений с небольшой глубиной вложенности некоторые компиляторы вырезают даже без поддержки global optimization.
                Ответить

              • ava abatishchev 03.02.2011 10:14 # +1

                ага! никак не может узнать.

                здесь цикл,
                потом оператор, статический метод, метод экземпляра, метод экземпляра
                потом оператор, статический метод, метод экземпляра, метод экземпляра

                не смотря на код - или представляя себе другой код, как можно удалить из этого что-либо?

                давайте тогда сразу весь код на noop заменять и все дела :)
                Ответить

            • ava abatishchev 03.02.2011 10:12 # 0

              да, .NET 4.0, Release
              Ответить

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