正确返回列表

Question

如何正确返回列表？

我正在使用 OSMdroid 编写一个应用程序，我想使用“Place”类的变量“经度”和“纬度”（在这篇文章的底部）最终在“onPostExecute”方法中使用它们，对吧我在其中设置了“PLACEHOLDERS”。

Android Studio 想让我修改“return loadXmlFromNetwork(urls[0]);”这行代码，它会执行下面的代码，但我不知道具体如何（下面的方法在同一个类中）。

private class DownloadXmlTask extends AsyncTask<String, Void, String> {
    @Override
    protected String doInBackground(String... urls) {
        try {
            //Here I want to recieve the list
            return loadXmlFromNetwork(urls[0]);       
        } catch (IOException e) {
            return getResources().getString(R.string.connection_error);
        } catch (XmlPullParserException e) {
            return getResources().getString(R.string.xml_error);
        }

    }

    @Override
    protected void onPostExecute(List<Place> result) {

        MapView map = (MapView) findViewById(R.id.map);
        IMapController mapController = map.getController();
        mapController.setZoom(17);
        //Here I want to use the latitude and longitude variables of the List
        GeoPoint myLocation = new GeoPoint(PLACEHOLDER(latitude), PLACEHOLDER(longitude)); 
        mapController.animateTo(myLocation);

    }
}

这是我第一次收到名单的地方：

private List<Place> loadXmlFromNetwork(String urlString) throws XmlPullParserException, IOException {
    InputStream stream = null;  // Instantiate the parser
    XMLParser XMLParser = new XMLParser();
    List<Place> places = null; 

    try {
        stream = downloadUrl(urlString);
        places = XMLParser.parse(stream);  // Makes sure that the InputStream is closed after the app is finished using it.
    } finally {
        if (stream != null) {
            stream.close();
        }
    }

    return places;
}

这是我的 Place 课程：

public class Place {
    private String longitude;
    private String latitude;
    private String place_id;

    public String getLongitude() {
        return longitude;
    }
    public void setLongitude(String longitude) {
        this.longitude = longitude;
    }

    public String getLatitude() {
        return latitude;
    }
    public void setLatitude(String latitude) {
        this.latitude = latitude;
    }

    public String getPlace_id() {
        return place_id;
    }
    public void setPlace_id(String place_id) {this.place_id = place_id;}

    @Override
    public String toString() {
        return "ID: " + place_id + "\n" + "Longitude: " + longitude + "\n" + "Latitude: " + latitude;
}

Answer 1

AsyncTask 的三种参数类型是AsyncTask<Params, Progress, Result>

（见AsyncTask documentation）

如果你想要一个List<Place>作为结果，你必须替换

private class DownloadXmlTask extends AsyncTask<String, Void, String> {

与

private class DownloadXmlTask extends AsyncTask<String, Void, List<Place>> {

您的doInBackground() 方法必须返回List<Place> 而不是String。然后，您调用 loadXmlFromNetwork(urls[0]) 的行将工作，因为 loadXmlFromNetwork() 确实返回 List<Place>，但您的 catches 子句中的另外两个返回行将不再编译，因为它们返回 String。例如，您必须更改它们并返回 null。

编辑： 正如@Code-Apprentice 所说，不应忽略异常。您可以使用布尔标志以便稍后检查它们。

private class DownloadXmlTask extends AsyncTask<String, Void, List<Place>> {
    private boolean mConnectionError;
    private boolean mXMLError;

    @Override
    protected List<Place> doInBackground(String... urls) {
        try {
            //Here I want to recieve the list
            return loadXmlFromNetwork(urls[0]);       
        } catch (IOException e) {
            mConnectionError = true;
            return null;
        } catch (XmlPullParserException e) {
            mXMLError = true;
            return null;
        }

    }

    @Override
    protected void onPostExecute(List<Place> result) {

        if (result != null) {
            MapView map = (MapView) findViewById(R.id.map);
            IMapController mapController = map.getController();
            mapController.setZoom(17);
            //Here I want to use the latitude and longitude variables of the List
            GeoPoint myLocation = new GeoPoint(PLACEHOLDER(latitude), PLACEHOLDER(longitude)); 
            mapController.animateTo(myLocation);

        } else {
            // An error happened, check mConnectionError and
            // mXMLError in order to display an error message.

        }
    }
}

Answer 2

最简单的解决方法是改变

private class DownloadXmlTask extends AsyncTask<String, Void, String>

到

private class DownloadXmlTask extends AsyncTask<String, Void, List<Place>>

AsyncTask 的最后一个参数是结果类型。您将需要一种不同的方式来传达错误情况。例如，您可以添加一些布尔字段以用作标志和适当的 gettter 来访问它们。此外，AsyncTask 不应对 UI 中显示的错误消息负责。这违反了单一职责原则。