0

Пытаюсь зарелизить приложение выдает ошибку, а так все работает четко. Как мне кажется ругается DatePickerFragment. Что это? Как пофиксить?

Тут может быть проблема?

//Календарь возвращает текущее время
    final Calendar calendar = Calendar.getInstance();
    //Добавим к времени час если указана только дата (без времени), для срабатывания.
    calendar.set(Calendar.HOUR_OF_DAY, calendar.get(Calendar.HOUR_OF_DAY) + 1 );


    //Слушатель для выбора Даты
    etDate.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            //Проверка на длину текста в поле
            if (etDate.length() == 0) {
                etDate.setText(" ");
            }
            DialogFragment datePickerFragment = new DatePickerFragment() {
                @Override
                public void onDateSet(DatePicker view, int year, int monthOfYear, int dayOfMonth) {
                    calendar.set(Calendar.YEAR, year);
                    calendar.set(Calendar.MONTH, monthOfYear);
                    calendar.set(Calendar.DAY_OF_MONTH, dayOfMonth);
                    etDate.setText(Utils.getDate(calendar.getTimeInMillis()));
                }
                //При нажатии "Отмена" текст не устанавливался
                @Override
                public void onCancel(DialogInterface dialog) {
                    etDate.setText(null);
                }
            };
            datePickerFragment.show(getFragmentManager(), "DatePickerFragment");
        }
    });
    //Слушатель для выбора Времени
    etTime.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            if (etTime.length() == 0){
                etTime.setText(" ");
            }
            DialogFragment timePickerFragment = new TimePickerFragment() {
                @Override
                public void onTimeSet(TimePicker view, int hourOfDay, int minute) {
                    calendar.set(Calendar.HOUR_OF_DAY, hourOfDay);
                    calendar.set(Calendar.MINUTE, minute);
                    calendar.set(Calendar.SECOND, 0);
                    etTime.setText(Utils.getTime(calendar.getTimeInMillis()));
                }

                @Override
                public void onCancel(DialogInterface dialog) {
                    etTime.setText(null);
                }
            };
            timePickerFragment.show(getFragmentManager(), "TimePickerFragment ");
        }
    });

    builder.setPositiveButton(R.string.dialog_add, new DialogInterface.OnClickListener() {

        @Override
        public void onClick(DialogInterface dialog, int which) {
            task.setTitle(etTitle.getText().toString());
            task.setStatus(ModelTask.STATUS_CURRENT);
            if (etDate.length() != 0 || etTime.length() != 0 ){
                task.setDate(calendar.getTimeInMillis());

                AlarmHelper alarmHelper = AlarmHelper.getInstance();
                alarmHelper.setAlarm(task);
            }
            task.setStatus(ModelTask.STATUS_CURRENT);
            addingTaskListener.onTaskAdded(task);
            dialog.dismiss();
        }
    });

    builder.setNegativeButton(R.string.dialog_cancel, new DialogInterface.OnClickListener() {
        @Override
        public void onClick(DialogInterface dialog, int which) {
            addingTaskListener.onTaskAddingCancel();
            dialog.cancel();
        }
    });

Класс DatePickerFragment

public class DatePickerFragment extends DialogFragment
    implements DatePickerDialog.OnDateSetListener {


@Override
public Dialog onCreateDialog(Bundle savedInstanceState) {

    Calendar c = Calendar.getInstance();
    int year = c.get(Calendar.YEAR);
    int month = c.get(Calendar.MONTH);
    int day = c.get(Calendar.DAY_OF_MONTH);

    return new DatePickerDialog(getActivity(), this, year, month, day);
}

@Override
public void onDateSet(DatePicker view, int year, int monthOfYear, int dayOfMonth) {

}

Фрагмент вынесен отдельно...

Сделал как сказали проверка показала на этот кусок кода, ниже укажу строчку

    @Override
public void addTask(ModelTask newTask, boolean saveToDB) {
    int position = -1;
    ModelSeparator separator = null;

    for (int i = 0; i < adapter.getItemCount(); i++) {
        if (adapter.getItem(i).isTask()) {
            ModelTask task = (ModelTask) adapter.getItem(i);
            if (newTask.getDate() < task.getDate()) {
                position = i;
                break;
            }
        }
    }

    if (newTask.getDate() != 0) {
        Calendar calendar = Calendar.getInstance();
        calendar.setTimeInMillis(newTask.getDate());
        if (calendar.get(Calendar.YEAR) < Calendar.getInstance().get(Calendar.YEAR)) {
            newTask.setDateStatus(ModelSeparator.TYPE_OVERDUE);
            if (!adapter.containsSeparatorOverdue) {
                adapter.containsSeparatorOverdue = true;
                separator = new ModelSeparator(ModelSeparator.TYPE_OVERDUE);
            }
        } else if (calendar.get(Calendar.YEAR) > Calendar.getInstance().get(Calendar.YEAR)) {
            newTask.setDateStatus(ModelSeparator.TYPE_FUTURE);
            if (!adapter.containsSeparatorFuture) {
                adapter.containsSeparatorFuture = true;
                separator = new ModelSeparator(ModelSeparator.TYPE_FUTURE);
            }
        } else if (calendar.get(Calendar.DAY_OF_YEAR) < Calendar.getInstance().get(Calendar.DAY_OF_YEAR) &&
                calendar.get(Calendar.YEAR) == Calendar.getInstance().get(Calendar.YEAR)) {
            newTask.setDateStatus(ModelSeparator.TYPE_OVERDUE);
            if (!adapter.containsSeparatorOverdue) {
                adapter.containsSeparatorOverdue = true;
                separator = new ModelSeparator(ModelSeparator.TYPE_OVERDUE);
            }
        } else if (calendar.get(Calendar.DAY_OF_YEAR) == Calendar.getInstance().get(Calendar.DAY_OF_YEAR) &&
                calendar.get(Calendar.YEAR) == Calendar.getInstance().get(Calendar.YEAR)) {
            newTask.setDateStatus(ModelSeparator.TYPE_TODAY);
            if (!adapter.containsSeparatorToday) {
                adapter.containsSeparatorToday = true;
                separator = new ModelSeparator(ModelSeparator.TYPE_TODAY);
            }
        } else if (calendar.get(Calendar.DAY_OF_YEAR) == Calendar.getInstance().get(Calendar.DAY_OF_YEAR) + 1 && // <-- Ругается на эту строку на кусок после "==" - Сообщение такое - "Must be one of: Calendar.SUNDAY, Calendar.MONDAY, Calendar.TUESDAY, Calendar.WEDNESDAY, Calendar.THURSDAY, Calendar.FRIDAY, Calendar.SATURDAY less... (Ctrl+F1)"
                calendar.get(Calendar.YEAR) == Calendar.getInstance().get(Calendar.YEAR)) {
            newTask.setDateStatus(ModelSeparator.TYPE_TOMORROW);
            if (!adapter.containsSeparatorTomorrow) {
                adapter.containsSeparatorTomorrow = true;
                separator = new ModelSeparator(ModelSeparator.TYPE_TOMORROW);
            }
        } else if (calendar.get(Calendar.DAY_OF_YEAR) > Calendar.getInstance().get(Calendar.DAY_OF_YEAR) + 1 &&
                calendar.get(Calendar.YEAR) == Calendar.getInstance().get(Calendar.YEAR)) {
            newTask.setDateStatus(ModelSeparator.TYPE_FUTURE);
            if (!adapter.containsSeparatorFuture) {
                adapter.containsSeparatorFuture = true;
                separator = new ModelSeparator(ModelSeparator.TYPE_FUTURE);
            }
        }
    }
5
  • Где и при каких действиях возникает и отображается ошибка?
    – ЮрийСПб
    5 дек 2016 в 21:14
  • При попытки генерировать проект из "Дебаг" в "релиз" (Андроид Студио) 5 дек 2016 в 22:04
  • Я думаю, что ошибка сия намекает на то, что где-то у вас фрагмент объявлен не в отдельном файле и не в виде врутреннего статического класса, а в виде анонимного класса. По идее, если вы вынесите этот фрагмент в отдельный файл то проблема исчезнет
    – ЮрийСПб
    5 дек 2016 в 22:08
  • Вы добавили большой кусок кода, но что-то строчки, на которую анализатор ругается, я не вижу
    – lllyct
    6 дек 2016 в 11:28
  • Я мучаюсь с этой же проблемой уже который день. @Eugene Zaychenko, вы решили проблему? 2 дек 2018 в 13:21

2 ответа 2

0

У вас где-то объявлен фрагмент как анонимный или не статический вложенный класс. Так делать нехорошо, по тому что система может прибить активность вместе с фрагментом, например, при сворачивании, а потом восстановить. При восстановлении может создаться новый экземпляр фрагмента и для этого будет использоваться конструктор по умолчанию без параметров. У вложенных не статических классов все конструкторы неявно принимают в качестве параметра экземпляр класса (грубо говоря, для пояснения ситуации подойдет), поэтому все фрагменты должны объявляться как отдельные классы, или как статические вложенные классы. Т.е. вместо

class MyFragment extends Fragment {

объявленного внутри другого класса, должны быть

static class MyFragment extends Fragment {

И да, вы не сможете так просто ссылаться на экземпляр внешнего класса, его придется как-то иначе получать.

Ну и еще такого кода надо избегать:

new MyFragment() {
    // some methods overriding
}

Это и есть анонимный класс. И он неявно ссылается на класс, в котором вы его объявили.

Плюс ко всему у каждого фрагмента обязательно должен быть конструктор без параметров.

18
  • Я добавил код в котором может быть проблема, но тут вроде все правильно. Фрагмент вынесен в отдельный файл, а при нажатии создаем его новый экземпляр. Тут есть ошибка? 6 дек 2016 в 10:37
  • @EvgeniyZaychenko c DatePickerFragment вроде бы все нормально, но ошибка может возникать из-за другого фрагмента. Попробуйте в студии Analyze->Inspect Code, в настройках InspectionProfile поставьте галочку только для Fragment not instantiatable и запустите проверку. По идее вы должны увидеть не только абстрактное сообщение об ошибке, но и конкретную строку или по крайней мере класс.
    – lllyct
    6 дек 2016 в 10:45
  • И еще не забудьте предварительно клонировать профиль Inspections и так или иначе вернуть все обратно. Оно вам еще пригодится и будет подсвечивать всякие разные ошибки.
    – lllyct
    6 дек 2016 в 10:47
  • Сделал как написали, выдала 1 строчку, помню она ругалась на нее раньше, но я что-то нажал (выбрал один из предложенных вариантов решения) и подчеркивание исчезло, но не так все просто :)) В коде указал строку на которую ругается. 6 дек 2016 в 11:19
  • В последнем блоке кода передвиньте на центр горизонтальный бегуок прокрутки 6 дек 2016 в 11:30
0

Добавь в build.gradle

android {
    lintOptions {
        abortOnError false
    }
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.