Почему не происходит заполнение переменных из потока?

Question

[HubMan17]

Столкнулся с проблемой, что паршу сайт в android studio на java, узнал, что в основном потоке парсить нельзя и нужно делать это из нового потока, но вот проблема, не получается произвести запись данных в переменные из потока или что-то делаю не так, подскажите пожалуйста, как осуществить запись в данные класса из созданного потока, спасибо

public class MainActivity extends AppCompatActivity {

ListView listView;

MyListAdapter adapter;
ArrayList<String> URL_pict = new ArrayList<String>();


Document doc, doc1;
Elements content;
ArrayList<String> titleList = new ArrayList<String>();

public ArrayList<String> title = new ArrayList<String>();
public ArrayList<String> subTitle = new ArrayList<String>();
public ArrayList<String> imageID  = new ArrayList<String>();

@SuppressLint("MissingInflatedId")
@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);


    new NewThread().execute();

    listView = (ListView) findViewById(R.id.listView_id);

    adapter = new MyListAdapter(MainActivity.this, title, subTitle, imageID);
    listView.setAdapter(adapter);
}

public class NewThread extends AsyncTask<String, Void, String> implements com.example.test_listview.NewThread {

    @Override
    protected String doInBackground(String... arg){

        try {
            doc = Jsoup.connect("...").get();
            content = doc.select("p, img");
            titleList.clear();

            titleList.add(doc.title());
            Elements table = doc.getElementsByClass("item-page");
            Element page = table.get(0);
            Elements links = page.select("a[href]");

            // picture_preview
            Elements pict_srcs = page.select("img[src]");

            // preview
            for (Element link : links) {
                //previewPict
                int count = 0;
                String pict_src = "some_links" + pict_srcs.get(count).attr("src").toString();
                URL_pict.add(pict_src);
                count++;

                doc1 = Jsoup.connect("..." + link.attr("href").toString()).get();

                // get header of news
                Elements titles = doc1.getElementsByClass("item-page");
                Element title2 = titles.get(0);
                Elements head = title2.select("h2");


                // main text of news
                Elements news_texts = doc1.getElementsByTag("p");
                Element news_text = news_texts.get(10);

                title.add("doc.title()");
                subTitle.add("news_text.text()");
                imageID.add("https://c4.wallpaperflare.com/wallpaper/71/196/981/digital-art-minimalism-nature-hills-wallpaper-preview.jpg");
                break;
            }

        } catch (IOException e) {
            e.printStackTrace();
        }

        return null;
    }
}

Comments

[Dmitry Roo]

В какие переменные вы ждёте запись?

[HubMan17]

Dmitry Roo, Например в эти три title, subTitle, imageID, в которые я для проверки сам даже указываю данные, но всегда что-то не работает, но если сделать то же самое в OnCreate, всё работает отлично

Answer 1

[Dmitry Roo]

Проблема, видимо, в том, что данные в новом потоке приходят позже, чем вы их подставляете в

adapter = new MyListAdapter(MainActivity.this, title, subTitle, imageID);

Т.е. у вас метод onCreate уже выполнен, а doInBackground еще даже не запустился. Соответственно и данные, которые он получил, уже никому не нужны.

Comments

[HubMan17]

а как тогда быть

[Dmitry Roo]

HubMan17, видимо, осваивать реактивное програмирование. Вот, что я нагуглил:
https://habr.com/ru/company/mobileup/blog/326962/
https://habr.com/ru/post/306746/

[HubMan17]

Помогло удаление объявления адаптера в onCreate и его объявление в функции самого потока, которое выглядит так

@Override
        public void onPostExecute(String result) {
            adapter = new MyListAdapter(MainActivity.this, title, subTitle, imageID);
            listView.setAdapter(adapter);
        }

[Сергей Горностаев]

Проблема ещё в том, что изменять из потока состояние - очень плохая идея. Для получения данных из потока в AsyncTask стоит переопределить метод onPostExecute. Да и вообще AsyncTask уже deprecated.