Добрый вечер. Я не знаю что не так??? ТОЛЬКО НЕ РЕШЕНИЕ!!!

Спасибо. Все решил только почему private List<Task> tasks=new ArrayList<>(); private List<String> names=new ArrayList<>(); Ведь мы делаем private class TaskDataProvider implements DbDataProvider<Task> { @Override public void refreshAllData(Map criteria) { List<Task> s= new ArrayList<>(); s=MockDB.getFakeTasks(criteria); for (Task o :s) { tasks.add(o); } } } private class NameDataProvider implements DbDataProvider<String> { @Override public void refreshAllData(Map criteria) { List<String> s= MockDB.getFakeNames(criteria); for (String s1 :s) { names.add(s1); } } } И так инициализируем?

можно проще инициализировать

tasks = MockDB.getFakeTasks(criteria);

а поле так и оставить

private List<Task> tasks;

Получается, что так инициализируем. Хотя не понимаю как это работает, ведь мы не выделяем память для полей класса Solution через оператор new...

.new есть в MockDB в инициализации поля FAKE_NAMES. метод .getFakeTasks(criteria) как раз возвращает ссылку на этот объект.

А когда мы присваем в такс мы ведь тоже иницилизируем. Зачем тогда инициализировать?

ты про метод refreshAllData(Map criteria) ?

Нет. Про это private class TaskDataProvider implements DbDataProvider<Task> { @Override public void refreshAllData(Map criteria) { List<Task> s= new ArrayList<>(); s=MockDB.getFakeTasks(criteria); for (Task o :s) { tasks.add(o); } } } private class NameDataProvider implements DbDataProvider<String> { @Override public void refreshAllData(Map criteria) { List<String> s= MockDB.getFakeNames(criteria); for (String s1 :s) { names.add(s1); } } }

здесь

List<Task> s=   new ArrayList<>();
s=MockDB.getFakeTasks(criteria);

ты инициализируешь s новым объектом через .new и в следующей строке снова инициализируешь эту же переменную другим объектом через метод MockDB.getFakeTasks(criteria); При этом ранее созданный объект отдается на съедение GC. Эти две строки можно сократить, думаю ты знаешь как ) затем в for ты добавляешь в список (names | tasks) элементы. Ровно для этого тебе и нужно было их сразу инициализировать, тк в null ты ничего не добавишь. А теперь скажи зачем расходовать столько ресурсов если вместо твоей новой переменной s можно было просто подставить tasks и присвоить в эту переменную готовый объект (сразу со всеми элементами, если они есть)

Согласен можно сократить, но я не об этом. Если бы список tasks=null; Тогда да я бы ничего не добавил бы. Но когда private List<Task> tasks; То я потом public void refreshAllData(Map criteria) { List<Task> s= new ArrayList<>(); s=MockDB.getFakeTasks(criteria); for (Task o :s) { tasks.add(o); } } Ведь так я добавляю в tasks или я не понимаю?

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

Отлично. Я подумал что вообще сам собой веду диалог. Возвращаясь к вопросу почему писал ошибку? Если все равно добавляется "по одному кирпичу из одного камаза в другой".

писал ошибку когда было так

private List<Task> tasks;
private List<String> names;

или так

private List<Task> tasks=new ArrayList<>();
private List<String> names=new ArrayList<>();

Первый вариант. А 2 вариант когда наименование другое тоже ошибка.

первый вариант, как я понял, приаттачен. там ты добавляешь, но поле null. по этому варианту Vit подсказал в чем дело. про наименование что-то не пойму о чем речь )