Passing ArrayList

Question

I am trying to pass an ArrayList of type Album to another activity in Android Studio.

The problem is that the Lastfm API I am using does not implement Parcelable.

I tried making my own Album class and extending their Album class but I got the error

"there is no default constructor available in 'de.umass.lastfm.Album'

Quiz.java - intent.getParcel... not working as Album is not Parcelable

public class Quiz extends AppCompatActivity  {

private static TextView tv_quiz;
private static EditText et_quiz;
private ArrayList<Album> albums;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_quiz);

    Intent intent = getIntent();
    albums = intent.getParcelableArrayListExtra(MainActivity.ALBUMS_LIST);
}
}

The calling portion of my MainActivity.java.

Intent intent = new Intent(this, Quiz.class);
    intent.putExtra(ALBUMS_LIST, allAlbums);
    intent.putExtra(DIFFICULTY, difficulty);
    startActivity(intent);

Is there any way I can get around this?

Thanks anyway

Vishal kumar singhvi · Answer 1 · 2019-01-30T17:29:50.190

0

Here is the solution open activity using this code

Bundle bundle = new Bundle();
bundle.putSerializable("data", allAlbums);
bundle.putString(DIFFICULTY", difficulty);
Intent intent = new Intent(this, Quiz.class);
    intent.putExtra(bundle);
    startActivity(intent);

get intent data

albums = (ArrayList<Album>) getIntent().getExtras().getSerializable("data");

updated Album Class

public class Album implements java.io.Serializable {

//your code
}

edited Jan 30 '19 at 17:29

answered Jan 29 '19 at 19:39

Vishal kumar singhvi

936
8
17

This initially compiled but did not work. Still giving Parcel error with the Album class - java.lang.RuntimeException: Parcel: unable to marshal value Album[name='Play', .... – MySupersuit Jan 30 '19 at 15:42
oh i forgot to add one more line in your Album updated answer check again:) – Vishal kumar singhvi Jan 30 '19 at 17:28
yeah if I could change the Album class it would be grand but I can't so this won't work either. Thanks though – MySupersuit Jan 31 '19 at 12:10

score 0 · Accepted Answer · answered Jan 29 '19 at 19:45

I took a look at the api you linked, and Album looks extremely difficult to parcel. I would say that you might be better off simply re-loading the list in the next Activity (rather than trying to pass the list).

The only way to "construct" a new Album instance is through the getInfo() static factory method. You could create a new class AlbumWrapper that is parcelable, send a list of AlbumWrapper through the Intent, and then use getInfo() on the other side to re-fetch the albums.

public class AlbumWrapper implements Parcelable {

    // insert CREATOR here

    public final String artist;
    public final String mbid;

    public AlbumWrapper(Album a) {
        this.artist = a.getArtist();
        this.mbid = a.getMbid();
    }

    private AlbumWrapper(Parcel in) {
        this.artist = in.readString();
        this.mbid = in.readString();
    }

    @Override
    public void writeToParcel(Parcel dest, int flags) {
        dest.writeString(artist);
        dest.writeString(mbid);
    }

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

You can put your list of Albums in the intent like this:

ArrayList<AlbumWrapper> wrappers = new ArrayList<>();

for (Album album : albums) {
    AlbumWrapper wrapper = new AlbumWrapper(album);
    wrappers.add(wrapper);
}

intent.putParcelableArrayListExtra("ALBUM_WRAPPERS", wrappers);

And then in your next activity you can do something like:

List<AlbumWrapper> wrappers = getIntent().getParcelableArrayListExtra("ALBUM_WRAPPERS");
List<Album> albums = new ArrayList<>();

for (AlbumWrapper wrapper : wrappers) {
    Album album = Album.getInfo(wrapper.artist, wrapper.mbid, YOUR_API_KEY);
    albums.add(album);
}

2 Answers2