Search code examples
androidparse-platformandroid-asynctaskcustom-lists

Android How to implement AsyncTask in my custom Listview


When I ran my code I always get a log about Skipped 303 frames! The application may be doing too much work on its main thread. How to implement AsyncTask in this code below, and how to show a progressbar, while fetching data from one of my Parse.com's database. I would appreciate any help.


Here is my Activity's code:

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

    Bundle extras = getIntent().getExtras();
    final String key = extras.getString("KEY");
    ListView lvDoctorsName = (ListView) findViewById(R.id.lvDoctorsName);

    ParseQueryAdapter.QueryFactory<ParseObject> factory = new ParseQueryAdapter.QueryFactory<ParseObject>() {
        public ParseQuery create() {
            ParseQuery query = new ParseQuery("doctors");
            query.whereContains("name", key);
            return query;
        }
    };
    CustomLayout urgentAdapter = new CustomLayout(this, factory);
    lvDoctorsName.setAdapter(urgentAdapter);

    TextView empty = (TextView) findViewById(R.id.empty_list_item);
    lvDoctorsName.setEmptyView(empty);

    itemClickListener();
}

public void itemClickListener() {
    ListView lvDoctorsName = (ListView) findViewById(R.id.lvDoctorsName);
    lvDoctorsName.setOnItemClickListener(new AdapterView.OnItemClickListener() {
        @Override
        public void onItemClick(AdapterView<?> parent, View view, int position, long id) {

            ParseObject item = (ParseObject) parent.getAdapter().getItem(position);
            String objectID = item.getObjectId().toString();
            Intent i = new Intent();
            i.setClass(getApplicationContext(), DoctorPage.class);
            //i.putExtra("new_variable_name",value);
            i.putExtra("objectID", objectID);
            startActivity(i);
        }
    });

}

And here my custom layout:

public class CustomLayout extends ParseQueryAdapter<ParseObject> {


public CustomLayout(Context context, QueryFactory<ParseObject> queryFactory) {
    super(context, queryFactory);
}


@Override
public View getItemView(ParseObject object, View v, ViewGroup parent) {
    if (v == null) {
        v = View.inflate(getContext(), R.layout.row, null);
    }
    super.getItemView(object, v, parent);

    TextView titleTextView = (TextView) v.findViewById(R.id.text1);
    titleTextView.setText(object.getString("name"));

    TextView titleTextView2 = (TextView) v.findViewById(R.id.text2);
    titleTextView2.setText(object.getString("city"));

    return v;
}

Solution

  • One problem I see is that you are calling findViewById() every time getItemView() is called; findViewById() is an expensive operation. You should implement something like the ViewHolder pattern to avoid this.