0

Нужно загрузить изображение по URL и отобразить отрисовать его в Canvas. Загрузка изображения в AsyncTask никак не меняет ситуацию. Делаю следующим образом:

class Lesson extends View {

    public Lesson(Context context) {
        super(context);
    }

    public InputStream downloadImageFromUrl() throws Exception {
        
        URL url = new URL("https://mysite.ru/image.png");
        HttpsURLConnection httpsURLConnection = (HttpsURLConnection) url.openConnection();

        httpsURLConnection.setDoInput(true);
        httpsURLConnection.connect();

        return httpsURLConnection.getInputStream();
    }

    @Override
    protected void onDraw(Canvas canvas) {
        super.onDraw(canvas);

        try {
            Bitmap bitmap = BitmapFactory.decodeStream(downloadImageFromUrl());
        } catch (Exception e) {
            e.printStackTrace();
        }

        canvas.drawBitmap(bitmap, 0, 0, null);
    }
}

Возникает ошибка: : Attempt to invoke virtual method 'boolean android.graphics.Bitmap.isRecycled()' on a null object reference. Почему возникает эта ошибка и как можно её решить без использования SurfaceView?

10
  • Возможный дубликат вопроса: Как починить android.os.NetworkOnMainThreadException
    – woesss
    Commented 11 дек. 2022 в 22:09
  • 1
    И вообще загружать/создавать картинку в каждом вызове onDraw() - очень плохая идея. Правильно создать её заранее однажды, а в onDraw() просто рисовать.
    – woesss
    Commented 11 дек. 2022 в 22:17
  • Имеете ввиду, что её нужно сохранить и после этого отрисовывать в onDraw?
    – Thony
    Commented 11 дек. 2022 в 22:23
  • 1
    Чем не подходит Glide.with(context).load("https://mysite.ru/image.png").into(imageView);? Commented 12 дек. 2022 в 1:54
  • 1
    Исключение выкидывает попытка работы с сетью в UI-потоке - об этом я намекнул в самом начале (в смысле это исключение вы ловите и печатаете, а битмап остаётся неинициализированным, что потом кидает NPE). Это подразумевает что вы не можете получить картинку синхронно здесь и сейчас. Вы должны загрузить её в фоновом потоке, передать вашему классу, а после уже рисовать.
    – woesss
    Commented 12 дек. 2022 в 15:26

1 ответ 1

1
  1. Андроид запрещает выполнять сетевые операции в главном потоке. И собственно любые потенциально длительные операции выполнять в главном (он же UI) потоке - очень плохая практика. Это блокирует "отзывчивость" приложения, а то и всей системы. Поэтому подобные задачи следует выполнять в фоновых потоках (асинхронно). Сделать это можно множеством навороченных и не очень способов, в основе которых лежит класс потока Thread.
  2. метод onDraw() является частью подготовки кадра и в нём делать что-то длительное и тем более каждый раз - тоже очень плохая идея. Нужно сократить логику этого метода до минимально необходимой для вывода кадра - в идеале, ничего кроме вызовов canvas.drawXXX() там быть не должно

Пример "на коленке", надеюсь поймёте принцип

class Lesson extends View {

    // объявляем поле, в котором будем хранить ссылку на загруженный битмап
    private Bitmap bitmap;

    public Lesson(Context context) {
        super(context);
        // сразу при создании класса начинаем загрузку
        new Thread(() -> {
            try(InputStream stream = downloadImageFromUrl()) {
                bitmap = BitmapFactory.decodeStream(stream);
                if (bitmap == null) {
                     Log.e("Lesson", "что-то пошло не так");
                } else {
                    // вызываем перерисовку
                    postInvalidate();
            } catch (Exception e) {
                e.printStackTrace();
            }
        }).start();
    }

    public InputStream downloadImageFromUrl() throws Exception {
        
        URL url = new URL("https://mysite.ru/image.png");
        HttpsURLConnection httpsURLConnection = (HttpsURLConnection) url.openConnection();

        httpsURLConnection.setDoInput(true);
        httpsURLConnection.connect();

        return httpsURLConnection.getInputStream();
    }

    @Override
    protected void onDraw(Canvas canvas) {
        super.onDraw(canvas);
        // проверяем есть ли картинка (возможно она ещё не загружена)
        if (bitmap != null) {
            canvas.drawBitmap(bitmap, 0, 0, null);
        }
    }
}
1
  • Спасибо за ответ, я уже разобрался и забыл отписаться. У меня похожее решение (только Thread вынес в отдельную функцию и перерисовываю с помощью invalidate).
    – Thony
    Commented 14 дек. 2022 в 16:12

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.