Refactoring.

Author: michael-rapp

library/src/main/java/de/mrapp/android/dialog/adapter/RecyclerViewAdapterWrapper.java

@@ -26,6 +26,7 @@
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import androidx.recyclerview.widget.RecyclerView;
+
 import de.mrapp.android.dialog.R;
 import de.mrapp.android.dialog.model.ListDialog.OnItemSelectedListener;
 import de.mrapp.android.util.ThemeUtil;
@@ -35,8 +36,7 @@
  * A wrapper that encapsulates a recycler view adapter in order to manage the selection states of
  * the adapter's list items.
  *
- * @param <VH>
- * The type of the view holder of the encapsulated adapter
+ * @param <VH> The type of the view holder of the encapsulated adapter
  * @author Michael Rapp
  * @since 5.0.0
  */
@@ -52,12 +52,10 @@ public interface OnItemClickListener {
  /**
  * The method, which is invoked, when a list item has been clicked.
  *
- * @param adapter
- * The adapter, the list item belongs to, as an instance of the class {@link
- * RecyclerViewAdapterWrapper}. The adapter may not be null
- * @param position
- * The position of the list item, which has been clicked, as an {@link Integer}
- * value
+ * @param adapter The adapter, the list item belongs to, as an instance of the class {@link
+ * RecyclerViewAdapterWrapper}. The adapter may not be null
+ * @param position The position of the list item, which has been clicked, as an {@link Integer}
+ * value
  */
  void onItemClick(@NonNull RecyclerViewAdapterWrapper<?> adapter, int position);
 
@@ -72,21 +70,18 @@ public interface ChoiceMode {
  /**
  * Returns, whether the list item at a specific position is currently selected, or not.
  *
- * @param position
- * The position of the list item, whose selection state should be returned, as an
- * {@link Integer} value
+ * @param position The position of the list item, whose selection state should be returned, as an
+ * {@link Integer} value
  * @return True, if the list item is selected, false otherwise
  */
  boolean isItemChecked(int position);
 
  /**
  * Sets whether the list item at a specific position should be selected, or not.
  *
- * @param position
- * The position of the list item, whose selection state should be changed, as an
- * {@link Integer} value
- * @param checked
- * True, if the list item should be selected, false otherwise
+ * @param position The position of the list item, whose selection state should be changed, as an
+ * {@link Integer} value
+ * @param checked True, if the list item should be selected, false otherwise
  * @return True, if the selection of the list item has been changed, false otherwise
  */
  boolean setItemChecked(int position, boolean checked);
@@ -101,7 +96,16 @@ public static class SingleChoiceMode implements ChoiceMode {
  /**
  * The position of the currently selected list item.
  */
- private int checkedItem = 0;
+ private int checkedItem;
+
+ /**
+ * Creates a new choice mode, which allows only a single list item to be selected at once.
+ *
+ * @param checkedItem The position of the list item, which should be selected by default
+ */
+ public SingleChoiceMode(final int checkedItem) {
+ this.checkedItem = checkedItem;
+ }
 
  @Override
  public final boolean isItemChecked(final int position) {
@@ -130,6 +134,22 @@ public static class MultipleChoiceMode implements ChoiceMode {
  */
  private final Set<Integer> checkedItems = new HashSet<>();
 
+ /**
+ * Creates a new choice mode, which allows multiple list items to be selected at once.
+ *
+ * @param checkedItems A boolean array, which specifies whether the list items at a specific
+ * position is selected or not
+ */
+ public MultipleChoiceMode(@Nullable final boolean[] checkedItems) {
+ if (checkedItems != null) {
+ for (int i = 0; i < checkedItems.length; i++) {
+ if (checkedItems[i]) {
+ this.checkedItems.add(i);
+ }
+ }
+ }
+ }
+
  @Override
  public final boolean isItemChecked(final int position) {
  return checkedItems.contains(position);
@@ -166,8 +186,7 @@ public final boolean setItemChecked(final int position, final boolean checked) {
  /**
  * A wrapper, which encapsulates a view holder.
  *
- * @param <T>
- * The type of the encapsulated view holder
+ * @param <T> The type of the encapsulated view holder
  */
  public static class ViewHolderWrapper<T extends RecyclerView.ViewHolder>
  extends RecyclerView.ViewHolder {
@@ -180,12 +199,10 @@ public static class ViewHolderWrapper<T extends RecyclerView.ViewHolder>
  /**
  * Creates a new wrapper, which encapsulates a specific view holder.
  *
- * @param itemView
- * The view, the wrapper corresponds to, as an instance of the class {@link View}.
- * The view may not be null
- * @param wrappedViewHolder
- * The encapsulated view holder as an instance of the generic type T. The view
- * holder may not be null
+ * @param itemView The view, the wrapper corresponds to, as an instance of the class {@link View}.
+ * The view may not be null
+ * @param wrappedViewHolder The encapsulated view holder as an instance of the generic type T. The view
+ * holder may not be null
  */
  public ViewHolderWrapper(@NonNull final View itemView, @NonNull final T wrappedViewHolder) {
  super(itemView);
@@ -234,8 +251,7 @@ public T getWrappedViewHolder() {
  /**
  * Creates a listener, which allows to select a list item when it has been clicked.
  *
- * @param position
- * The position of the list item as an {@link Integer} value
+ * @param position The position of the list item as an {@link Integer} value
  * @return The listener, which has been created, as an instance of the type {@link
  * View.OnClickListener}. The listener may not be null
  */
@@ -259,11 +275,9 @@ public void onClick(View v) {
  * Creates and returns a runnable, which allows to change the selection state of a specific
  * {@link Checkable}.
  *
- * @param checkable
- * The checkable, whose selection state should be changed, as an instance of the type
- * {@link Checkable}. The checkable may not be null.
- * @param checked
- * True, if the checkable should be selected, false otherwise
+ * @param checkable The checkable, whose selection state should be changed, as an instance of the type
+ * {@link Checkable}. The checkable may not be null.
+ * @param checked True, if the checkable should be selected, false otherwise
  * @return The runnable, which has been created, as an instance of the type {@link Runnable}.
  * The runnable may not be null
  */
@@ -284,15 +298,12 @@ public void run() {
  * Creates a new wrapper that encapsulates a recycler view adapter in order to manage the
  * selection states of the adapter's list items.
  *
- * @param context
- * The context, which should be used by the wrapper, as an instance of the class {@link
- * Context}. The context may not be null
- * @param wrappedAdapter
- * The encapsulated recycler view adapter as an instance of the class
- * RecyclerView.Adapter. The adapter may not be null
- * @param choiceMode
- * The choice mode, which should be used by the adapter, as an instance of the type
- * {@link ChoiceMode}. The choice mode may not be null
+ * @param context The context, which should be used by the wrapper, as an instance of the class {@link
+ * Context}. The context may not be null
+ * @param wrappedAdapter The encapsulated recycler view adapter as an instance of the class
+ * RecyclerView.Adapter. The adapter may not be null
+ * @param choiceMode The choice mode, which should be used by the adapter, as an instance of the type
+ * {@link ChoiceMode}. The choice mode may not be null
  */
  public RecyclerViewAdapterWrapper(@NonNull final Context context,
  @NonNull final RecyclerView.Adapter<VH> wrappedAdapter,
@@ -308,9 +319,8 @@ public RecyclerViewAdapterWrapper(@NonNull final Context context,
  /**
  * Sets the listener, which should be notified, when a list item has been clicked.
  *
- * @param listener
- * The listener, which should be set, as an instance of the type {@link
- * OnItemClickListener} or null, if no listener should be notified
+ * @param listener The listener, which should be set, as an instance of the type {@link
+ * OnItemClickListener} or null, if no listener should be notified
  */
  public final void setOnItemClickListener(@Nullable final OnItemClickListener listener) {
  this.itemClickListener = listener;
@@ -319,9 +329,8 @@ public final void setOnItemClickListener(@Nullable final OnItemClickListener lis
  /**
  * Sets the listener, which should be notified, when a list item has been selected.
  *
- * @param listener
- * The listener, which should be set, as an instance of the type {@link
- * OnItemSelectedListener} or null, if no listener should be notified
+ * @param listener The listener, which should be set, as an instance of the type {@link
+ * OnItemSelectedListener} or null, if no listener should be notified
  */
  public final void setOnItemSelectedListener(@Nullable final OnItemSelectedListener listener) {
  this.itemSelectedListener = listener;
@@ -352,9 +361,8 @@ public final RecyclerView.Adapter<VH> getWrappedAdapter() {
  /**
  * Returns, whether the list item at a specific position is currently selected, or not.
  *
- * @param position
- * The position of the list item, whose selection state should be returned, as an {@link
- * Integer} value
+ * @param position The position of the list item, whose selection state should be returned, as an {@link
+ * Integer} value
  * @return True, if the list item is selected, false otherwise
  */
  public final boolean isItemChecked(final int position) {
@@ -364,11 +372,9 @@ public final boolean isItemChecked(final int position) {
  /**
  * Sets, whether the list item at a specific position should be selected, or not.
  *
- * @param position
- * The position of the list item, whose selection state should be changed, as an {@link
- * Integer} value
- * @param checked
- * True, if the list item should be selected, false otherwise
+ * @param position The position of the list item, whose selection state should be changed, as an {@link
+ * Integer} value
+ * @param checked True, if the list item should be selected, false otherwise
  */
  public final void setItemChecked(final int position, final boolean checked) {
  if (choiceMode.setItemChecked(position, checked)) {
@@ -402,9 +408,10 @@ public final void onBindViewHolder(@NonNull final ViewHolderWrapper<VH> holder,
  wrappedAdapter.onBindViewHolder(wrappedViewHolder, position);
  View view = holder.itemView;
  view.setOnClickListener(createItemClickListener(position));
+ View wrappedView = wrappedViewHolder.itemView;
 
- if (view instanceof Checkable) {
- handler.post(createCheckableRunnable((Checkable) view, isItemChecked(position)));
+ if (wrappedView instanceof Checkable) {
+ handler.post(createCheckableRunnable((Checkable) wrappedView, isItemChecked(position)));
  }
  }