3

(2016.3.15更新)

先週奇妙な問題に遭遇しました。これについてあなたと話したいと思います。

問題のシナリオ：</p>

AppCompatActivity に searchView があります。テキストが変更されるたびに、アクティビティ内のフラグメントは " getSupportFragmentManager().beginTransaction().replace(R.id.fragment_layout, fragment).commit();" に置き換えられます。

フラグメントには、SearchThread と呼ばれるスレッドがあり、onActivityCreated(). 要するに、そのgetActivity().runOnUiThread(new Runnable{...})中にメソッドがあります。"new Runnable()" の内容に関係なく、searchView のテキストが急速に変化すると NullPointerException が発生し、このフラグメントが頻繁に再確立されます。

ログキャット:

03-15 20:12:25.912/cn.example.app E/AndroidRuntime: FATAL EXCEPTION: Thread-10820  
   Process: cn.example.app, PID: 31532  
   java.lang.NullPointerException  
       at cn.example.app.homepage.GymFragment$searchThread.run(GymFragment.java:257)  
       at java.lang.Thread.run(Thread.java:841)

しかし、runOnUiThread の代わりに sendMessage(searchHandler.obtain...) を使用すると、すべてがうまくいきます!!

GymFragment.Java：</p> 

@Override  
public void onActivityCreated(Bundle savedInstanceState){  
    super.onActivityCreated(savedInstanceState);
    new Thread(new SearchThread()).start();  
}  
class SearchThread implements Runnable{  
        @Override  
        public void run() {  
        String s = "";  
            try {  
                Thread.sleep(4000);  
            } catch (Exception e) {;}  
            //searchHandler.sendMessage(searchHandler.obtainMessage(0, s));//Correct   
            getActivity().runOnUiThread(new Runnable() { //Throw NullPointer Exception or pool-1-thread-1 (with ExecutorService) 
                @Override  
                public void run() {  
                    ;  
                }  
            });  
        }  
}

SearchActivity.Java :

class queryChangeListener implements SearchView.OnQueryTextListener{
    ...
    @Override
    public boolean onQueryTextChange(String newText) {
        currentSearchTip = newText;
        if (newText != null && newText.length() > 0) {
            searchDelayed(newText);
        }
        return true;
    }
}
private Handler searchHandler = new Handler(){
    @Override
    public void handleMessage(Message msg) {
        if (msg == null)
            return;
        String searchText = (String) msg.obj;
        if (currentSearchTip != null && currentSearchTip.isEmpty()==false) {
            GymFragment fragment = GymFragment.newInstance(searchText);
            getSupportFragmentManager().beginTransaction().replace(R.id.fragment_layout, fragment).commit();
        }
    }
};
private ScheduledExecutorService scheduledExecutor = Executors.newScheduledThreadPool(10);
private String currentSearchTip;
public void searchDelayed(String newText) {
    scheduledExecutor.schedule(new SearchThread(newText),500, TimeUnit.MILLISECONDS);
}
class SearchThread implements Runnable {
    String newText;
    public SearchThread(String newText){
        this.newText = newText;
    }
    public void run() {
        if (newText != null && newText.equals(currentSearchTip)) {
            searchHandler.sendMessage(searchHandler.obtainMessage(0, newText));
        }
    }
}

ソリューション：</p>

1) "getActivity()..." をコメントアウトします。代わりに、コメント化された "sendMessage(searchhandler.obtain..)" を使用します。

2) getActivity().runOnUiThread() を実行する前に、まず getActivity()==null を判断します。

2016.3.14:

[いくつか興味深いもの:)]

  1. 「runOnUiThread」のソースコード

runOnUIThread(Runnable r)のソースコード：</p> 

public final void runOnUiThread(Runnable action) {  
        if (Thread.currentThread() != mUiThread) {  
            mHandler.post(action);  
        } else {  
            action.run();  
        }  
}

実際、runOnUiThread は mHandler.post(Runnable r) を呼び出します。

post(Runnable r) の内容を見てみましょう:

public final boolean post(Runnable r)  {    
       return  sendMessageDelayed(getPostMessage(r), 0);    
}

getPostMessage(Runnable r) では：</p> 

private final Message getPostMessage(Runnable r) {    
        Message m = Message.obtain();    
        m.callback = r;    
        return m;    
}

次に Message.obtain() で：</p> 

public final Message obtainMessage()  {    
      return Message.obtain(this);    
}

public static Message obtain(Message orig) {    
       Message m = obtain();    
       m.what = orig.what;    
       m.arg1 = orig.arg1;    
       m.arg2 = orig.arg2;    
       m.obj = orig.obj;    
       m.replyTo = orig.replyTo;    
       if (orig.data != null) {    
           m.data = new Bundle(orig.data);    
       }    
       m.target = orig.target;    
       m.callback = orig.callback;    

       return m;    
}  

/**   
     * Return a new Message instance from the global pool. Allows us to   
     * avoid allocating new objects in many cases.   
     */    
public static Message obtain() {    
      synchronized (sPoolSync) {    
            if (sPool != null) {    
                Message m = sPool;    
                sPool = m.next;    
                m.next = null;    
                sPoolSize--;    
                return m;    
            }    
      }    
      return new Message();  
 }

2.「sendMessage(Message msg)」のソースコード：</p> 

public final boolean sendMessage(Message msg)  {    
       return sendMessageDelayed(msg, 0);    
}

ここでわかるように、sendMessage(Message msg) の実装は、post(Runnable r) の実装とほぼ同じです。この違いは、getPostMessage(Runnable r) に関連している可能性があります。

つまり、簡単に言えば、実際に searchHandler.SendMessage(searchHandler.obtainMessage(0,str)) が正しく実行される結果となる一方で、同時に getActivity().runOnUiThread(new Runnable{...}) の例外が発生します。それらを繰り返し実行しましたか？どうもありがとう！

4

1 に答える 1