Диалоговое окно занятости вызывает странную обработку кода

Подытожу для вас ситуацию:

  • У меня есть кнопка (btnChooseEp), и при ее нажатии появляется AlertDialog.
  • Когда что-то выбирается в AlertDialog, должны быть оценены три оператора if.
  • Пока они оцениваются, появляется диалоговое окно ProgressDialog. Это указывает на то, что приложение «занято».
  • После оценки этих операторов ProgressDialog должен исчезнуть.

Моя проблема описана под кодом.

Весь блок кода показан здесь:

ProgressDialog getTracklistProgressDialog = null;

...

    Button btnChooseEp = (Button)findViewById(R.id.btnChooseEp);
    btnChooseEp.setOnClickListener(new OnClickListener()
    {           
        public void onClick(View v)
        {               
            final AlertDialog.Builder builder = new AlertDialog.Builder(GA.this);
            builder.setTitle(getText(R.string.chooseEpisode));
            builder.setItems(episodes, new DialogInterface.OnClickListener()
            {
                public void onClick(DialogInterface dialog, final int pos)
                {                       
                    getTracklistProgressDialog = ProgressDialog.show(GA.this, "Please wait...",
                            "Retrieving tracklist...", true);

                    new Thread()
                    {
                        public void run()
                        {
                            try
                            {                       
                                String str1, epURL;

                                if(pos < 9)
                                {
                                    str1 = getResources().getString(R.string.epNo1);        
                                    epURL = String.format(str1, pos+1);
                                    setTracklist(epURL, tracklist);                 
                                }
                                else if(pos < 100)
                                {
                                    str1 = getResources().getString(R.string.epNo2);
                                    epURL = String.format(str1, pos+1);
                                    setTracklist(epURL, tracklist);
                                }
                                else if(pos >= 100)
                                {
                                    str1 = getResources().getString(R.string.epNo3);
                                    epURL = String.format(str1, pos+1);
                                    setTracklist(epURL, tracklist);
                                }
                            }
                            catch(Exception e)
                            {}

                            // Remove progress dialog
                            getTracklistProgressDialog.dismiss();                               
                        }
                    }.start();                          
                }
            });
            AlertDialog alertDialog = builder.create();
            alertDialog.show();
        }
    });

Не уверен, что нужно, но вот код функции setTracklist:

public void setTracklist(String string, TextView tv)
{
    try
    {
        tv.setText(getStringFromUrl(string));
    }
    catch(Exception e)
    {
        e.printStackTrace();
    } 
}

А код функции getStringFromUrl можно увидеть здесь: http://pastebin.com/xYt3FwaS

Теперь проблема: когда я еще не реализовал вещь ProgressDialog (которую я получил отсюда, кстати: http://www.anddev.org/tinytut_-_displaying_a_simple_progressdialog-t250.html), все работало нормально — функция setTracklist извлекает строку из текстового файла в Интернете. и устанавливает текст в TextView. Теперь, когда я добавил код ProgressDialog и поместил исходный код в оператор try, в TextView добавляется очень небольшая часть текстового файла — примерно 22-24 символа, не больше. «Занятый» ProgressDialog отображается просто отлично. Раньше это работало отлично; он был более чем способен загружать в TextView более 1300 символов. Я не знаю, имеет ли это какое-то отношение к теме - я много гуглил и не нашел ответа.

Итак, как мне заставить его загружать все данные, а не только небольшую часть?

(Между прочим, я хотел бы иметь возможность установить строку «setTracklist (epURL, tracklist);» под всеми операторами if, но тогда он говорит, что не может разрешить «epURL». Глупо писать то же самое. очередь 3 раза!)

Обновлено 25 января с текущим кодом:

final Handler uiHandler = new Handler();
final Button btnChooseEp = (Button)findViewById(R.id.btnChooseEp);
btnChooseEp.setEnabled(false);
btnChooseEp.setOnClickListener(new OnClickListener()
{           
        public void onClick(View v)
        {               
            builder.setTitle(getText(R.string.chooseEpisode));
            builder.setItems(episodes2, new DialogInterface.OnClickListener()
            {
                public void onClick(DialogInterface dialog, final int pos)
                {                       
                    replay.setVisibility(View.VISIBLE);
                    replayWeb.setVisibility(View.VISIBLE);

                    getTracklistProgressDialog = ProgressDialog.show(GA.this, "Please wait...",
                            "Retrieving tracklist...", true);

                    new Thread()
                    {
                        public void run()
                        {
                            try
                            {                       
                                String str1, epURL;

                                if(pos < 9)
                                {
                                    str1 = getResources().getString(R.string.gaEpNo1);      
                                    epURL = String.format(str1, pos+1);
                                    setTracklist2(epURL, tracklist);                    
                                }
                                else if(pos < 100)
                                {
                                    str1 = getResources().getString(R.string.gaEpNo2);
                                    epURL = String.format(str1, pos+1);
                                    setTracklist2(epURL, tracklist);
                                }
                                else if(pos >= 100)
                                {
                                    str1 = getResources().getString(R.string.gaEpNo3);
                                    epURL = String.format(str1, pos+1);
                                    setTracklist2(epURL, tracklist);
                                }
                            }
                            catch(Exception e)
                            {}

                            // Remove progress dialog
                            uiHandler.post(new Runnable()
                            { 
                                public void run()
                                { 
                                    getTracklistProgressDialog.dismiss();                                       
                                }
                            }
                            );                              
                        }
                    }.start();                          
                }
            });
            AlertDialog alertDialog = builder.create();
            alertDialog.show();
        }
    });

public void setTracklist2(final String string, final TextView tv)
{
    try
    {
        uiHandler.post(
                new Runnable()
                {
                    public void run()
                    {
                        try
                        {
                            tv.setText(getStringFromUrl(string));
                        }
                        catch (UnsupportedEncodingException e)
                        {
                            e.printStackTrace();
                        }
                    }
                });
    }
    catch(Exception e)
    {
        e.printStackTrace();
    } 
}

Примечания: «replay» и «replayWeb» — это всего лишь два TextView. «btnChooseEp» включается при нажатии другой кнопки.


person gosr    schedule 20.01.2011    source источник


Ответы (1)


Я предполагаю, что у вас странное поведение, потому что вы вызываете метод пользовательского интерфейса в потоке, отличном от пользовательского интерфейса. getTracklistProgressDialog.dismiss();
должен выполняться в потоке пользовательского интерфейса. Я предполагаю, что он падает, и ваш поток падает, а некоторые из ваших ресурсов остаются в плохом состоянии. Это объясняет, почему вы получаете разное количество символов.

Я бы попытался создать окончательный обработчик в вашем методе onCreate, который будет привязан к uiThread. В этом потоке вы можете позвонить

uiHandler.post( 
 new Runnable() { 
  public void run(){ 
    getTracklistPRogressDialog.dismiss();
  }
 }
);

Это быстро, так что это может быть синтаксически неправильно, проверьте свою идею.

Это лучшее, что я могу получить из того, что вы опубликовали. Если вы опубликуете больше кода, я могу попытаться запустить его, чтобы помочь вам.

Обновлять:

Кажется, я нашел вашу проблему:

Идея иметь еще один поток состоит в том, чтобы выполнять длительную работу там, но то, что у нас есть прямо сейчас, на самом деле выполняет длительную работу в потоке пользовательского интерфейса, что противоположно нашей цели. Что нам нужно сделать, так это переместить вызов getStringFromUrl(url) из вызова setTracklist в поток. Я бы переписал setTracklist следующим образом:

public void setTracklist(String tracklistContent, TextView tv)
{
    try
    {
      runOnUiThread(
        new Runnable() {
          public void run() {
            tv.setText(tracklistContent);
          }
         });
    }
    catch(Exception e)
    {
        e.printStackTrace();
    } 
}

Затем в вашем внутреннем методе onClick сделайте следующее:

            public void onClick(DialogInterface dialog, final int pos)
            {                       
                getTracklistProgressDialog = ProgressDialog.show(GA.this, "Please wait...",
                        "Retrieving tracklist...", true);

                new Thread()
                {
                    public void run()
                    {
                        try
                        {                       
                            String str1, epURL;

                            if(pos < 9)
                            {
                                str1 = getResources().getString(R.string.epNo1);        
                                epURL = String.format(str1, pos+1);
                                String tlContent = getStringFromUrl(epUrl);
                                setTracklist(epURL, tracklist);                 
                            }
                            else if(pos < 100)
                            {
                                str1 = getResources().getString(R.string.epNo2);
                                epURL = String.format(str1, pos+1);
                                String tlContent = getStringFromUrl(epUrl);
                                setTracklist(epURL, tracklist);
                            }
                            else if(pos >= 100)
                            {
                                str1 = getResources().getString(R.string.epNo3);
                                epURL = String.format(str1, pos+1);
                                String tlContent = getStringFromUrl(epUrl);
                                setTracklist(epURL, tracklist);
                            }
                        }
                        catch(Exception e)
                        {}

                        // Remove progress dialog
                        getTracklistProgressDialog.dismiss();                               
                    }
                }.start();                          
            }

Я так, мы делаем вызов веб-службы/url, прежде чем мы восстановим выполнение потока пользовательского интерфейса. Длительный поиск в Интернете выполняется в потоке bg, а затем в потоке пользовательского интерфейса происходит обновление пользовательского интерфейса. Думаю, это должно помочь.

person Nick Campion    schedule 20.01.2011
comment
Никогда раньше не имел дело с дескрипторами, но вот что я сделал: Поставил окончательный обработчик uiHandler = new Handler(); в методе onCreate (сразу после setContentView). Поместите свой код вместо одного оператора .dismiss() в мой код. Ошибок в IDE нет, но боюсь, что в приложении это не имеет значения. Как вы думаете, какой еще код может помочь? - person gosr; 21.01.2011
comment
Можете ли вы запустить это в режиме отладки на эмуляторе со своего рабочего места? Он должен автоматически прерываться, когда возникает исключение, которое укажет нам строку, вызывающую проблему. Я также вижу, что вы устанавливаете текстовое значение элемента пользовательского интерфейса в setTracklist, поэтому вы должны обернуть его в обработчик так же, как вы сделали отклонение. Все касания элементов пользовательского интерфейса должны происходить в одном потоке. - person Nick Campion; 21.01.2011
comment
На самом деле, я бы сначала обернул ваши элементы setTracklist, потому что вы проглатываете все исключения, я предполагаю, что это вызывает исключение Invocation, которое съедается вашим кодом. - person Nick Campion; 21.01.2011
comment
Если я поставлю setTracklist(epURL, tracklist); В коде uiHandler написано, что epURL не может быть разрешен. (На самом деле то же самое написано в последних двух строках моего первого поста.) Я не совсем понял, что вы имели в виду, заключая элементы setTracklist в свой последний ответ. - person gosr; 21.01.2011
comment
Хорошо, кажется, теперь это работает. Что я сделал: определил строковую переменную epURL в начале кода: static String epURL; (выше метод onCreate). Удален setTracklist(epURL, tracklist); из каждого оператора if и вместо этого поместите его в поток uiHandler. Не уверены, что это лучший способ? - person gosr; 21.01.2011
comment
Обновленный ответ с тем, что я имел в виду. - person Nick Campion; 21.01.2011
comment
Спасибо. Тем не менее, это имеет тот же визуальный эффект, что и то, что я описал в своем последнем комментарии. Я думал, что это работает правильно, но я заметил, что это не так. Вот что происходит: я нажимаю на элемент в AlertDialog, диалоговое окно исчезает, но индикатор выполнения не отображается, пока он не извлечет содержимое файла из Интернета, а затем сразу исчезает, и текст отображается в текстовом представлении. Таким образом, индикатор выполнения не имеет здесь эффекта занятости. - person gosr; 23.01.2011
comment
Не буду педантичным, но не могли бы вы обновить свой код до того, что вы сейчас используете. Я просмотрел его и не вижу ничего похожего на то, что он будет создавать ситуации, которые вы описываете. - person Nick Campion; 24.01.2011
comment
Вы должны простить меня, я не видел метод getStringFromUrl... я предполагаю, что вы выходите в Интернет, чтобы что-то сделать. Я обновлю свой ответ. - person Nick Campion; 25.01.2011
comment
Большое спасибо, Ник, за помощь, теперь все работает отлично! :) Я написал setTracklist(tlContent, tracklist); вместо setTracklist(epURL, tracklist);, но это, конечно, то, что вы имели в виду. - person gosr; 26.01.2011