Должен ли внутренний класс DialogFragment быть статическим или нет?

В примере нет этих аннотаций, потому что класс находится в собственном файле. Это означает, что он независим от действия, использующего фрагмент.

В вашем случае ваш фрагмент находится внутри действия и не использует статический модификатор. Это означает, что он привязан к экземпляру Activity.

Зависимость фрагмента от экземпляра Activity — плохая идея, оба этих класса имеют сложные жизненные циклы (особенно потому, что Activity довольно часто уничтожаются и воссоздаются) и должны быть независимы друг от друга.

Вам нужно будет сделать модификатор EnableGpsDialogFragment static.

public static class EnableGpsDialogFragment extends DialogFragment {

Класс static не зависит от экземпляра окружающего класса, поэтому предупреждение исчезнет.

Дополнительные сведения см. в руководстве по Java по вложенным классам.

Изменить в ответ на ваше редактирование: теперь, когда классы не зависят от экземпляра друг друга, вам нужно будет получить экземпляр YourActivity, чтобы вы могли вызывать enabledLocationSettings() одним из способов приведения и будет работать только в том случае, если EnableGpsDialogFragment используется только YourActivity:

@Override
public void onClick(DialogInterface dialog, int which) {
  enableLocationSettings();
}

to

@Override
public void onClick(DialogInterface dialog, int which) {
  ((YourActivity)getActivity()).enableLocationSettings();
}

Если этот фрагмент будет использоваться несколькими действиями, вам следует вместо этого создать interface для реализации каждым действием.

Все DialogFragments должны быть общедоступными и, если это внутренний класс, статическими. Они также должны иметь общедоступный конструктор без аргументов и полагаться только на setArguments() для передачи параметров.

Невыполнение этого требования в течение некоторого времени приводило к появлению предупреждения Lint, которое вы могли бы подавить, если бы действительно хотели, но, начиная с библиотеки поддержки Android v25 и выше, вы фактически получите исключение, если попытаетесь показать DialogFragment, который не t соблюдать эти правила:

java.lang.IllegalStateException: Fragment TestActivity$TestDialogFrament must be a public static class to be properly recreated from instance state.

Причина, как уже говорилось, в том, что ОС должна иметь возможность воссоздать все фрагменты на случай, если что-то вроде ситуации с нехваткой памяти заставит ее уничтожить фрагменты, когда приложение будет переведено в фоновый режим. Когда приложение снова выводится на передний план, должна быть возможность воссоздать фрагменты из сериализованного состояния приложения, что невозможно для нестатических внутренних классов, отличных от экземпляра окружающего внешнего класса, и повторного создания. -творение не делается из этого контекста.

К сожалению, это усложняет работу с диалогами, поскольку обычно очень удобно просто создать анонимный подкласс, который переопределяет onCreateDialog. Однако такой диалог вообще невозможно воссоздать.

Так не должно быть !

С моей точки зрения, я не хочу, чтобы мой DialogFragment (ваш NetworkConnectionError) был статическим, потому что я хочу иметь возможность вызывать из него переменные или методы моего содержащего класса (Activity).
Так что это не будет статический. Но я также не хочу генерировать memoryLeaks.
Итак, каково решение?
Просто, когда вы входите в onStop, убедитесь, что вы убили свой DialogFragment, это так просто. Итак, код выглядит примерно так:

public class CarActivity extends AppCompatActivity{

/**
 * The DialogFragment networkConnectionErrorDialog 
 */
private NetworkConnectionError  networkConnectionErrorDialog ;
//...  your code ...//
@Override
protected void onStop() {
    super.onStop();
    //invalidate the DialogFragment to avoid stupid memory leak
    if (networkConnectionErrorDialog != null) {
        if (networkConnectionErrorDialog .isVisible()) {
            networkConnectionErrorDialog .dismiss();
        }
        networkConnectionErrorDialog = null;
    }
}
/**
 * The method called to display your dialogFragment
 */
private void onDeleteCurrentCity(){
    FragmentManager fm = getSupportFragmentManager();
     networkConnectionErrorDialog =(DeleteAlert)fm.findFragmentByTag("networkError");
    if(networkConnectionErrorDialog ==null){
        networkConnectionErrorDialog =new DeleteAlert();
    }
    networkConnectionErrorDialog .show(getSupportFragmentManager(), "networkError");
}

И таким образом вы избегаете утечек памяти (потому что это плохо) и гарантируете, что у вас нет гребаного статического фрагмента, который не может получить доступ к полям и методам вашей активности. Это хороший способ справиться с этой проблемой, с моей точки зрения.