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

    +110

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    int strnlen(const char *s, int size)
{
        int i;
        for(i=0; i<size; i++) if(!*s) break;
        return i;
}

    Запостил: tz-lom, 18 Января 2013

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

    • ava Psionic 18.01.2013 14:44 # −1

      Опа - Индус стайл!!!
      Ответить

      • ava tz-lom 18.01.2013 14:46 # 0

        На самом деле это из Китая :)
        Ответить

        • ava Psionic 18.01.2013 14:49 # 0

          А что мешало автору использовать аналогичную функцию из библиотеки поставки? Я Правда видел китайскую реализацию спринтфа - но там особая ситуация он для 16 битных строк.
          Ответить

          • ava tz-lom 18.01.2013 14:50 # 0

            а вы ещё посмотрите внимательно что код на самом деле делает
            Ответить

            • ava roman-kashitsyn 18.01.2013 15:04 # +9

              Греет комнату, видимо, в китае бывает довольно холодно
              P.S. Только собирать нужно без оптимизации
              Ответить

              • ava inkanus-gray 18.01.2013 15:16 # +3

                Я слоупок. Пока не прочитал этот комментарий, не заметил, что указатель в цикле не изменяется.
                Ответить

            • ava Psionic 18.01.2013 15:36 # 0

              Да я понял что возвращает сайз, а не ищет индекс терминирующего символа,
              Ответить

              • ava bormand 18.01.2013 16:06 # +4

                Не совсем. Если первый символ нулевой - то вернет 0, в остальных случаях size.
                Ответить

          • ava absolut 18.01.2013 15:37 # +4

            вроде как из-за китайцев как раз юникод в 2 байта не укладывается.
            Ответить

      • ava guest 19.01.2013 14:44 # +1

        > Опа - Индус стайл!!!
        http://www.youtube.com/watch?v=9bZkp7q19f0
        Ответить

        • ava bormand 19.01.2013 15:21 # +3

          1.2 млрд... скорее бы перевалило через 2147483648, хочется узнать, не поставил ли там гугл 32-битный знаковый инт.
          Ответить

    • ava Dummy00001 18.01.2013 16:41 # +2

      кто-то что-то например с линуха или бсд портировал. в GNU libc, в POSIX, и в фрибсд есть strnlen(), по описанию соответсвующая реализации сверху. но strnlen() не является частью С стандарта и поэтому на некоторых платформах (например солярка) отсутствует.
      Ответить

    • ava someone 18.01.2013 17:03 # 0

      Зачем это вообще нужно?
      Ответить

      • ava bormand 18.01.2013 17:07 # 0

        > strnlen - determine the length of a fixed-size string
        Видимо неудачный порт чего-то ГНУтого и юзающего strnlen на какую-нибудь говноштудию, которая срала на POSIX.
        Ответить

        • ava Xom94ok 19.01.2013 13:01 # 0

          Я не особо крут во многопоточном программировании, но это явно очень хитрый мьютекс со счетчиком, правда пришлось немного его допилить.
          #include <thread>
#include <iostream>

int strnlen(volatile const char *s, int size)
{
        int i;
        for(i=0; i<size; i++) if(!*s) break;
        return i;
}

int main(int, char **) 
{
	volatile char mutex = 'a';

	std::thread thread1([&mutex]()
	{
		std::cout << std::hex << strnlen(&mutex, std::numeric_limits<int>::max()) << std::dec << std::endl;
	});

	std::thread thread2([&mutex]()
	{
		std::this_thread::sleep_for(std::chrono::milliseconds(100));
		mutex = '\0';
	});

	thread1.join();
	thread2.join();
	return 0;
}

          В одиннадцатой студии на Горе и5 в релизе при первом запуске вывело 4а8ф76ц.
          Ответить

          • ava bormand 19.01.2013 13:10 # +1

            volatile при многопоточном программировании только усугубляют ситуацию, но никак не помогают ;)

            Use barriers, Luke.

            P.S. Ай блин, старый дурак я, забыл что визуалка для защиты от не прочитавших про барьеры сует эти самые барьеры во все volatile сохранения\загрузки...
            Ответить

          • ava bormand 19.01.2013 13:37 # +1

            Неплохая статья про бесполезность volatile в многопоточных ситуациях: http://software.intel.com/en-us/blogs/2007/11/30/volatile-almost-useless-for-multi-threaded-programming

            "...the use of volatile accomplishes nothing but to prevent the compiler from making useful and desirable optimizations, providing no help whatsoever in making code "thread safe". " - David Butenhof
            Ответить

            • ava Xom94ok 19.01.2013 16:02 # +2

              Спасибо за ссылочку, будем ковырять :) Когда регистрировался на говнокоде, то даже и не подозревал, что он станет для меня дополнительным источником образования. PROFIT
              Ответить

              • ava bormand 19.01.2013 16:06 # 0

                > Когда регистрировался на говнокоде, то даже и не подозревал, что он станет для меня дополнительным источником образования. PROFIT
                Я тоже... но благодаря местным обитателям каждый день получаю какие-то новые знания или интересные ссылки ;)
                Ответить

          • ava guest 19.01.2013 14:52 # −1

            Тарасокод
            Ответить

    • ava guest 19.01.2013 14:48 # 0

      Забавно, но такая программа в Delphi\Pascal - UB.
      Значение переменной цикла после цикла не определено. Если точнее то определено, что в диапазоне: i=a, i=a-1, i=a+1
      Ответить

      • ava guest 19.01.2013 14:50 # 0

        Присмотрелся. все ещё хуже. s не меняется
        Ответить

      • ava bormand 19.01.2013 15:26 # 0

        > Значение переменной цикла после цикла не определено.
        Пасцаль такой пасцаль. Ради микрооптимизации сделали весьма полезное значение UB'ом, а я думал что только кресты этим славятся...

        P.S. Можно пруф от Вирта?
        Ответить

        • ava defecate-plusplus 19.01.2013 16:35 # 0

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

          • ava bormand 19.01.2013 17:01 # +1

            Ну вот наверное: ISO 7185:1990.

            UPD: Блин всего 92 страницы. Куда уж ему до крестостандартов.
            Ответить

            • ava guest 19.01.2013 18:15 # +4

              Ты ещё лиспа не видел.
              Ответить

              • ava roman-kashitsyn 19.01.2013 20:22 # +1

                коммон лиспа. у схемы меньше, чем у паскаля, 59 страниц, если не считать аппендиксы (http://www.r6rs.org/final/r6rs.pdf)
                Ответить

                • ava guest 19.01.2013 23:29 # 0

                  CLispGovno now #
                  Я именно о схеме. Ну и смотри r5rs, так как rs6r сильно раздута.
                  Я даже больше скажу: Прочитав её стандарт - не нужно читать дополнительных книг по нему (по крайней мере это не критически необходимо, если хоть какие-то языки видел). Так как эти самые 59 страниц стандарта содержут и коментарии к стандарту на уровне туториала по языку на понятном уровне. А попробуйка прочитать стандарт крестов и сразу начать писать на них. Хуй начнешь писать и через полгода после нескольких книг.

                  И специально для тебя, Роман: Погугли Scala Workshitsheet.
                  Ответить

                  • ava roman-kashitsyn 19.01.2013 23:45 # +1

                    > смотри r5rs, так как rs6r сильно раздута
                    В пятерке нет никакого аналога модулей. Язык без нормальной системы модулей серьёзно воспринимать довольно сложно (даже php чудом попал в мэйнстрим, посмотрел по сторонам и наговнявкал неймспейсы).

                    > Scala Worksheet
                    > There are 5 Eclipse plugins...
                    дальше не читал
                    Ответить

          • ava bormand 19.01.2013 17:04 # +1

            UPD2: Собственно цитата из стандарта: After a for-statement is executed, other than being left by a goto-statement, the control-variable shall be undefined.
            Ответить

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