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

    +160

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    std::string ScriptInfo::GetDebugInfo() const
{
    char sz[256];
    sprintf(sz, "%s ('%s' script id: %u)", GetScriptCommandName(command).c_str(), GetScriptsTableNameByType(type).c_str(), id);
    return std::string(sz);
}

    Может, не совсем говнокод, но меня от подобного коробит.

    Запостил: Oops, 17 Октября 2010

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

    • ava absolut 17.10.2010 16:08 # +1

      тогда stringstream
      Ответить

    • ava Анонимус 17.10.2010 16:57 # 0

      >>char sz[256];
      :))) а почему не 42? Паскалист детектед?
      Ответить

      • ava absolut 17.10.2010 17:04 # +1

        Какая связь с паскалистом ? Максимальная длина строки?
        Ответить

        • ava Анонимус 17.10.2010 18:18 # 0

          ага)
          Ответить

          • ava absolut 17.10.2010 20:37 # +2

            Ну это не показатель. Просто привычнее как-то давать размеры кратные степени двойки. И минимально достаточные для результирующей стоки.
            Ответить

      • ava bugmenot 18.10.2010 05:53 # −2

        fail. TP-string - 255 чаров, не 256
        Ответить

        • ava absolut 18.10.2010 10:41 # −2

          256-ой нулевой терминатор. Так что информационных 255.
          selffail, bugmenot :)
          Ответить

          • ava Govnoeb 18.10.2010 13:48 # −4

            В памяти ЭВМ строка занимает количество байтов, на единицу большее ее длины. Нулевой байт строки содержит ее длину.

            http://wm-help.net/books-online/print-page/79769/79769-11.html

            ++fail;
            Ответить

            • ava absolut 18.10.2010 20:22 # 0

              Я говорил про С-строку из ГК, а не про TP-строку.
              Ответить

            • ava bugmenot 19.10.2010 02:48 # −2

              TP-строка символов "вы ебанитесь!" предваряется символом "\r" и это символизирует
              Как бы смягчающим и увлажняющим обстоятельством является то, что это был первый managed тип броланда и придумано это был в 1980-х. Но не пример для подражания ни разу

              мм, учебник по трубопаскакалю на сайтике "в помощь уеб-мастеру" :)
              Ответить

        • ava Анонимус 19.10.2010 20:26 # 0

          вебкил, перелогинтесь)
          Ответить

    • ava rudvil 17.10.2010 17:58 # −2

      Жесть, из функций возвращающих std::string брать const char* чтобы потом склеить их sprintf'ом в массив чаров(char sz[256];) и из всего этого опять создавать std::string... ебануться.

      std::string ScriptInfo::GetDebugInfo() const
{
  return GetScriptCommandName(command) +
  " ('" + GetScriptsTableNameByType(type) + "' script id: " +
  boost::lexical_cast<std::string>(id) + ")";
}
      Ответить

      • ava bober_maniac 17.10.2010 18:02 # +1

        Капитан какбэ заявляет, что буст говно.
        Ответить

      • ava AxisPod 18.10.2010 07:13 # +7

        Ну тоже так еще какаха. Есть std::stringstream для сбора строк в кучу и к тому же работает быстрее, чем конкатенатить строки. Да и использования boost::lexical_cast<std::string> в этом контексте очень большая какаха, т.к. работает примерно так
        int id;
std::string out;
std::stringstream ss;
ss << id;
ss >> out;

        Получаем
        std::string ScriptInfo::GetDebugInfo() const
{
  std::stringstram ss;
  ss
    << GetScriptCommandName(command)
    << " ('"
    << GetScriptsTableNameByType(type)
    << "' script id: "
    << id
    << ")";
  return ss.str();
}
        Ответить

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