web-dev-qa-db-ja.com

Android：データ（エクストラ）をフラグメントに渡す

Androidプログラミングは初めてで、ParcelableのArrayListをフラグメントに渡すときに問題が発生します。これは、起動されるアクティビティです（正常に動作しています！）feedlistは、パーセル可能なMusicのArrayListです。

Intent in = new Intent(context, ListMusicsActivity.class);

in.putExtra("arrayMusic", feedList);
activity.startActivity(in);

フラグメントアクティビティonCreate（）メソッド：

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

    if(savedInstanceState != null)
    {
        ListMusicFragment frag = new ListMusicFragment();
        frag.setArguments(getIntent().getExtras());
    }
}

フラグメントコード：

public class ListMusicFragment extends SherlockFragment{

private ArrayList<Music> listMusics = new ArrayList<Music>();
private ListView listMusic;


@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container, 
    Bundle savedInstanceState)
{
    listMusics = (ArrayList<Music>) getArguments().getSerializable("arrayMusic");
    View view = inflater.inflate(R.layout.musiclistview, container, false);
    listMusic = (ListView) view.findViewById(R.id.musicListView);
    listMusic.setAdapter(new MusicBaseAdapter(getActivity(), listMusics));

    return view;
}
}

問題はラインにあると思う

listMusics = (ArrayList<Music>) getArguments().getSerializable("arrayMusic");

最後に、これは私の音楽クラスです。

public class Music implements Parcelable{

private String url;
private String artist;
private String title;
private String duration;
private String info_url;
private String lyrics;


public Music(String url, String artist, String title, 
    String duration, String lyrics, String info_url)
{
    this.url = url;
    this.artist = artist;
    this.title = title;
    this.duration = duration;
    this.lyrics = lyrics;
    this.info_url = info_url;
}

public Music(Parcel in)
{
    url = ParcelUtils.readString(in);
    artist = ParcelUtils.readString(in);
    title = ParcelUtils.readString(in);
    duration = ParcelUtils.readString(in);
    info_url = ParcelUtils.readString(in);
    lyrics = ParcelUtils.readString(in);
}

public String getUrl()
{
    return url;
}

public String getArtist()
{
    return artist;
}

public String getTitle()
{
    return title;
}

public String getDuration()
{
    return duration;
}

public String getLyrics()
{
    return lyrics;
}

public String getInfo()
{
    return info_url;
}

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


@Override
public void writeToParcel(Parcel dest, int flags)
{
    ParcelUtils.writeString(dest, url);
    ParcelUtils.writeString(dest, artist);
    ParcelUtils.writeString(dest, title);
    ParcelUtils.writeString(dest, duration);
    ParcelUtils.writeString(dest, lyrics);
    ParcelUtils.writeString(dest, info_url);
}

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

    public Music createFromParcel(Parcel in)
    {
        return new Music(in);
    }

    public Music[] newArray(int size)
    {
        return new Music[size];
    }
};
}

このコードを実行すると、発生する問題はFragment onCreateView（）メソッドのJava.lang.NullPointerExceptionです。誰かが私が失敗している場所を確認するために正しい方向に私を指し示した場合、私はたくさん感謝します。

EDIT：問題解決：この行をフラグメントActivity onCreate（）メソッドに追加する必要がありました（そうでなければgetArguments（）はnullを返します）：

getSupportFragmentManager().beginTransaction()
    .add(Android.R.id.content, frag).commit();

そして、これをフラグメントコードに追加します。

@Override
    public void onActivityCreated(Bundle savedInstanceState)
{
    super.onActivityCreated(savedInstanceState);

    Bundle bundle = getArguments();
    if(bundle != null)
    {
        listMusics = bundle.getParcelableArrayList("arrayMusic");
        listMusic.setAdapter(new MusicBaseAdapter(getActivity(), listMusics));
    }
}

ここで、listMusicsはArrayList of Parcelable Musicです。

javaandroidandroid-intentnullpointerexception
76
pluralism

二つのこと。まず、フラグメントに正しく渡したいデータを追加しているとは思わない。フラグメントに渡す必要があるのは、意図ではなくバンドルです。たとえば、int値をフラグメントに送信する場合、バンドルを作成し、intをそのバンドルに入れて、フラグメントが作成されたときに使用する引数としてそのバンドルを設定します。

Bundle bundle = new Bundle();
bundle.putInt(key, value);
fragment.setArguments(bundle);

2番目にその情報を取得するには、フラグメントに送信される引数を取得する必要があります。次に、識別したキーに基づいて値を抽出します。あなたのフラグメントの例：

Bundle bundle = this.getArguments();
if (bundle != null) {
    int i = bundle.getInt(key, defaulValue);
}

何を取得するかは、何を置くかによって変わります。また、デフォルト値は通常nullですが、そうである必要はありません。その引数にデフォルト値を設定したかどうかによります。

最後に、onCreateViewでこれを実行できるとは思わない。フラグメントのonActivityCreatedメソッド内でこのデータを取得する必要があると思います。私の理由は次のとおりです。 onActivityCreatedは、基礎となるアクティビティが独自のonCreateメソッドを完了した後に実行されます。アクティビティのonCreateメソッド中に取得したい情報をバンドル内に配置する場合、フラグメントのonCreateViewの間に存在しません。 onActivityCreatedでこれを使用してみて、後でListViewの内容を更新してください。

180
Rarw

私はSerializable =定型コードなしを好みます。他のフラグメントまたはアクティビティにデータを渡す場合 Parcelableへの速度差 は重要ではありません。

また、FragmentまたはActivityのヘルパーメソッドを常に提供します。これにより、どのデータを渡す必要があるかが常にわかります。ここにListMusicFragmentの例：

private static final String EXTRA_MUSIC_LIST = "music_list";

public static ListMusicFragment createInstance(List<Music> music) {
    ListMusicFragment fragment = new ListMusicFragment();
    Bundle bundle = new Bundle();
    bundle.putSerializable(EXTRA_MUSIC_LIST, music);
    fragment.setArguments(bundle);
    return fragment;
}

@Override
public View onCreateView(...) { 
    ...
    Bundle bundle = intent.getArguments();
    List<Music> musicList = (List<Music>)bundle.getSerializable(EXTRA_MUSIC_LIST);
    ...
}
2
artkoenig

メモリにデータが重複していないため、バンドルを選択した理由は簡単です。フラグメントのinit publicメソッドで構成されます

private ArrayList<Music> listMusics = new ArrayList<Music>();
private ListView listMusic;


public static ListMusicFragment createInstance(List<Music> music) {
    ListMusicFragment fragment = new ListMusicFragment();
    fragment.init(music);
    return fragment;
}

public void init(List<Music> music){
    this.listMusic = music;
}

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container, 
    Bundle savedInstanceState)
{

    View view = inflater.inflate(R.layout.musiclistview, container, false);
    listMusic = (ListView) view.findViewById(R.id.musicListView);
    listMusic.setAdapter(new MusicBaseAdapter(getActivity(), listMusics));

    return view;
}
}

つまり、フラグメントのインスタンスを作成し、initメソッドによって（uは必要に応じて呼び出すことができます）、フラグメントのインスタンスへのシリアル化によってコピーを作成せずにリストの参照を渡します。これは非常に便利です。リスト内の何かを変更すると、アプリの他の部分やもちろんもちろんそれを取得できるため、使用するメモリが少なくて済むからです。

1
Ivan