POST запрос и массив?

Question

[Ockonal]

Здравствуйте, нужно вручную на с++ составить пост-запрос и отправить. Он должен включать в себя массив чисел. Вот в этом и проблема:

int *binary = new int[bufferLength];<br>
...<br>
<br>
std::string data;<br>
std::stringstream out(data);<br>
out << "POST /push1_pub?id=game\r\n";<br>
out << "Host: http://localhost\r\n";<br>
out << "Content-Length: ";<br>
out << bufferLength << "\r\n";<br>
out << "\r\n";<br>
out << reinterpret_cast<char*>(binary);<br>
<br>
if (socket.send(data.c_str(), data.size()) == -1)<br>
{<br>
    std::cout << "Failed to send headers\n";<br>
}<br>
else<br>
{<br>
   // Получаю ответ от сервера<br>
}<br>

Не могу понять, как правильно передать массив int-элементов, какой размер в Content-Length указывать. В коде, что я показал, сервер не отвечает, судя по всему, ждет еще информации. Если убрать приведение к char*, то всегда Bad Request.



Подскажите, пожалуйста.

Answer 1

[Riateche]

Во-первых, при таком подходе могут быть проблемы с кодировкой. Не любые символы можно писать в теле запроса, всё зависит от Content-encoding. Но точно не знаю, надо пробовать.

Самым надежный способ — честно преобразовать все числа в строки и эти строки записать в запрос, разделяя, например, пробелом. Не знаю, подходит ли это решение к вашей задаче, но так будет проще и проверять корректность запроса, и принимать данные на сервере.

Если же делать, как вы пытаетесь (записать в запрос байты, которыми представляются целые числа), то записывать в строку нужно так:

out.write(reinterpret_cast<char*>(binary), bufferLength*sizeof(int));

Тогда запишется массив целиком. А у вас получается, что байты пишутся до тех пор, пока не встретится нулевой байт. А в представлении int-а нулевые байты встречаются часто. В Content_length нужно указать количество байт в теле запроса, т.е. bufferLength*sizeof(int).

Comments

[Ockonal]

Я так делаю, потому что в передаваемой строке только и только числа, на клиенте приходится менять порядок байт, но с чтением проблем не. Спасибо за идею.

Answer 2

[Melkij]

Я немного совсем не знаю плюсы, но вы уверены, что reinterpret_cast указателя на массив реинтерпретирует весь массив, а не только первый элемент? И при том опять же весь массив запишется в поток?

Сделайте std::cout << data; посмотрите, что записалось в поток и с каким content-length

Answer 3

[barker]

Очевидно, что длина в поле Content-Length никак не может задаваться в каких-то там абстрактных размером int-ах, как у вас. Задаётся в байтах. Почитайте длину вывода, на sizeof(int) умножьте, или как-то так. И почему старт-строка не по стандарту, может, потому и Bad Request?

Comments

[barker]

Не по стандарту — в смысле, почему метод и uri только, где HTTP/версия?

[Melkij]

К слову да, Host ведь тоже указан неверно. http:// там не пишется.

[Ockonal]

Вроде можно было не задавать, дописал на всякий случай. На sizeof(int) умножать не надо. Во всяком случае, я так понимаю.
Попытался просто передать массив в поток, без reinterpret_cast, запрос отправляется, все ок. Но, просматривая заголовки, вижу:

Headers: POST /push1_pub?id=game HTTP/1.1
User-Agent: Lotosoft soccer client
Host: localhost
Accept: */*
Content-Type: application/octet-stream
Content-Length: 4

0x165d790

Вот чувствую, это точно не то. Ведь это я передаю просто указатель на массив?

[barker]

Если у вас каждый int — отдельный символ, то не надо, да. Если же вы инты хотите передать просто как битовый поток, то не пойму почему не надо. Вообще, просто непонятно что и в каком виде вы хотите получить. Что должно быть? Символьное представление? raw дата? Если raw то в каком именно виде оно у вас в изначальном массиве представлено в массиве binary?

[barker]

Понятно, что это просто получаем указатель на массив, т.е. на первый элемент. Потому что int *binary это просто указатель, с чего вы взяли, что stringstream его развернёт именно в тот вид (а какой, кстати?), который вам ндо?

Answer 4

[Monnoroch]

Вот это:

out << reinterpret_cast<char*>(binary);

Записывает в поток указатель на char*. Не массив. Указатель.
Чтобы записывало массив, надо в цикле выводить элементы.
Не уверен, но может можно так:

out << std::string(reinterpret_cast<char*>(binary));

Comments

[ixSci]

Нет, он в поток записывает нуль-терминированную строку, на которую указывает char*.

[Monnoroch]

Спасибо.

Answer 5

[ixSci]

Покажите как, и самое главное чем Вы заполняете int массив.