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

    +138

    1. 1
    memset(cb->chars, cb->width * cb->height, ' ');

    Мой однострочный эпик фейл.

    Запостил: bormand, 27 Апреля 2012

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

    • ava ctm 27.04.2012 07:04 # 0

      а что такое cb?
      Ответить

      • ava bormand 27.04.2012 08:08 # 0

        character buffer
        struct CharBuffer {
    unsigned int width;
    unsigned int height;
    char *chars;
};
        Ответить

    • ava guest 27.04.2012 09:51 # 0

      Не аллоцирован указатель chars? Другого говна не вижу.
      Ответить

      • ava bormand 27.04.2012 09:56 # 0

        Нет, говно именно в этой строке.
        Ответить

      • ava absolut 27.04.2012 10:02 # +5

        аргументы перепутаны местами
        Ответить

        • ava bormand 27.04.2012 10:11 # +1

          Ага. Был бы второй аргумент хотя бы char а не int, компилятор сразу выдал бы предупреждение.
          Ответить

        • ava TarasB 27.04.2012 11:38 # −4

          И няшаблядский кокомпилятор даже ничего не заподозрил, приводя целое к указателю!
          Ответить

          • ava bormand 27.04.2012 11:42 # 0

            Где? Перепутали то int и size_t который тоже int.
            Ответить

            • ava TarasB 27.04.2012 12:01 # −3

              Ой, поправочка:
              И няшаблядский кокомпилятор даже ничего не заподозрил, приводя 4байтовое целое к байту!
              Ответить

              • ava absolut 27.04.2012 12:50 # +3

                где ты байт-то нашёл?
                Ответить

              • ava bormand 27.04.2012 13:26 # +1

                void *memset(void *s, int c, size_t n);
                Где тут байт?
                Ответить

                • ava TarasB 27.04.2012 15:12 # −5

                  показать все, что скрытоА почему int c, он заполняет по байту же, а не по 4 байта?

                  Короче, криворукая няшаблядская библиотека. Кстати, библиотеку можно считать частью компилятора? Ну, чтобы слово "кокомпилятор" было куда ввернуть.
                  Ответить

                  • ava TheCalligrapher 27.04.2012 21:34 # +10

                    В стандартной библиотеке языке С по историческим причнам не бывает параметров типа 'char', 'short', 'float' и т.п. Минимальными типами параметров является 'int' и 'double'. Происходит это потому, что на заре языка С не требовалось предварительного объявления функций с указанием прототипа. Функции можно было вызывать без знания прототипа, что заставляло компилятор "угадывать" фактические типы параметров по типам передаваемых аргументов (и предпологать, что тип возврата - 'int'). При передаче целочисленного аргумента типа 'char', 'short' или 'int' автоматически подразумевалось, что соответствующий параметр имеет тип 'int'. Поэтому в определениях библиотечных функций всегда используются именно те типы параметров, которые "угадает" компилятор при вызове необъявленной функции.

                    Возможность вызова функций без предварительного объявления была исключена в стандарте С99. Но даже стандарт С99 не требует наличия прототипа для вызова функции. Например, для С99 достаточно такого объявления 'memset'
                    void *memset();

                    и функцую уже можно вызывать. При этом компилятор опять же будет "угадывать" типа параметров как описано выше.
                    Ответить

                    • ava guest 27.04.2012 22:12 # +5

                      Это первый вывод типов и количества параметров, посоны.
                      Функциональные языки оказались не первыми. А вывод количества параметров функции - функциональные языки до сих пор не осилили.
                      Ответить

          • ava roman-kashitsyn 27.04.2012 11:47 # +1

            перепутаны второй и третий аргумент, а не первый и второй.
            Ответить

        • ava 63F45EF45RB65R6VR 27.04.2012 11:58 # −17

          показать все, что скрытоа если бы вы проверяли свой код PVS-Studio то она бы сразу выдала один из двух warning'ов

          V512 A call of the 'memset' function will lead to underflow of the buffer 'chars'
          V512 A call of the 'memset' function will lead to overflow of the buffer 'chars'

          и вообще это супер анализатор по выявлению ошибок при переносе на x64 или с линуксячьего
          LP64 на вендовый LLP64
          Ответить

    • ava Lure Of Chaos 27.04.2012 10:35 # 0

      фейл с порядком аргументов?
      Ответить

    • ava someone 27.04.2012 12:04 # +1

      Это C, детка.

      Какая-нибудь Java просто не дала бы преобразовать int в byte без явного каста.
      Ответить

      • ava bormand 27.04.2012 13:25 # 0

        Где тут байт? В прототипе int.
        Ответить

        • ava someone 27.04.2012 13:29 # 0

          А это к вопросу о кривизне рук авторов стандартной библиотеки - почему у них байт для заполнения массива передаётся как int.
          Ответить

          • ava bormand 27.04.2012 13:31 # 0

            А с этим полностью согласен, о чем и писалось выше.

            http://govnokod.ru/10058#comment136594
            Ответить

          • ava absolut 27.04.2012 13:54 # +3

            http://goo.gl/k8rnq
            Ответить

            • ava inkanus-gray 27.04.2012 14:40 # +6

              То есть это сделано для совместимости с криворукими макаками, которые не подключают инклюдники, в которых описан прототип функции, потому что для неизвестных типов Си выбирает int? Мало того, что Си позволяет использовать неописанные функции, так ещё и библиотеки под криворукость подстроены.
              Ответить

      • ava Lure Of Chaos 28.04.2012 00:24 # +2

        ага, а Reader.read() возвращает int, хотя сам класс читает _символьные_ потоки
        Ответить

        • ava someone 30.04.2012 23:03 # 0

          Это чтобы можно было вернуть -1.
          Ответить

          • ava Lure Of Chaos 01.05.2012 02:26 # 0

            аналогично можно было бы возвращать \0
            Ответить

            • ava someone 01.05.2012 02:36 # +4

              \0 может быть вполне валидным символом, действительно прочитанным из потока.
              Ответить

              • ava Lure Of Chaos 01.05.2012 09:24 # −1

                (char)-1?
                Ответить

                • ava eth0 01.05.2012 10:00 # 0

                  Не помню где видел, но в каком-то языке при чтении чаровского значения был таки отдельный символ за пределами чара для признака конца файла. Соответственно, в потоке данных этот символ попасться не мог и проблема не существовала. Но подробности вылетели из головы.
                  Ответить

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