Android: передать данные (дополнительные) в фрагмент

Я новичок в программировании на Android, и у меня возникают проблемы при передаче ArrayList Parcelable в фрагмент. Это действие, которое запускается (работает хорошо!), Где feedlist представляет собой массив ArrayList из произвольной музыки.

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

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

Метод Activity 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");

Наконец, это мой класс Music:

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

Когда я запускаю этот код, проблема, которую я получаю, - это java.lang.NullPointerException в методе Fragment onCreateView(). Я был бы признателен, если бы кто-то указал мне в правильном направлении, чтобы увидеть, где я терплю неудачу.

РЕДАКТИРОВАТЬ: проблема решена: мне просто нужно было добавить эту строку в метод 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 из Parcelable Музыка.

ОТВЕТЫ

Ответ 1

Две вещи. Сначала я не думаю, что вы добавляете данные, которые хотите передать фрагменту правильно. То, что вам нужно передать фрагменту, - это расслоение, а не намерение. Например, если мне нужно отправить значение int в фрагмент, я бы создал пучок, поместил int в этот пакет, а затем установил этот пакет как аргумент, который будет использоваться при создании фрагмента.

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

Во-вторых, чтобы получить эту информацию, вам нужно получить аргументы, отправленные фрагменту. Затем вы извлекаете значение на основе ключа, с которым вы его идентифицировали. Например, в вашем фрагменте:

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

Что вы получаете изменения в зависимости от того, что вы положили. Также значение по умолчанию обычно null, но не обязательно. Это зависит от того, установлено ли значение по умолчанию для этого аргумента.

Наконец, я не думаю, что вы можете сделать это в onCreateView. Я думаю, вы должны получить эти данные в своем методе onActivityCreated. Мое рассуждение таково. onActivityCreated запускается после того, как базовый актив завершил свой собственный метод onCreate. Если вы размещаете информацию, которую хотите получить в комплекте, задерживая ваш метод onCreate, он не будет существовать во время фрагмента onCreateView. Попробуйте использовать это в onActivityCreated и просто обновите содержимое ListView позже.

Ответ 2

Я предпочитаю 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);
    ...
}

Ответ 3

Существует простая причина, почему я предпочитаю связывать из-за отсутствия дубликатов данных в памяти. Он состоит из открытого для init метода для фрагмента

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

В двух словах вы создаете экземпляр фрагмента an методом init (u можете называть его как хотите) вы передаете ссылку своего списка, не создавая копию путем сериализации в экземпляр фрагмента. Это очень полезно, потому что, если вы что-то измените в списке, вы получите его в других частях приложения, и, конечно же, вы используете меньше памяти.

Ответ 4

отличный ответ от @Rarw. Попробуйте использовать пакет для передачи информации из одного фрагмента в другой