BookListing project. Each book search overlaps previous results. How do you clear ListView?

Question

This is my onClick information:

        Button searchButton = (Button) findViewById(R.id.search_button);
    searchButton.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {

                BookListFragment bookListFragment = new BookListFragment();

                getSupportFragmentManager().beginTransaction()
                        .add(R.id.book_list_frame, bookListFragment).commit();

            }

    });

BookListFragment w/BookLoader:

    @Override
public Loader<List<BookListing>> onCreateLoader(int i, Bundle args) {

    return new BookLoader(this, GOOGLE_BOOKS_URL + searchedBook);
}


@Override
public void onLoadFinished(Loader<List<BookListing>> loader, List<BookListing> bookListings) {
    mAdapter.clear();


    if(bookListings != null && !bookListings.isEmpty()) {
        mAdapter.addAll(bookListings);
    }

}

@Override
public void onLoaderReset(Loader<List<BookListing>> loader) {
    mAdapter.clear();

}

I am able to get search results. But when searching again the previous results don't clear. They just continuously overlap.

First Search Screenshot Second Search Screenshot

score 2 · Accepted Answer · answered Oct 12 '17 at 17:38

 @Override
 public Loader<List<BookListing>> onCreateLoader(int i, Bundle args) {

 return new BookLoader(this, GOOGLE_BOOKS_URL + searchedBook);
 }


@Override
public void onLoadFinished(Loader<List<BookListing>> loader, 
    List<BookListing> bookListings) {
         mAdapter.clear();


if(bookListings != null && !bookListings.isEmpty()) {
    mAdapter.replace(bookListings);
    }

}
@Override
public void onLoaderReset(Loader<List<BookListing>> loader) {
mAdapter.clear();
 }

score 0 · Answer 2 · answered Oct 12 '17 at 17:14

0

When adding a BookListFragment, use replace() instead of add()

getSupportFragmentManager()
    .beginTransaction()
    .replace(R.id.book_list_frame, bookListFragment)
    .commit();

Hope it helps

answered Oct 12 '17 at 17:14

camilocons

31
2

score 0 · Answer 3 · answered Oct 12 '17 at 17:18

0

You have to use replace() instead of add()

 getSupportFragmentManager().beginTransaction()
                            .replace(R.id.book_list_frame, bookListFragment).commit();

That should do it

answered Oct 12 '17 at 17:18

JAAD

12,349
7
36
57

repetitive answer? – phpdroid Oct 12 '17 at 17:18
@phpdroid i was writing the answer when he posted – JAAD Oct 12 '17 at 17:19

score 0 · Answer 4 · answered Oct 12 '17 at 17:19

This is one solution. Remove all child views (rows) by first getting their parent (ListView).

public void removeView(View v) {
    if(v.getParent() != null) {
        ((ViewGroup) v.getParent()).removeView(v);
    }
}

Call removeView(v) for every row in your ListView. In this case all rows share one parent so try making that parent an instance field and then method above will be:

ListView listView;

. . .

public static void removeView(View v) {
    listView.removeView(v);
}

You can also remove all rows with one single call by:

listView.removeAllViews();

Hope this helps!

score 0 · Answer 5 · answered Oct 12 '17 at 17:21

0

difference between fragmentTransaction.add and fragmentTransaction.replace Just further adding onto CamiloCons answer, you're "adding" a new layer of fragments on top everytime you call add().

answered Oct 12 '17 at 17:21

whompum

69
2
12

Thanks! I knew it was something small just could not find an answer anywhere. – T. Rodgers Oct 12 '17 at 18:56

BookListing project. Each book search overlaps previous results. How do you clear ListView?

5 Answers5