Okay, I have read through nearly every single parcelable question on stack overflow and have tried a number of solutions in order to solve this problem. So overall I have an android project and I want to send 2 ArrayLists of Arrays from one class to another through Intent.
Note: this is not a duplicate of the nested parcelable thread here which does not deal with arrays. It is not a duplicate of this because I have made all nested classes parcelable. It is not a duplicate of this or this because I am not using an array list and also I recognize the error is a failure of initializing. It is not a duplicate of this because I do not need a parameter-less constructor in this situation (I have tested adding one and it doesn't change the error). It is also not a duplicate of this because it is not a parcel nested in a parcel.
I must be doing something obvious because the nullpointer error should only occur if my array has not been initialized. However, it is not possible for me to initialize my array length until I know how long I want it to be...
(as a side note perhaps someone could give me greater insight into the readTypedList class and what exactly XXX.Creator does because this undoubtedly could be part of the problem. Also for any future people with the same problem here is a link to the Parcel javadoc which was informative but did not solve my problem.)
So now I have a Display object. The Display object is solely used to store a String[] and allow it to be parceled. The line that breaks with the has been commented below, and should be obvious for anyone with parcel experience because it has clearly not been initialized:
class Display implements Parcelable {
String[] labels;
public Display(String[] labels) {
this.labels = labels;
}
protected Display(Parcel in) {
in.readStringArray(this.labels); // **** THIS LINE BREAKS
}
#Override
public void writeToParcel(Parcel dest, int flags) {
dest.writeStringArray(this.labels);
}
#Override
public int describeContents() {
return 0;
}
public static final Creator<Display> CREATOR = new Creator<Display>() {
#Override
public Display createFromParcel(Parcel in) {
return new Display(in);
}
#Override
public Display[] newArray(int size) {
return new Display[size];
}
};
}
So now I ask myself what I can do to solve this problem and investigated many other threads with promising names but they did not solve my issue.
Here was an answer I would have though would have helped, similarly here they suggested using createTypedList() instead of readTypedList().
However, I tried the first answer and also this and they didn't work because I don't know in advance how long I want my String[] to be, which just ends up leading instead to a bad array length error. And the second answer doesn't help because clearly I don't want to create a new typed list but instead use the typed list I already have, so by creating a new list I end up with a blank list and lose my data.
The root of all of these issues is the fact that my parcel is nested within another parcel and called via:
in.readTypedList(allChartLabels, Display.CREATOR);
Because it is nested, the CREATOR is being called in a way that very much limits my options and has led me to be unable to solve the issue.
Throughout various tests I have encountered a number of errors but no solutions... (the current error my code is throwing is Attempt to get length of null array error).
I know someone has a solution to this issue and for more details here is the rest of my code:
public class SummaryEntry implements Parcelable {
private ArrayList<Calculation> allChartValues; // NOTE: this is another nested parcel class which is basically a duplicate of Display but with float[]
private ArrayList<Display> allChartLabels;
public SummaryEntry() {
// initialize
allChartValues = new ArrayList<>();
allChartLabels = new ArrayList<>();
}
protected SummaryEntry(Parcel in) {
this();
// order in which we do this matters:
in.readTypedList(allChartLabels, Display.CREATOR);
in.readTypedList(allChartValues, Calculation.CREATOR);
}
#Override
public void writeToParcel(Parcel out, int i) {
// order in which we do this matters, must be same as reading typed lists
out.writeTypedList(allChartLabels);
out.writeTypedList(allChartValues);
}
/// ... getters and setters excluded because of irrelevance ///
/// PARCELABLE METHODS
#Override
public int describeContents() {
return 0;
}
public static final Creator<SummaryEntry> CREATOR = new Creator<SummaryEntry>() {
#Override
public SummaryEntry createFromParcel(Parcel in) {
return new SummaryEntry(in);
}
#Override
public SummaryEntry[] newArray(int size) {
return new SummaryEntry[size];
}
};
}
I appreciate you taking the time to read all the way down this long post. Ideally I am looking for a solution to the String[] initialization problem, or if someone could post their working code for a nested parcel including a array, or lastly perhaps someone could point out an easier way to achieve passing these items without nesting two parcels.
You can do it like below code:
Use writeArray & readArray like this:
Try with below updated code:
package com.myapplication;
import android.os.Parcel;
import android.os.Parcelable;
public class User implements Parcelable {
String name;
int age;
String [] array;
#Override
public int describeContents() {
return 0;
}
#Override
public void writeToParcel(Parcel dest, int flags) {
dest.writeString(this.name);
dest.writeInt(this.age);
dest.writeStringArray(this.array);
}
public User() {
}
protected User(Parcel in) {
this.name = in.readString();
this.age = in.readInt();
this.array = in.createStringArray();
}
public static final Parcelable.Creator<User> CREATOR = new Parcelable.Creator<User>() {
#Override
public User createFromParcel(Parcel source) {
return new User(source);
}
#Override
public User[] newArray(int size) {
return new User[size];
}
};
}
I am still unable to solve the second half of the original question (which was how to parcel arrays) but I was able to achieve a successful nested parcel implementation by redesigning my program slightly.
The new design breaks the implementation into 4 nested classes with a straight forward implementation. I also removed my arrays because I was unable to initialize them properly in the parcel.
EDIT: I did also stumble upon this website which automatically creates parcelables for you. Very useful and I wish I had known about it before!
Also note that apparently it is unnecessary to create parcel classes to parcel String and Float. You can instead call Float.class.getClassLoader() and String.class.getClassLoader() to use the parcel CREATORS for those classes.
Here is my code:
Parcelable class Q contains ArrayList of:
Parcelable class A which contains 2 Arraylists of Strings and Floats
By breaking down the parcel into smaller nested parcels I was able to incrementally test the parcel until I arrived at the finalized solution which was able to be passed through my Intent via this code:
ON THE SENDING END:
// create the parcel
Q parcel = new Q();
ArrayList<String> strings = new ArrayList<>();
ArrayList<Float> stats = new ArrayList<>();
// ... populate arraylists ... //
Intent I = new Intent(CURRENTACTIVITY.this, FUTUREACTIVITY.class);
// PUT THE THING
I.putExtra("Data", parcel);
startActivity(I);
ON THE RECIEVING END:
Q parcel = getIntent().getParcelableExtra("Data"); // get data
FULL CODE for the working parcel:
This code is very long and repetitive but the basic principle for setting up parcels is clear and easy to re-use if you want to create your own parcel.
// top level parcelable class Q
public class Q implements Parcelable {
private ArrayList<A> element;
public Q()
{
this.element = new ArrayList<A>();
}
public void addToA(A a)
{
element.add(a);
}
public ArrayList<A> getA() {
return element;
}
public void setA(ArrayList<A> a) {
this.element = a;
}
#Override
public int describeContents() {
return 0;
}
#Override
public void writeToParcel(Parcel dest, int flags) {
dest.writeTypedList(this.element);
}
private Q (Parcel in){
element = new ArrayList<A>();
in.readTypedList(this.element, A.CREATOR);
}
public static final Parcelable.Creator<Q> CREATOR = new Parcelable.Creator<Q>() {
public Q createFromParcel(Parcel in) {
return new Q(in);
}
public Q[] newArray(int size) {
return new Q[size];
}
};
}
// nested parcel object A
public class A implements Parcelable {
private ArrayList<Float> f;
private ArrayList<String> s;
public A() {
f = new ArrayList<>();
s = new ArrayList<>();
}
public A(ArrayList<Float> f, ArrayList<String> s) {
this.f = f;
this.s = s;
}
public ArrayList<String> getS() {
return s;
}
public void setS(ArrayList<String> s) {
this.s = s;
}
public ArrayList<Float> getF() {
return f;
}
public void setF(ArrayList<Float> f) {
this.f = f;
}
protected A(Parcel in) {
if (in.readByte() == 0x01) {
f = new ArrayList<>();
in.readList(f, Float.class.getClassLoader());
} else {
f = null;
}
if (in.readByte() == 0x01) {
s = new ArrayList<>();
in.readList(s, String.class.getClassLoader());
} else {
s = null;
}
}
#Override
public int describeContents() {
return 0;
}
#Override
public void writeToParcel(Parcel dest, int flags) {
if (f == null) {
dest.writeByte((byte) (0x00));
} else {
dest.writeByte((byte) (0x01));
dest.writeList(f);
}
if (s == null) {
dest.writeByte((byte) (0x00));
} else {
dest.writeByte((byte) (0x01));
dest.writeList(s);
}
}
#SuppressWarnings("unused")
public static final Parcelable.Creator<A> CREATOR = new Parcelable.Creator<A>() {
#Override
public A createFromParcel(Parcel in) {
return new A(in);
}
#Override
public A[] newArray(int size) {
return new A[size];
}
};
}
Related
I am trying to make a list of objects that are all of an abstract class, but each are there own class. This list needs to persistent so I figured I implement parcelable since I have done so in the past. Only not with different classes all of an abstract class.
I tried just making the abstract class parcelable but that can't have a creator that I am used to because (of course) you can't create an instance of it (because it is abstract). Reading around I noticed that people said you dont need a constructor in the abstract class, just in the subclasses.
AbstractFocusPower class
public abstract class AbstractFocusPower implements Parcelable {
private transient AppExtension app;
private ImplementSchool school;
private String name;
private int duration;
private int cost;
private int altCost;
private int requiredLevel;
private boolean isSelected;
private boolean isResonant;
private int nofSpirtBonusUsed;
/**
* Constructor for Focus Power with no alternative cost
*/
public AbstractFocusPower(AppExtension app, ImplementSchool school, String name, int requiredLevel, int duration, int cost, boolean isSelected) {
this.app = app;
this.school = school;
this.name = name;
this.requiredLevel = requiredLevel;
this.duration = duration;
this.cost = cost;
this.altCost = -1;
this.isSelected = isSelected;
this.isResonant = false;
}
// I cut out the other constructors
public abstract AbstractFocusPower makeCopy();
public abstract String getDescription();
// I cut out the getters and setters
#Override
public int describeContents() {
return 0;
}
#Override
public void writeToParcel(Parcel dest, int flags) {
dest.writeInt(this.school == null ? -1 : this.school.ordinal());
dest.writeString(this.name);
dest.writeInt(this.duration);
dest.writeInt(this.cost);
dest.writeInt(this.altCost);
dest.writeInt(this.requiredLevel);
dest.writeByte(this.isSelected ? (byte) 1 : (byte) 0);
dest.writeByte(this.isResonant ? (byte) 1 : (byte) 0);
dest.writeInt(this.nofSpirtBonusUsed);
}
protected AbstractFocusPower(Parcel in) {
int tmpSchool = in.readInt();
this.school = tmpSchool == -1 ? null : ImplementSchool.values()[tmpSchool];
this.name = in.readString();
this.duration = in.readInt();
this.cost = in.readInt();
this.altCost = in.readInt();
this.requiredLevel = in.readInt();
this.isSelected = in.readByte() != 0;
this.isResonant = in.readByte() != 0;
this.nofSpirtBonusUsed = in.readInt();
}
Sample subclass
public class AegisFocusPower extends AbstractFocusPower {
public AegisFocusPower(AppExtension app) {
super(app, ImplementSchool.ABJURATION, app.getString(R.string.focus_power_name_aegis), 0, 1, 1, false);
}
#Override
public String getDescription() {
return getApp().getString(R.string.focus_power_desc_aegis, (1+((int) Math.floor(getApp().getCurrentCharacter().getOccultistLevel()/6.0))));
}
#Override
public AegisFocusPower makeCopy() {
return new AegisFocusPower(getApp());
}
public AegisFocusPower(Parcel in) {
super(in);
}
public static final Parcelable.Creator<AegisFocusPower> CREATOR = new Parcelable.Creator<AegisFocusPower>() {
public AegisFocusPower createFromParcel(Parcel in) {
return new AegisFocusPower (in);
}
public AegisFocusPower [] newArray(int size) {
return new AegisFocusPower[size];
}
};
}
Code where I use it
Gson gsonFocusPowers = new Gson();
String jsonFocusPowers = sharedPreferences.getString(FOCUS_POWERS_GSON, null);
Type typeFocusPower = new TypeToken<ArrayList<AbstractFocusPower>>() {
}.getType();
ArrayList<AbstractFocusPower> focusPowers;
focusPowers = gsonFocusPowers.fromJson(jsonFocusPowers, typeFocusPower);
if (focusPowers != null) {
this.focusPowers.addAll(checkForNewFocusPowers(focusPowers));
} else {
this.focusPowers = getNewFocusPowerList();
}
Unfortunately this gives me an error which I don't know how to fix.
java.lang.RuntimeException: Unable to create application nl.rekijan.occultistmentalfocushelper.AppExtension: java.lang.RuntimeException: Unable to invoke no-args constructor for class nl.rekijan.occultistmentalfocushelper.mvc.focuspowers.AbstractFocusPower. Registering an InstanceCreator with Gson for this type may fix this problem.
Edit: Not sure why that post is a duplicate. For starters it doesn't have an accepted answer. The answer requires a 3rd party library. The question isn't about multiple subclasses under a single abstract.
have you tried registering a type adapter, something like Using Gson and Abstract Classes ? I always add adapters both for specific formatting (for dates, big decimals, anything where you usually require a very specific format) but also for sub-classing.
In this case however, no adapter is needed, this is.. straight on?
public abstract class AbstractFocusPower implements Parcelable {
// just some property needed to be pushed through a constructor
protected final String myString;
protected AbstractFocusPower(String myString) {
this.myString = myString;
}
}
and then the impl (yeah added toString(), hashCode() and equals() the way I like them to be in domain objects..):
public class AegisFocusPower extends AbstractFocusPower {
boolean imParcelled;
public AegisFocusPower(String myString) {
super(myString);
}
#Override //yup the interface impl
public void parcelMe() {
imParcelled = true;
}
#Override
public String toString() {
return new StringBuilder("{ imParcelled : ").append(imParcelled).append(", myString : ").append(myString).append(" }").toString();
}
#Override
public int hashCode() {
return toString().hashCode();
}
#Override
public boolean equals(Object other) {
if (other == this) {
return true;
} else if (other == null || !(other instanceof AegisFocusPower)) {
return false;
} else {
return other.hashCode() == hashCode();
}
}
}
and then I can run the following junit :
#Test
public void AegisFocusPowerToJsonAndBack(){
// single instance
AegisFocusPower ea = new AegisFocusPower("apa");
String json = GSON.toJson(ea);
assertEquals("{\"imParcelled\":\"false\",\"myString\":\"apa\"}", json);
AegisFocusPower backAtYa = (AegisFocusPower) GSON.fromJson(json, AegisFocusPower.class);
assertEquals(backAtYa, ea);
// A list
AegisFocusPower ea2 = new AegisFocusPower("bepa");
AegisFocusPower ea3 = new AegisFocusPower("cepa");
List<AegisFocusPower> powerList = new ArrayList<>();
powerList.add(ea2);
powerList.add(ea3);
String jsonList = GSON.toJson(powerList);
assertEquals("[{\"imParcelled\":\"false\",\"myString\":\"bepa\"},{\"imParcelled\":\"false\",\"myString\":\"cepa\"}]", jsonList);
List<AegisFocusPower> backAtYaz = Arrays.asList(GSON.fromJson(jsonList,AegisFocusPower[].class));
assertEquals(backAtYaz.get(0), ea2);
assertEquals(backAtYaz.get(1), ea3);
}
whereas GSON is initialized simply like
private static final Gson GSON = (new GsonBuilder()).registerTypeAdapter(Boolean.class, new JsonBooleanDeAndSerializer()).create();
and the type adapter registered for booleans which I use is irrelevant for your problem.
This is.. simple enough and would work for you too?
Check your imports. You may have mistakenly imported wrong class in your pojo. i.e. I have imported android.net.TransportInfo instead of my own TransportInfo class
I'm trying to pass a list of images to another activity via a Parcelable object in Android. I can convert the images down to a byte[] array fine.
However, I've got a list of these images so there's going to be more than one byte[] array.
Here is my code so far.
public class InvalidItem implements Parcelable {
public String id;
public ArrayList<byte[]> imageList = new ArrayList<>();
public InvalidItem(String id) {
this.id = id;
}
public InvalidItem(Parcel in) {
String[] data = new String[22];
in.readStringArray(data);
this.id= data[0];
this.imageList = data[1];
}
#Override
public int describeContents() {
return 0;
}
#Override
public void writeToParcel(Parcel dest, int flags) {
dest.writeStringArray(new String[]{
this.id,
String.valueOf(this.imageList)
});
}
public static final Parcelable.Creator CREATOR = new Parcelable.Creator() {
public InvalidItem createFromParcel(Parcel in) {
return new InvalidItem(in);
}
public InvalidItem[] newArray(int size) {
return new InvalidItem[size];
}
};
}
As you can see I write the string array which would work for one byte[] array. Is there a way I could have a list of byte[] arrays that I decode into one string and then encode when I want to show the images?
I've tried looking into solutions and people have suggested passing it in the bundle.
However, this application follows a step by step process and requires an object to store the data in.
I need a queue like CircularFifoQueue in Apache Commons but with the ability to iterate elements backward.
I.e. I need to insert elements in a fixed size queue with auto remove of old elements. But I need to iterate elements starting from the youngest element. Is there something available in Java?
I can use something like this (need also rewrite other insert methods like push, pushLast...) but would like to hear other available solutions (if they exist).
public class ConcurrentFixedSizeLinkedDeque<T> extends ConcurrentLinkedDeque<T> {
private int sizeLimit = Integer.MAX_VALUE;
public ConcurrentFixedSizeLinkedDeque() {
}
public ConcurrentFixedSizeLinkedDeque(Collection<? extends T> c) {
super(c);
}
public ConcurrentFixedSizeLinkedDeque(int sizeLimit) {
if(sizeLimit<0) sizeLimit=0;
this.sizeLimit = sizeLimit;
}
public ConcurrentFixedSizeLinkedDeque(Collection<? extends T> c, int sizeLimit) {
super(c);
if(sizeLimit<0) sizeLimit=0;
this.sizeLimit = sizeLimit;
}
public int getSizeLimit() {
return sizeLimit;
}
public void setSizeLimit(int sizeLimit) {
this.sizeLimit = sizeLimit;
}
#Override
public void addFirst(T e){
while(size()>=this.sizeLimit){
pollLast();
}
super.addFirst(e);
}
#Override
public void addLast(T e){
while(size()>=this.sizeLimit){
pollFirst();
}
super.addLast(e);
}
}
Hey I keep trying to print my hash table but even when I use my toArray and toString methods all it prints is the memory location. I'm really confused because I thought toArrays and toStrings were for this very problem. Any help would be appreciated.
I declare my hash table like this:
HashTable<Integer,EmployeeInfo> phone = new HashTable<>(300);
Again, when I try to do these all I get is the memory location and not the actual data that the hash table holds. For example when I do phone.toString() all I get is: hashing.HashTable#55f96302. Any help would be appreciated, thanks :).
package hashing;
import java.util.Arrays;
public class Hash {
public static void main(String[] args) {
// TODO Auto-generated method stub
HashTable<Integer,EmployeeInfo> phone = new HashTable<>(300);
System.out.println(phone.toString());
}
public static class HashTable<K,V>
{
private int numberEntries;
private static final int DEFAULT_CAPACITY = 5;
private HashHolder<K,V>[] hashTable;
private int tableSize;
public HashTable()
{
this(DEFAULT_CAPACITY);
}
public HashHolder<K,V>[] toArray()
{
int size = hashTable.length;
#SuppressWarnings("unchecked")
HashHolder<K,V>[] tempArray = Arrays.copyOf(hashTable,size);//(HashHolder<K,V>[]) new Object[size];
return tempArray;
}
public HashTable(int initCapacity)
{
//Note: I'm not implementing a rigorous check for capacity constraints here. I probably should...
#SuppressWarnings("unchecked")
HashHolder<K,V>[] temp = (HashHolder<K,V>[]) new HashHolder[initCapacity];
hashTable = temp;
numberEntries = 0;
}
}
public static class HashHolder<K,V>
{
private K hashKey;
private V data;
private HashHolder<K,V> next;
private States state;
private enum States {CURRENT,REMOVED};
#Override
public String toString()
{
if(state != States.REMOVED)
return String.format("%s: %s", hashKey.toString(), data.toString());
else
return String.format("REMOVED");
}
public HashHolder(K key, V value)
{
hashKey = key;
data = value;
state = States.CURRENT;
}
}
}
Your HashTable class needs its own toString() method that produces some reasonable output. Otherwise, you just get the default toString behavior from Object that prints the more-or-less useless hashing.HashTable#55f96302 you are seeing.
I have a bit trouble implementing Parcelable. Here's how I did it:
public class Player implements Parcelable{
String name;
int score;
#Override
public int describeContents() {
// TODO Auto-generated method stub
return 0;
}
#Override
public void writeToParcel(Parcel dest, int flags) {
dest.writeString(name);
dest.writeInt(score);
}
public Player(Parcel source){
score = source.readInt();
name = source.readString();
}
}
public class MyCreator implements Parcelable.Creator<Player> {
#Override
public Player createFromParcel(Parcel source) {
return new Player(source);
}
#Override
public Player[] newArray(int size) {
return new Player[size];
}
}
This was the whole code implementing Parcelable. Now I'm trying to create a new class object:
Player newPlayer = new Player(null);
newPlayer.name = text;
newPlayer.score = 0;
playersParceledData.add(newPlayer);
zacniIgro.putParcelableArrayListExtra("parceledData", playersParceledData);
This is the line that is bothering me:
Player newPlayer = new Player(null);
Is the fact that I just insrted "null" okay? Or do I have to insert something else between those ()? I was following this example and this isn't explained in it. It says that a new object should be created like this:
Player newPlayer = new Player();
But I am not allowed to do this since I made a constructor.
Create an additional constructor for use cases you're not using Parcel to construct your object, for example
public class Player implements Parcelable{
/* ... */
public Player(Parcel source){
/* ... */
}
public Player(int score, String name){
this.score = score;
this.name = name;
}
}
Then you can construct objects both from Parcel and using an int and an String:
final Player aPlayer = new Player(10000, "SomeRandomPlayerName");
Also, you read the int and the String in inverse order as you write them.