0

Использую следующий код:

int mret;
char buffer[2048];
string response;

mret = recv(s, buffer, sizeof(buffer), 0);
response = string(buffer);
fill_n(buffer, sizeof(buffer), NULL);

При этом все объявления делаю за пределами функции. Однако, если перенести их внутрь функции (что мне сейчас и нужно), то происходит не приятная ситуация: responce вместо того, что бы подстроить свой size автоматически, равным размерам текста из buffer (допустим 200), ставит size равным 2058, заполняя остальное пространство знаками вопроса ?.

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

4
  • fill_n(buffer, sizeof(buffer), NULL); - что здесь делает NULL? 24 фев 2018 в 18:57
  • @AnT тихо поживает. А что с ним не так? 24 фев 2018 в 18:59
  • 1
    NULL предназначен для использования в указательных контекстах. А вы его в char запихивать пытаетесь. Какой смысл запихивать null pointer constant в char? В современном С++ макро NULL может быть определено как nullptr. Что тогда будет с вашим fill_n? Лучше уж напишите fill_n(buffer, sizeof(buffer), false);. Это false там тоже странно выглядеть будет, но это намного логичнее, чем ваш NULL. 24 фев 2018 в 19:02
  • @AnT окей, как скажете 24 фев 2018 в 19:06

1 ответ 1

3

Если string является ::std::string, то конструктор будет принимать на вход оканчивающуюся нулем строку, а вы ей передаете буфер потенциально полностью забитый мусором. Инициализация вне функции видимо работает так как глобальные переменные всегда инициализируются нулями. Соответственно перед присваиванием надо проверить результат вызова recv и поместить в строку именно полученное количество байт:

if((0 < mret) && (mret <= sizeof(buffer))
{
    response.assign(buffer, static_cast<size_t>(mret));
}
else
{
    // обработка ошибок...
}

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.