Возврат ArrayList из метода с помощью залпа

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

    private ListView mTripList;
public ArrayList<TripItem> tripItems;
private JSONArray unclaimedTrips;
private TripSelectAdapter adapter;



public void getTrips() {

        JsonArrayRequest req = new JsonArrayRequest(url, new Response.Listener<JSONArray> () {  

        @Override
        public void onResponse(JSONArray response) {

            // Public Array
            unclaimedTrips = response;

            tripItems = new ArrayList<TripItem>();


            // Optional if you want to manipulate the data in some way
            if (unclaimedTrips != null) {

                for (int i = 0; i < unclaimedTrips.length(); i++) {

                    try {
                        JSONObject item = unclaimedTrips.getJSONObject(i);

                        int tripID = item.getInt("trip_id");
                        int claimID = item.getInt("claim_id");
                        String payrollCode = item.getString("payroll_code");
                        String tripDate = item.getString("trip_date");
                        String tripPurpose = item.getString("trip_purpose");

                        TripItem tripItem = new TripItem(tripID, claimID, payrollCode, tripDate, tripPurpose);

                        tripItems.add(tripItem);

                    } catch (JSONException e) {
                        e.printStackTrace();
                    }
                }

                refreshData();
            }

        }


    }, new Response.ErrorListener() {
        @Override
        public void onErrorResponse(VolleyError error) {
            unclaimedTrips = null;
            refreshData();

        }
    }) {

       @Override
       public Map<String, String> getHeaders() throws AuthFailureError {
           HashMap<String, String> headers = new HashMap<String, String>();
                headers.put("Accept", "application/json");
                headers.put("Content-Type", "application/json");
           return headers;
       };
    };


    // add the request object to the queue to be executed
    MyApplication.getInstance().addToRequestQueue(req);
}


public void refreshData() {

    if (tripItems.isEmpty()) {
        mEmptyList.setVisibility(android.view.View.VISIBLE);
    } else {
        mEmptyList.setVisibility(android.view.View.GONE);                   
    }

    adapter = new TripSelectAdapter(this, thisContext, tripItems);
    mTripList.setAdapter(adapter);
}

Ваша проблема в том, что вы пытаетесь вернуть что-то, что еще не заполнено, вызов асинхронный, поэтому может потребоваться некоторое время, чтобы заполнить дату в вашем ArrayList<Cities>, поэтому он пуст.

Вы должны удалить эту строку return cities; и не забудьте создать эту final ArrayList<Cities> cities=new ArrayList<>(); вне вашего метода и удалить возврат ArrayList<Cities> в void.

И затем вы можете создать обратный вызов, чтобы уведомить, что данные заполнены в вашем ArrayList<Cities>, если вы пытаетесь заполнить их в RecyclerView, я рекомендую вам установить адаптер в методе onResponse().

Чтобы сделать обратный вызов, вы можете взглянуть на этот ответ или на этот Gist