2011-12-29 16 views
6

Quello che sto tentando di fare è quando si riceve un messaggio c2dm, avviare un servizio che richiede la posizione per "x" quantità di tempo e poi consegna tale posizione al nostro server. Il messaggio c2dm avvia correttamente il servizio e la posizione GPS si attiva, ma non si aggiorna mai. Si limita a stare lì per il periodo di tempo specificato (attualmente 12 secondi) nel thread e non fa nulla. Sto usando lo stesso identico codice da qualche altra parte nella mia app (non come servizio) e funziona perfettamente. Che cosa sto facendo di sbagliato?Avvio di LocationManager come servizio Android

Avvia il servizio quando si riceve un messaggio c2dm.

context.startService(new Intent(context, ServicePingLocation.class)); 

Questo è il codice per il servizio stesso. Tutto ciò che viene chiamato è "onCreate" e "onStart".

public class ServicePingLocation extends Service implements LocationListener { 

private final String DEBUG_TAG = "[GPS Ping]"; 
private boolean xmlSuccessful = false; 
private boolean locationTimeExpired = false; 

private LocationManager lm; 
private double latitude; 
private double longitude; 
private double accuracy; 

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

    latitude = location.getLatitude(); 
    longitude = location.getLongitude(); 

    accuracy = location.getAccuracy(); 
} 

@Override 
public void onProviderDisabled(String provider) { 
    Log.d(DEBUG_TAG, "onProviderDisabled"); 
    Toast.makeText(
      getApplicationContext(), 
      "Attempted to ping your location, and GPS was disabled.", 
      Toast.LENGTH_LONG).show(); 
} 

@Override 
public void onProviderEnabled(String provider) { 
    Log.d(DEBUG_TAG, "onProviderEnabled"); 
    lm.requestLocationUpdates(LocationManager.GPS_PROVIDER, 10000, 10f, this); 
} 

@Override 
public void onStatusChanged(String provider, int status, Bundle extras) { 
    Log.d(DEBUG_TAG, "onStatusChanged"); 

} 

@Override 
public void onCreate() { 
    Log.d(DEBUG_TAG, "onCreate"); 
} 

@Override 
public void onDestroy() { 
    Log.d(DEBUG_TAG, "onDestroy"); 
} 

@Override 
public IBinder onBind(Intent intent) { 
    Log.d(DEBUG_TAG, "onBind"); 

    return null; 
} 

@Override 
public void onStart(Intent intent, int startid) { 
    Log.d(DEBUG_TAG, "onStart"); 

    lm = (LocationManager) this.getSystemService(Context.LOCATION_SERVICE); 

    lm.requestLocationUpdates(LocationManager.GPS_PROVIDER, 10000, 10f, this); 
    lm.requestLocationUpdates(LocationManager.NETWORK_PROVIDER, 10000, 
      300f, this); 

    Log.d(DEBUG_TAG, lm.toString()); 

    new SubmitLocationTask(ServicePingLocation.this).execute(); 
} 

private void locationTimer() { 

    new Handler().postDelayed(new Runnable() { 
     // @Override 
     @Override 
     public void run() { 
      locationTimeExpired = true; 
     } 
    }, 12000); 
} 

private class SubmitLocationTask extends AsyncTask<String, Void, Boolean> { 

    /** application context. */ 
    private Context context; 

    private Service service; 

    public SubmitLocationTask(Service service) { 
     this.service = service; 
     context = service; 
    } 

    @Override 
    protected void onPreExecute() { 
     locationTimer(); // Start 12 second timer 
    } 

    @Override 
    protected void onPostExecute(final Boolean success) { 

     if (success && xmlSuccessful) { 
      lm.removeUpdates(ServicePingLocation.this); 
      onDestroy(); 
     } else { 
      if (!GlobalsUtil.DEBUG_ERROR_MSG.equals("")) 
       Toast.makeText(getBaseContext(), 
         GlobalsUtil.DEBUG_ERROR_MSG, Toast.LENGTH_SHORT) 
         .show(); 
      GlobalsUtil.DEBUG_ERROR_MSG = ""; 
     } 
    } 

    @Override 
    protected Boolean doInBackground(final String... args) { 
     try { 

      DateFormat df = null; 
      df = new SimpleDateFormat("M/d/yy h:mm a"); 
      Date todaysDate = new Date();// get current date time with 
              // Date() 
      String currentDateTime = df.format(todaysDate); 

      while ((accuracy > 100f || accuracy == 0.0) 
        && !locationTimeExpired) { 
       // We just want it to sit here and wait. 
      } 

      return xmlSuccessful = SendToServerUtil.submitGPSPing(
        0, longitude, 
        latitude, accuracy, currentDateTime); 
     } catch (Exception e) { 

      return false; 
     } 
    } 
} 

}

[Edit] Risolto il problema che stavo avendo. Il codice funzionava davvero. Ho aggiunto il gestore di rete, ho regolato il metodo onDestroy() per fermare il servizio e ottimizzato il tempo impiegato per catturare il segnale GPS.

Grazie per i consigli, CommonsWare

+1

12 secondi potrebbero non essere abbastanza lunghi in tutte le circostanze. Potresti dare un'occhiata al mio ['LocationPoller'] (https://github.com/commonsguy/cwac-locpoll), che è progettato per scenari come questo. – CommonsWare

+0

Ciao Ryan, in realtà sto lavorando a qualcosa di simile e non riesco a far funzionare il mio codice. Lo sto testando con il tuo codice e non viene compilato. Qual è la variabile "xmlSuccessful" e le classi "SendToServerUtil", "GlobalsUtil" nel codice? –

+0

xmlSuccessivo è il valore booleano che ho utilizzato per gestire la risposta REST dopo aver inviato la posizione al server. SendToServerUtil, è un file di utilità che avevo, che conteneva tutte le mie chiamate di comunicazione del server (ad esempio, "SendLocation"). GlobalsUtil è solo una classe di utilità generale in cui ho archiviato tutte le mie variabili globali. Quest'ultima probabilmente non è una buona pratica, ma è stata una delle mie prime app reali ... vivi e impari, giusto? In entrambi i casi, è possibile rimuovere tutti questi riferimenti e non si dovrebbero avere problemi. – RyanInBinary

risposta

0

Risolto il problema che stavo avendo. Il codice funzionava davvero. Ho aggiunto il gestore di rete, ho regolato il metodo onDestroy() per fermare il servizio e ottimizzato il tempo impiegato per catturare il segnale GPS.

Grazie per il consiglio, CommonsWare

Problemi correlati