Search code examples
androidjsonandroid-volleyandroid-gps

BasicNetwork.performRequest: Unexpected response code 400 in android


I am trying to build an application where I use JSON to get the current latitude and longitude and also the address from it, but as soon as I click on the button I get an error in my log:

E/Volley: [5807] BasicNetwork.performRequest: Unexpected response code 400 for https://maps.googleapis.com/maps/api/geocode/json?latlng=0.00.0

The code is given below :

Gps3Activity.java

    public class Gps3Activity extends AppCompatActivity {
    private Button display;
    private LocationManager locationManager;
    private LocationListener locationListener;
    private RequestQueue requestQueue;
    private double lat;
    private double lng;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_gps3);

        display = (Button) findViewById(R.id.displayL);
        requestQueue = Volley.newRequestQueue(this);

        locationManager = (LocationManager) getSystemService(Context.LOCATION_SERVICE);
        locationListener = new myLocationlistener();
        if (ActivityCompat.checkSelfPermission(this, Manifest.permission.ACCESS_FINE_LOCATION) != PackageManager.PERMISSION_GRANTED && ActivityCompat.checkSelfPermission(this, Manifest.permission.ACCESS_COARSE_LOCATION) != PackageManager.PERMISSION_GRANTED) {

            return;
        }
        locationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER, 1000, 0, locationListener);

        display.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {

                Log.e("Latitude", String.valueOf(lat));
                Log.e("Longitude", String.valueOf(lng));

                JsonObjectRequest request = new JsonObjectRequest("http://maps.googleapis.com/maps/api/geocode/json?latlng="+ lat+","+lng +"&sensor=true", new Response.Listener<JSONObject>() {

                    @Override
                    public void onResponse(JSONObject response) {
                        try {
                            String address = response.getJSONArray("results").getJSONObject(0).getString("formatted_address");
                            //textViewCity.setText(address);
                            Toast.makeText(getApplicationContext(), "City : " + address, Toast.LENGTH_LONG);
                        } catch (JSONException e) {
                            e.printStackTrace();
                        }
                    }
                }, new Response.ErrorListener() {
                    @Override
                    public void onErrorResponse(VolleyError error) {

                    }
                });
                requestQueue.add(request);
            }
        });
    }

    private class myLocationlistener implements LocationListener {
        @Override
        public void onLocationChanged(Location location) {
            if (location != null) {
                lat = location.getLatitude();
                lng = location.getLongitude();
            }
        }

        @Override
        public void onStatusChanged(String s, int i, Bundle bundle) {

        }

        @Override
        public void onProviderEnabled(String s) {

        }

        @Override
        public void onProviderDisabled(String s) {

        }
    }
}

logcat

07-30 01:18:33.287 9750-9750/com.example.shaloin.gps2 W/System.err: org.json.JSONException: Index 0 out of range [0..0)
07-30 01:18:33.288 9750-9750/com.example.shaloin.gps2 W/System.err:     at org.json.JSONArray.get(JSONArray.java:293)
07-30 01:18:33.288 9750-9750/com.example.shaloin.gps2 W/System.err:     at org.json.JSONArray.getJSONObject(JSONArray.java:521)
07-30 01:18:33.288 9750-9750/com.example.shaloin.gps2 W/System.err:     at com.example.shaloin.gps2.Gps3Activity$1$1.onResponse(Gps3Activity.java:63)
07-30 01:18:33.288 9750-9750/com.example.shaloin.gps2 W/System.err:     at com.example.shaloin.gps2.Gps3Activity$1$1.onResponse(Gps3Activity.java:58)

Also I am not able to figure out as to why am I getting the values of latitude and longitude as 0.0.

Can anyone help ? Thank you :)


Solution

  • I just had to change the format of JsonObjectRequest(). The correct format is given below as suggested by @ZeekHuge

    JsonObjectRequest(int method,
                  String url,
                  JSONObject jsonRequest,
                  Response.Listener<JSONObject> listener,
                  Response.ErrorListener errorListener)
    

    The correct code is given below :

    public class Gps3Activity extends AppCompatActivity {
    
    private static final int MY_PERMISSIONS_REQUEST_SEND_SMS =0 ;
    private Button display;
    private TextView displayLocation;
    private LocationManager locationManager;
    private LocationListener locationListener;
    private RequestQueue requestQueue;
    private double lat;
    private double lng;
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_gps3);
    
        display = (Button) findViewById(R.id.displayL);
        displayLocation=(TextView)findViewById(R.id.location1);
        requestQueue = Volley.newRequestQueue(this);
    
        locationManager = (LocationManager) getSystemService(Context.LOCATION_SERVICE);
        locationListener = new myLocationlistener();
        if (ActivityCompat.checkSelfPermission(this, Manifest.permission.ACCESS_FINE_LOCATION) != PackageManager.PERMISSION_GRANTED && ActivityCompat.checkSelfPermission(this, Manifest.permission.ACCESS_COARSE_LOCATION) != PackageManager.PERMISSION_GRANTED) {
    
            return;
        }
        locationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER, 1000, 0, locationListener);
    
        display.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
    
                getLocation(lat,lng);
    
                Log.e("Latitude", String.valueOf(lat));
                Log.e("Longitude", String.valueOf(lng));
                Log.e("city",address);
                Toast.makeText(getApplicationContext(), "City : " + address, Toast.LENGTH_LONG);
                displayLocation.setText("City : "+address);
    }
        });
    }
    
    private class myLocationlistener implements LocationListener {
        @Override
        public void onLocationChanged(Location location) {
            if (location != null) {
                lat = location.getLatitude();
                lng = location.getLongitude();
                //Toast.makeText(getApplicationContext(),"Latitude: "+lat+"\nLongitude: "+lng,Toast.LENGTH_LONG).show();
                getLocation(lat,lng);
            }
        }
    
        @Override
        public void onStatusChanged(String s, int i, Bundle bundle) {
    
        }
    
        @Override
        public void onProviderEnabled(String s) {
    
        }
    
        @Override
        public void onProviderDisabled(String s) {
    
        }
    }
    
    public void getLocation(double latitude,double longitude){
        JsonObjectRequest request = new JsonObjectRequest(Request.Method.GET,
                "http://maps.googleapis.com/maps/api/geocode/json?latlng="+ latitude+","+longitude +"&sensor=true",
                new Response.Listener<JSONObject>() {
    
                    @Override
                    public void onResponse(JSONObject response) {
                        try {
                            address = response.getJSONArray("results").getJSONObject(0).getString("formatted_address");
                            //textViewCity.setText(address);
                            //Toast.makeText(getApplicationContext(), "City : " + address, Toast.LENGTH_LONG);
                        } catch (JSONException e) {
                            e.printStackTrace();
                        }
                    }
                }, new Response.ErrorListener() {
            @Override
            public void onErrorResponse(VolleyError error) {
    
            }
        });
        requestQueue.add(request);
    }
    }
    

    Although by using this method it takes a lot of time to display the location.