ListView заставляет мое приложение работать очень медленно, хотя я создал Holder Class

Question

Я совершенно отчаялся из-за этой проблемы, и я пробовал все идеи, которые вы все публиковали в других темах.

Проблема в том, что моя программа выполняет массивные вызовы метода getView (), и я реализовал парадигму Холдера, но она все еще ужасно медленная, может, кто-то может меня просветить?

Это мой адаптеркласс:

    import java.util.ArrayList;

import android.content.Context;
import android.location.Location;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.ArrayAdapter;
import android.widget.TextView;

    public class IconListViewAdapterPharmacy extends ArrayAdapter<PharmacyListItem> 
    {

    private ArrayList<PharmacyListItem> items;
    private Context cont;
    private Location l;

    public IconListViewAdapterPharmacy(Context cont, int textViewResourceId, ArrayList<PharmacyListItem> items, Location l)
    {
        super(cont, textViewResourceId, items);
        this.cont = cont;
        this.items = items;
        this.l = l;
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) 
    {
        View v = convertView;

        PharmacyListItem o = items.get(position);
        ViewHolder holder = new ViewHolder();

        if (v == null) 
        {
            LayoutInflater vi = (LayoutInflater)cont.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
            v = vi.inflate(R.layout.icon_row_pharmacy, null);

            holder.tt = (TextView) v.findViewById(R.id.pharmacy_row_toptext);
            holder.address = (TextView) v.findViewById(R.id.pharmacy_row_address);
            holder.distance = (TextView) v.findViewById(R.id.pharmacy_row_distance);
            holder.telephone = (TextView) v.findViewById(R.id.pharmacy_row_telephone);

         // associate the holder with the view for later lookup
            v.setTag(holder);
        }    

        else
        {
            holder = (ViewHolder)convertView.getTag();
        }

        holder.tt.setText(o.getName());     

        String add = "" + o.getAddress();
        holder.address.setText("" + add.substring(0, add.lastIndexOf(",")));

        holder.telephone.setText("" + o.getTelephone());

        String distancia = "" + o.getDistance();

        if(l != null)
        holder.distance.setText("Distancia: " + distancia);

        return v;
    }

        static class ViewHolder 
        {
            TextView tt;
            TextView address;
            TextView distance;
            TextView telephone;
        }
    }

Класс PharmacyListItem:

    import android.location.Location;

    public class PharmacyListItem 
    {    
private String name;
private double lat;
private double lng;
private String telephone;
private double distance;

public PharmacyListItem(Location l, String name, double lat, double lng , String telephone)
{
    this.name = name;
    this.lng = lng;
    this.lat = lat;
    this.telephone = telephone;

    Location l2 = new Location(l);

    l2.setLatitude(lat);
    l2.setLongitude(lng);

    distance = l.distanceTo(l2);
}

public String getName()
{
    return name;
}

public String getAddress()
{
    return GeoLocator.getAddress(lat, lng);
}

public String getTelephone()
{
    return telephone;
}


public String getDistance()
{
    return (""+distance).substring(0, (""+distance).indexOf(".")) + "m.";
}

public double getDist()
{
    return distance;
}

}

Любые идеи будут высоко оценены!

Спасибо всем!

РЕДАКТИРОВАТЬ Я вставляю сюда мой файл .xml:

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout
xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="fill_parent"
android:layout_height="?android:attr/listPreferredItemHeight"
android:padding="6dip"
android:layout_marginRight="4dip"
android:background="@drawable/bg_detalle_1"> 


      <ImageView
        android:id="@+id/arrow2" 
        android:layout_width="wrap_content"
        android:layout_centerVertical="true"
        android:layout_height="wrap_content" 
        android:layout_alignParentRight="true"
        android:layout_alignParentBottom="true"   
        android:singleLine="true"
        android:ellipsize="marquee"
        android:layout_marginRight="4dip"/>

      <LinearLayout android:padding="10dip" android:layout_width="match_parent" android:layout_height="wrap_content" android:id="@+id/linearLayout3" android:orientation="vertical">
          <LinearLayout android:layout_height="wrap_content" android:id="@+id/linearLayout1" android:layout_width="match_parent">
              <TextView android:gravity="center" android:textColor="#525252" android:layout_height="fill_parent" android:singleLine="true" android:ellipsize="marquee" android:text="" android:textStyle="bold" android:id="@+id/pharmacy_row_toptext" android:layout_width="wrap_content" android:layout_marginRight="6dip"></TextView>
          </LinearLayout>
          <LinearLayout android:layout_height="wrap_content" android:id="@+id/linearLayout2" android:layout_width="match_parent">
                  <TextView android:layout_height="fill_parent" android:text="TextView" android:id="@+id/pharmacy_row_address" android:layout_width="wrap_content" android:layout_marginRight="6dip"></TextView>
          </LinearLayout>

          <LinearLayout android:layout_height="wrap_content" android:layout_width="match_parent" android:id="@+id/linearLayout5">
              <TextView android:gravity="center" android:layout_height="fill_parent" android:text="TextView" android:singleLine="true" android:ellipsize="marquee" android:layout_width="wrap_content" android:id="@+id/pharmacy_row_telephone"></TextView>
          </LinearLayout>

          <LinearLayout android:layout_gravity="right" android:layout_height="wrap_content" android:id="@+id/linearLayout5" android:layout_width="match_parent">
              <TextView  android:layout_height="fill_parent" android:text="TextView" android:singleLine="true" android:ellipsize="marquee" android:layout_width="wrap_content" android:id="@+id/pharmacy_row_distance"></TextView>
          </LinearLayout>
      </LinearLayout>          

</LinearLayout>

Answer 1

Судя по опубликованному коду, проблема не в вашем адаптере или коде ListView.Вы используете вызовы GeoLocation API из вашего метода getview, которые, как известно, являются медленными и ресурсоемкими.Вам, вероятно, следует взглянуть на эту часть вашего кода для оптимизации.Ваш адаптер и код ListView абсолютно исправны и оптимизированы.

Answer 2

Это действительно старый поток, но я обнаружил большую причину для медленной прокрутки ListView.

Атрибут android: ellipsize = "marquee" очень сильно влияет на производительность ListView.Проще говоря, без него маслянистая плавная прокрутка, но с ней, чертовски медленная.

Я обнаружил это, когда использовал Show Overdraw в GPU в настройках разработчика.

Надеюсьэто поможет будущим читателям этой темы.

Answer 3

Вместо этого

public IconListViewAdapterPharmacy(Context cont, int textViewResourceId, ArrayList<PharmacyListItem> items, Location l)
{
    super(cont, textViewResourceId, items);
    this.cont = cont;
    this.items = items;
    this.l = l;
}

сделать это

public IconListViewAdapterPharmacy(Context cont, int textViewResourceId, ArrayList<PharmacyListItem> items, Location l)
{
    super(cont, textViewResourceId, items);
    this.l = l;
}

Затем в элементах доступа getView использовать getItem(position) вместо items.get(position) и получить контекстный вызов getContext().

Не знаю, улучшит ли это производительность.Но не пытайтесь.Также это правильный путь.

Редактировать:
Позволяет получить адрес и обновить интерфейс позже, как вы, если у вас было изображение профиля, которое нужно загрузить и показать.

String add = "" + o.getAddress();
if (TextUtils.IsEmpty(add)) {
    // download the geolocation. pass the position to the downloader thread which will update the item at position with address, after fetching is complete.
    new GetAddressTask(position).execute(o);
} 

Это освободит поток пользовательского интерфейса, чтобы сконцентрироваться на отображении списка.

class GetAddressTask extends Asynctask<.......> {
     int mPosition = -1;
     public void GetAddressTask(int position) {
         mPosition = position;
     }
     String doInBackground(parameters) {
         return GeoLocation.getaddress(parameter[0]);
     }

     void onPostExecute(String result) { // this runs on UI thread.
        // use mPosition here.
        // update addapter item with this address thats in 'result' variable.
       // and call adapter.notifyDatasetInvalidated()
     }

Answer 4

Вы должны переместиться ViewHolder holder = new ViewHolder();, чтобы оказаться внутри секции if(v==null).

Answer 5

Попробуйте сбросить параметр типа <PharmacyListItem> в вашем классе IconListViewAdapterPharmacy. Вы всегда можете изменить тип items на ArrayList или Object[] и использовать конструктор

IconListViewAdapterPharmacy(Context context, int resource, Object[] objects){...}

Я столкнулся с подобной проблемой при расширении класса ArrayAdapter<String> для моего обычая Adapter. Когда я опустил параметр типа <String> и просто расширил ArrayAdapter, все работало намного быстрее. Я не совсем уверен, что понимаю, почему ...