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

    +989

    1. 1
    char* mData=new char[MAX_STRING_LENGTH+1];

    Кто сможет перечислить все ошибки в этом говнокоде?

    Запостил: Говногость, 11 Ноября 2011

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

    • ava Говногость 11.11.2011 08:39 # 0

      MAX_STRING_LENGTH - константа.
      Ответить

      • ava greno 11.11.2011 08:44 # +2

        В плюсах разбираюсь плохо. Но если она большая и данные лучше разместить на куче?
        Ответить

    • ava bugmenot 11.11.2011 09:10 # +7

      всё нормально в этом коде, урсус.
      Ответить

      • ava lucidfoxGovno 11.11.2011 10:51 # −3

        всё нормально, в этом коде усрус.
        Ответить

        • ava ursus 16.11.2011 09:49 # 0

          наговор! урсус тут не при чём!!!!
          Ответить

          • ava lucidfoxGovno 16.11.2011 10:51 # −1

            Как не причем? Ты пришёл, так что причем, даже когда тебя не звали.
            Ответить

          • ava TarasB 16.11.2011 12:42 # +5

            СМОТРИТЕ ВСЕ ЭТО ЖЕ УРСУС!!!!!
            Ответить

            • ava eth0 16.11.2011 13:10 # 0

              В каком треде раздача автографов?
              Ответить

            • ava ursus 16.11.2011 13:37 # −1

              так, вот не надо этого урсусопоклонства.
              кстати, можете стать обожателями моей жжжшечки
              http://ursusalbus.livejournal.com
              а на лурке про говнокод статью на снос поставили - печаль (друзья сегодня хвастались что нашли меня).
              Ответить

    • ava defecate-plusplus 11.11.2011 09:30 # +2

      ошибка может быть только если
      #define MAX_STRING_LENGTH (-1)

      либо один дибил не сказал другому, что MAX_STRING_LENGTH - это включая замыкающий \0
      Ответить

      • ava roman-kashitsyn 11.11.2011 09:36 # +2

        возможно, Говногость считает, что не использовать строки - это ошибка.
        Ответить

    • ava Говногость 11.11.2011 09:37 # −3

      В зависимости от контекста, конечно, но:
      1)Возможный перерасход памяти, если MAX_STRING_LENGTH больше обычно используемой длинны.
      2)Частично требует ручной работы со строкой.
      3)Код не безопасен с точки зрения исключений.
      4)Массив кастится до указателя и поэтому теряется его длинна. То есть, не смотря на то, что длинна константная, обобщенными методами (например через sizeof или шаблон) получить длину массива нельзя.
      5)Нет автоматического контроля за выходом индекса массива за диапазон MAX_STRING_LENGTH.
      6)Лучше использовать std::string.
      7) +1 не нужен. Нумерация массива начинается с 0. В любом случае, не всегда можно понять по коду MAX_STRING_LENGTH включает \0 или нет. Забыл ли +1 автор или нет? То есть, не всегда возможно определить есть ли ошибка или нет.
      Ответить

      • ava bugmenot 11.11.2011 09:43 # +1

        > длинна
        и эти люди запрещают нам ковырять в носу!
        Ответить

      • ava Setry 11.11.2011 10:23 # +1

        исходя из твоей логики: весь язык С - это говнокод >_<
        Ответить

        • ava bugmenot 11.11.2011 10:25 # +1

          кресты же
          Ответить

          • ava Setry 11.11.2011 10:40 # +1

            я понимаю, что кресты
            или "все что нормально для С, плохо для С++"?


            не говоря уже о том, что далеко не всегда "лучше использовать std::string"
            Ответить

            • ava bugmenot 11.11.2011 12:20 # 0

              > все что нормально для С, плохо для С++
              В случае тезиса о std::string так и есть

              ой и дырявая же каптча...
              Ответить

          • ava absolut 11.11.2011 10:44 # +2

            совместимость же. и по говну тоже
            Ответить

      • ava absolut 11.11.2011 10:43 # +3

        большинство проблем вытекает из C-style, хотя код заявлен как С++
        Ответить

      • ava ctm 11.11.2011 10:54 # +2

        1. про MAX_STRING_LENGTH нам ничего не известно - почему мы должны думать, что ошибка?
        2. может требуется - хз для чего?
        3. по 1 строке не видно. можно расставить try/catch чтобы было безопасно.
        4. а куда денешься, если C-style, опять же его использование может для чего еще нужно?
        5. в коде вообще нет обращений по индексу.
        6. лучше конечно, но откуда нам знать, можно ли?
        7.вообще не ошибка, т.к. будет работать как для включенного \0, так и не включенного. хотя, 1 лишний байт в куче не гуд конечно.
        Ответить

      • ava saab 12.11.2011 02:07 # 0

        используйте жаву.
        Ответить

      • ava ursus 16.11.2011 09:52 # +1

        кстате говногость прав
        Ответить

    • ava absolut 11.11.2011 10:39 # 0

      >Кто сможет перечислить все ошибки в этом говнокоде?
      я почему-то сразу вспомнил про http://govnokod.ru/3130
      Ответить

    • ava istem 11.11.2011 10:50 # +5

      > Кто сможет перечислить все ошибки в этом говнокоде?
      Вы имеете ввиду сайт?
      Ответить

    • ava Lure Of Chaos 11.11.2011 12:21 # +4

      Мужики, не обижайтесь...
      Ответить

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