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

    +78

    1. 1
    2. 2
    3. 3
    4. 4
    int strlenw(const wchar_t *s)
{
	return (int)((int)(strlen(s)) % sizeof(wchar_t));
};

    Даже и не знаю что сказать. Это было в доставшемся мне проекте. Мне кажется это говнокод.

    Запостил: Говногость, 11 Января 2010

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

    • ava guest 11.01.2010 13:02 # −6.8

      показать все, что скрытоГде здесь С++?
      Ответить

    • ava guest 11.01.2010 13:11 # 0

      Ну (int) это понятно — многие компиляторы (а особенно msvc) выдают предупреждение о несоответствии типов size_t и int
      А так, да, sort of говнокод.
      Ответить

    • ava Altravert 11.01.2010 13:27 # +1

      Согласен, говнокод, даже улыбнуло) Не сам придумал? Чтобы реально такое где-то написать надо быть знатным извращенцем.
      Ответить

    • ava guest 11.01.2010 14:52 # +1

      Похоже, думать автору кода было лень и он напихал приведения типов куда только можно. Внешний (int) не нужен.

      Ещё странно, зачем такая функция вообще нужна (остаток деления строки на длину wchar_t). Чтобы узнать, можно ли представить строку как wchar-строку?
      Ответить

      • ava Altravert 11.01.2010 15:05 # −1

        Ты о чем? И так wchar передается.
        Ответить

        • ava guest 11.01.2010 15:17 # −1

          Передаётся-то wchar, а strlen считает количество char-ов.

          Обычно размер wchar -- два байта. Если остаток от деления длины строки (в char-ах, они же байты) на размер wchar нулевой, значит строка валидная (в ней помещается целое число wchar-ов). Если ненулевой, значит строка кривая.

          А вообще я о том, что мне непонятно, зачем эта функция используется.
          Ответить

          • ava Altravert 12.01.2010 08:21 # 0

            strlen("string") = 6.
            6 % sizeof(wchar) = 0.

            strlen("stringPUMPURUM!") = 15.
            15 % sizeof(wchar) = 1.

            И?
            Ответить

            • ava Nex 12.01.2010 14:16 # 0

              И в чем прикол, что ты приводишь в пример 2 строки char*? Вбей что-то в стиле
              strlen(L"p\u0x00ff\u0xff00prara-pararam-pam-pam") и посмотри результат.

              f.ex.
              int _tmain(int argc, _TCHAR* argv[])
{
	wchar_t *param=L"p\u0x00ff\u0xff00prara-pararam-pam-pam";
	printf("strlen((const char*)param)=%d\nwcslen((const wchar_t*)param)=%d", strlen((const char*)param),wcslen((const wchar_t*)param));
	int n=10;
	scanf("\n%d",n);
	return 0;
}


              резалт:
              strlen((const char*)param)=1
              wcslen((const wchar_t*)param)=36
              Ответить

    • ava guest 11.01.2010 15:49 # +2

      индус ниасилил wcslen()
      К тому же если будет символ вида 0x00 0xXX, то strlen выдаст неправильный результат
      Ответить

      • ava cdta 11.01.2010 20:47 # 0

        да и strlen вроде на wchar_t * поругается варнингом - если его не исправлять то непонятно зачем все эти пляски с (int)
        Ответить

    • ava TarasB 11.01.2010 23:33 # +1

      Нормальный компилятор сведёт это к
      strlen(s)&1
      Но извращение дикое, жаль, что нынче модно такое оправдывать "логичностью" и "читабельностью".
      Ответить

      • ava guest 12.01.2010 10:22 # −2

        Ты дурочко - sizeof(wchar_t) == 2
        Ответить

        • ava guest 12.01.2010 10:57 # 0

          Фигвам. GCC под linux-ом считает sizeof(wchar_t) == 4. Более того, указание -fshort-wchar конечно приводит к использованию 16-битовых unicode-символов, но валит программу по SIGSEGV при первом же обращении к любой wchar-функции.
          Ответить

          • ava guest 12.01.2010 12:14 # −3

            Так про альтернативные оси никто не говорил :) Пользуйтесь православной виндой
            Ответить

            • ava guest 12.01.2010 17:48 # +1

              Винда и есть альтернативная.
              Ответить

            • ava guest 13.01.2010 08:14 # +1

              православной виндой??

              еретик!!
              Ответить

              • ava guest 13.01.2010 10:46 # +1

                +1
                Ответить

              • ava guest 13.01.2010 14:02 # 0

                <[:|||:]>
                Конечно, в винде иконки есть и службы проводятся.
                <\[:|||:]>
                Ответить

                • ava Altravert 13.01.2010 16:43 # 0

                  Кто-то когда-то сравнивал винду и линух -- службы и демоны.
                  Ответить

            • ava xaionaro 19.02.2010 07:27 # +1

              Сам ты альтернативная ОС. Си как раз таки родился в UNIX, а не в windows, между прочим.
              Ответить

        • ava TarasB 13.01.2010 01:24 # 0

          i % sizeof(wchar_t) == i % 2 == i & 1
          Кто из нас дурочко?
          Ответить

    • ava guest 11.01.2010 23:51 # 0

      Может это какая-то попытка получить длинну строки? Просто чувак протупил и / заменил на % ?
      Ответить

      • ava Говногость 12.01.2010 13:52 # 0

        У меня тоже есть такое подозрение. Эта функция используеться только пару раз в проекте и судя по использующему её коду - так оно и есть.
        Ответить

        • ava Nex 12.01.2010 15:43 # 0

          если там нулевой байт до конца строки попадётся, то облом. по-любому не только знак менять надо.
          Ответить

          • ava Altravert 12.01.2010 17:29 # +1

            Эмм.. Если попадется нулевой байт тебе strlen менять надо)
            Ответить

            • ava guest 13.01.2010 14:14 # 0

              wchar_t как бы содержит 2 байта, один из которых может быть вполне нулевым
              Ответить

              • ava guest 13.01.2010 15:43 # 0

                strlen об этом не знает.
                и не 2 там байта.
                Ответить

                • ava Altravert 13.01.2010 16:45 # 0

                  Про не да байта уже говорить бессмысленно: люди читают с конца походу.
                  Ответить

          • ava guest 12.01.2010 17:40 # 0

            Человек не случайно это говно выложил. Хорошо, что хоть не весь проект на этом построен. :)
            Ответить

            • ava Altravert 12.01.2010 17:50 # +1

              А ты откуда знаешь? Может он копипастил себе в калькулятор для института и случайно вставил в браузер, а потом в панике нажал кнопку!.. Амфетамины они такие...
              Ы)
              Ответить

            • ava guest 13.01.2010 10:54 # 0

              Дак выше об этом сказано. Читай внимательнее.
              Ответить

    • ava guest 07.03.2010 01:03 # +0.2

      wchat_t обычно используется для UTF-16. следовательно, для английских символов, каждый двухбайтный символ будет содержать байт с нулем. соответственно strlen заткнется на первом же символе и вернет 1 или 0 (в зависимости от Big/Little Endian)
      Ответить

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