Форум программистов, компьютерный форум CyberForum.ru

Программирование Android

Войти
Регистрация
Восстановить пароль
 
JavJun
12 / 10 / 3
Регистрация: 27.08.2015
Сообщений: 235
#1

Остановка потока - Android

04.09.2016, 18:45. Просмотров 254. Ответов 10
Метки нет (Все метки)

Доброго времени суток.
Пишу первое приложение под андроид, поэтому прошу сильно не пинать.
Вопрос в следующем: на данный момент при нажатии button основной поток не останавливается, не запускаться следующий и listview не обновляется исходя из нового запроса. Потратил часов 7, и перепробовал уже все что дал гугл, но так ничего и не работает.
Буду очень-очень рад любой помощи.
Код:
Кликните здесь для просмотра всего текста
Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
package ru.startandroid.webparser;
 
import android.app.AlertDialog;
import android.content.DialogInterface;
import android.content.Intent;
import android.net.Uri;
import android.os.Bundle;
import android.app.Activity;
import android.os.AsyncTask;
import android.os.Parcelable;
import android.view.View;
import android.widget.AdapterView;
import android.widget.ArrayAdapter;
import android.widget.Button;
import android.widget.EditText;
import android.widget.ListView;
import android.widget.ProgressBar;
 
import org.jsoup.Jsoup;
import org.jsoup.nodes.Document;
import org.jsoup.nodes.Element;
import org.jsoup.select.Elements;
 
import java.io.IOException;
import java.net.URLEncoder;
import java.util.ArrayList;
 
public class MainActivity extends Activity {
 
    public ArrayList<String> infoList = new ArrayList<>();
    public ArrayList<String> linkList = new ArrayList<>();
    public ArrayAdapter infoAdapter;
    public ListView lv;
    public ProgressBar progressBar;
    public String userRequest;
    EditText eText;
    Button btn;
    NewThread thread;
 
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
 
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
 
        eText = (EditText) findViewById(R.id.editText1);
        btn = (Button) findViewById(R.id.button);
        lv = (ListView) findViewById(R.id.listView1);
        progressBar = (ProgressBar) findViewById(R.id.progressBar);
        progressBar.setVisibility(ProgressBar.INVISIBLE);
        infoAdapter = new ArrayAdapter<>(MainActivity.this, R.layout.list_item, R.id.song_name, infoList);
 
        btn.setOnClickListener(new View.OnClickListener() {
            public void onClick(View v) {
                thread.stopIt();
                userRequest = eText.getText().toString();
                infoList.clear();
                linkList.clear();
                infoAdapter.notifyDataSetChanged();//*
                thread = new NewThread();
                thread.execute();
                progressBar.setVisibility(ProgressBar.VISIBLE);
            }
        });
 
        lv.setOnItemClickListener(new AdapterView.OnItemClickListener() {
            @Override
            public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
                final String downloadLink = linkList.get(position);
 
                AlertDialog.Builder alertDialog = new AlertDialog.Builder(MainActivity.this);
                alertDialog.setTitle("Downloading...");
 
                alertDialog.setMessage("Download " + infoList.get(position) + " ?");
                alertDialog.setPositiveButton("Yes", new DialogInterface.OnClickListener() {
                    public void onClick(DialogInterface dialog, int which) {
                        Intent myIntent = new Intent(Intent.ACTION_VIEW, Uri.parse(downloadLink));
                        startActivity(myIntent);
                    }
                });
 
                alertDialog.setNegativeButton("No", new DialogInterface.OnClickListener() {
                    public void onClick(DialogInterface dialog, int which) {
                        dialog.cancel();
                    }
                });
                alertDialog.show();
            }
        });
 
    }
 
 
    public class NewThread extends AsyncTask<String, Integer, String> {
 
        @Override
        protected String doInBackground(String... arg) {
 
 
            try {
                String url = String.format("http://zaycev.net/search.html?query_search=%s",
                        URLEncoder.encode(userRequest, "UTF-8"));
                String useragent = "Mozilla/5.0 (Windows NT 10.0; Win64; x64) " +
                        "AppleWebKit/537.36 (KHTML, like Gecko) " +
                        "Chrome/52.0.2743.116 Safari/537.36";
 
                Elements nextPage;
                int number = 0;
 
                    do {
                        Document doc = Jsoup.connect(url).userAgent(useragent).timeout(5000).get();
                        nextPage = doc.select("[class$=pager__item_last]");
                        Elements resultOfSearch = doc.getElementsByClass("search-page__tracks");
                        Elements elements = resultOfSearch.select("div[data-dkey]");
 
 
                        for (Element element : elements) {
                            String artName = element.select(".musicset-track__artist").text();
                            String trackName = element.select(".musicset-track__track-name").text();
                            String trackDur = element.select(".musicset-track__duration").text();
 
                            Document link = Jsoup
                                    .connect(element.select(".musicset-track__download-link")
                                            .attr("abs:href"))
                                    .userAgent(useragent)
                                    .timeout(5000)
                                    .get();
 
                            String trackLink = link.select("#audiotrack-download-link").attr("abs:href");
 
                            infoList.add(artName + " - " + trackName + ": " + trackDur);
                            linkList.add(trackLink);
                            number++;
                            publishProgress();
 
                        }
                        url = nextPage.attr("abs:href");
 
 
                    } while (nextPage.hasAttr("href") && number <= 200);
 
            } catch (IOException e) {
                e.printStackTrace();
 
            }
            return null;
        }
 
 
        @Override
        protected void onProgressUpdate(Integer... values) {
 
            lv.setScrollContainer(false);
            Parcelable state = lv.onSaveInstanceState();
            infoAdapter.notifyDataSetChanged(); //*
            lv.setAdapter(infoAdapter);
            lv.onRestoreInstanceState(state);
        }
 
        @Override
        protected void onPostExecute(String result) {
            progressBar.setVisibility(ProgressBar.INVISIBLE);
        }
 
        public void stopIt() {
            if (thread == null) {
                return;
            }
            thread.cancel(false);
        }
    }
}
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
korshun84
13 / 68 / 14
Регистрация: 08.07.2014
Сообщений: 354
05.09.2016, 13:04     Остановка потока #2
Цитата Сообщение от JavJun Посмотреть сообщение
thread.stopIt();
....
thread = new NewThread();
Тебя ничего не смущает в такой конструкции ?
Паблито
резкий
1943 / 1683 / 520
Регистрация: 12.05.2014
Сообщений: 6,018
Завершенные тесты: 1
05.09.2016, 13:24     Остановка потока #3
ты не можешь в джаве просто взять и прервать Thread
пока асинктаск не отработает - другой не запустится
JavJun
12 / 10 / 3
Регистрация: 27.08.2015
Сообщений: 235
05.09.2016, 14:21  [ТС]     Остановка потока #4
korshun84,
thread.stopIt(); - останавливаю поток (пытаюсь=), при повторном нажатии на button
....
thread = new NewThread(); - запускаю новый поток согласно нового запроса

Добавлено через 50 секунд
Паблито, но все таки возможно ли что-то сделать?
korshun84
13 / 68 / 14
Регистрация: 08.07.2014
Сообщений: 354
05.09.2016, 14:24     Остановка потока #5
Цитата Сообщение от Паблито Посмотреть сообщение
ты не можешь в джаве просто взять и прервать Thread
пока асинктаск не отработает - другой не запустится
Да это ладно.
У него же в первую очередь сначала идет обращение к методу класса, который еще не инициализирован, не ?

Добавлено через 2 минуты
Цитата Сообщение от JavJun Посмотреть сообщение
thread.stopIt(); - останавливаю поток (пытаюсь=), при повторном нажатии на button
Я понимаю, что ты пытаешься сделать, другой вопрос, насколько это корректно.
В первую очередь имею в виду, что ты пытаешься обратиться к методу класса, который еще не инициализирован.
JavJun
12 / 10 / 3
Регистрация: 27.08.2015
Сообщений: 235
05.09.2016, 14:34  [ТС]     Остановка потока #6
korshun84, ну я в методе stopIt() добавил проверку на случай если это первое нажатие на кнопку "if (thread == null)"

Добавлено через 1 минуту
Но полностью согласен, криво написано. Пытался уже хотя бы что-то сделать что бы заработало как нужно...
korshun84
13 / 68 / 14
Регистрация: 08.07.2014
Сообщений: 354
05.09.2016, 14:40     Остановка потока #7
Цитата Сообщение от JavJun Посмотреть сообщение
я в методе stopIt() добавил проверку на случай если это первое нажатие на кнопку "if (thread == null)"
Ты в этот метод не попадаешь, потому что экземпляр класса не создан.
Проверку надо делать в onClick()
Паблито
резкий
1943 / 1683 / 520
Регистрация: 12.05.2014
Сообщений: 6,018
Завершенные тесты: 1
05.09.2016, 14:44     Остановка потока #8
я не читал внимательно код, а сразу перешел к сути, нет смыла выискивать что там у него инициализировано, а что нет
пытаться остановить асинктаск - неблагодарное занятие
JavJun
12 / 10 / 3
Регистрация: 27.08.2015
Сообщений: 235
05.09.2016, 14:50  [ТС]     Остановка потока #9
Паблито, но любая же задача имеет решение. Может кардинально что-то изменить? Просто опыта у меня нет, есть только гугл, но гугл ничего не показывает, если не знаешь что искать)
ExFau$t
111 / 99 / 13
Регистрация: 08.05.2012
Сообщений: 683
05.09.2016, 20:05     Остановка потока #10
JavJun, необязательно "убивать" поток, если нужно, чтобы он закончился. Укажи условия при котором он завершится успешно досрочно, например, при выполнении условия делать break из цикла.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
07.09.2016, 22:10     Остановка потока
Еще ссылки по теме:

Аргументы Async потока Android
Остановка выполнения таймера (CountDownTimer) из другого класса Android
Android Остановка / воспроизведение аудио
Android Остановка анимации
Android Не работает остановка потока!

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
xaat
43 / 16 / 6
Регистрация: 05.12.2015
Сообщений: 65
07.09.2016, 22:10     Остановка потока #11
AsyncTask имеет метод cancel

И свою задачу завершать "вручную" при его вызове

Добавлено через 20 минут

Есть метод
isCancelled(), если cancel вызывался он вернет true
Соответственно в

Java
1
2
3
4
5
6
7
8
9
   
 protected String doInBackground(Void... params) {
         do {
 
               If( isCancelled()){
                    return null;
                     } 
 
            } while (nextPage.hasAttr("href") && number <= 200);
Yandex
Объявления
07.09.2016, 22:10     Остановка потока
Ответ Создать тему
Опции темы

Текущее время: 18:03. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru