Переместить логику из адаптера во фрагмент - ОШИБКА: не удается найти идентификаторы вида - PullRequest
Новая
       39

Переместить логику из адаптера во фрагмент - ОШИБКА: не удается найти идентификаторы вида

0 голосов
/ 02 декабря 2018

ПРОБЛЕМА: Я пытаюсь объединить свои методы медиаплеера (который находится внутри моего адаптера) с моим фрагментом (где я хочу, чтобы мои объекты медиаплеера).
Почему? - Потому что я хочу, чтобы мой адаптер НЕ содержал какие-либо методы / логику.Это должно ТОЛЬКО раздувать взгляды и больше ничего.

Я попытался переместить мои методы MediaPlayer (который находится внутри моего адаптера) в мой фрагмент, но когда я пытаюсь найти мои playB и stopB внутри моего фрагмента (который должен воспроизводить и остановить песню), это дает мнеnullexception.Я посмотрел на мой XML-файл (где мой идентификатор playB + stopB) и все было в порядке.

Мой адаптер: 

public class ListViewAdapter extends BaseAdapter {

//Create variables
MediaPlayer mediaPlayer;
Song currentSong;
int layout;
ArrayList<Song> arrayList;
Context context;

//Constructor
public ListViewAdapter(int layout, ArrayList<Song> arrayList, Context context) {
    this.layout = layout;
    this.arrayList = arrayList;
    this.context = context;
}

//ViewHolder class holding my views
private class Viewholder {
    TextView artistTxt, songNameTxt;
    ImageView playB, stopB;
    CircleImageView artistImg;
}

@Override
public int getCount() {
    return arrayList.size();
}

@Override
public Object getItem(int position) {
    return null;
}

@Override
public long getItemId(int position) {
    return 0;
}

@Override
public View getView(int position, View view, ViewGroup parent) {

    //Create viewholder variable
    final Viewholder viewholder;

    //Check if view is null
    if (view == null) {

        //Create new ViewHolder object
        viewholder = new Viewholder();

        //Inflate my view
        LayoutInflater layoutInflater = (LayoutInflater) context.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
        view = layoutInflater.inflate(R.layout.listview_customlayout, null);

        //Find my view id's
        viewholder.artistImg = view.findViewById(R.id.artistImgBackgroundDetail);
        viewholder.artistTxt = view.findViewById(R.id.artistTxt);
        viewholder.songNameTxt = view.findViewById(R.id.songNameTxt);
        viewholder.playB = view.findViewById(R.id.playB);
        viewholder.stopB = view.findViewById(R.id.stopB);

        //Set my view to viewholder
        view.setTag(viewholder);
    } else {
        viewholder = (Viewholder) view.getTag();
    }

    //Assign song to my arraylist
    final Song song = arrayList.get(position);

    //Set my views to their resources
    viewholder.artistImg.setImageResource(song.getArtistImg());
    viewholder.artistTxt.setText(song.getArtist());
    viewholder.songNameTxt.setText(song.getSongName());

    return view;

Как видите, оченьпростой адаптер, где я настраиваю свой собственный список.

Это мой класс фрагмента, где я ** не могу найти идентификаторы вида: **

MediaPlayer mediaPlayer;
Song currentSong;
int position;
ImageView playB, stopB;

@Nullable
@Override
public View onCreateView(@NonNull final LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) {
    View view = inflater.inflate(R.layout.fragment_main, container, false);

    //Actionbar
    ((AppCompatActivity) getActivity()).getSupportActionBar().setTitle("Song list");

    //Find my listview
    songListView = view.findViewById(R.id.songListView);
    playB = getActivity().findViewById(R.id.playB);
    stopB = getActivity().findViewById(R.id.stopB);

    //create a new arraylist object
    arrayList = new ArrayList<>();


    adapter = new ListViewAdapter(R.layout.listview_customlayout, arrayList, getActivity());

    //Set my listview to my custom adapter
    songListView.setAdapter(adapter);

    //Click on a specific song from my list
    songListView.setOnItemClickListener(new ListViewClickListener());


    final Song song = arrayList.get(position);


    //get all songs
    mediaPlayer = MediaPlayer.create(getActivity(), song.getSong());

    //Play button click performed
    playB.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {


            //Checks if my current song is null and creates a new song
            if (currentSong == null) {
                mediaPlayer = MediaPlayer.create(getActivity(), song.getSong());
            }

            //if mediaplayer is not null and my current song is not equal to the new song i clicked on
            if (mediaPlayer != null && currentSong != song) {

                //resets the mediaplayer and creates a new song from the position in the list
                mediaPlayer.reset();

                mediaPlayer = MediaPlayer.create(getActivity(), song.getSong());
                playB.setImageResource(R.drawable.play_orange);

                mediaPlayer.start();
                playB.setImageResource(R.drawable.pause_orange);
            } else {
                mediaPlayer.pause();
                playB.setImageResource(R.drawable.play_orange);
            }

            //check if current song is null or the newly clicked song is equal to my current song
            //if true then assign the newly clicked song as my CURRENT one
            //--so it doesnt play the same song for every single one
            if (currentSong == null || song != currentSong) {
                currentSong = song;
            }
        }
    });

    //Stop song when click performed
    stopB.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            //stops my current song and make it null
            if (currentSong != null) {
                mediaPlayer.stop();
                mediaPlayer.release();

                currentSong = null;
                playB.setImageResource(R.drawable.play_orange);
            }
        }
    });


    //return my view
    return view;

Я хочу уйтиmy ListViewAdapter to только раздувают просмотры , поэтому это означает, что i не должен содержать функции.

Любое предложение приветствуется, поскольку я пытаюсь лучше понять, что я делаю:)

Если вам нужна дополнительная информация, пожалуйста, сообщите мне.Я с удовольствием постараюсь прояснить это, насколько это возможно.

Спасибо.

Ответы [ 2 ]

0 голосов
/ 04 декабря 2018

Просто попробуйте это сделать интерфейс

ClickUpdate.java 

public interface ClickUpdate {
    void getClicks();
}

Внутри

ListViewAdapter.java 

public static ClickUpdate clickupadte;
public static void setonConnectionListener(ClickUpdate clicklisten) {
            ListViewAdapter.clickupadte = clicklisten
        }

Также нажмите кнопку stopB внутри listviewadapter 

viewholder.stopB.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                clickupadte.getClicks();
            }
        });

Затем, наконец, внутри

класс фрагмента внутри инициализации oncreateview 

ListViewAdapter.setonConnectionListener(this);

также реализует ClickUpdate, затем переопределенные методы генерируют внутри класса фрагмента с именем getClicks, я думаю, что внутри этого метода дают 

 if (currentSong != null) {
            mediaPlayer.stop();
            mediaPlayer.release();

            currentSong = null;
            //playB.setImageResource(R.drawable.play_orange);//this will  cause null point error,if occured to avoid the error pass viewholder.playB inside clickupadte.getClicks(); function
        }
0 голосов
/ 04 декабря 2018

В шаблоне MVVM предполагается, что адаптеры связаны с представлением, а не с моделью представления.Итак, что вы потенциально можете сделать, это сделать адаптер внутренним классом класса фрагмента.Если у вас медиаплеер глобальная переменная, вы можете использовать один и тот же экземпляр внутри адаптера и фрагмента, если это то, что вы ищете.

Надеюсь, это поможет!

...