CameraUpdateFactory.newLatLngBounds is not workinf all the time

Question

I'm having some problema to move camera on android google maps. I have that function: ``` protected void centralizeMapToBounds(final LatLng southwest, final LatLng northeast, final LatLng taxi) { final int padding = 30; Location loc1 = new Location(""); loc1.setLatitude(southwest.latitude); loc1.setLongitude(southwest.longitude); Location loc2 = new Location(""); loc2.setLatitude(northeast.latitude); loc2.setLongitude(northeast.longitude); Location loc3 = new Location(""); loc3.setLatitude(taxi.latitude); loc3.setLongitude(taxi.longitude); float d1 = loc1.distanceTo(loc2); float d2 = loc1.distanceTo(loc3); float d3 = loc2.distanceTo(loc3); LatLngBounds bounds = null; try { if(d1 > d2){ if(d1 > d3){ ICabDebug.i(TAG, "Maior distancia d1: " + d1); if(southwest.latitude > northeast.latitude) bounds = new LatLngBounds(southwest,northeast); else bounds = new LatLngBounds(northeast, southwest); }else{ ICabDebug.i(TAG, "Maior distancia d3: " + d3); if(northeast.latitude > taxi.latitude) bound…

Accepted Answer

Not sure what you want to achieve with the code in your question. It's a bit messy.

If you want to show area containing 3 LatLngs, you are better of using

```
LatLngBounds bounds = LatLngBounds.builder().include(latLng1).include(northeast).include(taxi).build();
map.moveCamera(CameraUpdateFactory.newLatLngBounds(bounds, padding));
```

This would replace all your ifs and try-catches with a few lines.

Btw. You shouldn't be doing try-catches in your code. A programming error is not something you catch and try something different, but change the code instead.

CameraUpdateFactory.newLatLngBounds is not workinf all the time

Solution

Problem