Estou me deparando com esse aviso do Android Studio. Warning:(40, 5) Do not place Android context classes in static fields; this is a memory leak (and also breaks Instant Run) Eu tenho um botão que abre um fragment DatePicker para selecionar a data.
// Botão para chamar o DatePicker Fragment na Activity principal
btnSelecionaData.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View view) {
DialogFragment newFragment = new DatePickerFragment();
newFragment.show(getSupportFragmentManager(), "datePicker");
}
});
Ao selecionar a data eu passo essas informações para um EditText que fica na classe que chamou esse DatePicker Fragment com um .setText() mas para isso funcionar o EditText precisa ser static e por isso o Android Studio fica dando esse aviso.
// Classe DatePickerFragment
public static class DatePickerFragment extends DialogFragment
implements DatePickerDialog.OnDateSetListener {
@Override
public Dialog onCreateDialog(Bundle savedInstanceState) {
// Use the current date as the default date in the picker
final Calendar c = Calendar.getInstance();
int year = c.get(Calendar.YEAR);
int month = c.get(Calendar.MONTH);
int day = c.get(Calendar.DAY_OF_MONTH);
// Create a new instance of DatePickerDialog and return it
return new DatePickerDialog(getActivity(), this, year, month, day);
}
public void onDateSet(DatePicker view, int year, int month, int day) {
// Do something with the date chosen by the user
//######### ESTE É O EDITTEXT STATIC DA ACTIVITY PRINCIPAL, O QUAL EU PRECISO SETAR COM A DATA ASSIM QUE O USUÁRIO CLICAR NO OK DO FRAGMENT
etAniver.setText(day+"/"+(month+1)+"/"+year);
//#########
}
}
Não entendo ainda 100% o funcionamento da linguagem, por isso pergunto... Tem problema deixar assim como está?
Para mim parece que não tem problema, o valor da variável é necessária apenas enquanto o usuário está na activity pois assim que ele clicar em salvar os dados serão salvos no banco e o valor da variável não terá mais importância. Ao meu entender essa String vai ficar na memoria, depois será substituída pelo novo valor quando o usuário cadastrar outro cliente. Portanto a memoria ocupada por 1 variável estática seria insignificante... seria isso? Ou os dados que passam por essa variável se sobrepõem na memoria?
DatePickerFragment
está sendo usado pra alguma coisa? Por que não usar diretamente oDatePickerDialog
?etAniver
?