获取Android循环内多个位置的天气状态

时间:2019-02-10 06:21:38

标签: java android location okhttp weather-api

我正在为此开发一个天气应用程序,因为我正在使用Dark sky API,在其中我想知道我存储在ArrayList<LatLng>中的位置的天气状况。

我正在使用OKHttp来解析来自API的JSON数据,因此我尝试在for循环中循环整个获取过程,但未提供所需的输出。

private void beginTask(ArrayList<LatLng> arrayLis) { 
    //arraylis contains list of locations(LatLng)
    m = 0;
    startTask = true;

    for (int i = 0;i<arrayLis.size();i++) {
        double latitude = ((LatLng)arrayLis.get(i)).latitude;
        double longitude = ((LatLng)arrayLis.get(i)).longitude;
        String url = "https://api.darksky.net/forecast/APIKEY/"
                +latitude+","+longitude+"?units=si";
        LatLng mylatlng = new LatLng(latitude,longitude);
        startProcess(url,mylatlng);

        Log.i("GGGTT",""+latitude+", "+longitude);
    }
}

private void startProcess(String myurl, final LatLng myLatlng){
    OkHttpClient httpClient = new OkHttpClient();
    Request request = new Request.Builder()
            .url(myurl)
            .build();

    Call call  = httpClient.newCall(request);
    call.enqueue(new Callback() {
        @Override
        public void onFailure(Call call, IOException e) {
        }

        @Override
        public void onResponse(Call call, Response response) throws IOException {
            String data = response.body().string();
            Log.i("DATASS",data);
            if (response.isSuccessful()){
                try {
                    getCurrentDetails(data,myLatlng);
                } catch (JSONException e){
                    e.printStackTrace();
                }
            }
        }
    });
}

private void getCurrentDetails(String data,LatLng myLatlng) throws JSONException{
    JSONObject main = new JSONObject(data);
    double la = main.getDouble("latitude");
    double lon = main.getDouble("longitude");
    JSONObject currently = main.getJSONObject("currently");
    String summary = currently.getString("summary");
    double temperature = currently.getDouble("temperature");
    String icon = currently.getString("icon");

    LatLng latLngo = new LatLng(la,lon);
    ter.add(latLngo);
    weatherInfo.add(icon);

    // Output here is not in the same order that I have passed 

    Log.i("LETSCHECK",""+la+", "+lon +icon);
}

我将值传递为:

  1. 19.21111,73.07729
  2. 19.20238,73.06582
  3. 19.19383,73.05362
  4. 19.18848,73.04221

但是getCurrentDetails方法中的输出顺序不同

  1. 19.19383,73.05362
  2. 19.20238,73.06582
  3. 19.18848,73.04221
  4. 19.21111,73.07729

我认为该方法不会在上一个循环完成之前等待。

是否有任何解决方案可以在不更改其顺序的情况下获取存储在ArrayList中的所有位置的天气状况?

编辑

您好,我已经通过此方法按顺序获取数据并且可以正常工作,但是,还有一个问题是我希望显示4次数据,因为ArrayList中有四个LatLng,并且可以正常工作,但是当我尝试读取存储在另一个数组中的提取数据,它仅显示2项而不是4。

private void getCurrentDetails(String data,LatLng myLatlng) throws JSONException{

    JSONObject main = new JSONObject(data);
    double la = main.getDouble("latitude");
    double lon = main.getDouble("longitude");
    JSONObject currently = main.getJSONObject("currently");
    String summary = currently.getString("summary");
    double temperature = currently.getDouble("temperature");
    String icon = currently.getString("icon");

    //Log.i("LETSCHECK",""+la+", "+lon +icon+",k");

    loopi++;
    Log.i("LETSCHECK",""+loopi);
    if (loopi < arrayList.size()) {
        getItAgain();
    } else if (loopi == arrayList.size()){
        for (String l:weatherInfo){
            Log.i("LETSCHECK",l); 
            //expected 4 items to show but its showing only 2 items
        }
    }

    Log.i("LETSCHECK",""+la+", "+lon +icon);
    weatherInfo.add(icon); 
}

private void getItAgain() {
    double latitude = ((LatLng)arrayList.get(loopi)).latitude;
    double longitude = ((LatLng)arrayList.get(loopi)).longitude;
    String url = "https://api.darksky.net/forecast/74d8feeda5ecf5ee6667d034778b239d/"
            +latitude+","+longitude+"?units=si";
    LatLng mylatlng = new LatLng(latitude,longitude);
    startProcess(url,mylatlng);
}             

3 个答案:

答案 0 :(得分:0)

您正在运行异步代码,由于每个HTTP调用都需要一定的时间(与其他时间无关),所以它们的回调没有一个接一个地返回是合理的,因此实际上可能收到了您的第三个调用的响应在第一个之前。

您可以使用RxJava(及其运算符),也可以改进当前代码。对于您的startProcess,将for循环中的i作为索引传递,并在函数之外创建一个数组。只要从服务器获得响应,就可以将其存储在数组的第i位。每次调用后,您可以检查是否已通过计数器或其他方法获取了所有值。最好将所有这些内容移到其自己的类中,以使其包含且可测试。

要注意的一件事是您将如何处理错误,因为您的呼叫可能无法成功接收。

答案 1 :(得分:0)

网络调用是异步的,这是您在此处遇到的预期行为。如果要按照传递的顺序对它们进行排序,则可能需要一种机制来等待上一个的网络调用完成,然后再次调用下一个。但是,这可能会增加等待服务器响应的延迟,因为您不会同时从服务器获取多个响应。

如果要等待响应以保持排序方式,则可能只需要执行以下操作。

// Declare a global variable of the list of your locations. 
ArrayList<LatLng> arrayLis; 
int i = 0; 

// Just start the first task without the loop. 
double latitude = ((LatLng)arrayLis.get(i)).latitude;
double longitude = ((LatLng)arrayLis.get(i)).longitude;
String url = "https://api.darksky.net/forecast/APIKEY/"
        +latitude+","+longitude+"?units=si";
LatLng mylatlng = new LatLng(latitude,longitude);
startProcess(url,mylatlng);

现在在getCurrentDetails函数中,您可能只需要执行以下操作。

private void getCurrentDetails(String data,LatLng myLatlng) throws JSONException{
    JSONObject main = new JSONObject(data);
    double la = main.getDouble("latitude");
    double lon = main.getDouble("longitude");
    JSONObject currently = main.getJSONObject("currently");
    String summary = currently.getString("summary");
    double temperature = currently.getDouble("temperature");
    String icon = currently.getString("icon");

    LatLng latLngo = new LatLng(la,lon);
    ter.add(latLngo);
    weatherInfo.add(icon);

    // Now initiate the next call
    i++;
    if(i < arrayLis.size()) getItAgain();

    Log.i("LETSCHECK",""+la+", "+lon +icon);
}

public void getItAgain() {
    // Just start the first task without the loop. 
    double latitude = ((LatLng)arrayLis.get(i)).latitude;
    double longitude = ((LatLng)arrayLis.get(i)).longitude;
    String url = "https://api.darksky.net/forecast/APIKEY/"
            +latitude+","+longitude+"?units=si";
    LatLng mylatlng = new LatLng(latitude,longitude);
    startProcess(url,mylatlng);
}

更新

我不明白为什么它显示2个结果而不是4个结果。但是,我认为您需要按以下方式修改代码。

private void getCurrentDetails(String data,LatLng myLatlng) throws JSONException {

    JSONObject main = new JSONObject(data);
    double la = main.getDouble("latitude");
    double lon = main.getDouble("longitude");
    JSONObject currently = main.getJSONObject("currently");
    String summary = currently.getString("summary");
    double temperature = currently.getDouble("temperature");
    String icon = currently.getString("icon");

    // Move this upto here
    weatherInfo.add(icon);

    loopi++;
    Log.i("LETSCHECK",""+loopi);
    if (loopi < arrayList.size()) {
        getItAgain();
    } else {
        for (String l:weatherInfo) {
            Log.i("LETSCHECK",l); 
            //expected 4 items to show but its showing only 2 items
        }
    }
}

答案 2 :(得分:0)

这是示例代码。每次成功调用之后,从数组中删除当前的proced项,然后执行另一个API请求。

private void startProcess(ArrayList<LatLong> elementList){
LatLong myLatlng = elementList.get(0);
OkHttpClient httpClient = new OkHttpClient();
Request request = new Request.Builder()
        .url(myurl)
        .build();
Call call  = httpClient.newCall(request);
call.enqueue(new Callback() {
    @Override
    public void onFailure(Call call, IOException e) {

    }

    @Override
    public void onResponse(Call call, Response response) throws IOException {

        String data = response.body().string();
        Log.i("DATASS",data);
        if (response.isSuccessful()){
            try {
                getCurrentDetails(data,myLatlng);
                elementList.remove(0)
                startProcess(elementList)
            }catch (JSONException e){
                e.printStackTrace();
            }


        }
    }

});

}