«Занятый» диалог прогресса вызывает странную обработку кода - PullRequest
Новая
       9

«Занятый» диалог прогресса вызывает странную обработку кода

1 голос
/ 21 января 2011

Позвольте мне обобщить ситуацию для вас:

  • У меня есть кнопка (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 (которая у меня естьотсюда btw: http://www.anddev.org/tinytut_-_displaying_a_simple_progressdialog-t250.html), все работало нормально - функция setTracklist извлекает строку из текстового файла в Интернете и устанавливает текст в TextView. Теперь, когда я добавил код ProgressDialog и поместилИсходный код в операторе try, в TextView добавляется только очень небольшая часть текстового файла - примерно 22-24 символа, не более. «Занятый» ProgressDialog отображается просто отлично. До этого он отлично работал;Они могут загружать более 1300 символов в TextView.Я не знаю, имеет ли это какое-либо отношение к потоку - я много гуглил и не нашел ответа.

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

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

Обновлено 25/1 с текущим кодом: 

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» включается при нажатии другой кнопки.

1 Ответ

2 голосов
/ 21 января 2011

Я предполагаю, что вы получаете странное поведение, потому что вы вызываете метод пользовательского интерфейса в потоке без пользовательского интерфейса.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, а затем в потоке пользовательского интерфейса происходит обновление пользовательского интерфейса.Думаю, это должно помочь.

...