从 Retrofit rxjava 中的 onNext 获取数据

Question

这里的列表（部分是 list.get(9) ）是一个字符串数组列表。我试图从用户那里获取所有用户名并将其添加到列表中，然后在文本视图中显示第 10 个名称。

当我运行应用程序时，它会在我下面链接的图像中引发错误，错误是 textView.setText(list.get(9)) 中的索引超出范围。但是当我评论这一行时它起作用，并使用按钮 onclick 为我的 textview 设置文本。

这是我示例中的 api。 https://jsonplaceholder.typicode.com/users

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    //init api
    Retrofit retrofit = RetrofitClient.getInstance();
    btnOk = findViewById(R.id.btn_ok);
    myAPI = retrofit.create(IMyAPI.class);
    textView = findViewById(R.id.show);

    //fetchData()

    fetchUserData();

    textView.setText(list.get(9));
    btnOk.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            textView.setText(list.get(9));
        }
    });

}

private void fetchUserData() {
    myAPI.getUsers()
            .subscribeOn(Schedulers.io())
            .observeOn(AndroidSchedulers.mainThread())
            .subscribe(new Observer<ArrayList<User>>() {
                @Override
                public void onSubscribe(Disposable d) {

                }

                @Override
                public void onNext(ArrayList<User> users) {
                    String usName ;
                    for(int i = 0; i < users.size(); i++){
                        usName = posts.get(i).getUsername();
                        list.add(usName);
                    }
                }

                @Override
                public void onError(Throwable e) {

                }

                @Override
                public void onComplete() {

                }
            });

}

Error in line textView.settext

Answer 1

您的代码中有一些错误可能导致IndexOutOfBoundsException。

我会在你的代码中加入一些 cmets：

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    //init api
    Retrofit retrofit = RetrofitClient.getInstance();
    btnOk = findViewById(R.id.btn_ok);
    myAPI = retrofit.create(IMyAPI.class);
    textView = findViewById(R.id.show);

    fetchUserData();


    // this line should be removed cause you didn't get yet the result of fetchUserData(). Instead you should do this on the onNext method.
    // textView.setText(list.get(9)); 

    btnOk.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            // Before accessing the list, you must ensure your list contains at least 9 elements. Otherwise it could crash.
            // textView.setText(list.get(9));

            // This would be the correct way
            if (list != null && list.size > 9) {
               textView.setText(list.get(9));
            }
        }
    });

}

private void fetchUserData() {
    myAPI.getUsers()
            .subscribeOn(Schedulers.io())
            .observeOn(AndroidSchedulers.mainThread())
            .subscribe(new Observer<ArrayList<User>>() {
                @Override
                public void onSubscribe(Disposable d) {

                }

                @Override
                public void onNext(ArrayList<User> users) {
                    String usName ;
                    for(int i = 0; i < users.size(); i++){
                        usName = posts.get(i).getUsername();
                        list.add(usName);
                    }
                    // This is the place where you are safe to access the users list, so any View component you want to update with the users list should be done here.
                }

                @Override
                public void onError(Throwable e) {

                }

                @Override
                public void onComplete() {

                }
            });

}

请记住，您正在使用 Rx 进行 asynchronous 调用，在对结果进行任何操作之前，您应该等待调用 onNext 方法。

Answer 2

好的，问题是myAPI.getUsers() 在io 线程中运行，这需要时间来完成并用用户名填充list。另一方面，textView.setText(list.get(9)); 会立即触发。所以正确的流程是把textView.setText(list.get(9));放到onNext或者onComplete里面，这些方法会在myAPI.getUsers()完成时触发

Answer 3

您在 onCreate 方法中调用 textView.setText(list.get(9)); 而 list 仍然为空，这会导致 IndexOutOfBoundsException 。将其从onCreate 中移除并放入onNext 方法中：

public void onNext(ArrayList<User> users) {
     String usName ;
     for(int i = 0; i < users.size(); i++){
         usName = posts.get(i).getUsername();
         list.add(usName);
     }
     textView.setText(list.get(9));
}