Android：不知道为什么我的方法没有返回 ArrayList<HashMap<String, String>>答案

【问题标题】：Android:Not sure why my method isnt returning ArrayList<HashMap<String, String>>Android：不知道为什么我的方法没有返回 ArrayList<HashMap<String, String>>
【发布时间】：2014-08-18 13:15:07
【问题描述】：

嘿，我的问题是，在我的 AsyncTask 方法 doInBackground 中，我定义了一个返回类型为 ArrayList<HashMap<String, String>>，然后在我的代码中，我返回了类型为 ArrayList<HashMap<String, String>> 的 menuItems。我见过类似的情况，它说并非所有出口都支持返回ArrayList<HashMap<String, String>>，但查看我的代码，我看不到除了ArrayList<HashMap<String, String>> 以外的任何地方，任何帮助都会非常感激。这是我的 doInBackground 代码

protected ArrayList<HashMap<String, String>> doInBackground(String...petrolPriceURL){

                 ArrayList<HashMap<String, String>> menuItems = new ArrayList<HashMap<String, String>>();

            {

                for(int i = 0; i < 100; i++){


                    publishProgress(1);
                    try {
                        Thread.sleep(100);
                    } catch (InterruptedException e) {
                        // TODO Auto-generated catch block
                        e.printStackTrace();
                    }
                }


                String urlString = petrolPriceURL.toString();
                String result = "";
                InputStream anInStream = null;
                int response = -1;
                URL url = null;

                try {
                    url = new URL(urlString);
                } catch (MalformedURLException e) {
                    // TODO Auto-generated catch block
                    return null;
                }
                URLConnection conn = null;
                try {
                    conn = url.openConnection();
                } catch (IOException e) {
                    // TODO Auto-generated catch block
                    return null;
                }

                // Check that the connection can be opened
                if (!(conn instanceof HttpURLConnection))
                    try {
                        throw new IOException("Not an HTTP connection");
                    } catch (IOException e) {
                        // TODO Auto-generated catch block
                        return null;
                    }
                try
                {
                    // Open connection
                    HttpURLConnection httpConn = (HttpURLConnection) conn;
                    httpConn.setAllowUserInteraction(false);
                    httpConn.setInstanceFollowRedirects(true);
                    httpConn.setRequestMethod("GET");
                    httpConn.connect();
                    response = httpConn.getResponseCode();
                    // Check that connection is OK
                    if (response == HttpURLConnection.HTTP_OK)
                    {
                        // Connection is OK so open a reader 
                        anInStream = httpConn.getInputStream();
                        InputStreamReader in= new InputStreamReader(anInStream);
                        BufferedReader bin= new BufferedReader(in);

                        // Read in the data from the RSS stream
                        String line = new String();
                        while (( (line = bin.readLine())) != null)
                        {
                            result = result + "\n" + line;
                            Log.v(TAG, "index=" + result);

                        }
                    }
                }
                catch (IOException ex)
                {
                        try {
                            throw new IOException("Error connecting");
                        } catch (IOException e) {
                            // TODO Auto-generated catch block
                            e.printStackTrace();
                        }

                        Handler parser = new Handler();
                        String xml = result.toString(); // getting XML
                        Document doc = parser.getDomElement(xml); // getting DOM element

                        NodeList nl = doc.getElementsByTagName(KEY_FUEL);
                        // looping through all item nodes <item>
                        for (int i = 0; i < nl.getLength(); i++) {
                            // creating new HashMap
                            HashMap<String, String> map = new HashMap<String, String>();
                            Element e = (Element) nl.item(i);
                            // adding each child node to HashMap key => value
                            map.put(KEY_HIGHEST, parser.getValue(e, KEY_HIGHEST));
                            map.put(KEY_AVERAGE, parser.getValue(e, KEY_AVERAGE));
                            map.put(KEY_LOWEST, "Rs." + parser.getValue(e, KEY_LOWEST));
                            map.put(KEY_LINK, parser.getValue(e, KEY_LINK));

                            // adding HashList to ArrayList
                            menuItems.add(map);
                        }
                        return menuItems;
                }
            }

       }

我得到的错误是

This method must return a result of type ArrayList<HashMap<String,String>>  PetrolPriceActivity.java    /PetrolpriceTestProject/src/org/me/myandroidstuff   line 78

重新评估的代码：

 @Override
protected ArrayList<HashMap<String, String>> doInBackground(String...petrolPriceURL) {

    for(int i = 0; i < 100; i++) {
        publishProgress(1);
        try {
            Thread.sleep(100);
        } catch (InterruptedException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
    }


    String urlString = petrolPriceURL.toString();
    String result = "";
    InputStream anInStream = null;
    int response = -1;
    URL url = null;

    try {
        url = new URL(urlString);
    } catch (MalformedURLException e) {
        // TODO Auto-generated catch block
        return null;
    }
    URLConnection conn = null;
    try {
        conn = url.openConnection();
    } catch (IOException e) {
        // TODO Auto-generated catch block
        return null;
    }

    // Check that the connection can be opened
    if (!(conn instanceof HttpURLConnection)) {
        try {
            throw new IOException("Not an HTTP connection");
        } catch (IOException e) {
            // TODO Auto-generated catch block
            return null;
        }
    }

    try {
        // Open connection
        HttpURLConnection httpConn = (HttpURLConnection) conn;
        httpConn.setAllowUserInteraction(false);
        httpConn.setInstanceFollowRedirects(true);
        httpConn.setRequestMethod("GET");
        httpConn.connect();
        response = httpConn.getResponseCode();
        // Check that connection is OK
        if (response == HttpURLConnection.HTTP_OK) {
            // Connection is OK so open a reader
            anInStream = httpConn.getInputStream();
            InputStreamReader in= new InputStreamReader(anInStream);
            BufferedReader bin= new BufferedReader(in);

            // Read in the data from the RSS stream
            String line = new String();
            while (( (line = bin.readLine())) != null) {
                result = result + "\n" + line;
                Log.v(TAG, "index=" + result);


                ArrayList<HashMap<String, String>> menuItems = new ArrayList<HashMap<String, String>>();

                Handler parser = new Handler();
                String xml = result.toString(); // getting XML
                Document doc = parser.getDomElement(xml); // getting DOM element

                NodeList nl = doc.getElementsByTagName(KEY_FUEL);
                // looping through all item nodes <item>
                for (int i = 0; i < nl.getLength(); i++) {
                    // creating new HashMap
                    HashMap<String, String> map = new HashMap<String, String>();
                    Element e = (Element) nl.item(i);
                    // adding each child node to HashMap key => value
                    map.put(KEY_HIGHEST, parser.getValue(e, KEY_HIGHEST));
                    map.put(KEY_AVERAGE, parser.getValue(e, KEY_AVERAGE));
                    map.put(KEY_LOWEST, "Rs." + parser.getValue(e, KEY_LOWEST));
                    map.put(KEY_LINK, parser.getValue(e, KEY_LINK));

                    // adding HashList to ArrayList
                    menuItems.add(map);
                    return menuItems;
                }
            }
        }
    } catch (IOException ex) {
        try {
            throw new IOException("Error connecting");
        } catch (IOException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
    }
    //return menuItems;
}

【问题讨论】：

只有在出现IOException 时才会返回List...
所有代码中的第一个应该是编译时异常，因为我可以在代码开始时有两个{。同样，在您的第一个 try 块内的线程睡眠时间结束后，您不会返回任何内容。代码中的最后一个 return 语句来自您的 catch 块内
Deb 是我的代码而不是我的 catch 块......
catch (IOException ex) { try { throw new IOException("Error connected"); } catch (IOException e) { // TODO 自动生成的 catch 块 e.printStackTrace(); }
这不是我的catch块结束了吗？等等，我想我明白你的意思了。我试试看

标签： java android arraylist android-asynctask

【解决方案1】：

您不会在 try {} 块中返回任何内容：

try {
                // Open connection
                HttpURLConnection httpConn = (HttpURLConnection) conn;
                httpConn.setAllowUserInteraction(false);
                httpConn.setInstanceFollowRedirects(true);
                httpConn.setRequestMethod("GET");
                httpConn.connect();
                response = httpConn.getResponseCode();
                // Check that connection is OK
                if (response == HttpURLConnection.HTTP_OK)
                {
                    // Connection is OK so open a reader
                    anInStream = httpConn.getInputStream();
                    InputStreamReader in= new InputStreamReader(anInStream);
                    BufferedReader bin= new BufferedReader(in);

                    // Read in the data from the RSS stream
                    String line = new String();
                    while (( (line = bin.readLine())) != null)
                    {
                        result = result + "\n" + line;
                        Log.v(TAG, "index=" + result);

                    }
                }
            }

这意味着如果没有引发IOException，则不会返回任何内容。

您的catch (IOException ex) 子句中似乎有太多逻辑，这可能表明您应该重构您的代码（尝试将整个menuItems 从这个catch 子句中取出）。

编辑以下重新评估的代码：

让我们看看你的最后一个 try-catch 块，这也是你方法的结尾：

try {
        // Open connection
        HttpURLConnection httpConn = (HttpURLConnection) conn;

        // remainder omitted

    } catch (IOException ex) {
        try {
            throw new IOException("Error connecting");
        } catch (IOException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
    }
    //return menuItems;

在 try 块中，您确实返回了 menuItems（ArrayList<HashMap<String, String>> 类型），但是您在在 for 循环中执行此操作，这将导致循环在返回 @987654329 之前只重复一次@object 给调用者，结束方法调用（因为它可能不是你想要的，你应该把 menuItems 从 for 循环中取出）。

然后想想如果在返回menuItems 对象之前抛出 IOException 会发生什么（不能在之后发生）？异常将在您的 catch 子句中捕获，您在该子句中抛出另一个异常只是为了在打印当前堆栈跟踪之后立即捕获它，然后...nothing。该函数将结束而不返回任何编译器所抱怨的内容。

所以你应该找到一种方法来让 menuItems 无论如何都返回（提示：由于 java 变量是块范围的，从 for 循环中获取 menuItems 声明可能还不够。但你在你最后一行代码的好轨道！）

【讨论】：

这段代码只是为了获取结果值。代码的下一部分是所有内容的组合。我正在尝试返回 menuItems。在编辑此功能之前，我确实返回了结果。这个例子后面的代码是我寻找回报的地方
是的，我看到您正在此 try 块中查找结果值，但您需要使其返回 ArrayList> 类型的内容，或者您需要继续正常程序流程（在此 try 块中没有抛出 IOException）并最终返回 ArrayList> 类型的内容（或将捕获的异常重新抛出给调用者）。
我在编辑我的问题时更改了我的代码。现在我将下面的代码添加到了尝试中。我将如何完成这项工作？一个帮助我理解的例子将不胜感激
不，我认为您无意中试图编辑我的答案。请编辑您的问题，我很乐意查看。
没问题。只是一个简单的问题，您使用的是 Eclipse 或 IntellijIdea 之类的 IDE 吗？我发现你的代码格式很奇怪，正如 Deb 所说，你的方法周围有一些无用的代码块。这个缩进很容易出错，我建议你把它清理一下。我将对您问题中的代码进行编辑，以向您展示我的意思。