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

    +1014

    1. 1
    2. 2
    3. 3
    4. 4
    if (strlen(Uid.c_str()) > 0)
{
  // ...
}

    собственно, std::string Uid;

    Запостил: JeremyW, 27 Октября 2011

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

    • ava Psionic 27.10.2011 16:33 # 0

      Да ладно, просто if (strlen(Uid.c_str()))? Тут чуть-чуть плохо пахнет.
      Ответить

    • ava Dummy00001 27.10.2011 16:40 # +1

      это еще что. у нас в одном проекте код напичкан `if (str.c_str() != NULL)` после присваивания строк. когда первый раз увидел, мне понадобилось несколько минит что бы понять что именно этим хотели сказать.

      ЗЫ к слову. не стоит забывать что код сверху, не смотря на то что навярняка говно, все же имеет смысл в некоторых ситуациях: std::string может содержать '\0'.
      Ответить

      • ava absolut 27.10.2011 16:44 # +3

        >несколько минит
        интересно ... в какой букве опечатка
        Ответить

      • ava Kirinyale 27.10.2011 17:30 # +1

        И какой же смысл имеет использование strlen на строке, для которой \0 - допустимый символ? Разве что разбитие на подстроки, если \0 - разделитель (где-то в винде я такое извращение видел).
        Ответить

        • ava Dummy00001 27.10.2011 17:44 # +2

          да. если часть бинарного буфера внутри std::string есть строка с 0 терминатором.

          я как-то раз std::string использовал для парсинга двоичного протокола. до strlen() я не опускался, но std::string::find_first_of и его друзьями я пользовался.
          Ответить

        • ava bot-minurast 27.10.2011 20:59 # +6

          Щас придёт Тарас и скажет, что сишные строки отстой, а паскалевские строки - кул.
          Ответить

        • ava bugmenot 27.10.2011 23:08 # 0

          там есть даже хуже, http://msdn.microsoft.com/en-us/library/windows/desktop/aa365240(v=vs.85).aspx#movefile_write_th rough
          Ответить

      • ava gegMOPO4 27.10.2011 18:53 # 0

        Я тупой. Мне нескольких минит не хватило.
        Ответить

      • ava AnimeGovno-_- 27.10.2011 21:05 # −2

        >if (str.c_str() != NULL)
        Мне сразу стало все ясно, если это не std::w\string, а свой кривой велосипед.
        Ответить

        • ava Dummy00001 28.10.2011 01:23 # 0

          при чем тут std::wstring?

          c_str() метод может вернуть NULL только если выделение памяти обломалось. но если выделение памяти обломалось, то будет брошен эксепшн.
          Ответить

          • ava delirium 29.10.2011 14:35 # 0

            g++ -fno-exceptions
            Ответить

            • ava Dummy00001 29.10.2011 14:40 # 0

              С++ без исключений является не С++, а каким то новым кастрированым языком.

              которым я к слову пользовался - в те времена когда просто добавления try/catch делало програму в пару раз медлнее. мало приятного когда прога слетает полностью по первому брошеному исключению в какой внешней либе.
              Ответить

            • ava Dr_Offset 01.11.2011 18:43 # +1

              -fno-exceptions выключает исключения только в текущем коде. Исключения, которые бросает libstdc++ им не выключаются. Скомпилируй с этим ключом и убедись.
              #include <limits>

int main()
{
    char * p = new char[std::numeric_limits<unsigned long>::max()];
}
              Ответить

    • ava AnimeGovno-_- 02.11.2011 12:21 # −4

      И конечно же опять с ГК.ру...
      Мне вот интересно, говнодев филиал говнокода или наоборот?
      Ответить

    • ava guest 22.11.2011 16:56 # 0

      Где здесь С++?
      Ответить

    • ava guest8 09.04.2019 11:49 # −999

      показать все, что скрытоvanished
      Ответить

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