Android: получение ошибки Только исходный поток, создавший иерархию представлений, может касаться своих представлений

Я работал над этой проблемой весь день, и я готов рвать на себе волосы. Я нашел несколько ответов здесь и в Интернете, в которых говорится, что это вызвано попыткой сделать что-то с представлением в потоке (а не в потоке пользовательского интерфейса). Но я пробовал все идеи (обработчик/новый поток), которые я видел, и до сих пор не могу заставить их работать. Я много лет программировал на C в качестве хобби, а теперь я новичок в Java/Android. Я программирую с Eclipse и платформой Android 2.1. Я хочу, чтобы мое приложение работало с как можно большим количеством телефонов Android, и я думаю, что все функции, которые я использую, совместимы с API 1. Я также видел, что есть что-то под названием AsyncTask, но не вызовет ли это проблемы с у кого старые телефоны?

Итак, вот что делает мое приложение. Когда я нажимаю кнопку, приложение подключается к веб-сайту и загружает канал xml/rss. Затем он анализирует его и помещает данные в список с помощью специального адаптера, который я создал. Загрузка и анализ могут занять от 1 до 15 секунд, поэтому я хотел добавить диалоговое окно прогресса. После добавления этого я начал получать сообщение об ошибке в заголовке этого поста. Приложение выполняет загрузку успешно (мой пример XML-файла в Интернете содержит 8 записей, поэтому он очень мал), но затем я вижу ошибку перед отображением списка. Поэтому я думаю, мне нужно точно знать, какая часть представления вызывает ошибку, а затем как ее исправить.

Вот код (я удалил весь свой тестовый код за последние несколько часов, поэтому он чистый и будет менее запутанным для всех вас... и для меня):

@SuppressWarnings("serial")
public class ClubMessageList extends ListActivity implements Serializable
{
private static final String TAG = "DGMS News";
private ArrayList<CMessage> m_messages = null;
private MessageAdapter m_adapter;
private ProgressDialog m_ProgressDialog = null; 
private Runnable downloadMessages;
// Need handler for callbacks to the UI thread
final Handler mHandler = new Handler();

@SuppressWarnings("unchecked")
@Override
public void onCreate(Bundle icicle)
{
    Log.i(TAG, "Starting the ClubMessageList activity");
    super.onCreate(icicle);

    setContentView(R.layout.list);
    setTitle("DGMS News - Clubs");

    try
    {
        // check to see if we already have downloaded messages available in the bundle
        m_messages = (ArrayList<CMessage>) ((icicle == null) ? null : icicle.getSerializable("savedMessages"));

        // if there are no messages in the bundle, download them from the web and then display them
        if (m_messages == null)
        {
            m_messages = new ArrayList<CMessage>();
            this.m_adapter = new MessageAdapter(this, R.layout.row_club, (ArrayList<CMessage>) m_messages);
            setListAdapter(this.m_adapter);

            downloadMessages = new Runnable(){
                public void run() {
                    getMessages();
                }
            };
            Thread thread =  new Thread(null, downloadMessages, "DownloadMessages");
            thread.start();
            m_ProgressDialog = ProgressDialog.show(ClubMessageList.this,    
                  "Please wait...", "Retrieving 2010 Show data ...", true);
        }
        else // messages were already downloaded, so display them in the listview (don't download them again)
        {
            Log.i("DGMS News", "Starting activity again. Data exists so don't retrieve it again.");
            m_adapter = new MessageAdapter(this, R.layout.row_club, (ArrayList<CMessage>) m_messages);
            this.setListAdapter(m_adapter);
        }
    }
    catch (Throwable t)
    {
        Log.e("DGMS News",t.getMessage(),t);
    }
}

private Runnable returnRes = new Runnable()
{
    public void run()
    {
        if(m_messages != null && m_messages.size() > 0)
        {
            m_adapter.notifyDataSetChanged();
            for(int i=0;i<m_messages.size();i++)
                m_adapter.add(m_messages.get(i));
        }
        m_ProgressDialog.dismiss();
        m_adapter.notifyDataSetChanged();
    }
};

private void getMessages()
{
    try
    {
        m_messages = new ArrayList<CMessage>();
        ClubFeedParser parser = ClubFeedParserFactory.getParser();
        m_messages = parser.parse();
        for(int i = 0; i < m_messages.size(); i++)
            m_adapter.add(m_messages.get(i));
    }
    catch (Exception e)
    { 
        Log.e("DGMS News", e.getMessage());
    }
    runOnUiThread(returnRes);
}

protected void onSaveInstanceState(Bundle outState)
{
    super.onSaveInstanceState(outState);
    outState.putSerializable("savedMessages", (Serializable) m_messages);
}

@Override
protected void onListItemClick(ListView l, View v, int position, long id)
{
    super.onListItemClick(l, v, position, id);
    Intent intent = new Intent(ClubMessageList.this, ClubDetails.class);
    // Add all info about the selected club to the intent
    intent.putExtra("title", m_messages.get(position).getTitle());
    intent.putExtra("location", m_messages.get(position).getLocation());
    intent.putExtra("website", m_messages.get(position).getLink());
    intent.putExtra("email", m_messages.get(position).getEmail());
    intent.putExtra("city", m_messages.get(position).getCity());
    intent.putExtra("contact", m_messages.get(position).getContact());
    intent.putExtra("phone", m_messages.get(position).getPhone());
    intent.putExtra("description", m_messages.get(position).getDescription());

    startActivity(intent);
}

private class MessageAdapter extends ArrayAdapter<CMessage> implements Serializable
{
    private ArrayList<CMessage> items;

    public MessageAdapter(Context context, int textViewResourceId, ArrayList<CMessage> items)
    {
        super(context, textViewResourceId, items);
        this.items = items;
    }

    public View getView(int position, View convertView, ViewGroup parent)
    {
        View v = convertView;
        if (v == null)
        {
            LayoutInflater vi = (LayoutInflater)getContext().getSystemService(Context.LAYOUT_INFLATER_SERVICE);
            v = vi.inflate(R.layout.row_club, null);

            CMessage m = items.get(position);
            if (m != null)
            {
                TextView ltt = (TextView) v.findViewById(R.id.ltoptext);
                TextView rtt = (TextView) v.findViewById(R.id.rtoptext);
                TextView lbt = (TextView) v.findViewById(R.id.lbottext);

                if (ltt != null)
                    ltt.setText(m.getTitle());

                if (rtt != null)
                    rtt.setText(m.getLocation());

                if (lbt != null)
                    lbt.setText(m.getCity() + ", CO");

                //if (rbt != null)
                    ; // not used in this list row
            }
        }
        return v;
    }
}
}

Как я уже сказал, весь этот код работал нормально, пока я не добавил диалоговое окно прогресса, которое я нашел на другом веб-сайте несколько дней назад.

Я ценю любую помощь, хотя я уже зашел на веб-сайт разработчиков Android, чтобы посмотреть на потоки, обработчики и т. д., и это только запутывало меня все больше и больше. Фактические изменения кода были бы потрясающими. :-) У меня сегодня голова болит после просмотра стольких сайтов.

Спасибо!

Боб


person Bobert    schedule 11.04.2011    source источник


Ответы (2)


              runOnUiThread(new Runnable() {
            public void run() {
                  m_adapter.notifyDataSetChanged();

                  m_adapter.add(m_messages.get(i));

                 m_ProgressDialog.dismiss();
                 m_adapter.notifyDataSetChanged();


            }
        });

Весь ваш код пользовательского интерфейса должен находиться в runOnUiThread. Ошибка, которую вы получаете, заключается в том, что вы пытаетесь обновить пользовательский интерфейс из другого потока, кроме потока пользовательского интерфейса активности.

этот поток в вашем коде вызывает проблему.

               private Runnable returnRes = new Runnable()
             {
                public void run()
                {
              if(m_messages != null && m_messages.size() > 0)
             {
                m_adapter.notifyDataSetChanged();
                for(int i=0;i<m_messages.size();i++)
                m_adapter.add(m_messages.get(i));
              }
           m_ProgressDialog.dismiss();
           m_adapter.notifyDataSetChanged();
         }
          };
person bHaRaTh    schedule 11.04.2011
comment
@ user593424: Спасибо! Это исправило это. И на самом деле я дважды загружал список в свой код, поэтому я это тоже исправил. Очень ценю! - person Bobert; 15.04.2011

Используйте асинтаск. На данный момент почти ни один телефон не использует Android 1.0/1.1 (единственным телефоном, который когда-либо поставлялся с Android ‹ 1.5, был HTC G1, и большинство из них были обновлены OTA очень давно). Если вам действительно нужна поддержка этих устройств, вы можете использовать идентичные файлы UserTask. Дополнительные сведения см. в этой статье. И посмотрите около дюжины SO-вопросов о том, чтобы убедиться, что вы обновляете пользовательский интерфейс из основного потока.

person Yoni Samlan    schedule 11.04.2011
comment
Привет Йони! Спасибо за ваш ответ. Я попробую ваше предложение, как только смогу. Если я смогу заставить его работать, я заменю им существующий код. :-) - person Bobert; 15.04.2011