Socket Error

[ernilos]

Hi guys! I'm making my own Socket class(my private server development...) but i have one error on this:
char *HSocket::Recv()
{
char tmp[256*256];
int tmplen = 256*256;
int resize;
do{
resize = ::recv(this->socket,(char*)tmp,tmplen,0);
char *resized = new char[resize];
sprintf(resized,"%s",tmp);
delete [] tmp;
return resized;
}while(resize > 0);
};
But i didn't get all buffer, normaly i only get the first 20 or 15 char's... Why?

[Schlüsselbein]

Wtf is this piece of **** supposed to do?
Do yourself a favour and use a network library like boost::asio.

[Kingrap]

Originally Posted by Schlüsselbein

Wtf is this piece of **** supposed to do?
Do yourself a favour and use a network library like boost::asio.

boost lib is ****.. D:

[.SkyneT.]

Originally Posted by Kingrap

boost lib is ****.. D:

Imo you should get banned from this section for this post.

I bet boost is much better than every piece of
code you wrote, or will write.

[Padmak]

Ever tried to implement a good working network lib with boost::asio?
Pain in the A**

Padmak

[MrSm!th]

Originally Posted by ernilos

Hi guys! I'm making my own Socket class(my private server development...) but i have one error on this:
char *HSocket::Recv()
{
char tmp[256*256];
int tmplen = 256*256;
int resize;
do{
resize = ::recv(this->socket,(char*)tmp,tmplen,0);
char *resized = new char[resize];
sprintf(resized,"%s",tmp);
delete [] tmp;
return resized;
}while(resize > 0);
};
But i didn't get all buffer, normaly i only get the first 20 or 15 char's... Why?

You cannot delete[] a local array, it is used for dynamically allocated buffers.

[Delinquenz]

Originally Posted by Padmak

Ever tried to implement a good working network lib with boost::asio?
Pain in the A**

Padmak

Word! I'm still hoping that <network> which got announced for C++14 won't be such a pain.

[Padmak]

Well... isn't asio this <network> lib? Everything i can find about C++14 looks a lot like asio

Padmak

[Schlüsselbein]

I think boost::asio does a very good job at being a low level(!) modern written async network lib.
Its simply that asynchronous programming per se is a difficult and complex task.

Theres only one point about asio that bothers me: the lack of a good official tutorial. The documentation is very sporadic. Imho the examples are the best resource for learning boost::asio.

[Padmak]

I hope it'll be better documented as soon as it's officially standardized. But until then, i'll go with hating it for its complexity

Padmak

[Delinquenz]

There was a nice for the new <network> library which suggested an tcp stream where you can shift in and out with << and >>. That would be nice.

[Schlüsselbein]

which suggested an tcp stream where you can shift in and out with << and >>. That would be nice.

I dont think thats even worth mentioning cause thats what every cpp-programmer should expect. Its a network _stream_ and therefore should support operator<</>>.

Imho cpp-netlib has a good attempt: (idk if its still maintained).

[Delinquenz]

Imho cpp-netlib has a good attempt:

Last update was 1 year ago and it's as far as i see it just supports HTTP.

[artfulwave]

Originally Posted by ernilos

But i didn't get all buffer, normally i only get the first 20 or 15 char's... Why?

Show me your send function.send() might not send all the bytes as the kernel decided not to send all the data out in one pack.
maybe you can do it this way.

int SendData(SOCKET socket, char *buffer, int *length)
{
int Total = 0;
int BytesLeft = *len;
int n;

while(Total < *length) {

n = send(s, buffer+Total, BytesLeft, 0);
if (n == -1) { break; }
Total += n;
BytesLeft -= n;
}

*length = Total;

return n == -1? -1 : 0;
}

And you can call the function like that.

char buf[20] = "Send all data to ...";

int buflen = strlen(buf);

if((SendData(socket,buffer, &buflen) == -1)
{

//Failed to send all the data
cout << "Some data had n`t been sent:" << buflen << endl;

}

Take note :i am not sure if that is the matter i am just guessing.

[ernilos]

Originally Posted by artfulwave

Show me your send function.send() might not send all the bytes as the kernel decided not to send all the data out in one pack.
maybe you can do it this way.



And you can call the function like that.



Take note :i am not sure if that is the matter i am just guessing.

The client works perfectly, isn't coded by me(I'm developing p.server) ><