Espero que isto resolva o seu problema sem criar outros.
public class CadAnuncio extends AppCompatActivity {
private static Double latiatual, longatual;
private static String cidade;
// ...
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_cad_anuncio);
// ...
GetCidade getCidade = new GetCidade(this);
getCidade.execute();
// ...
}
// ...
/* Inner classes não estáticas mantêm uma referência da classe do nível
de cima. Como esta é uma classe AsyncTask e pelo comportamento de
background dela, ela pode acabar sendo executada por mais tempo
que a própria activity, mentendo a referêcia à classe em que ela
está aninhada e, portanto, vazando memória.
Sendo assim, para evitar esses vazamentos, declaramos esta inner
class estática.
*/
private static class GetCidade extends AsyncTask<String, Integer, String> {
// Não podemos manter uma referência forte da activity
// pois o contexto pode facilmente se tornar inválido
// quando a tela for rotacionada, por exemplo
private WeakReference<Activity> weakActivity;
private LocationManager locationManager;
GetCidade(Activity activity) {
this.weakActivity = new WeakReference<>(activity);
locationManager = (LocationManager)getSystemService(Context.LOCATION_SERVICE);
}
@Override
protected void onPreExecute() {
super.onPreExecute();
// Ótima hora pra mostrar ProgressDialogs
}
@Override
protected String doInBackground(String... strings) {
if (locationManager == null) {
return null;
}
@SuppressLint("MissingPermission")
Location location = locationManager.getLastKnownLocation(LocationManager.NETWORK_PROVIDER);
/* Considere declarar e consumir essas variáveis dentro da AsyncTask
Por causa do comportamento assíncrono, você "receberá" esses
valores um tempo depois, e o fluxo da aplicação não
parará esperando por eles.
*/
latiatual = location.getLatitude();
longatual = location.getLongitude();
Verso.getInstance().setup(CadAnuncio.this);
GeoInfo geoInfo = Verso.getInstance().getGeoInfo(latiatual, longatual);
return geoInfo.getCityName();
}
@Override
protected void onProgressUpdate(Integer... progress) {
super.onProgressUpdate(progress);
int progresso = progress[0];
// Se precisar, use-o para atualizar o
// progresso de algum progress widget na UI.
}
@Override
protected void onPostExecute(String resultado) {
// Recuperar a referência da activity
Activity activity = weakActivity.get();
// Verificar se podemos mecher na UI
if (activity != null && !activity.isFinishing()) {
// Feche diálogos aqui
if (resultado != null) {
cidade = resultado;
// use cidade globalmente
} else {
new AlertDialog.Builder(activity)
.setIcon(R.drawable.ic_warning)
.setTitle("Falhou")
.setMessage("Não deu")
.show();
}
}
// else: não podemos fazer mais nada :(
}
@Override
protected void onCancelled() {
super.onCancelled();
Toast.makeText(weakActivity.get(), "Cancelado", Toast.LENGTH_SHORT).show();
}
}
}
Edit: se você quiser realmente executar o método getCidade() em uma thread diferente, receber o valor de cidade e ainda manter o fluxo, pode fazer algo assim:
public class CadAnuncio extends AppCompatActivity {
private String cidade;
// ...
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_cad_anuncio);
// ...
new Thread(() -> { // Java 8: https://developer.android.com/studio/write/java8-support?hl=pt-br
cidade = getCidade();
if (!isFinishing()) {
runOnUiThread(() -> {
// Continue o fluxo aqui,
// faça algo com 'cidade', chame outros métodos etc.
// Já estamos de volta da thread principal
});
}
}).start();
}
private String getCidade() {
AtomicReference<String> resultado = new AtomicReference<>();
AtomicBoolean completo = new AtomicBoolean(false);
Location location = locationManager.getLastKnownLocation(LocationManager.NETWORK_PROVIDER);
latiatual = location.getLatitude();
longatual = location.getLongitude();
Verso.getInstance().setup(CadAnuncio.this);
GeoInfo geoInfo = Verso.getInstance().getGeoInfo(latiatual, longatual);
resultado.set(geoInfo.getCityName());
completo.set(true);
while (true) { // Salvo por rodar eu outra thread
if (completo.get()) {
return resultado.get();
}
}
}