11

I use the following code to read BigEndian information using BinaryReader but I'm not sure if it is the efficient way of doing it. Is there any better solution?

Here is my code:

// some code to initialize the stream value
// set the length value to the Int32 size
BinaryReader reader =new BinaryReader(stream);
byte[] bytes = reader.ReadBytes(length);
Array.Reverse(bytes);
int result = System.BitConverter.ToInt32(temp, 0);

3 Answers 3

13

BitConverter.ToInt32 isn't very fast in the first place. I'd simply use

public static int ToInt32BigEndian(byte[] buf, int i)
{
  return (buf[i]<<24) | (buf[i+1]<<16) | (buf[i+2]<<8) | buf[i+3];
}

You could also consider reading more than 4 bytes at a time.

Sign up to request clarification or add additional context in comments.

3 Comments

thanks It's very interesting but to be sure that I got the idea correctly, can you explain how we can read more than 4 bytes.
Just call ReadBytes with a larger length, and then use different is to read integers at different positions in the array. But that's an optimization you should only take after benchmarking.
Yes, I've used something like this for some of my projects (albeit not in C#). But I had expected BinaryReader/Writer to have endian control built in, which would make it much simpler.
7

As of 2019 (in fact, since .net core 2.1), there is now

byte[] buffer = ...;

BinaryPrimitives.ReadInt32BigEndian(buffer.AsSpan());

Documentation

Implementation

Comments

1

You could use IPAddress.NetworkToHostOrder, but I have no idea if it's actually more efficient. You'd have to profile it.

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.