我有一个实现监听器的 Activity 。我担心的是 可以重新创建 Activity ,然后回调将有一个引用 到一个空的对象。
这意味着我们必须用一个新的引用来更新 Controller 新创建的 Activity 。
What pattern is best to use even if the callbacks are async? Is there perhaps a safe way to update the controllers reference in a thread > safe way.
或
Should one rather use a Headless fragment and use the onAttach method get the updated reference.
或
Should one rather not use these patterns and use a Handler for all your callbacks?
我怀疑我的 updateListener 方法在所有情况下都不起作用,例如
1) init 正忙,正要调用回调,标记为 *10*
2) Activity 被重新创建并更新 Controller 一个新的引用,但 updateListener 方法被阻止,因为回调即将发生。
3) 回调执行并失败,因为监听器引用变量已过时。
public class Controller {
UserActionListener listener
static Controller instance;
public static synchronized Controller getInstance(UserActionListener listener) {
if (instance == null) {
instance = new Controller();
}
this.listener = listener;
return instance;
}
private Controller() {
//empty, enforce getInstance
}
private init() {
// do some very long running operation in a separate thread.
//.... on completion we update the UI
synchronized(Controller.class) {
/*10*/ listener.handle("SHOW DIALOG");
}
}
public void updateListener(UserActionListener listener) {
synchronized(Controller.class) {
this.listener = listener;
}
}
public class MainActivity extends AppCompatActivity implements UserActionListener {
static Controller controller;
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
controller = Controller.getInstance(this);
if (savedInstanceState == null) {
//do not run on re-create
controller.init();
}
}
@Override
protected void onPostResume() {
super.onPostResume();
controller.updateListener(this);
}
@Override
public void handleAction(String userAction) {
switch (userAction) {
case "SHOW DIALOG" :
Toast.makeText(getActivity(),"Hello",Toast.LENGTH_LONG).show();
}
最佳答案
您问题的直接答案是简单的订阅模式。 在您调用的 Activity 上:
@Override
public void onStart(){
controller.updateListener(this);
}
@Override
public void onStop(){
controller.updateListener(null);
}
在调用监听器上的任何内容之前,在 Controller 内部检查是否为 null。 但逻辑上存在根本性缺陷。
使用以下代码:
static Controller controller;
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
controller = new Controller(this);
}
引用 Activity 的静态 Controller 正在泄漏 Activity ,避免它被垃圾收集。
此外,即使 Controller 是静态的,每次创建 Activity 时都会创建一个新 Controller ,同样在 Controller init()
内部,您具有以下内容:
// do some very long running operation
//....
这意味着这个运行时间非常长的操作是:
- 在 UI 线程中运行。这将阻止您的应用程序初始化,用户会认为它已损坏,系统可能会向用户显示一条消息,要求关闭它。
- 没有什么可以保证您的进程在“非常长的操作”完成之前不会被用户或系统终止。如果你想运行一个长时间的操作,你必须使用
服务
。
关于android - 当 Android Activity 的回调在方向更改后变为 null 时,更新回调引用的正确方法是什么,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/32057712/