1. Си / Говнокод #17638

    +143

    1. 1
    2. 2
    3. 3
    char value[60] = {0};
int data;
snprintf(value, sizeof(value-1), "|%7u\n", data);

    coverity полагал что заменить sprintf на snprintf было бы безопаснее

    Запостил: Yeiradohr, 13 Февраля 2015

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

    • ava guest 14.02.2015 00:05 # +1

      >sizeof(value-1)
      Ответить

    • ava 1024-- 14.02.2015 00:45 # +2

      Ха-ха, код зайдёт за пределы массива на 512-битном компьютере.
      И запишет туда десятичное представление мусора.
      Ответить

    • ava absolut 15.02.2015 08:22 # 0

      Адресная арифметика - зло
      Ответить

      • ava codemonkey 16.02.2015 11:05 # 0

        >Я ниасилил адресную арифметику
        FTFY bro
        Ответить

        • ava bormand 16.02.2015 11:30 # +1

          Адресная арифметика зло.

          Осиливший адресную арифметику.
          Ответить

          • ava kegdan 16.02.2015 13:16 # +3

            зло - не зло, а байтоебить приходится
            Ответить

            • ava bormand 16.02.2015 14:35 # +1

              Ну и goto тоже зло, но в сишке юзать приходится. Тут главное не злоупотреблять.
              Ответить

              • ava kegdan 16.02.2015 15:47 # 0

                из разряда - jmp тоже зло, но в асме приходится юзать

                goto помогает выпрыгивать из n-ого цикла без всяких флагов
                Ответить

                • ava 1024-- 16.02.2015 15:57 # 0

                  Да goto из цикла даже у нас в высокоуровневом жс есть. Кажется, в одном из прошлых холиваров goto для циклов помиловали.
                  Ответить

                  • ava kegdan 16.02.2015 15:59 # 0

                    главное юзать с умом.

                    Так я высокоуровневый сишарпей
                    Ответить

                    • ava bormand 16.02.2015 16:02 # 0

                      > высокоуровневый сишарпей
                      80лвл?
                      Ответить

                    • ava 1024-- 16.02.2015 16:03 # 0

                      У меня не выйдет. Опыта нет.
                      Про метки в жс вспоминаю только во время обсуждений на ГК и, когда мне встречаются тесты вида "ололо если ти крутой жабошкриптер, то ты знаешь, сколько будет []+true*8-4+'3'|0".

                      P.S. А вообще, ошибся. Там "break метка", а не goto.
                      Ответить

        • ava roman-kashitsyn 16.02.2015 11:31 # 0 

          http://golang.org/doc/faq#no_pointer_arithmetic
          Пайк и Томпсон - определённо неосиляторы
          Ответить

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