11

以前使用していた場所から、メソッドのみをResourceCursorAdapter使用newViewbindViewている場所に切り替えました。SimpleCursorAdaptergetView

今、私はにエラーがありますonLoaderFinished。それは私のアダプターとカーソルオブジェクトの両方で私に与えますがnullではありNullPointerExceptionませ。以下にすべてのコードを投稿します。どんな助けでも大歓迎です(引き抜くためにあまり髪が残っていません)。adapter.swapCursor(cursor)

import android.annotation.SuppressLint;
import android.content.Context;
import android.database.Cursor;
import android.net.Uri;
import android.os.Bundle;
import android.provider.ContactsContract;
import android.support.v4.app.FragmentActivity;
import android.support.v4.app.LoaderManager;
import android.support.v4.content.CursorLoader;
import android.support.v4.content.Loader;
import android.support.v4.widget.ResourceCursorAdapter;
import android.util.Log;
import android.util.SparseBooleanArray;
import android.view.LayoutInflater;
import android.view.View;
import android.view.View.OnClickListener;
import android.view.ViewGroup;
import android.view.Window;
import android.widget.CheckBox;
import android.widget.ListView;
import android.widget.TextView;

public class ContactSelect extends FragmentActivity implements LoaderManager.LoaderCallbacks<Cursor> {
private static final int LOADER_ID = 1;
private MyAdapter adapter;
private ListView list;
private View row;
private SparseBooleanArray checkedState = new SparseBooleanArray();

@SuppressLint({ "NewApi", "NewApi" })
@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    requestWindowFeature(Window.FEATURE_INDETERMINATE_PROGRESS);
    setContentView(R.layout.activity_contact_select);       

    adapter = new MyAdapter(this, R.layout.contacts_select_row, null, null, null, 0);     

    getSupportLoaderManager().initLoader(LOADER_ID, null, this);        

    list = (ListView)findViewById(R.id.list);                 

    list.setAdapter(adapter);
    list.setEmptyView(findViewById(R.id.empty));    

}   

@SuppressLint("NewApi")
public Loader<Cursor> onCreateLoader(int arg0, Bundle arg1) {
    final String projection[] = new String[]{ContactsContract.Contacts._ID, ContactsContract.Contacts.DISPLAY_NAME};
    final Uri uri = ContactsContract.Contacts.CONTENT_URI;

    final String selection = ContactsContract.Contacts.HAS_PHONE_NUMBER + "=1" + 
    " AND " + ContactsContract.Contacts.IN_VISIBLE_GROUP + " =1";

    final String order = ContactsContract.Contacts.DISPLAY_NAME + " COLLATE LOCALIZED ASC";

    final CursorLoader loader = new CursorLoader(this, uri, projection, selection, null, order);

    return loader;      
}

public void onLoadFinished(Loader<Cursor> loader, Cursor cursor) {
    for(int i=0;i<cursor.getCount();i++){
        checkedState.put(i, false);
    }

    adapter.swapCursor(cursor);                 
}

public void onLoaderReset(Loader<Cursor> loader) {
    adapter.swapCursor(null);       
}

private class MyAdapter extends SimpleCursorAdapter implements OnClickListener{
    private CheckBox markedBox;
    private TextView familyText;
    private Context context;
    private Cursor cursor;

    public MyAdapter(Context context, int layout, Cursor c, String[] from,
            int[] to, int flags) {
        super(context, layout, c, from, to, flags);

        this.context = context;
        this.cursor = getCursor();
    }

    @Override
    public View getView(int position, View view, ViewGroup group) {

        final LayoutInflater li = (LayoutInflater)getSystemService(Context.LAYOUT_INFLATER_SERVICE);
        row = li.inflate(R.layout.contacts_select_row, group, false);

        view.setTag(cursor.getPosition());
        view.setOnClickListener(this);

        familyText = (TextView)view.findViewById(R.id.contacts_row_family_name);
        markedBox = (CheckBox)view.findViewById(R.id.contacts_row_check);
        familyText.setText(cursor.getString(cursor.getColumnIndexOrThrow(ContactsContract.Contacts.DISPLAY_NAME)));

        boolean currentlyChecked = checkedState.get(cursor.getPosition());
        markedBox.setChecked(currentlyChecked);     


        setProgressBarIndeterminateVisibility(false);

        return super.getView(position, view, group);
    }

    public void onClick(View view) {
        int rowId = (Integer)view.getTag();
        Log.d("OnClick", String.valueOf(rowId));
        boolean currentlyChecked = checkedState.get(rowId);
        markedBox.setChecked(!currentlyChecked);
        checkedState.put(rowId, !currentlyChecked);
        Log.d("checkedState", "checkedState(" + rowId + ") = " + checkedState.get(rowId));
    }       
      }     
}
4

2 に答える 2

12

swapCursorクラスのメソッドを呼び出すと、コンストラクターに提供された配列(4番目のパラメーター)から列の名前を、列のインデックスを表す整数の配列にSimpleCursorAdapterマップする関数がトリガーされます。カーソルから列名を表す配列のコンストラクターを渡すと、String後でマッピングを試行するときにスローされます(これは、列名配列を使用する実際のメソッドであるメソッドに表示される必要があります)。 )。nullMyAdapterStringNullPointerExceptionswapCursorNullPointerExceptionfindColumnsString

解決策は、有効なString配列を渡すことですint。データを配置するビューのIDを表す配列に対してもこれを行うことができます。

String[] from = {ContactsContract.Contacts.DISPLAY_NAME};
int[] to = {R.id.contacts_row_family_name, R.id.contacts_row_check};
adapter = new MyAdapter(this, R.layout.contacts_select_row, null, from, to, 0);

あなたが何をしようとしているのかわかりませんが、getViewメソッドの実装は完全に正しくありません。

メソッドの通常のgetView処理(レイアウトの作成、ビューの検索、データのバインディング)を実行してから、スーパークラス(?!?)からビューを返すだけで、おそらく何も含まれていないデフォルトのレイアウトが表示されます。

メソッドのgetView記述方法はあまり効率的ではないため、ビューのリサイクルとビューホルダーのパターンを調べることをお勧めします。

cursor.getPosition()カーソルを正しい位置に移動しないため、目的の処理が実行されません。デフォルトでは、カーソルベースのアダプタがメソッドでこれを行いますが、getViewメソッドを上書きすると、カーソルの位置を移動するのがあなたの仕事になります。

getView メソッドを終了し、2つのメソッドを使用するnewView必要がありますbindView。これらのメソッドは、ロジックをより適切に分離できるためです。

于 2012-09-08T12:48:29.547 に答える
2

adapter = new MyAdapter(this、R.layout.contacts_select_row、null、null、null、0);

ヌルカーソルを渡すMyAdapterクラス

  public MyAdapter(Context context, int layout, Cursor c, String[] from,
            int[] to, int flags) {
        super(context, layout, c, from, to, flags);

        this.context = context;
        this.cursor = getCursor();
    }   
于 2012-09-02T11:51:31.833 に答える