Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

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'
        }
    };
}

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.