Как мне получить Android Spinner Text для обновления? - PullRequest
2 голосов
/ 21 июня 2020

У меня внутри активности есть счетчик.

spinner

The top spinner bar is to act as a title, I then have a navigation bar below that and in the bottom half I have a fragment container.

The spinner contains a list of all of the exercises which it is passed.

enter image description here

After clicking one of the items inside the spinner drop down, I would like the spinner text to be updated, however no change occurs...

I get the following messages:

W/e.exerciseappv: Accessing hidden field Landroid/widget/AbsListView;->mIsChildViewEnabled:Z (greylist, reflection, allowed)

D/OpenGLRenderer: endAllActiveAnimators on 0xc6707410 (DropDownListView) with handle 0xc6e4a5f0

How can I get my basic spinner working (alternatively does anyone have a work-around) ?

Activity



import android.content.Intent;
import android.os.Bundle;
import android.util.Log;
import android.view.MenuItem;
import android.view.View;
import android.widget.AdapterView;
import android.widget.ArrayAdapter;
import android.widget.Spinner;
import android.widget.Toast;
import androidx.annotation.NonNull;
import androidx.appcompat.app.AppCompatActivity;
import androidx.fragment.app.Fragment;
import androidx.lifecycle.ViewModelProviders;
import com.example.exerciseappv5.Fragments.ExerciseGraphFragment;
import com.example.exerciseappv5.Fragments.RecordExerciseFragment;
import com.example.exerciseappv5.Fragments.RecordExerciseHistoryFragment;
import com.example.exerciseappv5.ViewModels.ChildExerciseViewModel;
import com.google.android.material.bottomnavigation.BottomNavigationView;

import java.util.ArrayList;
import java.util.List;

public class RecordExerciseActivity2 extends AppCompatActivity implements AdapterView.OnItemSelectedListener {

    List allChildExerciseNames = new ArrayList<>();
    public static final String PARENT_EXERCISE_ID = "-999";
    public static final String EXTRA_DATE = "com.example.exerciseappv4.EXTRA_DATE";
    public static final String EXTRA_WEEK_DATES = "1";
    public static String EXTRA_JUNCTIONID = "EXERCISE_JUNCTION_ID";
    int parentExerciseID;
    private ChildExerciseViewModel childExerciseViewModel;
    String firstExerciseName;
    String selectedExercise;

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

        getSupportActionBar().hide();

        Intent intent = getIntent();
        if (intent.hasExtra(PARENT_EXERCISE_ID)) {
            parentExerciseID = Integer.parseInt(intent.getStringExtra(PARENT_EXERCISE_ID));
        }

        BottomNavigationView bottomNav = findViewById(R.id.top_navigation);
        bottomNav.setOnNavigationItemSelectedListener(navListener);

        getSupportFragmentManager().beginTransaction().replace(R.id.fragment_container2, new RecordExerciseFragment()).commit();

        //getSupportActionBar().setHomeAsUpIndicator(R.drawable.ic_back);
        childExerciseViewModel = ViewModelProviders.of(this).get(ChildExerciseViewModel.class);
        childExerciseViewModel.getChildExerciseNameFromParentID(parentExerciseID).observe(this, this::setChildExerciseName);
        childExerciseViewModel.getAllchildExercisesFromParentID(parentExerciseID).observe(this, this::getAllChildExercisesFromParentID);


        Spinner spinner =  findViewById(R.id.spinner1);
        ArrayList spinnerStringArray = new ArrayList<>();
        //Add your data to your array
        spinnerStringArray.addAll(allChildExerciseNames);
        ArrayAdapter spinnerAdapter = new ArrayAdapter(this,
               android.R.layout.simple_spinner_dropdown_item, allChildExerciseNames);
        spinnerAdapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
        spinner.setAdapter(spinnerAdapter);
        spinner.setOnItemSelectedListener(this);
    }

    @Override
    public void onItemSelected(AdapterView<?> parent, View view, int position, long id) {
        selectedExercise = parent.getItemAtPosition(position).toString();
        Toast.makeText(parent.getContext(), selectedExercise, Toast.LENGTH_SHORT).show();
        Log.i("spinner item clicked ", selectedExercise);
    }

    @Override
    public void onNothingSelected(AdapterView<?> parent) {

    }

    private BottomNavigationView.OnNavigationItemSelectedListener navListener =
            new BottomNavigationView.OnNavigationItemSelectedListener() {
                @Override
                public boolean onNavigationItemSelected(@NonNull MenuItem item) {
                    Fragment selectedFragment = null;

                    switch (item.getItemId()) {
                        case R.id.nav_track:
                            selectedFragment = new RecordExerciseFragment();
                            break;
                        case R.id.nav_history:
                            selectedFragment = new RecordExerciseHistoryFragment();
                            break;
                        case R.id.nav_exercise_list:
                            selectedFragment = new ExerciseGraphFragment();
                            break;
                    }
                    getSupportFragmentManager().beginTransaction().replace(R.id.fragment_container2, selectedFragment).commit();
                    return true;
                }
            };

    private void setChildExerciseName(String childExerciseName) {
        firstExerciseName = childExerciseName;
    }

    public String getSelectedExercise() {
        return selectedExercise;
    }

    private void getAllChildExercisesFromParentID(List allChildExercisesReceived) {
        allChildExerciseNames.addAll(allChildExercisesReceived);
    }
}


Activity Layout XML

<?xml version="1.0" encoding="utf-8"?>
  

build.Gradle

apply plugin: 'com.android.application'

android {
    compileSdkVersion 29
    buildToolsVersion "29.0.3"

    defaultConfig {
        applicationId "com.example.exerciseappv5"
        minSdkVersion 16
        targetSdkVersion 29
        versionCode 1
        versionName "1.0"

        testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
    }

    buildTypes {
        release {
            minifyEnabled false
            proguardFiles getDefaultProguardFile('proguard-android-optimize.txt'), 'proguard-rules.pro'
        }
    }
    compileOptions {
        sourceCompatibility = 1.8
        targetCompatibility = 1.8

        sourceCompatibility JavaVersion.VERSION_1_8
        targetCompatibility JavaVersion.VERSION_1_8
    }
}

dependencies {

    implementation 'com.github.barteksc:android-pdf-viewer:2.8.2'

    def lifecycle_version = "1.1.1"
    def room_version = "1.1.1"

    implementation fileTree(dir: 'libs', include: ['*.jar'])
    implementation 'androidx.appcompat:appcompat:1.1.0'
    implementation 'com.google.android.material:material:1.1.0-rc01'
    implementation 'androidx.cardview:cardview:1.0.0'




    implementation 'androidx.constraintlayout:constraintlayout:1.1.3'
    testImplementation 'junit:junit:4.12'
    androidTestImplementation 'androidx.test.ext:junit:1.1.1'
    androidTestImplementation 'androidx.test.espresso:espresso-core:3.2.0'

    implementation 'androidx.recyclerview:recyclerview:1.0.0'
    implementation 'android.arch.lifecycle:extensions:1.0.0-alpha1'
    annotationProcessor 'android.arch.lifecycle:compiler:1.1.1'

    implementation 'android.arch.persistence.room:runtime:1.0.0-alpha1'
    annotationProcessor 'android.arch.persistence.room:compiler:1.0.0-alpha1'
}

Ответы [ 5 ]

1 голос
/ 24 июня 2020

В настоящее время вы выполняете всю настройку Spinner в onCreate(): вы создаете экземпляр адаптера с (возможно, пустым) списком данных и вызываете spinner.setAdapter(spinnerAdapter);

После этого вы меняете только данные list: каждый раз, когда вызывается getAllChildExercisesFromParentID(List<String>), вы добавляете пару String s в список данных, который вы передали в адаптер. Но вы никогда не вызываете notifyDatasetChanged() на адаптере, и я думаю, что именно здесь все начинает идти не так.

Среда выполнения не выдает Exception (как это было бы в случае с ListView вместо Spinner, если я правильно помню) - вы просто видите пустой (!) Spinner с непустым раскрывающимся списком, и выбор элемента не имеет никакого эффекта.

Итак, чтобы исправить вашу проблему, вы можете воссоздать новый адаптер для Spinner при вызове getAllChildExercisesFromParentID(List<String>). В этом случае вам нужно сделать Spinner полем вашего Activity, чтобы к нему можно было получить доступ с помощью этого метода:

private Spinner spinner;

Инициализируйте его в onCreate(), но не устанавливайте адаптер еще:

spinner =  findViewById(R.id.spinner1);
spinner.setOnItemSelectedListener(this);

Создайте новый экземпляр адаптера для Spinner, когда вы получите данные:

private void getAllChildExercisesFromParentID(List<String> allChildExercisesReceived){
    // maybe removing old data is a good idea?
    allChildExerciseNames.clear();

    allChildExerciseNames.addAll(allChildExercisesReceived);
 
    ArrayAdapter<String> spinnerAdapter = new ArrayAdapter<String>(this,
            android.R.layout.simple_spinner_dropdown_item, allChildExerciseNames);
    spinnerAdapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
    spinner.setAdapter(spinnerAdapter);

}

Или вы позволите своему Activity иметь поле private ArrayAdapter<String> spinnerAdapter;

Инициализируйте Spinner полностью в onCreate():

Spinner spinner =  findViewById(R.id.spinner1);
spinnerAdapter = new ArrayAdapter<String>(this,
            android.R.layout.simple_spinner_dropdown_item, allChildExerciseNames);
    spinnerAdapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
spinner.setAdapter(spinnerAdapter);

spinner.setOnItemSelectedListener(this);

... и обновите список данных (и, следовательно, адаптер), когда вы получите данные:

private void getAllChildExercisesFromParentID(List<String> allChildExercisesReceived){
    // remove old data 
    allChildExerciseNames.clear();

    allChildExerciseNames.addAll(allChildExercisesReceived);
    spinnerAdapter.notifyDataSetChanged();
}

Мне больше нравится вторая версия, потому что в ней не нужно создавать новый объект каждый раз при обновлении данных - это может помочь улучшить производительность (но в этом случае влияние на современные устройства будет едва заметно , так что это больше вопрос стиля кода)

Кстати, конечно, вы можете и должны сохранить Spinner в своем макете - AppCompatSpinner был всего лишь моей попыткой избавиться от странного "Landroid / widget / AbsListView ... ", которое только что появилось в одновременно с проблемой Spinner.

0 голосов
/ 24 июня 2020

Мне удалось программно воспроизвести ваш сценарий в API 29. Я считаю, что таким образом можно получить поведение счетчика по умолчанию. Чтобы воспроизвести ваш сценарий, я переопределил реализацию getView() по умолчанию объекта ArrayAdapter (я сделал это со своим настраиваемым макетом simple_spinner_item_custom, но он также должен работать с android.R.layout.simple_spinner_item):

ArrayAdapter<String> spinnerAdapter = new ArrayAdapter<String>(this,
                R.layout.simple_spinner_item_custom, spinnerStringArray) {
            @NonNull
            @Override
            public View getView(int position, @Nullable View convertView, @NonNull ViewGroup parent) {
                View view = super.getView(position, convertView, parent);
                // get a reference to the textview from the item layout
                TextView tv = view.findViewById(R.id.custom_text_1);
                // I manually set the text to empty like this to reproduce :D
                // tv.setText("");
                // the fix I would like to suggest
                tv.setText(spinnerStringArray[position]);
                return view;
            }
        };  

Я также считаю, что теперь вы можете установить onClickListener() в этом view объекте, чтобы получить onItemSelectedListener() подобное поведение.

0 голосов
/ 24 июня 2020

Должно работать из коробки. Я думаю, что есть проблема с эмулятором или старая зависимость в проекте. Подтвердите и на реальном устройстве. Могут быть альтернативы, которые мне тоже не нравятся!

Например,

// .....
@Override
public void onItemSelected(AdapterView<?> parent, View view, int position, long id) {
    selectedExercise = parent.getItemAtPosition(position).toString();
    Toast.makeText(parent.getContext(), selectedExercise, Toast.LENGTH_SHORT).show();
    Log.i("spinner item clicked ", selectedExercise);
    updateAdapter(selectedExercise)
}

// ....
private void updateAdapter(@Nullable final String selected) {
   final List<String> array = new ArrayList<>(allChildExerciseNames);

   if (!TextUtils.isEmpty(selected)) {
       array.remove(selected)
       array.set(0, selected)
}
    
   final Spinner spinner =  findViewById(R.id.spinner1);
   final ArrayAdapter<String> spinnerAdapter = new 
         ArrayAdapter<String>(this,  android.R.layout.simple_spinner_dropdown_item, array);
   

   spinnerAdapter.setDropDownViewResource(
       android.R.layout.simple_spinner_dropdown_item);
   spinner.setAdapter(spinnerAdapter, array);
   spinner.setOnItemSelectedListener(this);
}
//...

Ps. Вы можете избавиться от некоторых переменных выше, поместив их как Activity состояние класса. Это решение было добавлено по Вашему запросу в качестве альтернативы. По крайней мере, вы можете использовать его, пока мы не выясним настоящую проблему.

0 голосов
/ 24 июня 2020

enter image description hereI copy and pasted your activity and XML from above compiled and ran on pixel 3xl emulator and it is working for me.

Note - I have only commented and updated sections like framents, model and styles (in XML) that I didn't had. Please check if style of the spinner is the issue.

You can provide a custom UI to spinner as below.

 ArrayAdapter spinnerGenderAdapter = new ArrayAdapter(getActivity(),
            R.layout.spinner_item, sp_gender);
    spinnerGenderAdapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
    mGenderSpinner.setAdapter(spinnerGenderAdapter);
    mGenderSpinner.setSelection(0);

spinner_item.xml

'<?xml version="1.0" encoding="utf-8"?>

'

Activity Gradle

0 голосов
/ 21 июня 2020

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

if(view instanceof Spinner) {
  (Spinner)view.setSelection(position);
}

Если ваш onItemSelected используется только вашим счетчиком, вы можете взять весь код из onItemSelected и установите onClickListener непосредственно для вашего счетчика, таким образом проверка instanceof не нужна.

...