Question

我正在尝试实现Parcleable，以便我可以将其添加为额外的（这是一项要求）。

这是类SwinImage的精简版，它只显示相关细节，即从Parcleable派生的部分：

package com.example.kevin.imagemetadata;
import android.os.Parcel;
import android.os.Parcelable;

public class SwinImage implements Parcelable
{
    public SwinImage(String imageName, String location, String[] keywords, String imageDate, boolean share, String email, int rating)
    {
        update(imageName, location, keywords, imageDate, share, email, rating);
    }

    //A constructor for when we havent assigned any metadata.
    public SwinImage(String imageName)
    {
    }

    public void update(String imageName, String location, String[] keywords, String imageDate, boolean share, String email, int rating)
    {
    }

    @Override
    public String toString()
    {
    }

    private void storeImageDetails() {
    }

    @Override
    //We don't need it - but we are forced to due to interface.
    public int describeContents()
    {
        return 0;
    }

    @Override
    public void writeToParcel(Parcel out, int flags)
    {
        out.writeString(imageName);
        out.writeString(location);

        out.writeStringArray(keywords);

        out.writeString(imageDate);

        //Can write boolean array but not boolean...
        boolean[] temp = {share};

        out.writeBooleanArray(temp);
        out.writeString(email);

        out.writeInt(rating);
    }

    public static final Parcelable.Creator<SwinImage> CREATOR = new Parcelable.Creator<SwinImage>()
    {

        @Override
        public SwinImage createFromParcel(Parcel parcel)
        {
            return new SwinImage(parcel);
        }

        @Override
        public SwinImage[] newArray(int i)
        {
            return new SwinImage[i];
        }
    };

    //THE PRIVATE CONSTRUCTOR - FOR INTERNAL USE ONLY.
    private SwinImage(Parcel parcel)
    {
        imageName = parcel.readString();
        location = parcel.readString();

        parcel.readStringArray(keywords);

        imageDate = parcel.readString();

        boolean[] tempArr = new boolean[1];
        parcel.readBooleanArray(tempArr);
        share = tempArr[0];

        email = parcel.readString();
        rating = parcel.readInt();
    }

}

好的......所以，在调用类中，我这样做：

public void ClickedImage(View v)
{
    Intent i = new Intent(this, MetaDataActivity.class);

    switch (getResources().getResourceEntryName(v.getId()))
    {
        case "burgerView":
            i.putExtra("SENT_IMAGE", burger); //IT IMPLEMENTS PARCLEABLE THEREFORE WE CAN PASS IT WHERE IT LOOKS FOR A PARCEL
            break;
        case "pitaView":
            i.putExtra("SENT_IMAGE", pita);
            break;
        case "pizzaView":
            i.putExtra("SENT_IMAGE", pizza);
            break;
        case "steakView":
            i.putExtra("SENT_IMAGE", steak);
            break;
    }

    startActivityForResult(i, GET_AND_SET_METADATA_REQUEST);
}

我在最后设置了一个断点，并且确实可以看到它在意图中是正确的：

The Intent showing an Extra

但是，当谈到它时：

@Override
protected void onCreate(Bundle savedInstanceState)
{
    Log.d("Does", "Does this work");

    Intent i;
    i = getIntent();
    image = i.getExtras().getParcelable("SENT_IMAGE");

    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_meta_data);
}

永远不存在。对image的分配使程序崩溃，我无法通过调试器看到我的SwinImage甚至存在于intent中。

我做错了什么？为什么会崩溃？我已经尝试了几个小时的解决方案（主要是改变了我的分配方式 - 以及一些＆＃34;解决方案＆＃34;已经阻止它崩溃，但是image被指定为空）

任何人都可以提供一些见解吗？感谢。

编辑：

我尝试将getIntent移到onCreate之后无效。

编辑＃2：堆栈跟踪：

http://pastebin.com/raw/XJbPxHRv

和steak等类型：

public class MainActivity extends AppCompatActivity {

    static final int GET_AND_SET_METADATA_REQUEST = 1;  // The request code
    SwinImage burger, pita, pizza, steak;

Answer 1

在以SwinImage为参数的Parcel构造函数中，执行此操作：

parcel.readStringArray(keywords);

此操作失败，因为它尝试将String的数组读入keywords，这是null，因为您没有先将其初始化。

当您致电readStringArray()时，您会向其传递一个String现有数组的引用，并将String中的Parcel数组复制到其中。< / p>

注意readStringArray()和writeStringArray()只有在您传递给writeStringArray()的数组引用不是null时才能正常工作，并且如果数组中的元素数量是传递给readStringArray()与您传递给writeStringArray()的数组中的元素数量完全相同。

您应该使用没有这些限制的readStringArray()，而不是createStringArray()。像这样使用它：

keywords = parcel.createStringArray();

Answer 2

你不能在getIntent()之前致电super.onCreate() - 此时没有意图。

在super来电之前移动getIntent来电。

<强>更新

完成上述修复后，您的堆栈跟踪会显示问题：

NullPointerException: Attempt to get length of null array
at android.os.Parcel.readStringArray(Parcel.java:1026)
at com.example.kevin.imagemetadata.SwinImage.<init>(SwinImage.java:108)

似乎keywords是null，因此尝试从包裹中读取它会崩溃。确保keywords永远不为null，或者放置一个布尔值，表示是否可以读取关键字。另外，根据这个：How to use writeStringArray() and readStringArray() in a Parcel你应该使用createStringArray而不是readStringArray，这要简单得多。

一般说明：您应该始终仔细阅读崩溃堆栈跟踪，以便更好地理解代码无法正常工作的原因，通常这是修复损坏代码的最佳方法。

开始活动

2 个答案: