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

    +109

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if (File.Exists(local)) {
                    if (Connect()) {
                        if (ftpConnection.Exists(remote)) {
//code
                        }
                    }
                }

    Запостил: Irdis, 27 Июня 2011

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

    • ava guest 27.06.2011 13:04 # −1

      Как же, по вашему, это должно выглядеть?
      Ответить

      • ava absolut 27.06.2011 13:13 # +2

        Очевидное первое предположение:
        if (File.Exists(local) && Connect() && ftpConnection.Exists(remote)) { ... }
        Ответить

        • ava guest 27.06.2011 13:31 # −3

          Это не повод суда постить этот говнокод. Уныло.

          >Connect() && ftpConnection.Exists(remote)
          В C# условия в if'е в каком порядке выполняются?
          Если Connect() поднимает соединение в ftpConnection, то я бы не стал писать и надеяться, что условия выполнятся в правильном порядке.
          Ответить

          • ava Irdis 27.06.2011 13:36 # −2

            как угодно, блин, выполняются
            сначала первое потом последние...
            а вот из-за таких неуверенных как вы приходиться любоваться
            }}}}}}}}} бесит же
            Ответить

          • ava absolut 27.06.2011 13:42 # −2

            Условие в ифе выполняется в порядке, установленном приоритетами операций, использованных в данном ифе.
            В данном случае выполнение будет осуществляться слева направо.
            Ответить

            • ava guest 27.06.2011 20:13 # −4

              >В данном случае выполнение будет осуществляться слева направо.
              В том же си или с++ в старом стандарте это будет не так
              Ответить

              • ava absolut 27.06.2011 21:39 # +2

                а как? желательно с примерчиком.
                Ответить

              • ava gegMOPO4 28.06.2011 18:01 # +1

                Это неправда. Ленивость &&, || и ?: гарантирована языком.
                Ответить

              • ava guest 01.07.2011 22:58 # 0

                Ваш пост расстраивает Sequence Points (а им поболе 10 лет уже, или речь об ооочень старом стандарте?).
                Ответить

            • ava gegMOPO4 28.06.2011 18:00 # +1

              Приоритет тут совершенно ни при чём. Приоритет заканчивается на этапе синтаксического разбора.

              А играет роль — ленивость операторов.
              Ответить

          • ava SmackMyBitchUp 27.06.2011 14:42 # 0

            Плюсом, как я понимаю, вычисления ленивые (спасибо местным, что научили правильно выражаться).
            Ответить

            • ava guest 27.06.2011 15:21 # −1

              Текущий говнокод (Current) не менее ленивый, чем во втором комменте
              Ответить

              • ava SmackMyBitchUp 27.06.2011 19:23 # 0

                Зато чего стоят операторные скобки в курренте.
                Ответить

                • ava guest 27.06.2011 20:14 # 0

                  >(Current)
                  >Текущий говнокод
                  Написал, что-бы не подумал кто чего...
                  Ответить

    • ava KirAmp 27.06.2011 15:23 # 0

      Очень напоминает C1 в ЕГЭ по информатике (исправить ошибку) =)
      Ответить

      • ava RaZeR 28.06.2011 13:33 # 0

        Там обычно надо поправить ошибку в проверке на принадлежность точки к некоторому отсечённому куску графика.
        И что самое интересное, написанный тобой код на бумажке они сравнивают с неким "эталоном".
        Интересно мне посмотреть на этот эталон...

        P.S.: Да, я сдавал эту хуиту.
        Ответить

    • ava musuk 30.06.2011 12:46 # 0

      ИМХО загонять кучу логики в один оператор - тот ещё говнокод, особенно, когда потом приходится добавлять параметры в функции - условия IF'ов разразастаются в кучу строк. Many if vs compex condition в данном случае есть холивар. Помнится, когда появился LINQ все тоже увлеклись засовыванием безумной логики в один запрос. Ничего, первое же сопровождение эту привычку отбило: стали временные переменные создавать.

      Что мне здесь реально не нравится, это мутная функция Connect() и поле ftpConnection. Как бы не очевидно, куда Connect() и что ftpConnection != null, например. Короче тут явно нужен рефакторинг.
      Ответить

    • ava guest8 08.04.2019 21:01 # −999

      показать все, что скрытоvanished
      Ответить

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