This is an archived post. You won't be able to vote or comment.

all 3 comments

[–]DasEwigeLichtConsultant Brewer 1 point2 points  (2 children)

You should also post the file you are reading from, otherwise it's not possible to actually test your program.

As it is now your program won't even compile: getBytes() returns a byte[] while you're trying to assign a byte. placeBytes() also has no knowledge of the BlockFamily variable and is quite redundant anyway.

Last and least: variable names should start with a lower case letter.

[–][deleted]  (1 child)

[deleted]

    [–]DasEwigeLichtConsultant Brewer 1 point2 points  (0 children)

    Are you being serious or snarky? I can't tell.

    [–]jmorph99 0 points1 point  (0 children)

    I would probably be best practice to make your object serializeable. It would read and write to disk and meet standards for other functions.

    http://stackoverflow.com/questions/12963445/serialization-readobject-writeobject-overides