Re: change function

From:

tonydee <tony_in_da_uk@yahoo.co.uk>

Newsgroups:

comp.lang.c++

Date:

Wed, 3 Feb 2010 17:58:42 -0800 (PST)

Message-ID:

<b74f8c14-be79-4b51-aa9b-4d0df5d29b28@t34g2000prm.googlegroups.com>

On Jan 30, 7:14 am, "Larry" <dontmewit...@got.it> wrote:

Hi,

  I would like to turn this function:

std::string Socket::ReceiveBytes() {
  std::string ret;
  char buf[1024];

  while (1) {
    u_long arg = 0;
    if (ioctlsocket(s_, FIONREAD, &arg) != 0)
      break;

    if (arg == 0)
      break;

    if (arg > 1024) arg = 1024;

    int rv = recv (s_, buf, arg, 0);
    if (rv <= 0) break;

    std::string t;

    t.assign (buf, rv);
    ret += t;
  }

  return ret;

}

into something like:

bool Socket::ReceiveBytes(char * buffer, const int buffersize);

how should I change it internally?

thanks

I think you're making this a lot more complicated than it need be.
The existing recv() or read() functions provide almost this interface
already. If you want to avoid them blocking when no input is
available, set the socket to nonblocking mode before calling them -
you only need do that once instead of checking before every recv().

Cheers,
Tony