ListView clickEvent выбирает неправильный идентификатор строки БД - Помогите!

У меня есть ListView, который заполняет данные из БД в комплексный ListView. Строка в столбце gotoURL базы данных — это URL-адрес для загрузки ссылки в WebView с использованием намерения putExtra. Когда я нажимаю на ListItem, чтобы перейти к следующему действию, мой метод получает правильные строковые данные, но извлекает строку из неправильной строки базы данных. Согласно моему 'log.i(...);' указывает в DDMS, что выбрана правильная строка идентификатора, но строка из столбца извлекается из идентификатора первой строки, а не из выбранного идентификатора строки (???) - это происходит для любого выбранного ListItem.

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

Моя деятельность:

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    requestWindowFeature(Window.FEATURE_NO_TITLE);
    setContentView(R.layout.list_view2);

    activityTitle = (TextView) findViewById(R.id.titleBarTitle);
    activityTitle.setText("ADVISORY CIRCULATORS");

    displayResultList();

    final ListView lv = getListView();
    lv.setTextFilterEnabled(true);
    lv.setClickable(true);

    lv.setOnItemClickListener(new AdapterView.OnItemClickListener() {

        @Override
        public void onItemClick(AdapterView<?> a, View v, int pos,
                long id) {

            String url = "";
            lv.getItemIdAtPosition(pos);
            TextView tv = (TextView) lv.findViewById(R.id.dummy);
            url = (String) tv.getTag();

            LLog.i("tag", "ID:" + id + "URL: " + url + " selected");
            Intent i = new Intent(List_AC.this, DocView.class);
            i.putExtra("url", url);
            startActivity(i);
        }
    });
}

private void displayResultList() {

    if (android.os.Environment.getExternalStorageState().equals(
            android.os.Environment.MEDIA_MOUNTED)) {
        extStorageDirectory = Environment.getExternalStorageDirectory()
                .toString();

        File dbfile = new File(extStorageDirectory
                + "/Aero-Technologies/flyDroid/dB/flyDroid.db");

        SQLiteDatabase db = SQLiteDatabase.openOrCreateDatabase(dbfile,
                null);

        Cursor databaseCursor = db.rawQuery(
                "SELECT * FROM AC_list ORDER BY `label` ASC", null);

        Adapter_AC databaseListAdapter = new Adapter_AC(this,
                R.layout.list_item, databaseCursor, new String[] { "label",
                        "title", "description", "gotoURL" }, new int[] {
                        R.id.label, R.id.listTitle, R.id.caption,
                        R.id.dummy });

        databaseListAdapter.notifyDataSetChanged();
        this.setListAdapter(databaseListAdapter);

        } else if (android.os.Environment.getExternalStorageState().equals(
            android.os.Environment.MEDIA_UNMOUNTED)) {
            Log.i("tag", "SDCard is NOT writable/mounted");
            Alerts.sdCardMissing(this);
        }
    }
}

Мой адаптер:

public class Adapter_AC extends SimpleCursorAdapter {


static Cursor dataCursor;
private LayoutInflater mInflater;

public Adapter_AC(Context context, int layout, Cursor dataCursor,
        String[] from, int[] to) {
    super(context, layout, dataCursor, from, to);
    this.dataCursor = dataCursor;
    mInflater = LayoutInflater.from(context);
}

public View getView(int position, View convertView, ViewGroup parent) {

    ViewHolder holder;

    if (convertView == null) {
        convertView = mInflater.inflate(R.layout.list_item, null);

        holder = new ViewHolder();
        holder.text1 = (TextView) convertView.findViewById(R.id.label);
        holder.text2 = (TextView) convertView.findViewById(R.id.listTitle);
        holder.text3 = (TextView) convertView.findViewById(R.id.caption);
        holder.text4 = (TextView) convertView.findViewById(R.id.dummy);

        holder.text4.setVisibility(View.GONE);

        convertView.setTag(holder);

    } else {
        holder = (ViewHolder) convertView.getTag();
    }

    dataCursor.moveToPosition(position);

    int label_index = dataCursor.getColumnIndex("label");
    String label = dataCursor.getString(label_index);

    int title_index = dataCursor.getColumnIndex("title");
    String title = dataCursor.getString(title_index);

    int description_index = dataCursor.getColumnIndex("description");
    String description = dataCursor.getString(description_index);

    int goto_index = dataCursor.getColumnIndex("gotoURL");
    String gotoURL = dataCursor.getString(goto_index);

    holder.text1.setText(label);
    holder.text2.setText(title);
    holder.text3.setText(description);
    //holder.text4.setText(gotoURL);
    holder.text4.setTag(gotoURL);

    return convertView;
}

static class ViewHolder {
    TextView text1;
    TextView text2;
    TextView text3;
    TextView text4;
}

}

person Community    schedule 03.05.2011    source источник
comment
Почему вы задаете один и тот же вопрос тремя разными способами? Я уже ответил, почему вы получаете неправильную строку в другом вопросе (stackoverflow.com/questions/5838765/).   -  person Brian Dupuis    schedule 04.05.2011
comment
@ColdForged Я пробовал вашу реализацию, но мне ничего не возвращается (log cat показывает ноль). Да, вопрос похож, но код изменился. По крайней мере, теперь с тем, что у меня есть, возвращается строка. Кроме того, я повторно отправляю, потому что поток не перемещается в начало очереди «без ответа», когда комментарий или редактирование были сделаны (плохой дизайн со стороны ОС - он находится в том же месте/порядке).   -  person    schedule 04.05.2011
comment
Таким образом, посты застаиваются через несколько часов и больше не попадают на счетчик. Я благодарю вас за вашу помощь, и я все еще продолжаю искать обходной путь с помощью вашего метода (он кажется более логичным). Поскольку я был вынужден пройти крутую кривую обучения (новый для Java и Android) для разработки этого проекта, и нет никаких реальных ресурсов, кроме ОС, я должен повторно отправить свои вопросы, когда я внесу изменения, которые ближе. Еще раз спасибо, и если вы готовы продвинуть меня дальше своим ответом, я был бы очень признателен за помощь.   -  person    schedule 04.05.2011


Ответы (1)


ОК, ваш onItemClick имеет несколько неправильных вещей (см. встроенные комментарии):

 lv.setOnItemClickListener(new AdapterView.OnItemClickListener() {
        @Override
        public void onItemClick(AdapterView<?> a, View v, int pos,
                long id) {

            String url = "";
            lv.getItemIdAtPosition(pos); //you're never storing this value, but that's not the problem


            //This "lv" is the main problem: you're asking the LISTVIEW for the TextView
            //That's why you always get the first one. You should be asking the current view (v) instead
            TextView tv = (TextView) lv.findViewById(R.id.dummy); 
            url = (String) tv.getTag();

            LLog.i("tag", "ID:" + id + "URL: " + url + " selected");
            Intent i = new Intent(List_AC.this, DocView.class);
            i.putExtra("url", url);
            startActivity(i);
        }
    });

Однако почему вы пытаетесь использовать данные в держателе, а не получать резервную копию?

person dmon    schedule 03.05.2011
comment
Спасибо!! @dmon Это было так. Из-за моих знаний о Java и Android это единственный способ вернуть данные любого типа (по крайней мере, что-то). Если у вас есть лучший метод, пожалуйста, опубликуйте его как еще один ответ и внимательно посмотрите на остальную часть моей деятельности и код адаптера. Я думаю, что у @ColdForged есть правильная идея здесь но он вернул бы мои данные как null. Спасибо еще раз. - person ; 04.05.2011