views:

213

answers:

5

Hi,

I'm trying to get an image from an url using a byte stream. But i get this error message:

This stream does not support seek operations.

This is my code:

byte[] b;
HttpWebRequest myReq = (HttpWebRequest)WebRequest.Create(url);
WebResponse myResp = myReq.GetResponse();

Stream stream = myResp.GetResponseStream();
int i;
using (BinaryReader br = new BinaryReader(stream))
{
    i = (int)(stream.Length);
    b = br.ReadBytes(i); // (500000);
}
myResp.Close();
return b;

What am i doing wrong guys?

A: 

The length of a stream can not be read from the stream since the receiver does not know how many bytes the sender will send. Try to put a protocol on top of http and send i.e. the length as first item in the stream.

schoetbi
+1  A: 

If the server doesn't send a length specification in the HTTP header, the stream size is unknown, so you get the error when trying to use the Length property.

Read the stream in smaller chunks, until you reach the end of the stream.

Guffa
+1  A: 

You can't reliably ask an HTTP connection for its length. It's possible to get the server to send you the length in advance, but (a) that header is often missing and (b) it's not guaranteed to be correct.

Instead you should:

  1. Create a fixed-length byte[] that you pass to the Stream.Read method
  2. Create a List<byte>
  3. After each read, call List.AddRange to append the contents of your fixed-length buffer onto your byte list

Note that the last call to Read will return fewer than the full number of bytes you asked for. Make sure you only append that number of bytes onto your List<byte> and not the whole byte[], or you'll get garbage at the end of your list.

Tim Robinson
Hi, I'm kind of confused now. how should my code look like then?
Yustme
@Yustme, `var buffer = new List<byte>(); while (true) { byte[] tmpBuffer = new byte[1024]; int bytesRead = br.Read(tmpBuffer, 0, tmpBuffer.length); buffer.AddRange(tmpBuffer.Take(bytesRead)); if (bytesRead < tmpBuffer.length) { break; } }`
strager
Or use a StreamReader
Kragen
+3  A: 

You probably want something like this. Either checking the length fails, or the BinaryReader is doing seeks behind the scenes.

HttpWebRequest myReq = (HttpWebRequest)WebRequest.Create(url);
WebResponse myResp = myReq.GetResponse();

byte[] b = null;
using( Stream stream = myResp.GetResponseStream() )
using( MemoryStream ms = new MemoryStream() )
{
  int count = 0;
  do
  {
    byte[] buf = new byte[1024];
    count = stream.Read(buf, 0, 1024);
    ms.Write(buf, 0, count);
  } while(stream.CanRead && count > 0);
  b = ms.ToArray();
}

edit:

I checked using reflector, and it is the call to stream.Length that fails. GetResponseStream returns a ConnectStream, and the Length property on that class throws the exception that you saw. As other posters mentioned, you cannot reliably get the length of a HTTP response, so that makes sense.

ngoozeff
Hi ngoozeff, thanks, that did the trick! Seems to be working!
Yustme
The stream length fails but the response.ContentLength does return a valid length.
Andrew Robinson
A: 

Use a StreamReader instead:

HttpWebRequest myReq = (HttpWebRequest)WebRequest.Create(url);
WebResponse myResp = myReq.GetResponse();

StreamReader reader = new StreamReader(myResp.GetResponseStream());
return reader.ReadToEnd();

(Note - the above returns a String instead of a byte array)

Kragen
(... which then can be converted to a byte array using functions provided by the Encoding classes.)
Markus