Как сократить потребление оперативной памяти у приложения?

Question

[Роман Султанов]

Использую Jsoup для парсинга списка новостей и вывожу их через ListView (вид новости: ImageView + TextView), но я стал замечать, что мое приложение жрет слишком много памяти (видно из скриншота выше) при своем безобидном весе (1.9 MB). Помогите найти и устранить причину. Виновата ли в этом сама библиотека jsoup?

PS: для вывода изображений из сети использую библиотеку Picasso.

Answer 1

[IceJOKER]

О святые единороги, как вы себе представляете помощь от простых смертных, у которых нет сверхспособностей, чтоб узнать что да как у вас там???
Приложение с весом и меньше 1-го мб может на колени поставить телефон, так что не в размере дело, а в коде, который вы пишите и которого тут нет, чтоб хоть как-то вам помочь.

Comments

[Роман Султанов]

public class NewThread extends AsyncTask<String, Void, ArrayList<String>> {

        @Override
        protected ArrayList<String> doInBackground(String... arg) {

            Document doc;
            urlIncomplete = "http://miped.ru/page/";
            url = urlIncomplete + pageNum;

            try {
                doc = Jsoup.connect(url).get();
                imageList.clear();
                titleList.clear();
                linkList.clear();
                image = doc.select("div.story div.story_c img");
                title = doc.select("div.story div.story_title a");
                link = doc.select("div.story div.story_title a");
                for (Element images : image) {
                    imageList.add(images.attr("src"));
                }
                for (Element titles : title) {
                    titleList.add(titles.text());
                }
                for (Element links : link) {
                    linkList.add(links.attr("href"));
                }
            } catch (IOException e) {
                e.printStackTrace();
            }
            return null;
        }

        @Override
        protected void onPostExecute(ArrayList<String> result) {
            if (pageNum != 1) {
                prevPage.setVisibility(View.VISIBLE);
            } else {
                prevPage.setVisibility(View.GONE);
            }

            news.setAdapter(adapter);

            news.setOnItemClickListener(new AdapterView.OnItemClickListener() {
                @Override
                public void onItemClick(AdapterView<?> parent, View itemClicked, int position,
                                        long id) {
                    int idClicked;
                    idClicked = position;
                    Intent browseIntent = new Intent(Intent.ACTION_VIEW, Uri.parse(linkList.get(idClicked)));
                    browseIntent.putExtra("title", titleList.get(idClicked));
                    startActivity(browseIntent);
                }
            });

            news.setOnItemLongClickListener(new AdapterView.OnItemLongClickListener() {
                public boolean onItemLongClick(AdapterView<?> parent, View itemClicked, int position,
                                               long id) {
                    int idClicked;
                    idClicked = position;
                    @SuppressWarnings("deprecation")
                    ClipboardManager copylink = (ClipboardManager) getSystemService(CLIPBOARD_SERVICE);
                    copylink.setText(titleList.get(idClicked) + ": " + linkList.get(idClicked));
                    copy = Toast.makeText(getApplication(), R.string.copy,
                            Toast.LENGTH_SHORT);
                    copy.show();
                    return true;
                }
            });
        }
    }

[Александр]

news.setAdapter(adapter); - интересует, как реализован адаптер. Скорее проблема в нем.
далее по коду:
copy = Toast.makeText(getApplication(), R.string.copy,
Toast.LENGTH_SHORT);
copy.show();
зачем вы создаете переменную???
Toast.makeText(getApplication(), R.string.copy, Toast.LENGTH_SHORT).show();

[Александр]

и еще пропустил:
int idClicked;
idClicked = position;
тоже:
int idClicked = position;

[Роман Султанов]

cry_san: обыкновенный адаптер, ImageView + TextView.

[Александр]

Адаптер в виде класса?
Скиньте код адаптера (если конечно еще нужна помощь).

[Роман Султанов]

cry_san:

package ru.sezex.miped.adapter;

import android.support.v7.app.ActionBarActivity;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.BaseAdapter;
import android.widget.ImageView;
import android.widget.TextView;

import com.squareup.picasso.Picasso;

import java.util.ArrayList;

import ru.sezex.miped.R;

public class NewsAdapter extends BaseAdapter {
    ActionBarActivity context;
    public ArrayList<String> image;
    public ArrayList<String> title;

    public NewsAdapter(ActionBarActivity context,
                       ArrayList<String> image,
                       ArrayList<String> title) {
        super();
        this.context = context;
        this.image = image;
        this.title = title;
    }

    public int getCount() {
        return title.size();
    }

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

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

    private class ViewHolder {
        ImageView imgViewPreview;
        TextView txtViewTitle;
    }

    public View getView(int position, View convertView, ViewGroup parent) {
        ViewHolder holder;
        if (convertView == null) {
            convertView = LayoutInflater.from(context).inflate(R.layout.main_item, parent, false);
            holder = new ViewHolder();
            holder.imgViewPreview = (ImageView) convertView.findViewById(R.id.preview);
            holder.txtViewTitle = (TextView) convertView.findViewById(R.id.title);
            convertView.setTag(holder);
        } else {
            holder = (ViewHolder) convertView.getTag();
        }

        Picasso.with(context).load(image.get(position)).into(holder.imgViewPreview);
        holder.txtViewTitle.setText(title.get(position));
        return convertView;
    }
}

[Александр]

Роман Султанов: Извините за долгое ожидание.
Смотрите из официальной документации пикассо:

Place Holders
Picasso supports both download and error placeholders as optional features.
Picasso.with(context)
.load(url)
.placeholder(R.drawable.user_placeholder)
.error(R.drawable.user_placeholder_error)
.into(imageView);

Далее:

Image Transformations
Transform images to better fit into layouts and to reduce memory size.
Picasso.with(context)
.load(url)
.resize(50, 50)
.centerCrop()
.into(imageView)

[Роман Султанов]

cry_san: что вы хотите этим сказать? Картинки отображаются и без доп. параметров.

[Александр]

Естественно отображаются и без доп параметров. Какие размеры картинок вы загружаете в память? А какой размер отображаемого imgViewPreview? Picasso без доп. параметров (т.е. без оптимизации) хранит кеш и в памяти устройста и в оперативной памяти. Вы ведь парсите список новостей и изображения берете оттуда же?

[Роман Султанов]

cry_san: картинка - 450x300, ImageView - match_parent/150dp.

[Александр]

Просто попробуйте для примера Picasso..resize(50, 50)
Сравните графики потребления памяти