Получить статус погоды в нескольких местах внутри цикла for android

Сетевые вызовы являются асинхронными, и это ожидаемое поведение, которое у вас здесь. Если вы хотите, чтобы они сортировались по мере их прохождения, вам может потребоваться механизм ожидания завершения сетевого вызова предыдущего вызова, а затем повторный вызов следующего. Однако это может увеличить задержку ожидания ответа от сервера, поскольку вы не получаете несколько ответов от сервера одновременно.

Если вы хотите дождаться ответа, чтобы сохранить отсортированный способ, вам может потребоваться сделать следующее.

// 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
        }
    }
}

Вы используете асинхронный код, и причина, по которой их обратные вызовы не возвращаются один за другим, логична, поскольку каждый HTTP-вызов занимает некоторое время, не связанное с другими, поэтому ответ на ваш третий вызов может фактически быть получен до первого. один.

Вы можете либо использовать RxJava (и его операторы), либо улучшить текущий код. Для вашего startProcess передайте i из цикла for в качестве индекса и создайте массив вне нашей функции. Всякий раз, когда вы получаете ответ от сервера, вы можете сохранить его в i позиции вашего массива. После каждого вызова вы можете проверить, все ли значения были получены (с помощью счетчика или чего-то еще). Гораздо лучше переместить все это в отдельный класс, чтобы он содержался и тестировался.

Следует отметить, как вы собираетесь обрабатывать ошибки, потому что ваши звонки могут быть не приняты успешно.

Это пример кода. После каждого успешного вызова удалите текущий элемент обработки из массива и выполните другой запрос 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();
            }


        }
    }

});

}