Serviceを作りました。コードが汚いと言われました。どこが問題でしょうか？

このServiceで、やりたいことはできるのですが、先輩にこれじゃ、改善しないと使えないって言われました。コードが汚いってことだと思うんですが、どこが問題でしょうか？

import android.app.Service;
import android.content.Intent;
import android.location.Criteria;
import android.location.LocationManager;
import android.os.Binder;
import android.os.IBinder;
import android.location.LocationListener;
import android.content.pm.PackageManager;
import android.location.Location;
import android.support.v4.app.ActivityCompat;
import android.os.Bundle;
import android.util.Log;
import com.google.firebase.database.DatabaseReference;
import com.google.firebase.database.FirebaseDatabase;
import com.google.firebase.auth.FirebaseAuth;

public class GpsService extends Service implements LocationListener{

    private final String TAG ="GpsService";
    final FirebaseDatabase database = FirebaseDatabase.getInstance();


    public class GpsServiceBinder extends Binder {
        public GpsService getService() {
            return GpsService.this;
        }
    }

    @Override
    public void onCreate() {
        super.onCreate();
    }

    public GpsService() {
    }

    //removeupdatesは必要
    //    @Override
    //    protected void onPause(){
    //        super.onPause();
    //        locationManager.removeUpdates(this);
    // mLocationManager.removeUpdates(listener);
    //    }

    protected void initLocationService() {

        if (ActivityCompat.checkSelfPermission(this, android.Manifest.permission.ACCESS_FINE_LOCATION) != PackageManager.PERMISSION_GRANTED && ActivityCompat.checkSelfPermission(this, android.Manifest.permission.ACCESS_COARSE_LOCATION) != PackageManager.PERMISSION_GRANTED) {
            return;
        }
        if (ActivityCompat.checkSelfPermission(this, android.Manifest.permission.ACCESS_FINE_LOCATION) != PackageManager.PERMISSION_GRANTED &&
                ActivityCompat.checkSelfPermission(this, android.Manifest.permission.ACCESS_COARSE_LOCATION) != PackageManager.PERMISSION_GRANTED) {
            return;
        }

        Criteria criteria = new Criteria();
        criteria.setAccuracy(Criteria.ACCURACY_FINE); //setAccuracyは内部では、https://stackoverflow.com/a/17874592/1709287の用にHorizontalAccuracyの設定に変換されている
        criteria.setPowerRequirement(Criteria.POWER_HIGH);
        criteria.setAltitudeRequired(false);
        criteria.setSpeedRequired(true);
        criteria.setCostAllowed(true);
        criteria.setBearingRequired(false);
        criteria.setHorizontalAccuracy(Criteria.ACCURACY_HIGH);
        criteria.setVerticalAccuracy(Criteria.ACCURACY_HIGH);


        LocationManager locationManager = (LocationManager) getSystemService(LOCATION_SERVICE);
        String bestProvider = locationManager.getBestProvider(criteria, true);
        locationManager.requestLocationUpdates(bestProvider, 6000, 1, this);
    }

    @Override
    public void onLocationChanged(Location location) {
        Log.d(TAG,"onLocationChanged");
        refreshGeoLocation(location);
    }

    @Override
    public void onStatusChanged(String provider, int status, Bundle extras) {
    }

    @Override
    public void onProviderEnabled(String provider) {
    }

    @Override
    public void onProviderDisabled(String provider) {
    }

    Double forlatitude;
    Double forlongitude;
    boolean run=false;
    Double pchecklatitude = 0.0;
    Double mchecklatitude = 0.0;
    Double pchecklongitude = 0.0;
    Double mchecklongitude = 0.0;
    Integer count=0;

    protected void refreshGeoLocation(Location location) {
        Log.v(TAG,"refreshGeoLocation");
        String user = FirebaseAuth.getInstance().getCurrentUser().getUid();
        DatabaseReference ref = database.getReference().child("users").child(user);
        DatabaseReference latitudeRef = ref.child("latitude");
        DatabaseReference longitudeRef = ref.child("longitude");


        count++;
        if (count == 1) {
            forlatitude = location.getLatitude();
            forlongitude = location.getLongitude();
        }
        if (run == false) {
            pchecklatitude = forlatitude + 0.00025094944;
            mchecklatitude = forlatitude - 0.00025094944;
            pchecklongitude = forlongitude + 0.00025094944;
            mchecklongitude = forlongitude - 0.00025094944;

                Log.v(TAG,"LocationChange():"+count.toString());
                Log.v(TAG,"getLatitude():"+location.getLatitude()+"pchecklatitude"+pchecklatitude.toString());
                Log.v(TAG,"getLatitude():"+location.getLatitude()+"mchecklatitude"+mchecklatitude.toString());

            if (location.getLatitude() > pchecklatitude || location.getLatitude() < mchecklatitude &&
                    location.getLongitude() > pchecklongitude || location.getLongitude() < mchecklongitude) {

                latitudeRef.setValue(location.getLatitude());
                longitudeRef.setValue(location.getLongitude());
                run = true;

                Log.v(TAG,"移動検知:"+count.toString());
                Log.v(TAG,"getLatitude():"+location.getLatitude()+"pchecklatitude"+pchecklatitude.toString());
                Log.v(TAG,"getLatitude(:)"+location.getLatitude()+"mchecklatitude"+mchecklatitude.toString());

                //ここで保存用に追加
                forlatitude = location.getLatitude();
                forlongitude = location.getLongitude();
            }
        } else if (run == true) {
            latitudeRef.setValue(location.getLatitude());
            longitudeRef.setValue(location.getLongitude());
            Log.v(TAG,"追跡中:"+count.toString());
            Log.v(TAG,"getLatitude():"+location.getLatitude()+"pchecklatitude"+pchecklatitude.toString());
            Log.v(TAG,"getLatitude():"+location.getLatitude()+"mchecklatitude"+mchecklatitude.toString());
        }
    }

    @Override
    public IBinder onBind(Intent intent) {
        throw new UnsupportedOperationException("Not yet implemented");
    }

    private void startLocation() {
        initLocationService();
    }

    private void stopLocation() {
        //firebase.auth().signOut();これでログアウトはできるが、ログアウトはしない
        database.goOffline();
    }

    @Override
    public int onStartCommand(Intent intent, int flags, int startId) {

        String check =intent.getStringExtra("start/stop");
        if(check.equals("start")){
            startLocation();
        }else if(check.equals("stop")){
            stopLocation();
        }

        return super.onStartCommand(intent, flags, startId);
    }

    @Override
    public void onDestroy() {
        super.onDestroy();
    }

}