Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

Android Android Lists and Adapters (2015) Using Parcelable Data Reading Parcelable Data

There is a bug here: return new VideoGame(source); //doesn't work

Can't go through the question

VideoGame.java
public class VideoGame implements Parcelable {

    public String mTitle = "";
    public int mYear = 0;

    public VideoGame() {
        // intentionally blank
    }

    // getters and setters omitted for brevity!

    @Override
    public int describeContents() {
        return 0;
    }

    @Override
    public void writeToParcel(Parcel dest, int flags) {
        dest.writeString(mTitle);
        dest.writeInt(mYear);
    }

    public VideoGame(Parcel in) {
      in.readString();
      in.readInt();
        // Task 1 code here!
    }

    public static final Creator<VideoGame> CREATOR = new Creator<VideoGame>() {
        @Override
        public VideoGame createFromParcel(Parcel source) {

          return new VideoGame(source);
        }

        @Override
        public VideoGame[] newArray(int size) {
            // Task 3 code here!
            return new VideoGame[0];
        }
    };
}

3 Answers

AR Ehsan
AR Ehsan
7,912 Points

If this was helpful, feel free to mark as best answer

            @Override
        public VideoGame[] newArray(int size) {
            // Task 3 code here!
            return new VideoGame[0];
        }

I'm not even at Task 3 yet. Task 2 code is not passing through the compiler.

AR Ehsan
AR Ehsan
7,912 Points

Try the code below

AR Ehsan
AR Ehsan
7,912 Points

If this was helpful, feel free to mark as best answer

               public class VideoGame implements Parcelable {

    public String mTitle = "";
    public int mYear = 0;

    public VideoGame() {
        // intentionally blank
    }

    // getters and setters omitted for brevity!

    @Override
    public int describeContents() {
        return 0;
    }

    @Override
    public void writeToParcel(Parcel dest, int flags) {
        dest.writeString(mTitle);
        dest.writeInt(mYear);
    }

    public VideoGame(Parcel in) {
        // Task 1 code here!
       mTitle = in.readString();
     mYear = in.readInt();

    }

    public static final Creator<VideoGame> CREATOR = new Creator<VideoGame>() {
        @Override
        public VideoGame createFromParcel(Parcel source) {
            // Task 2 code here!
            return new VideoGame(source);
        }

       @Override
        public VideoGame[] newArray(int size) {  // <-- put the size back in there
            // Task 3 code here!
            return new VideoGame[size];  // <-- and use it here. Change your code to lower case 's'
        }
    };
}
Nicolas Hampton
Nicolas Hampton
44,638 Points

What Abdullah is saying in code is that your constructor for VideoGame is unfinished. You read both the parcel values (in order I might add, nice work) but you forgot to store them into mTitle and mYear. Also, in task three, as he's shown, use the value passed into newArray to set the VideoGame array size.

Thanks Nicolas and Abdullah, I do everything as you say, but the compiler still complains at TASK 2. I think there is a bug in a video course test. That is all I wanted to say. It works great for me on my computer.