Spinner setOnItemSelectedListener не работает должным образом с вызовами Room DB - PullRequest
Новая
       21

Spinner setOnItemSelectedListener не работает должным образом с вызовами Room DB

0 голосов
/ 02 января 2019

Проблема здесь в том, что после выбора нового типа пользователь прокручивает список вниз, затем снова копирует его, и тип возвращается к тому, который был до его изменения.

Эту проблему можно сформулировать следующим образом:

  1. Spinner каким-то образом сохраняет предыдущее значение от себя и установка
  2. функция SpIner OnItemClicked вызывается всякий раз, когда она связана Spinner, который должен быть ошибкой в ​​Android Spinner объект, потому что это не должно происходить, а не как Spinner объяснил в документах по Android (https://developer.android.com/guide/topics/ui/controls/spinner)

Например, у меня есть этот список типов:

  • Тип "Х"
  • Тип "Y"
  • Тип "Z"

Извините за длину, но мне, вероятно, следует включить всю мою реализацию. Я ищу возможные ошибки в своей реализации, прежде чем прибегнуть к удалению Spinner и использованию чего-то еще.

Проблема (я думаю) здесь в том, что setOnItemSelectedListener вызывается всякий раз, когда пользователь выполняет прокрутку и вносит изменения в соответствии с выбранным в данный момент элементом.

Я пробовал много разных решений; большинство из них включены здесь .

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

Используя ведение журнала, я проверил, что новое значение было установлено в БД правильно, ведя журнал до вызова update и после. 

    public class TypeFragment extends ListFragment implements AdapterView.OnItemClickListener {

    private static final String TAG = "TypeFragment";

    private TypeAdapter adapter;

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState){
        View view = inflater.inflate(R.layout.fragment_type_list, container, false);

        SwipeRefreshLayout refreshLayout = view.findViewById(R.id.swiperefresh);
        refreshLayout.setOnRefreshListener(() -> {
            if(refreshLayout.isRefreshing()){
                refreshLayout.setRefreshing(false);
            }
        });

        return view;
    }

    @Override
    public void onActivityCreated(Bundle savedInstanceState) {
        super.onActivityCreated(savedInstanceState);

        List<dbRun> typesList = ExampleDatabase.getExampleDatabase().getTypeDao().getTypes();

        ArrayList<String> typeOptions = new ArrayList<>();
        typeOptions.add(getString(R.string.type_x));
        typeOptions.add(getString(R.string.type_y));
        typeOptions.add(getString(R.string.type_z));

        ArrayAdapter<String> arrayAdapter = new ArrayAdapter<>(Objects.requireNonNull(getContext()),
                R.layout.support_simple_spinner_dropdown_item, typeOptions);

        adapter = new TypeAdapter(typesList, typeOptions, arrayAdapter);
        setListAdapter(adapter);
        adapter.notifyDataSetChanged();
        getListView().setOnItemClickListener(this);
    }

    @Override
    public void onItemClick(AdapterView<?> parent, View view, int position, long id) {}

    class TypeAdapter extends BaseAdapter {

        List<dbType> typesList;
        ArrayList<String> typeOptions;
        ArrayAdapter<String> arrayAdapter;

        public TypeAdapter(List<dbType> typesList, ArrayList<String> typeOptions, ArrayAdapter<String> arrayAdapter) {
            this.typesList = typesList;
            this.typeOptions = typeOptions;
            this.arrayAdapter = arrayAdapter;
        }

        public int getCount() {
            if (this.typesList != null) {
                return this.typesList.size();
            }
            return 0;
        }

        public Object getItem(int position) {
            return this.typesList.get(position);
        }

        public long getItemId(int position) {
            return this.typesList.get(position).id;
        }

        public View getView(int position, View convertView, ViewGroup parent) {
            if (convertView == null) {
                convertView = LayoutInflater.from(getContext()).inflate(R.layout.types_row_layout, parent, false);
            }

            TextView typeTV = convertView.findViewById(R.id.type_tv_label);
            Spinner typeSpinner = convertView.findViewById(R.id.type_spinner);

            typeSpinner.setAdapter(this.arrayAdapter);

            dbType currentType = this.typesList.get(position);

            // here I ideally would set the Type to the one currently in the DB
            //typeSpinner.setSelection(arrayAdapter.getPosition(ExampleDatabase.getExampleDatabase().getTypeDao().getType(currentType.id));

            // most popular solution from StackOverflow questions:
            typeSpinner.setSelected(false);
            typeSpinner.setSelection(Adapter.NO_SELECTION, true);

            // set a spinner with its' listener on each row in the list
            typeSpinner.post(() -> typeSpinner.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
                @Override
                public void onItemSelected(AdapterView<?> parent1, View view, int position1, long id) {
                    int item = typeSpinner.getSelectedItemPosition();
                    Log.d(TAG, "called onItemSelected, position " + position1 + " item: " + item);
                    ExampleDatabase.getExampleDatabase().getTypeDao().updateType(typeSpinner.get(item), currentType.id);
                    typeSpinner.setSelection(arrayAdapter.getPosition(arrayAdapter.getPosition(ExampleDatabase.getExampleDatabase().getTypeDao().getType(currentType.id)));
                }
                @Override
                public void onNothingSelected(AdapterView<?> parent1) {}
            }));

            return convertView;
        }
    }
}

Файлы базы данных:

DBTYPE 

@Entity(tableName="types")
public class dbType {
    @PrimaryKey(autoGenerate = true)
    @ColumnInfo(name = "id")
    public int id;

    @ColumnInfo(name = "type")
    public String type;

TypeDao 

@Dao
public interface TypeDao {
    @Query("SELECT * FROM types;")
    List<dbType> getTypes();

    @Query("SELECT type FROM types WHERE id = :typeID;")
    String getType(int typeID);

    @Query("UPDATE types SET type = :typeStr WHERE id = :typeID;")
    void updateType(String typeStr, int typeID);

ExampleDatabase исключая миграции для краткости: 

@Database(entities = {dbType.class})
@TypeConverters({dbTypeConverters.class})
public abstract class ExampleDatabase extends RoomDatabase {
    public abstract TypeDao getTypeDao()

    private static ExampleDatabase INSTANCE;

    public static CoriolisDatabase getCoriolisDatabase() {
        return INSTANCE;
    }

types_row_layout.xml 

<?xml version="1.0" encoding="utf-8"?>
<android.support.constraint.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">

    <TextView
        android:id="@+id/type_tv_label"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:textSize="16sp"
        android:padding="2dp"
        android:layout_marginLeft="2dp"
        android:layout_marginRight="2dp"
        android:layout_marginBottom="2dp"
        android:layout_marginTop="2dp"
        android:layout_gravity="center_vertical"
        android:text="@string/runs_list_runvolerror_column"
        app:layout_constraintStart_toEndOf="@id/netvolume_column"
        app:layout_constraintEnd_toStartOf="@id/run_info_btn"/>

    <android.support.constraint.Barrier
        android:id="@+id/barrier_run_row"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        app:barrierDirection="bottom"
        app:constraint_referenced_ids="netvolume_column, runvolerror_column"/>

    <Spinner
        android:id="@+id/type_spinner"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_gravity="start"
        android:layout_marginLeft="8dp"
        android:layout_marginStart="8dp"
        android:ellipsize="end"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toStartOf="@id/run_info_btn"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@id/barrier_run_row" />


</android.support.constraint.ConstraintLayout>

Редактировать - вот некоторые записи, чтобы убедиться, что вызовы БД работают, потому что добавление кода в основную часть выше, кажется, портит всю структуру / форматирование кода: 

// before update check:
Log.d(TAG, "DB type before updating: type ID is " + currentType.id + ", type is " + ExampleDatabase.getExampleDatabase().getTypeDao().getTypeWithID(currentType.id).type);
// update happens
Log.d(TAG, "DB type after updating: type ID is " + currentType.id + ", type is " + ExampleDatabase.getExampleDatabase().getTypeDao().getTypeWithID(currentType.id).type);

Эта регистрация показала, что тип был / изменен при прокрутке, и обновление базы данных зависло

Вывод при первоначальной прокрутке (обратите внимание, как он вызывается, когда на него не нажимают, и обновляет с тем же значением): 

DB type before updating: type ID is 57, type is Type X
DB type after updating: type ID is 57, type is Type X

Опять же, после нажатия и изменив значение на Тип Y: 

DB type before updating: type ID is 57, type is Type X
DB type after updating: type ID is 57, type is Type Y

Похоже, что он просто обновился ...

Опять же, прокрутка после его фактического обновления (обратите внимание, как он возвращается к типу X с этого момента): 

DB type before updating: type ID is 57, type is Type Y
DB type after updating: type ID is 57, type is Type X
...