AsyncTask.execute()未执行致应用崩溃问题求助

从你的崩溃堆栈日志可以直接定位核心问题：

java.lang.IllegalStateException: Cannot execute task: the task is already running.

这说明你复用了同一个loginBackgroundWorker实例多次调用execute()方法——AsyncTask的设计规则是：每个AsyncTask实例只能被执行一次，第一次调用execute()后，该实例的状态会变为RUNNING或FINISHED，再次调用就会触发这个异常。

另外你提到的"Toast先显示但AsyncTask逻辑没执行"的错觉，是因为AsyncTask.doInBackground()是在子线程异步执行的，主线程不会等待它完成，所以li.execute()调用后会立刻执行Toast.makeText()，此时后台任务其实刚刚启动，只是因为重复点击导致实例状态异常，最终引发崩溃。

修复方案 #

1. 每次点击创建新的AsyncTask实例 #

不要在onCreate中提前创建li实例，而是在点击登录按钮时每次都新建一个loginBackgroundWorker对象，这样每次点击都是独立的任务实例，不会有重复执行的问题：

修改后的onCreate点击事件代码：

protected void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.activity_login); 

    EditTextEmail =(EditText) findViewById(R.id.emailField); 
    EditTextPassword =(EditText) findViewById(R.id.passwordField); 
    loginButton = (CardView) findViewById(R.id.loginButton); 

    loginButton.setOnClickListener(new View.OnClickListener() { 
        @Override 
        public void onClick(View v) { 
            Email = EditTextEmail.getText().toString().trim(); 
            Password = EditTextPassword.getText().toString().trim(); 
            // 每次点击都创建新的AsyncTask实例
            loginBackgroundWorker li = new loginBackgroundWorker(LoginActivity.this);
            li.execute(Email,Password); 
            // 建议把这个Toast移到onPostExecute中，确保后台任务完成后再提示
            // Toast.makeText(LoginActivity.this,"Finish !",Toast.LENGTH_LONG).show(); 
        } 
    }); 
}

2. 优化用户体验：防止重复点击 #

为了避免用户短时间内多次点击按钮（即使创建新实例，频繁请求也会给服务器造成压力），可以在点击时禁用按钮，后台任务完成后再启用：

点击事件中添加按钮禁用逻辑：

loginButton.setOnClickListener(new View.OnClickListener() { 
    @Override 
    public void onClick(View v) { 
        Email = EditTextEmail.getText().toString().trim(); 
        Password = EditTextPassword.getText().toString().trim(); 
        
        // 禁用按钮，防止重复点击
        loginButton.setEnabled(false);
        
        loginBackgroundWorker li = new loginBackgroundWorker(LoginActivity.this);
        // 传递按钮实例给AsyncTask，方便在任务结束后重新启用
        li.setLoginButton(loginButton);
        li.execute(Email,Password); 
    } 
}); 

在loginBackgroundWorker中添加按钮引用并在onPostExecute启用：

public class loginBackgroundWorker extends AsyncTask<String,String,String> { 
    Context context; 
    public String email; 
    public String password; 
    public static final String login_url="http://192.168.0.104/login.php"; 
    private CardView loginButton; // 添加按钮引用

    public loginBackgroundWorker(Context context) { 
        this.context = context; 
    } 

    public void setLoginButton(CardView button) {
        this.loginButton = button;
    }

    // ... 其他方法保持不变 ...

    @Override 
    public void onPostExecute(String result) { 
        // 任务完成后重新启用按钮
        if(loginButton != null){
            loginButton.setEnabled(true);
        }

        Toast.makeText(context,"Onpost",Toast.LENGTH_LONG).show(); 
        String r=result.trim(); 
        Boolean aBoolean = true; 
        if (result.equals("1")) { 
            Toast.makeText(context,"Bien Connecté",Toast.LENGTH_LONG).show(); 
            aBoolean = false; 
        } else if (result.equals("")) {
            r = "Login ou mot de passe incorrect"; 
        } 
        if (aBoolean) { 
            Toast.makeText(context,r,Toast.LENGTH_LONG).show(); 
        }
        // 在这里添加完成提示，确保后台任务结束后才显示
        Toast.makeText(context,"Finish !",Toast.LENGTH_LONG).show();
    } 
}

3. 修复onPostExecute中的空指针风险 #

注意你的onPostExecute中直接使用result.trim()，如果doInBackground返回null（比如网络异常、URL错误），会触发空指针异常，建议先判断result是否为空：

修改onPostExecute的开头逻辑：

@Override 
public void onPostExecute(String result) { 
    if(loginButton != null){
        loginButton.setEnabled(true);
    }

    // 先判断result是否为空，避免空指针
    if(result == null){
        Toast.makeText(context,"网络请求失败，请检查连接",Toast.LENGTH_LONG).show();
        return;
    }

    Toast.makeText(context,"Onpost",Toast.LENGTH_LONG).show(); 
    String r=result.trim(); 
    Boolean aBoolean = true; 
    if (r.equals("1")) { // 用trim后的结果判断更准确
        Toast.makeText(context,"Bien Connecté",Toast.LENGTH_LONG).show(); 
        aBoolean = false; 
    } else if (r.isEmpty()) {
        r = "Login ou mot de passe incorrect"; 
    } 
    if (aBoolean) { 
        Toast.makeText(context,r,Toast.LENGTH_LONG).show(); 
    }
    Toast.makeText(context,"Finish !",Toast.LENGTH_LONG).show();
}

关键注意事项 #

• AsyncTask实例只能执行一次，多次调用execute()必然引发异常，每次任务都要创建新实例。
• AsyncTask是异步执行的，主线程不会等待它完成，所以不要在execute()之后立刻执行依赖任务结果的代码，相关逻辑要放在onPostExecute()中。
• 处理网络请求时一定要考虑异常情况（比如网络断开、URL错误），避免空指针或其他运行时异常。

内容的提问来源于stack exchange，提问作者Rachid Zaariou