1. C++ / Говнокод #8253

    +1005

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    memsize =
  strlen("cpy_dta.sh ");

if ( memsize >= 4096 ) {
  sprintf( errMsg, "[%s] - ExecScript string too long", func);
  return -7;
}

sprintf( args, "cpy_dta.sh " );

if ( 0 != ( rc = ::system ( args ) ) )
{
  // ...
}

    просто фейспалм.

    Запостил: Dummy00001, 20 Октября 2011

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

    • ava defecate-plusplus 20.10.2011 15:13 # +5

      C++ тут, видимо, только в одноразовом :: ?
      Ответить

      • ava absolut 20.10.2011 15:17 # 0

        он не одноразовый, он глобальный.
        но больше ++ нету.
        а по коду могу сказать, что похоже автор взял готовый шаблон и захардкодил имя файла.
        кстати, зачем пробел в 9 строке?
        Ответить

        • ava defecate-plusplus 20.10.2011 15:20 # 0

          там же, зачем и во второй
          Ответить

          • ava absolut 20.10.2011 15:43 # 0

            а во второй зачем?
            Ответить

            • ava alexoy 20.10.2011 15:49 # −10

              показать все, что скрытоДля стеков.
              Ответить

            • ava defecate-plusplus 20.10.2011 15:56 # +6

              бойцы помогали компилятору выровнять .data - сразу сделали литерал длиной 12 байт
              уверен они знали что делали
              при этом они точно использовали нормальный компилятор, который оба вхождения на 2 и 9 строках не будет дублировать, а разместит по одному адресу
              а проверка memsize >= 4096 специально нужна, чтобы поток отъел пару тактов у процессора, в то время как другой поток сделает высокопроизводительные вычисления над переменной args - тогда не надо ни мутексов, ни фенсов, ни тем более безумные ассемблерные вставки не понадобятся.
              кроме того, неизвестно какого типа переменная rc, возможно, это специальный объект с перегруженным оператором =, который перемножает матрицы со сложностью О(1) или даже О(0)

              а может они этим хотели зашифровать какое то послание потомкам, только вот какое?..
              Ответить

              • ava SmackMyBitchUp 20.10.2011 16:06 # +3

                Эт вы в своем докладе какую-то херень зашифровали О_о
                Ответить

              • ava roman-kashitsyn 20.10.2011 17:13 # +5

                а return -7; в шестой строке, видимо, символизирует семь смертных грехов?
                Ответить

      • ava Dummy00001 20.10.2011 15:27 # 0

        да. начинали писать прогу С++ники - дописывали Сишники.
        Ответить

        • ava defecate-plusplus 20.10.2011 15:30 # +3

          надо было программистов на коболе брать для проекта, девять женщин родят одного младенца за месяц
          Ответить

    • ava bugmenot 20.10.2011 17:20 # +3

      сру_дата
      Ответить

    • ava TheHamstertamer 21.10.2011 19:55 # 0

      >if ( 0 != ( rc = ::system(args) ) )
      Сравнения мастера Йоды? Нет пути!
      Ответить

      • ava guest 21.10.2011 20:36 # −1

        Сравнение это хорошее, но скрывает туман незнания причины истинные.
        Познать тебе много тонкостей языка нужно, мой юный падаван.
        http://images.wikia.com/starwars/images/e/e0/Yoda_SWSB.jpg
        Ответить

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