App crashes when I click on ocean or sea to get city name in maps

Question

Error

java.lang.IndexOutOfBoundsException: Invalid index 0, size is 0.

This is my code.

googleMap.setOnMapClickListener(new GoogleMap.OnMapClickListener() {
        @Override
        public void onMapClick(LatLng latLng) {

            Geocoder geocoder;

            geocoder = new Geocoder(MapsActivity.this, Locale.getDefault());

            try {
                addresses = geocoder.getFromLocation(latLng.latitude, latLng.longitude, 1); // Here 1 represent max location result to returned, by documents it recommended 1 to 5
                String address = addresses.get(0).getAddressLine(0); // If any additional address line present than only, check with max available address lines by getMaxAddressLineIndex()
                String city = addresses.get(0).getLocality();
                String state = addresses.get(0).getAdminArea();
                String country = addresses.get(0).getCountryName();
                String postalCode = addresses.get(0).getPostalCode();
                String knownName = addresses.get(0).getFeatureName();

                if (city == null) {
                    Toast.makeText(MapsActivity.this, "Please select proper location", Toast.LENGTH_SHORT).show();
                } else {
                    Toast.makeText(MapsActivity.this, "" + city, Toast.LENGTH_SHORT).show();
                }

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

       }
    });

Possible duplicate of [What is a Null Pointer Exception, and how do I fix it?](http://stackoverflow.com/questions/218384/what-is-a-null-pointer-exception-and-how-do-i-fix-it) — camelCaseCoder, Feb 15 '16 at 09:16
Post your code... your LogCat.... I don't know... something... — Aspicas, Feb 15 '16 at 09:16
You are trying to get a element from an array which has no elements. — Phantômaxx, Feb 15 '16 at 09:24
@CamelCaseCoder: This is an IndexOutOfBoundException, not a NullPointerException — Opiatefuchs, Feb 15 '16 at 09:26
@Opiatefuchs the question got updated. Although it's along the same lines where code is try to access value from objects that don't contain any value. It's a simple case of handling the failure situations. — camelCaseCoder, Feb 15 '16 at 09:30
holding no values is not the same as not initialized or rather a "null" object. — Opiatefuchs, Feb 15 '16 at 09:32

score 1 · Answer 1 · answered Feb 15 '16 at 09:29

GeoCoder might not return a result in all cases, since a sea or ocean might not have an address.

In this case the list of results called addresses will have 0 items, but you still try to access to first item.

You should do something like this:

addresses = geocoder.getFromLocation(latLng.latitude, latLng.longitude, 1);
if (addresses.size() == 0) { 
    Toast.makeText(MapsActivity.this, "Please select proper location", Toast.LENGTH_SHORT).show();
    return;
}
// ... rest of the code

App crashes when I click on ocean or sea to get city name in maps

1 Answers1