图片（base64）未正确上传

Question

不幸的是，当我尝试将一些图像从 android 设备上传到数据库时，我遇到了一些问题。 图像在一个文件夹中。此文件夹包含图像以及其他内容。我不知道图片的名称，我只需要上传图片（jpg）。在我上传图片之前，我需要使用 base64 对它们进行编码。

首先我从文件夹中获取 jpg 文件。然后我从图像名称中获取 ID。之后我通过base64对其进行编码：

Button upload = (Button) findViewById(R.id.upload);
upload.setOnClickListener(new View.OnClickListener() {
    public void onClick(View arg0) {
        String path = Environment.getExternalStorageDirectory().getPath();
        File dir = new File(path);
        File[] files = dir.listFiles();

        for (int i = 0; i < files.length; ++i) {
            if (files[i].getName().endsWith(".jpg")) {
                pics = new File(String.valueOf(files[i]));
                id = String.valueOf(files[i]);
                String sub = id.substring(id.lastIndexOf("/") + 1);
                int index = sub.indexOf("_");
                String book;
                if (index >= 0) {
                    book = sub.substring(0, index);
                    ID = book;
                    Log.e("ID", ID);
                }
                Bitmap imagex = BitmapFactory.decodeFile(pics.getAbsolutePath());
                ByteArrayOutputStream baos = new ByteArrayOutputStream();
                imagex.compress(Bitmap.CompressFormat.JPEG, 70, baos);
                byte[] b = baos.toByteArray();
                Image = Base64.encodeToString(b, Base64.DEFAULT);

                try {
                    new HttpAsyncTask(ID,Image,Nummer).execute("https://....");
                } catch (Exception e) {
                    Log.e("InputStream", e.getMessage());
                }

                Log.e("PICS", id);
            }
        }
    }
});

---

public String POST(String url) {
    InputStream inputStream;
    try {

        HttpClient httpclient = classxy.getNewHttpClient();

        HttpPost httpPost = new HttpPost(url);
        String json = "";

        JSONObject jsonObject = new JSONObject();

        jsonObject.put("bookId", ID);
        jsonObject.put("imageString", Image);
        jsonObject.put("imageNumber", Nummer);

        json = jsonObject.toString();

        StringEntity se = new StringEntity(json);

        httpPost.setEntity(se);

        httpPost.setHeader("Apikey", data);
        httpPost.setHeader("Modul", "upload_image");

        HttpResponse httpResponse = httpclient.execute(httpPost);
        inputStream = httpResponse.getEntity().getContent();


        if (inputStream != null)
            result = classxy.convertInputStreamToString(inputStream);
        else
            result = "Fehler!";
    } catch (Exception e) {
        Log.e("InputStream", e.getLocalizedMessage());
    }
    int num = Integer.parseInt(Nummer);
    num++;
    Nummer = Integer.toString(num);
    return result;
}

---

public class HttpAsyncTask extends AsyncTask<String, Void, String> {
    private final Object ID, Image, Nummer;

    public HttpAsyncTask(Object ID, Object Image, Object Nummer) {
        this.ID = ID;
        this.Image = Image;
        this.Nummer = Nummer;
    }

    protected String doInBackground(String... urls) {
        return POST(urls[0]);
    }

    protected void onPostExecute(String result) {
        if (result.matches("(.*)false(.*)")) {
            Toast.makeText(getApplicationContext(), "....", Toast.LENGTH_SHORT).show();
        } else {
            Toast.makeText(getApplicationContext(), "...", Toast.LENGTH_SHORT).show();
        }
        Log.e("RESPONSE", result);
    }
}

它确实通过 base64 对图像进行编码，并且确实上传了一些图像。不幸的是，它只上传第一张图片或多次上传一张图片。它永远不会以正确的顺序上传正确数量的图像。我已经在这个问题上坐了一段时间，无法弄清楚我做错了什么。

你能告诉我我做错了什么吗？

Answer 1

您的程序似乎根本不是线程安全的。

您的字段 ID、Image 和 Nummer 会随着 for 循环的每次迭代而更新。很可能循环在POST 第一次运行之前已经完成。您的观察将支持这一假设：

不幸的是，它只上传第一张图片或多次上传一张图片。

您可以通过记录对这些字段的每次访问来观察这一点。您会发现，它并没有像您期望的那样交替出现。

因此，您应该完全不使用这些字段来实现所有内容。而是使用局部变量并传递它们。如果您想将其用于多个上传，使用 Nummer 字段可能会很有用。但最好直接使用int：

upload.setOnClickListener(new View.OnClickListener() {
    public void onClick(View arg0) {
        String ID = "", Image;
        int Nummer = 0;
        [...]

        for (int i = 0; i < files.length; ++i) {
            if (files[i].getName().endsWith(".jpg")) {
                [...]

                try {
                    new HttpAsyncTask(ID,Image,Integer.toString(Nummer++)).execute("https://....");
                } catch (Exception e) {
                    Log.e("InputStream", e.getMessage());
                }

                Log.e("PICS", id);
            }
        }
    }
});

---

public String POST(String url, String ID, String Image, String Nummer) {
    InputStream inputStream;
    try {
        [...]
    } catch (Exception e) {
        Log.e("InputStream", e.getLocalizedMessage());
    }
    //int num = Integer.parseInt(Nummer);
    //num++;
    //Nummer = Integer.toString(num);
    return result;
}

---

public class HttpAsyncTask extends AsyncTask<String, Void, String> {
    private final String ID, Image, Nummer;

    public HttpAsyncTask(String ID, String Image, String Nummer) {
        this.ID = ID;
        this.Image = Image;
        this.Nummer = Nummer;
    }

    protected String doInBackground(String... urls) {
        return POST(urls[0], ID, Image, Nummer);
    }

    protected void onPostExecute(String result) {
        [...]
    }
}

Answer 2

在我的建议中不要直接从 for 循环调用 Asynctask，因为我们可以设置选择哪个图像的任何监视器。

所以请执行以下步骤：

1) 在 for 循环中获取所有图像 ID、名称和编号并将其存储到 ArrayList 中

2) 先检查ArrayList是否为空 如果没有，则获取第一个位置 ID、图像和编号

调用 new HttpAsyncTask(ID,Image,Integer.toString(Nummer++)).execute("https://....");

3) 在 HttpAsyncTask onPostExecute(String result) 方法中 首先删除第一个位置数据 然后创建

for loop (i=0;i<ArrayList.Size();i++) {
    ID=ArrayList first position data ID
    Image=ArrayList first position data IMAGE
    number=ArrayList first position data number

Call new HttpAsyncTask(ID,Image,Integer.toString(Nummer++)).execute("https://....");

}

所以这里先发送图片，然后再发送第二张图片，然后发送第三张图片，直到您的列表不为空，并且每次选择不同的图片。

就是这样……