Android 短信发送代码损坏

Question

好的，所以我正在制作一个 Android 应用程序，它能够多次发送短信，并且发送的消息量之间有一定的间隔。这部分起作用，但有时不起作用。例如，当我输入 20 条消息来发送时，它会发送 16 条或更多。另外，当我按下按钮时，它会冻结，直到完成发送短信。无论如何，有人可以告诉我我的代码有什么问题吗？我尝试了一些效果不佳的修复程序，并认为在多次失败的尝试后我会来这里寻求帮助。 PS 还有更多的代码，但这就是我相信问题所在的地方。感谢您的阅读 编辑：已修复！

Answer 1

该按钮处于“冻结”状态，因为 UI 线程正在发送所有 20 条短信，然后才能腾出时间来做其他事情。

我会研究 AsyncTask。这是 Android 轻松线程化的方式。本质上，您要做的就是使用 AsyncTask 在后台生成一个新线程。这会释放 UI 线程，使用户看起来一切正常（不会冻结），而 AsyncTask 则通过 doInBackground 在后台完成所有工作。然后，当任务完成时，使用 onPostExecute 通知 UI 任务已完成，并可能告诉用户一些信息。您可以在 onClick 中调用 MyAsyncTask.execute(smsDataHere)。

出于简单目的，AsyncTask 的链接相当全面，并且包含示例代码。

我不确定到底为什么它只发送 16 条左右的消息，但我可以立即看到您缺少一些花括号，并且它们与您想要的方式不太匹配他们到。我会先看看那里。另外，第二个 for 循环是否有原因？ size 始终为 1，因此每条消息仅发送两次。

编辑：
这是一些示例代码。我不会保证它在您的应用程序中有效，但它会向您展示 AsyncTask 的工作原理并为您提供一个起点。我还尝试解决花括号的任何问题并做了一些清理工作。我仍然建议您查看文档以确保您了解它是如何工作的。我希望这对你有帮助。

public void onClick(View v){                
    String phoneNo = putpn.getText().toString();
    String message = txtMessage.getText().toString();
    String txts = txtAmount.getText().toString();
    String inter = interval.getText().toString();

    if (msgCount > 21){
        Toast.makeText(getBaseContext(), "That is too many txts, the max is 20", Toast.LENGTH_SHORT).show();
    }
    else{
        MySmsTask.execute(phoneNo, message, txts, inter);
    }
}

private class MySmsTask extends AsyncTask<String, Void, Integer> {
    @Override
    protected Integer doInBackground(String... messageData) {
        int sentCount = 0;
        int size = 1;
        int delay = Integer.parseInt(messageData[3]);
        int msgCount = Integer.parseInt(messageData[2]);

        if (phoneNo.length()>0 && message.length()>0){
            for (sentCount = 0; sentCount < msgCount; sentCount++) {
                for (int i = 0; i < size; i++) {
                    sendSMS(messageData[0], messageData[1]);
                    Thread.sleep(delay * 1000);
                }
            }
        }
        return sentCount;
    }

    @Override
    protected void onPostExecute(Integer result) {
        Toast.makeText(getBaseContext(), "Sent " + sentCount + " messages", Toast.LENGTH_SHORT).show();
    }
}

private void sendSMS(String phoneNumber, String message){        
    PendingIntent pi = PendingIntent.getActivity(this, 0,
            new Intent(this, Object.class), 0);
    SmsManager sms = SmsManager.getDefault();
    sms.sendTextMessage(phoneNumber, null, message, null, null);
}