Пытаюсь написать создание по формату std::string
с помощью vsnprintf
, но что-то где-то пошло не так и программа падает во время вызова vsnprintf
.
inline std::string makeString(const char * pFormat, va_list vaList)
{
int nLength = strlen(pFormat) * 2;
std::unique_ptr<char[]> pFormatted;
while (true)
{
pFormatted.reset(new char[nLength]);
int nFormattedLength;
va_start(vaList, pFormat);
nFormattedLength = vsnprintf(pFormatted.get(), nLength, pFormat, vaList);
va_end(vaList);
if (nFormattedLength < 0 || nFormattedLength >= nLength)
nLength += std::abs(nFormattedLength - nLength + 1);
else
break;
}
return std::string(pFormatted.get());
}
inline std::string makeString(const char * pFormat, ...)
{
va_list vaList;
std::string result;
va_start(vaList, pFormat);
result = makeString(pFormat, vaList);
va_end(vaList);
return std::move(result);
}
int main()
{
std::cout << makeString("testing %d %s %d\n", 58, "ggg", 76) << std::endl;
}
По логике ошибка где-то в перевыделении памяти, поскольку следующий код работает корректно, но где именно - понять не могу
void test(const char * pFormat, ...)
{
va_list vaList;
char arr[1024];
va_start(vaList, pFormat);
vsnprintf(arr, 1024, pFormat, vaList);
va_end(vaList);
printf("%s", arr);
}
int main()
{
test("testing %d %s %d\n", 58, "ggg", 76);
}
va_start
внутри верхнегоmakeString
??? Вы же его уже сделали в нижнемmakeString
.nLength += std::abs(nFormattedLength - nLength + 1);
если можно простоnLength = nFormattedLength + 1;
va_list
, поэтому допустил ошибку думая, чтоva_start/va_end
локальны для каждой из функций. Если убрать второй вызов, то все работает. Спасибо. Прошу оформить как ответ