プログレスが表示されない

Question

実現したいこと

ソースコードの処理が終わるまでプログレスを表示したい。

発生している問題・分からないこと

処理はきちんと組み込んでいるつもりなのですがプログレスが表示されない。

該当のソースコード

    //データ検索
    @RequiresApi(api = Build.VERSION_CODES.O)
    private void SearchData(){
        // プログレスの表示
        mProgressDialog = new CustomProgressDialog(TimetableActivity.this);
        mProgressDialog.getWindow().addFlags(WindowManager.LayoutParams.FLAG_NOT_FOCUSABLE);
        mProgressDialog.show();

        //リスト更新
        getData(storeCode);
        mControlAdapter = new ControlAdapter(getApplicationContext(), mReserve, mReservationDataByDate);
        mControlList.setAdapter(mControlAdapter);
        Collections.sort(mReserve, itemComparator);
        mControlAdapter.notifyDataSetChanged();

        // プログレスが表示されていれば閉じる
        if (mProgressDialog != null && mProgressDialog.isShowing()) {
            mProgressDialog.dismiss();
        }
    }

public class CustomProgressDialog extends Dialog {
    /**
     * コンストラクタ
     *
     * @param context コンテキスト
     */
    public CustomProgressDialog(Context context) {
        super(context, R.style.Theme_CustomProgressDialog);

        // レイアウトを決定
        setContentView(R.layout.progress_activity);
    }
}

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

プログレスで検索して組み込んでみたのですが上手く動作しません。

補足

特になし

Comments

[jimbe]

何処に問題があるか分からない場合、コードの一部だけを出されても動かして確認することさえ出来ません。
言われている問題動作を確認出来るだけのコードやレイアウト・OSのバージョン等々、必要なものは全てご提示ください。

>プログレスで検索して組み込んでみた

どのようなサイトの情報を参考にされたのか、URLをご提示頂けると良いかもしれません。

[tanuki_]

ご回答ありがとうございます。
今後そのように致します。

Answer 1

ご教示頂いた回答を基に以下のように修正した所上手くいきました。

    private void SearchData(){
        // プログレスの表示
        mProgressDialog = new CustomProgressDialog(ReserveTimetableActivity.this);
        mProgressDialog.getWindow().addFlags(WindowManager.LayoutParams.FLAG_NOT_FOCUSABLE);
        mProgressDialog.show();

        new Thread(new Runnable() {
            @Override
            public void run() {
                //リスト更新
                getData(storeCode);
                mControlAdapter = new ControlAdapter(getApplicationContext(), mReserve, mReservationDataByDate);
                runOnUiThread(new Runnable() {
                    @Override
                    public void run() {
                        mControlList.setAdapter(mControlAdapter);
                        Collections.sort(mReserve, itemComparator);
                        mControlAdapter.notifyDataSetChanged();

                        // プログレスが表示されていれば閉じる
                        if (mProgressDialog != null && mProgressDialog.isShowing()) {
                            mProgressDialog.dismiss();
                        }
                    }
                });

            }
        }).start();

    }

Answer 2

おそらく

mProgressDialog.show();

でプログレスが表示されて

//リスト更新

で時間が掛かっている間プログレスが表示されていて処理が終わったらプログレスのダイアログを閉じる…とされたいのだと思いますが。

android は UI スレッドという単一のスレッドがアプリ全体の処理を行っていることはご理解されているでしょうか。

https://developer.android.com/guide/components/processes-and-threads?hl=ja#Threads

ダイアログ等の表示処理は実際には UI スレッドへの通知（処理要求）であって、実行したら即表示されるものではありません。
また View の操作によるイベントの発火もまた UI スレッドが処理していて、その場合イベントの処理が終わらない限り UI スレッドは次の処理は出来ません。
従って、もし SearchData の呼び出しがボタン操作等によるイベント処理メソッドからであれば、まさしく UI スレッド上で動いている状態であり、ダイアログの表示の要求は SearchData が終わらなければ実行されないことになります。
SearchData は表示・処理の後にダイアログを閉じるまで行っていますから、 SearchData が終わってダイアログを表示後即閉じとなり、結局何も起きていないような状態になるでしょう。

表ではプログレスが動き裏では更新処理が動くようにするというのであれば、マルチスレッドにする必要があると思います。
(参考にされた情報がどのようなものだったのかが気になります。)

---

取り合えずテキトウに想像しました。
(なお、ダイアログでプログレスを表示することはユーザが他の操作をすることを出来なくしてしまう可能性があるため推奨されません。)

MainActivity.java

import androidx.appcompat.app.AppCompatActivity;

import android.app.Dialog;
import android.os.*;
import android.view.*;
import android.widget.*;

import java.time.*;
import java.util.*;
import java.util.function.*;

public class MainActivity extends AppCompatActivity {
  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    ListView listView = findViewById(R.id.list_view);
    Adapter adapter = new Adapter();
    listView.setAdapter(adapter);

    EditText searchCodeEdit = findViewById(R.id.search_code);
    Button searchButton = findViewById(R.id.search_button);
    searchButton.setOnClickListener(e -> {
      String storeCode = searchCodeEdit.getText().toString();
      new AsyncProgressDialog<>(null, ()->getData(storeCode), adapter::setItemList).show();
    });
  }

  //時間のかかる処理を模倣
  private List<Item> getData(String storeCode) {
    String date = LocalDateTime.now().toString();
    int max = new Random().nextInt(10) + 3; //テキトウな数のデータを用意する
    List<Item> list = new ArrayList<>();
    for(int i=0; i<max; i++) {
      if(i != 0) {
        try {
          Thread.sleep(500); //[ms]
        } catch(InterruptedException e) {
          break; //中止
        }
      }
      list.add(new Item(storeCode, (i+1)+" : "+date));
    }
    return list;
  }

  private class AsyncProgressDialog<E> extends Dialog {
    AsyncProgressDialog(Runnable pre, Supplier<E> back, Consumer<E> post) {
      super(MainActivity.this);
      setContentView(R.layout.dialog_progress);

      setOnShowListener(d -> {
        if(pre != null) {
          try {
            pre.run();
          } catch(Throwable t) {
            dismiss();
            throw t;
          }
        }
        new Thread(() -> {
          try {
            E v = back.get();
            runOnUiThread(() -> {
              try {
                if(post != null) post.accept(v);
              } finally {
                dismiss();
              }
            });
          } catch(Throwable t) {
            runOnUiThread(() -> dismiss());
            throw t;
          }
        }).start();
      });
    }
  }

  private static class Item {
    final String code, text;
    Item(String code, String text) {
      this.code = code;
      this.text = text;
    }
  }

  private static class Adapter extends BaseAdapter {
    private List<Item> itemList = new ArrayList<>();
    private Comparator<Item> itemComparator = Comparator.comparing(a -> a.code);

    void setItemList(List<Item> itemList) {
      this.itemList = new ArrayList<>(itemList); //防御コピー
      Collections.sort(itemList, itemComparator);
      notifyDataSetChanged();
    }

    @Override
    public int getCount() {
      return itemList.size();
    }

    @Override
    public Object getItem(int position) {
      return itemList.get(position);
    }

    @Override
    public long getItemId(int position) {
      return 0;
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
      ViewHolder vh = convertView == null ? new ViewHolder(parent) : (ViewHolder)convertView.getTag();
      vh.bind(itemList.get(position));
      return vh.itemView;
    }

    private static class ViewHolder {
      final View itemView;
      private TextView text1, text2;

      ViewHolder(ViewGroup parent) {
        itemView = LayoutInflater.from(parent.getContext()).inflate(android.R.layout.simple_list_item_2, parent, false);
        itemView.setTag(this);
        text1 = itemView.findViewById(android.R.id.text1);
        text2 = itemView.findViewById(android.R.id.text2);
      }

      void bind(Item item) {
        text1.setText(item.code);
        text2.setText(item.text);
      }
    }
  }
}

layout/activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity">

  <TextView
      android:id="@+id/search_code_label"
      android:layout_width="0dp"
      android:layout_height="wrap_content"
      android:text="CODE:"
      app:layout_constraintStart_toStartOf="parent"
      app:layout_constraintBaseline_toBaselineOf="@id/search_code" />
  <EditText
      android:id="@+id/search_code"
      android:layout_width="0dp"
      android:layout_height="wrap_content"
      app:layout_constraintEnd_toEndOf="parent"
      app:layout_constraintStart_toEndOf="@id/search_code_label"
      app:layout_constraintTop_toTopOf="parent" />
  <Button
      android:id="@+id/search_button"
      android:layout_width="wrap_content"
      android:layout_height="wrap_content"
      android:text="search"
      app:layout_constraintEnd_toEndOf="parent"
      app:layout_constraintTop_toBottomOf="@id/search_code" />

  <ListView
      android:id="@+id/list_view"
      android:layout_width="0dp"
      android:layout_height="0dp"
      android:background="#d0ffff"
      app:layout_constraintBottom_toBottomOf="parent"
      app:layout_constraintEnd_toEndOf="parent"
      app:layout_constraintStart_toStartOf="parent"
      app:layout_constraintTop_toBottomOf="@id/search_button" />
</androidx.constraintlayout.widget.ConstraintLayout>

layout/dialog_progress.xml

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    xmlns:app="http://schemas.android.com/apk/res-auto">

  <ProgressBar
      android:layout_width="wrap_content"
      android:layout_height="wrap_content"
      app:layout_constraintBottom_toBottomOf="parent"
      app:layout_constraintEnd_toEndOf="parent"
      app:layout_constraintStart_toStartOf="parent"
      app:layout_constraintTop_toTopOf="parent" />
</androidx.constraintlayout.widget.ConstraintLayout>