Попытка вызвать прямой метод void для пустой ссылки на объект - PullRequest
Новая
       7

Попытка вызвать прямой метод void для пустой ссылки на объект

0 голосов
/ 03 апреля 2020

Почему я получаю ошибку Attempt to invoke direct method 'void asyncFinished(com.ats.atsver2.ui.db.classes.User)' on a null object reference. На onPostExecute я получаю результат, но когда я пытаюсь отправить его через делегата, я получаю ошибку. Пожалуйста, смотрите мой код ниже: 

public class UserRepository {
    private UserDao userDao;
    private User user;
    private LiveData<List<User>> allUsers;

    public UserRepository(Application application) {
        AtsDatabase database = AtsDatabase.getInstance(application);
        userDao = database.userDao();
        allUsers = userDao.getUsers();
    }

    private MutableLiveData<User> userSearched = new MutableLiveData<>();
    private void asyncFinished(User userResult) {
        if(userResult == null) return;
        Log.i("TAG", "asyncFinished: " + userResult.getLastName());
        userSearched.setValue(userResult);
    }

    public LiveData<User> getUserByUserName(String userName) {
        new GetUserByUserNameAsyncTask(userDao).execute(userName);
    }

    public LiveData<List<User>> getAllUsers() { return allUsers; }

    private static class InsertUserAsyncTask extends AsyncTask<User, Void, Void> {
        private UserDao userDao;

        private InsertUserAsyncTask(UserDao userDao) { this.userDao = userDao; }

        @Override
        protected Void doInBackground(User... users) {
            userDao.insert(users[0]);
            return null;
        }
    }

    private static class GetUserByUserNameAsyncTask extends AsyncTask<String, Void, User> {
        private LiveData<User> user;
        private UserDao userDao;
        private UserRepository delegate = null;

        private GetUserByUserNameAsyncTask(UserDao userDao) { this.userDao = userDao; }

        @Override
        protected User doInBackground(String... strings) {
             user = userDao.getUserByUserName(strings[0]); ======> I GET RESULT HERE 
             return null;
        }

        @Override
        protected void onPostExecute(User user) {
            super.onPostExecute(user);
            Log.i("TAG", "onPostExecute: "+user.getLastName());
            delegate.asyncFinished(user);
        }
    }
}

Этот делегат находится в том же классе, за исключением анонимного, так почему же это нулевой объект? Можете ли вы помочь мне сделать это правильно, пожалуйста? Спасибо.

Ответы [ 3 ]

2 голосов
/ 03 апреля 2020

Вы никогда не назначаете delegate экземпляру внешнего класса при создании нового GetUserByUserNameAsyncTask. Поскольку при вызове asyncFinished() он равен нулю, он бросит NPE. 

private static class GetUserByUserNameAsyncTask extends AsyncTask<String, Void, User> {
    private LiveData<User> user;
    private UserDao userDao;
    private UserRepository delegate;

    private GetUserByUserNameAsyncTask(UserDao userDao, UserRepository delegate) { 
        this.userDao = userDao; 
        this.delegate = delegate;
    }

    @Override
    protected User doInBackground(String... strings) {
         user = userDao.getUserByUserName(strings[0]); ======> I GET RESULT HERE 
         return null;
    }

    @Override
    protected void onPostExecute(User user) {
        super.onPostExecute(user);
        Log.i("TAG", "onPostExecute: "+user.getLastName());
        delegate.asyncFinished(user);
    }
}

Теперь вы можете создать его с вашим экземпляром UserRepository (this): 

public LiveData<User> getUserByUserName(String userName) {
    new GetUserByUserNameAsyncTask(userDao, this).execute(userName);
}
1 голос
/ 03 апреля 2020

вам нужно обновить код, чтобы он заработал 

private GetUserByUserNameAsyncTask(UserDao userDao , UserRepository delegate) {
 this.userDao = userDao; 
this.delegate = delegate ;
}
0 голосов
/ 03 апреля 2020

у вас есть объект delegate со значением null: 

private UserRepository delegate = null;

И никогда после того, как вы присвоите ему значение.

...