android中的索引超出范围异常

时间:2013-11-03 16:10:39

标签: java android multithreading thread-safety

我收到附加代码的以下错误。这里的问题是即使在退出“for”循环后线程仍在运行。我所做的只是点击按钮,我将一堆文件上传到云端。 ListofDocs[]是我存储我要上传的文件列表的地方。任何帮助表示赞赏。 TIA。

11-03 21:23:27.793: E/AndroidRuntime(24480): FATAL EXCEPTION: Thread-11897
11-03 21:23:27.793: E/AndroidRuntime(24480): java.lang.ArrayIndexOutOfBoundsException: length=12; index=12
11-03 21:23:27.793: E/AndroidRuntime(24480):    at com.projects.scloud.PicsList$1$1.run(PicsList.java:99)
11-03 21:23:27.793: E/AndroidRuntime(24480):    at java.lang.Thread.run(Thread.java:856)
for(i=0; i < ListofDocs.length; i++) {
          Log.i("Value of i: ", String.valueOf(i));
        MessageText.setText("Upload Documents");

        /************* Php script path ****************/
        upLoadServerUri = "http://www.androidexample.com/media/UploadToServer.php";
        Backup.setOnClickListener(new OnClickListener() {            
            @Override
            public void onClick(View v) {
                dialog = ProgressDialog.show(DocsList.this, "", "Uploading file...", true);
                new Thread(new Runnable() {
                    public void run() {
                        runOnUiThread(new Runnable() {
                            public void run() {
                                MessageText.setText("uploading started.....");
                            }
                        });                   

                        uploadFile(ListofDocs[i]);
                    }
                }).start();
            }
        });
      }

2 个答案:

答案 0 :(得分:1)

为了编译代码,i必须是实例变量,而不是局部变量。这意味着ListofDocs[i]将在线程主体实际执行的时间点引用该实例变量的值。但是,在外部循环已经i递增到ListofDocs.length之后,可能是。因此数组索引超出范围。

您不应该使用实例变量。相反,你应该使用final局部变量; e.g。

for(int i=0; i < ListofDocs.length; i++) {
    final int final_i = i;
    ...
        // in the anonymous inner class
        uploadFile(ListofDocs[final_i]);
}

答案 1 :(得分:0)

我已经将线程外部的循环更改为线程内部。由于Thread只执行一次,我的代码只执行一次并且正在递增数组元素。现在代码工作正常。