这个 Handler 类应该是静态的，否则可能会发生泄漏：IncomingHandler

如果 IncomingHandler 类不是静态的，它将引用您的 Service 对象。

同一线程的 Handler 对象都共享一个公共 Looper 对象，它们向该对象发布消息并从中读取。

由于消息中包含target Handler，只要消息队列中有带有target handler的消息，就不能对handler进行垃圾回收。如果处理程序不是静态的，则您的 Service 或 Activity 不能被垃圾收集，即使在被销毁之后也是如此。

这可能会导致内存泄漏，至少在一段时间内 - 只要消息留在队列中。除非您发布长时间延迟的消息，否则这不是什么大问题。

您可以将 IncomingHandler 设为静态并为您的服务添加 WeakReference：

static class IncomingHandler extends Handler {
    private final WeakReference<UDPListenerService> mService; 

    IncomingHandler(UDPListenerService service) {
        mService = new WeakReference<UDPListenerService>(service);
    }
    @Override
    public void handleMessage(Message msg)
    {
         UDPListenerService service = mService.get();
         if (service != null) {
              service.handleMessage(msg);
         }
    }
}

请参阅 Romain Guy 的此 post 以获取更多参考

正如其他人提到的那样，Lint 警告是因为潜在的内存泄漏。您可以通过在构造 Handler 时传递 Handler.Callback 来避免 Lint 警告（即您没有子类 Handler 并且没有 Handler 非静态内部类）：

Handler mIncomingHandler = new Handler(new Handler.Callback() {
    @Override
    public boolean handleMessage(Message msg) {
        // todo
        return true;
    }
});

据我了解，这不会避免潜在的内存泄漏。 Message 对象持有对 mIncomingHandler 对象的引用，该对象持有对 Handler.Callback 对象的引用，该对象持有对 Service 对象的引用。只要Looper消息队列中有消息，Service就不会被GC。但是，除非您在消息队列中有长时间延迟的消息，否则这不会是一个严重的问题。

这是一个使用弱引用和静态处理程序类来解决问题的通用示例（如 Lint 文档中所建议的那样）：

public class MyClass{

  //static inner class doesn't hold an implicit reference to the outer class
  private static class MyHandler extends Handler {
    //Using a weak reference means you won't prevent garbage collection
    private final WeakReference<MyClass> myClassWeakReference; 

    public MyHandler(MyClass myClassInstance) {
      myClassWeakReference = new WeakReference<MyClass>(myClassInstance);
    }

    @Override
    public void handleMessage(Message msg) {
      MyClass myClass = myClassWeakReference.get();
      if (myClass != null) {
        ...do work here...
      }
    }
  }

  /**
   * An example getter to provide it to some external class
   * or just use 'new MyHandler(this)' if you are using it internally.
   * If you only use it internally you might even want it as final member:
   * private final MyHandler mHandler = new MyHandler(this);
   */
  public Handler getHandler() {
    return new MyHandler(this);
  }
}

这种方式对我来说效果很好，通过将处理消息的位置保持在其自己的内部类中来保持代码清洁。

您希望使用的处理程序

Handler mIncomingHandler = new Handler(new IncomingHandlerCallback());

内部类

class IncomingHandlerCallback implements Handler.Callback{

        @Override
        public boolean handleMessage(Message message) {

            // Handle message code

            return true;
        }
}

在@Sogger's answer的帮助下，我创建了一个通用处理程序：

public class MainThreadHandler<T extends MessageHandler> extends Handler {

    private final WeakReference<T> mInstance;

    public MainThreadHandler(T clazz) {
        // Remove the following line to use the current thread.
        super(Looper.getMainLooper());
        mInstance = new WeakReference<>(clazz);
    }

    @Override
    public void handleMessage(Message msg) {
        T clazz = mInstance.get();
        if (clazz != null) {
            clazz.handleMessage(msg);
        }
    }
}

界面：

public interface MessageHandler {

    void handleMessage(Message msg);

}

我使用它如下。但我不是 100% 确定这是否是安全的。也许有人可以对此发表评论：

public class MyClass implements MessageHandler {

    private static final int DO_IT_MSG = 123;

    private MainThreadHandler<MyClass> mHandler = new MainThreadHandler<>(this);

    private void start() {
        // Do it in 5 seconds.
        mHandler.sendEmptyMessageDelayed(DO_IT_MSG, 5 * 1000);
    }

    @Override
    public void handleMessage(Message msg) {
        switch (msg.what) {
            case DO_IT_MSG:
                doIt();
                break;
        }
    }

    ...

}

我不确定，但您可以尝试在 onDestroy() 中将处理程序初始化为 null

我很困惑。我发现的示例完全避免了静态属性并使用 UI 线程：

    public class example extends Activity {
        final int HANDLE_FIX_SCREEN = 1000;
        public Handler DBthreadHandler = new Handler(Looper.getMainLooper()){
            @Override
            public void handleMessage(Message msg) {
                int imsg;
                imsg = msg.what;
                if (imsg == HANDLE_FIX_SCREEN) {
                    doSomething();
                }
            }
        };
    }

我喜欢这个解决方案的一点是尝试混合类和方法变量没有问题。