2
byte[] header = new byte[]{255, 216}; 

string ascii =  Encoding.ASCII.GetString(header);

I expect ASCII to be equal to be FFD8 (JPEG SOI marker)

Instead I get "????"

2
  • Where are you seeing "????" ?... Note that 0xFFD8 is not printable. Commented May 31, 2009 at 14:48
  • Edit the question adding your new info provided below. I think you asked the wrong question maybe? Perhaps best to make a whole new question where you ask the real question... "write a function which determines if a file is an image based only on header information" Commented May 31, 2009 at 18:46

5 Answers 5

11

In this case you'd be better to compare the byte arrays rather than converting to string.

If you must convert to string, I suggest using the encoding Latin-1 aka ISO-8859-1 aka Code Page 28591 encoding, as this encoding will map all bytes with hex values are in the range 0-255 to the Unicode character with the same hex value - convenient for this scenario. Any of the following will get this encoding:

Encoding.GetEncoding(28591)
Encoding.GetEncoding("Latin1")
Encoding.GetEncoding("ISO-8859-1")
Sign up to request clarification or add additional context in comments.

2 Comments

Cheers for the suggestions. Why would you compare the byte arrays?
Because it's binary data. JPEGs aren't text, so shouldn't be converted to text.
8

Yes, that's because ASCII is only 7-bit - it doesn't define any values above 127. Encodings typically decode unknown binary values to '?' (although this can be changed using DecoderFallback).

If you're about to mention "extended ASCII" I suspect you actually want Encoding.Default which is "the default code page for the operating system"... code page 1252 on most Western systems, I believe.

What characters were you expecting?

EDIT: As per the accepted answer (I suspect the question was edited after I added my answer; I don't recall seeing anything about JPEG originally) you shouldn't convert binary data to text unless it's genuinely encoded text data. JPEG data is binary data - so you should be checking the actual bytes against the expected bytes.

Any time you convert arbitrary binary data (such as images, music or video) into text using a "plain" text encoding (such as ASCII, UTF-8 etc) you risk data loss. If you have to convert it to text, use Base64 which is nice and safe. If you just want to compare it with expected binary data, however, it's best not to convert it to text at all.

EDIT: Okay, here's a class to help image detection method for a given byte array. I haven't made it HTTP-specific; I'm not entirely sure whether you should really fetch the InputStream, read just a bit of it, and then fetch the stream again. I've ducked the issue by sticking to byte arrays :)

using System;
using System.Collections.Generic;
using System.Collections.ObjectModel;
using System.Linq;

public sealed class SignatureDetector
{
    public static readonly SignatureDetector Png =
        new SignatureDetector(0x89, 0x50, 0x4e, 0x47);

    public static readonly SignatureDetector Bmp =
        new SignatureDetector(0x42, 0x4d);

    public static readonly SignatureDetector Gif =
        new SignatureDetector(0x47, 0x49, 0x46);

    public static readonly SignatureDetector Jpeg =
        new SignatureDetector(0xff, 0xd8);

    public static readonly IEnumerable<SignatureDetector> Images =
        new ReadOnlyCollection<SignatureDetector>(new[]{Png, Bmp, Gif, Jpeg});

    private readonly byte[] bytes;

    public SignatureDetector(params byte[] bytes)
    {
        if (bytes == null)
        {
            throw new ArgumentNullException("bytes");
        }
        this.bytes = (byte[]) bytes.Clone();
    }

    public bool Matches(byte[] data)
    {
        if (data == null)
        {
            throw new ArgumentNullException("data");
        }
        if (data.Length < bytes.Length)
        {
            return false;
        }
        for (int i=0; i < bytes.Length; i++)
        {
            if (data[i] != bytes[i])
            {
                return false;
            }
        }
        return true;
    }    

    // Convenience method
    public static bool IsImage(byte[] data)
    {
        return Images.Any(detector => detector.Matches(data));
    }        
}

8 Comments

lol, not this again... downvote removed due to your edit. Given the new information the author added now - it'd be best to properly write the IsFileImage method for him. You working on that? I'm not wasting my time if you are...
note, I didn't see who answered it this way. If i knew it was you, I would have commented and explained what he was trying to do... maybe still a downvote tho :P I thought it was a bad answer originally - but then again, it was a bad question too. :)
I'm not currently implementing IsFileImage, but might do later. Before the edit which talked about the JPEG part, the author just asked why it was showing question-marks instead of the expected characters (IIRC). Once again, I think a downvote is almost always worth a comment. Why would anyone else have gained less from the comment than I would have done?
I downvote a lot - I see bad answers and I want them below the better ones. Its just how it works. Requiring a comment would be annoying. Some answers aren't worthy of a comment. You can take that to uservoice, but it always gets rejected. Personally, I'll try to look to see if I'm downvoting Jon Skeet and write a little note. Most times it's not needed though - it was clear (even to you) what was wrong with your answer.
Also note Jon, there was no edit adding the JPEG comment. It was always there. You just missed it. I made a mistake once before too - it was painful. lol
|
1

If you then wrote:

Console.WriteLine(ascii)

And expected "FFD8" to print out, that's not the way GetString work. For that, you would need:

 string ascii = String.Format("{0:X02}{1:X02}", header[0], header[1]);   

1 Comment

It would then print "3F3F" - the biggest problem (IMO) is the fact that it's converted into text at all.
1

I once wrote a custom encoder/decoder that encoded bytes 0-255 to unicode characters 0-255 and back again.

It was only really useful for using string functions on something that isn't actually a string.

Comments

0

Are you sure "????" is the result?

What is the result of:

(int)ascii[0]
(int)ascii[1]

On the other hand, pure ASCII is 0-127 only...

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.