Как отобразить нужную строку в Spinner из БД sqlite в Android?

Question

[Chvalov]

Есть код который загружает из БД список в spinner

private static final String DB_NAME = "ovis_mill.sqlite3";
    //Заданее имена полей БД константами
    private static final String TABLE_NAME = "baudratespinner";
    private static final String BAUDRATE_ID = "_id";
    private static final String BAUDRATE_NAME = "baudrate";
    private static final String BAUDRATE_SELECTED = "selected";
    private SQLiteDatabase database;

    private ArrayList baudrate;

    Spinner BaudrateSpinner;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        setContentView(R.layout.general_settings);

        // Находим елементы
        BaudrateSpinner = (Spinner) findViewById(R.id.BaudrateSpinner);

        //Наш ключевой хелпер
        ExternalDbOpenHelper dbOpenHelper = new ExternalDbOpenHelper(this, DB_NAME);
        database = dbOpenHelper.getWritableDatabase();
        //Все, база открыта!

        // Загрузка данных спинера с БД
        loadSpinnerBaudrateData();

        //Используем адаптер и стандартный layout элемент для краткости
        ArrayAdapter<String> BaudratedataAdapter = new ArrayAdapter<String>(this, android.R.layout.simple_spinner_item, baudrate);
        BaudrateSpinner.setAdapter(BaudratedataAdapter);
    }

    // Загружаем с БД даныые и заносим их в список
    private void loadSpinnerBaudrateData() {
        baudrate = new ArrayList<String>();
        Cursor baudrateCursor = database.query(TABLE_NAME,
                new String[]{BAUDRATE_ID, BAUDRATE_NAME, BAUDRATE_SELECTED},
                null, null, null, null,
                BAUDRATE_NAME);
        baudrateCursor.moveToFirst();
        if (!baudrateCursor.isAfterLast()) {
            do {
                String name = baudrateCursor.getString(1);
                baudrate.add(name);
            } while (baudrateCursor.moveToNext());
        }
        baudrateCursor.close();
        database.close();   // Закрываем БД
    }

Бд выглядит вод так:

Как мне отображать спиннер в которой выбрана та строка которой selected равно 1 ?

Answer 1

[LeEnot]

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

private String loadSpinnerBaudrateData() {
        String selected;
        baudrate = new ArrayList<String>();
        Cursor baudrateCursor = database.query(TABLE_NAME,
                new String[]{BAUDRATE_ID, BAUDRATE_NAME, BAUDRATE_SELECTED},
                null, null, null, null,
                BAUDRATE_NAME);
        baudrateCursor.moveToFirst();
        if (!baudrateCursor.isAfterLast()) {
            do {
                String name = baudrateCursor.getString(1);
                if (baudrateCursor.getInt(baudrateCursor.getColumnIndex( BAUDRATE_SELECTED )) == 1){
                      selected = name;
                }
                baudrate.add(name);
            } while (baudrateCursor.moveToNext());
        }
        baudrateCursor.close();
        database.close();   // Закрываем БД
        return selected;
    }

а потом:

String compareValue = loadSpinnerBaudrateData();
int spinnerPosition = BaudratedataAdapter.getPosition(compareValue);
BaudrateSpinner.setSelection(spinnerPosition);

Но вообще это криво и неправильно...

Comments

[Chvalov]

Скажите какие проблемы есть в коде, так как в основном только учусь и делал по грубо говоря по примерам из сети

[Chvalov]

LeEnot а можно спиннеру задать setSelection чтобы выборка была из БД ?

[LeEnot]

Chvalov: ну для начала стоит заметить, что правильное решение может быть громоздким и не стоит забивать гвозди микроскопом. В вашем случае вы можете задавать значения спиннера просто массивом строк типа String []baudRate = {"9600", "19200", ... } . Далее сделать Listener, который будет при изменении позиции спиннера писать значение в SharedPreferences, а в onResume формы можно читать это значение и устанавливать спиннер в нужную позицию.
А можно и через базу данных, но тогда желательно:
1. Вынести схему базы данных в отдельный класс
2. Вынести методы работы с бд в Helper
3. Запросы к бд не следует выполнять в UI потоке, т.к. они могут тормозить отклик интерфейса. Для асинхронных запросов в бд в андроиде есть специальный класс - Loader (эту тему надо гуглить отдельно)
4. Нет смысла гонять данные между курсором и ArrayAdapter, используйте CursorAdapter. Он работает с БД напрямую.
5. Если вы грузите из таблицы несколько полей, под них лучше создать объект-POJO с нужными полями, геттерами-сеттерами. Это просто кусочек парадигмы MVC (часть model). Данные объекты потом проще сериализовывать и вообще передавать туда-сюда при нужде. Хотите подробнее, погуглите тему ORM.

[Chvalov]

LeEnot: Спасибо за ответ, почитал о SharedPreferences и понял что БД мне не нужна, заранее благодарен !

[LeEnot]

Chvalov: в андроиде вообще много вещей предусмотрено заранее ) удачи в изучении!

[Chvalov]

LeEnot: подскажите еще один момент чтобы не создавать новый вопрос, spinner.setPrompt работает на всех версиях Андроид ? А то даже если скопировать полностью код с startandroid.ru/ru/uroki/vse-uroki-spiskom/115-uro... заголовок все рамно не отображается, Android 4.2.2 и 5.1.1

[LeEnot]

Chvalov: нет, с версии андроида 3.0+ spinner prompt не поддерживается. Смотрите material design guidelines от гугла

[Chvalov]

LeEnot: Переделал вод так:

public class GeneralSettingsActivity extends Activity {

    String[] data = {"9600", "19200", "38400", "57600", "115200"};
    Spinner BaudrateSpinner;
    int BaudratePosition;

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

        ArrayAdapter<String> adapter = new ArrayAdapter<String>(this, android.R.layout.simple_spinner_item, data);
        adapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);

        BaudrateSpinner = (Spinner) findViewById(R.id.BaudrateSpinner);
        BaudrateSpinner.setAdapter(adapter);

        // Загружаем сохраненую позицию елемента спиннера
        SharedPreferences loadbaudrate = getSharedPreferences("BaudrateSavedFile",0);
        BaudratePosition = loadbaudrate.getInt("BaudrateSpinnerPosition",0);
        // выделяем элемент
        BaudrateSpinner.setSelection(BaudratePosition);

        // устанавливаем обработчик нажатия
        BaudrateSpinner.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
            @Override
            public void onItemSelected(AdapterView<?> parent, View view,
                                       int position, long id) {
                // Сохраняем позицию и значение baudrate
                SharedPreferences savebaudrate = getSharedPreferences("BaudrateSavedFile", MODE_PRIVATE);
                SharedPreferences.Editor editor = savebaudrate.edit();
                editor.putInt("BaudrateSpinnerPosition", position).putString("BaudrateSpinnerText",BaudrateSpinner.getSelectedItem().toString());
                editor.apply();
            }
            @Override
            public void onNothingSelected(AdapterView<?> arg0) {
            }
        });
    }
}

Но не могу понять как мне {"9600", "19200", "38400", "57600", "115200"}; переобразовать в integer, чтобы в другом активити я мог уже нормально работать.
Подскажите как можно их сохранить в типе int, а не string
Или может есть способ с самого начала задать елементы спиннера как int

Заранее благодарен.

[LeEnot]

Chvalov: любую строку можно преобразовать в integer:

int baudRate = Integer.parseInt(BaudrateSpinner.getSelectedItem().toString());

только если там будет что-то кроме цифр, вылетит исключение NumberFormatException