ListFragment OnListItemClickが呼び出されていない
ListFragmentを拡張するクラスがあり、OnListItemClickメソッドをオーバーライドしています。私も同じ方法で別のListFragmentでこれを行っています（そしてメソッドが呼び出されます）。リストアイテムをクリックしてもメソッドが呼び出されないのはなぜですか。
これがコードです：
package org.doraz.fdboard.activity;
import Java.sql.SQLException;
import Java.util.Collection;
import org.doraz.fdboard.FantasyDraftBoardApplication;
import org.doraz.fdboard.R;
import org.doraz.fdboard.activity.DraftBoardActivity.PlayerDetailsActivity;
import org.doraz.fdboard.domain.FantasyLeague;
import org.doraz.fdboard.domain.FantasyTeam;
import org.doraz.fdboard.domain.Player;
import org.doraz.fdboard.domain.Position;
import org.doraz.fdboard.repository.FantasyDraftBoardRepository;
import org.doraz.fdboard.view.PlayerAdapter;
import org.doraz.fdboard.view.PlayerCursorAdapter;
import Android.app.FragmentTransaction;
import Android.app.ListFragment;
import Android.app.ProgressDialog;
import Android.content.Intent;
import Android.database.Cursor;
import Android.os.AsyncTask;
import Android.os.Bundle;
import Android.util.Log;
import Android.view.View;
import Android.widget.Adapter;
import Android.widget.AdapterView;
import Android.widget.AdapterView.OnItemClickListener;
import Android.widget.ListView;
public class ListPlayersFragment extends ListFragment implements OnItemClickListener {
private final static String TAG = "ListPlayersFragment";
private boolean mDualPane;
private int curSelectedPlayerPosition = 0;
private PlayerCursorAdapter playerAdapter;
private QueryPlayersTask currentTask;
private FantasyDraftBoardRepository repository;
private FantasyLeague fantasyLeague;
private FantasyTeam fantasyTeam;
private Position position;
private ProgressDialog progressDialog;
/* (non-Javadoc)
* @see Android.app.ListFragment#onActivityCreated(Android.os.Bundle)
*/
@Override
public void onActivityCreated(Bundle savedInstanceState) {
super.onActivityCreated(savedInstanceState);
//Check for the view players view along with the pane
View teamListFragment = getActivity().findViewById(R.id.team_list_fragment);
mDualPane = teamListFragment != null && teamListFragment.getVisibility() == View.VISIBLE;
if(mDualPane) {
Log.i(TAG, "Setting list select mode to single");
getListView().setChoiceMode(ListView.CHOICE_MODE_SINGLE);
}
getListView().setSmoothScrollbarEnabled(false);
}
/* (non-Javadoc)
* @see Android.app.ListFragment#onListItemClick(Android.widget.ListView, Android.view.View, int, long)
*/
@Override
public void onListItemClick(ListView l, View v, int position, long id) {
super.onListItemClick(l, v, position, id);
Log.i(TAG, "[onListItemClick] Selected Position "+ position);
selectPlayer(position);
}
/* (non-Javadoc)
* @see Android.app.Fragment#onSaveInstanceState(Android.os.Bundle)
*/
@Override
public void onSaveInstanceState(Bundle outState) {
super.onSaveInstanceState(outState);
outState.putInt("curSelectedPlayerPosition", curSelectedPlayerPosition);
outState.putInt("fantasyLeague", fantasyLeague.getLeaguePID());
if(!Position.ALL.equals(position)) {
outState.putInt("position", position.getPositionPID());
}
if(!(FantasyTeam.ALL_AVAILABLE_TEAM.equals(fantasyTeam) || FantasyTeam.ALL_TEAM.equals(fantasyTeam))) {
outState.putInt("fantasyTeam", fantasyTeam.getTeamPID());
}
}
/**
* Selects the player at this position in the current list
* @param listPosition
*/
public void selectPlayer(int listPosition) {
curSelectedPlayerPosition = listPosition;
Player player = (Player) playerAdapter.getItem(listPosition);
Log.i(TAG, "Select Player ("+ listPosition +", "+ player.getName() +") called");
//Get the player url
String mPlayerUrl = player.getPlayerUrl();
Log.d(TAG, "Selected Player URL: "+mPlayerUrl);
if(mDualPane) {
if(getListView().getSelectedItemPosition() == listPosition) {
//Remove the selected item
return;
}
//Select the item
getListView().setItemChecked(listPosition, true);
Log.d(TAG, "Creating ViewPlayerFragment");
ViewPlayerFragment vpf = ViewPlayerFragment.newInstance(mPlayerUrl);
ListTeamsFragment ltf = (ListTeamsFragment) getFragmentManager().findFragmentById(R.id.team_list_fragment);
FragmentTransaction ft = getFragmentManager().beginTransaction();
ft.replace(R.id.player_web_view_fragment, vpf);
if(ltf != null && !ltf.isHidden()) {
//Hide the list of teams
ft.hide(ltf);
ft.addToBackStack(null);
}
ft.setTransition(FragmentTransaction.TRANSIT_FRAGMENT_OPEN);
ft.commit();
Log.d(TAG, "Committed to ViewPlayerFragment");
}
else {
Log.d(TAG, "Launching new activity to view player");
Intent intent = new Intent();
intent.setClass(getActivity(), PlayerDetailsActivity.class);
intent.putExtra("playerURL", mPlayerUrl);
startActivityForResult(intent, 0);
}
}
public void clearSelectedPlayer() {
Log.i(TAG, "Clearing selected player");
curSelectedPlayerPosition = -1;
//Clear the list view
getListView().clearChoices();
ViewPlayerFragment vpf = (ViewPlayerFragment) getFragmentManager().findFragmentById(R.id.player_web_view_fragment);
if(vpf != null) {
Log.d(TAG, "Closing ViewPlayerFragment");
//Close the ViewPlayersFragment
FragmentTransaction ft = getFragmentManager().beginTransaction();
ft.remove(vpf);
ft.setTransition(FragmentTransaction.TRANSIT_FRAGMENT_CLOSE);
ft.commit();
Log.d(TAG, "Closed ViewPlayerFragment");
}
}
/**
* Initializes the player adapter
*/
private void initializePlayerAdapter(Cursor cursor) {
if(playerAdapter != null)
return;
playerAdapter = new PlayerCursorAdapter(getActivity(), cursor, (DraftBoardActivity)getActivity(), repository);
setListAdapter(playerAdapter);
setEmptyText(getText(R.string.no_players_msg));
}
/**
* Initializes the player adapter
*/
public void setPlayersCursor(Cursor cursor) {
if(playerAdapter == null) {
initializePlayerAdapter(cursor);
}
else {
playerAdapter.changeCursor(cursor);
}
}
/**
* Drafts a player
*
* @param mPlayer the player to draft
* @param fantasyTeam the fantasy team
* @param value the draft value
*/
public void draftPlayer(Player mPlayer, FantasyTeam fantasyTeam, Double value) {
mPlayer.setFantasyTeam(fantasyTeam);
mPlayer.setDraftValue(value);
mPlayer.setDrafted(true);
fantasyTeam.draftPlayer(mPlayer);
try {
repository.savePlayer(mPlayer);
repository.saveFantasyTeam(fantasyTeam);
} catch (SQLException e) {
Log.e(TAG, "Error drafting player", e);
}
//Refresh the query
refresh();
}
/**
* Refreshes the players list
*/
public void refresh(){
if(fantasyLeague == null) {
fantasyLeague = ((FantasyDraftBoardApplication) (getActivity().getApplication())).getCurrentFantasyLeague();
}
if(fantasyTeam == null) {
fantasyTeam = FantasyTeam.ALL_AVAILABLE_TEAM;
}
if(position == null) {
position = Position.ALL;
}
if(currentTask != null) {
currentTask.cancel(true);
}
if(progressDialog != null) {
progressDialog.dismiss();
progressDialog = null;
}
progressDialog = ProgressDialog.show(getActivity(), null, "Loading...");
currentTask = new QueryPlayersTask(fantasyLeague, fantasyTeam, position, repository);
currentTask.execute();
}
/**
* Sets the fantasyLeague
* @param fantasyLeague the fantasyLeague to set
*/
public void setFantasyLeague(FantasyLeague fantasyLeague) {
this.fantasyLeague = fantasyLeague;
}
/**
* Sets the fantasyTeam
* @param fantasyTeam the fantasyTeam to set
*/
public void setFantasyTeam(FantasyTeam fantasyTeam) {
this.fantasyTeam = fantasyTeam;
}
/**
* Sets the position
* @param position the position to set
*/
public void setPosition(Position position) {
this.position = position;
}
/**
* Sets the repository
* @param repository the repository to set
*/
public void setRepository(FantasyDraftBoardRepository repository) {
this.repository = repository;
}
private class QueryPlayersTask extends AsyncTask<Integer, Integer, Cursor> {
private FantasyLeague fantasyLeague;
private FantasyTeam fantasyTeam;
private Position position;
private FantasyDraftBoardRepository repository;
public QueryPlayersTask(FantasyLeague fantasyLeague, FantasyTeam fantasyTeam, Position position, FantasyDraftBoardRepository repository) {
this.fantasyLeague = fantasyLeague;
this.fantasyTeam = fantasyTeam;
this.position = position;
this.repository = repository;
}
@Override
protected Cursor doInBackground(Integer... params) {
try {
return repository.queryForPlayersCursor(position, fantasyLeague, fantasyTeam);
} catch (SQLException e) {
Log.e("QueryPlayersTask", "Unable to query for players", e);
}
return null;
}
/* (non-Javadoc)
* @see Android.os.AsyncTask#onPostExecute(Java.lang.Object)
*/
@Override
protected void onPostExecute(Cursor result) {
super.onPostExecute(result);
if(!isCancelled()) {
//Update the player cursor
updatePlayerCursor(result);
}
}
}
/**
* Updates the player cursor
* @param c the player cursor
*/
private final void updatePlayerCursor(Cursor c){
Log.d(TAG, "Updating player cursor.");
if(playerAdapter == null)
initializePlayerAdapter(c);
else
playerAdapter.changeCursor(c);
if(progressDialog != null) {
progressDialog.dismiss();
progressDialog = null;
}
//Clear the current task
currentTask = null;
}
@Override
public void onItemClick(AdapterView<?> adapter, View arg1, int listPosition, long id) {
Log.d(TAG, "[onItemClick] Clicked item "+position);
selectPlayer(listPosition);
}
}
どんな援助も大歓迎です。他のいくつかのリスナーを実装し、それを各リスト項目に割り当てることで目的の効果を得ることができますが、これは正しい方法であり、機能する必要があります。なぜそれができないのか分かりません
前もって感謝します。
CheckBoxのような他のコンポーネントからの入力を盗むことができるレイアウト内のアイテムがある場合、そのコンポーネントはフォーカス不可能として定義する必要があります。
以下は、onListItemClickが思いどおりに起動できるようにするエレガントなソリューションです。また、子ビューがイベントを受信できるようにします（設定方法によっては、イベントを受信できない場合もあります）。これにより、チェックボックスのサブビューを保持できます。これがユーザーにとって直感的なエクスペリエンスであるかどうかを慎重に検討してください。
すべてのリストアイテムのルートViewGroupで、XMLのdescendantFocusability属性を設定します。
Android:descendantFocusability="blocksDescendants"
Android SDKのdescendantFocusabilityのドキュメント 。 Android 1.0以降、SDKの一部です。Android:descendantFocusabilityのその他の設定には、"beforeDescendants"および"afterDescendants"が含まれます。
descendantFocusabilityを設定するcluster_layout.xmlの例（ArrayAdapterを介してcluster_list.xmlに適用されるリストアイテム）：
<?xml version="1.0" encoding="utf-8"?>
<LinearLayout
xmlns:Android="http://schemas.Android.com/apk/res/Android"
xmlns:tools="http://schemas.Android.com/tools"
Android:id="@+id/cluster_layout"
Android:layout_width="match_parent"
Android:layout_height="wrap_content"
Android:orientation="horizontal"
Android:background="#e0e0e0"
Android:clickable="false"
Android:descendantFocusability="blocksDescendants" >
<FrameLayout
Android:id="@+id/cluster_sentiment_handle"
Android:layout_width="40dp"
Android:layout_height="match_parent"
Android:background="@color/handle_red" />
<!-- other elements -->
</LinearLayout>
Cluster_list.xmlの例。これは、IDが@ id/Android：listのListView要素を持つことで、ListFragmentでの使用が意図されていることを示すこと以外は、このソリューションには関係ありません。
<?xml version="1.0" encoding="utf-8"?>
<LinearLayout
xmlns:Android="http://schemas.Android.com/apk/res/Android"
Android:layout_width="match_parent"
Android:layout_height="match_parent"
Android:orientation="vertical"
Android:padding="8dp"
Android:background="#f6f6f6" >
<include layout="@layout/cluster_header" />
<ListView
Android:id="@id/Android:list"
Android:layout_width="match_parent"
Android:layout_height="match_parent"
Android:layout_gravity="top|center_horizontal"
Android:gravity="center_horizontal"
Android:divider="@Android:color/transparent"
Android:dividerHeight="8dp"
Android:fastScrollEnabled="true" />
</LinearLayout>
アプリをListActivityからListFragmentに変換した後、同じ問題が発生しました。
ListFragmentはアクティビティの唯一のフラグメント（タブなどはありません）でしたが、ListFragmentにはクリックが表示されませんでした。
問題は、私のアクティビティがまだListActivityとして定義されていることでした。これを修正したアクティビティに変更すると、ListFragmentはクリックを認識します。
public class MyActivity extends ListActivity {
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.my_layout);
}
になるはずだった：
public class MyActivity extends Activity {
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.my_layout);
}
<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:Android="http://schemas.Android.com/apk/res/Android"
Android:layout_width="fill_parent"
Android:layout_height="fill_parent"
Android:orientation="vertical" >
<fragment class="com.davidmarkscott.myapp.MyFragment"
Android:id="@+id/fragment"
Android:layout_width="match_parent"
Android:layout_height="match_parent" />
</LinearLayout>
他と同様に、私はListFragmentを持つCustomAdapterを持っています。これはgetView（）をオーバーライドして、各行のビューを提供します。返されたビューにリスナーを設定するとうまくいきました。例：
public class MyAdapter extends ArrayAdapter<SomeType> {
private Context context;
private List<SomeType> objects;
public MyAdapter(Context context, int resource, List<SomeType> objects) {
super(context, resource, objects);
this.context = context;
this.objects = objects;
}
@Override
public View getView(int position, View convertView, ViewGroup parent) {
LayoutInflater inflater = (LayoutInflater)context.getSystemService (Context.LAYOUT_INFLATER_SERVICE);
View rowView = inflater.inflate(R.layout.list_subscription_item, null);
TextView subTo = (TextView)rowView.findViewById(R.id.itemName);
SomeType sub = objects.get(position);
subTo.setText(sub.getName() + sub.getId());
rowView.setOnLongClickListener(new View.OnLongClickListener() {
@Override
public boolean onLongClick(View v) {
Toast.makeText(context, "from long click", Toast.LENGTH_SHORT).show();
return false;
}
});
rowView.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View v) {
Toast.makeText(context, "from click", Toast.LENGTH_SHORT).show();
}
});
return rowView;
}
}
私も同じ問題に遭遇しました。 ListFragment内のタブのタブコンテンツとしてインフレートされたTabHostを使用します。
私のTabHostの1つに3つのタブがあり、onListItemClick()が正しく呼び出されました。
しかし、他のTabHostには2つのタブしかないため、onListItemClick()は呼び出されませんでした。
そこで、上記のMarcelとH9kDroidに関連する作業を実装することにしました。ご回答有難うございます。
pdate：数時間いじってみたところ、リストアイテムのレイアウトとListViewのアダプターに問題があるようです。
リストアイテムのレイアウト内にチェックボックスが1つあり、チェックボックスの状態はアダプター内で切り替えられます。切り替え後、ListViewはめちゃくちゃになり、onListItemClick()への呼び出しを配信できないと思います。
レイアウトを変更し、チェックボックスを外すだけで問題なく動作します。
乾杯。
これまでの回避策のみ、FragmentとListViewを使用してからsetOnItemClickListener()を使用すると、ListFragmentとそのすべての便利さが廃止されます...まったく同じコード（クラス名のみが変更された）を含む新しいクラスを作成し、別のマシンでプロジェクトをビルドした後、「魔法のように」機能しました（それでも同じAPIレベル）。機能したコードの下。私はまた、通常ほとんどの問題を修正するproject-> cleanを試しましたが、それはいずれもうまくいきませんでした。
public class ContainerFragment extends ListFragment {
private ContainerFragmentListener listener;
public ContainerFragment(ContainerFragmentListener listener) {
super();
this.listener = listener;
}
@Override
public void onActivityCreated(Bundle savedInstanceState) {
super.onActivityCreated(savedInstanceState);
setListAdapter(new ContainerAdapter(getActivity()));
}
@Override
public void onListItemClick(ListView l, View v, int position, long id) {
super.onListItemClick(l, v, position, id);
Container container = (Container) getListAdapter().getItem(position);
listener.containerSelected(container.id);
}
}
@auselenが上で言ったことをエコーして、私の場合はAndroid:textIsSelectable="true"はTextViewに設定され、コールバックを盗みました。
これをfalseに設定すると問題が解決しました。
Listfragment.getListView（）。setOnItemClickListener（）を呼び出し、ListFragmentのonListItemClick（）もオーバーライドしていたため、onListItemClickが呼び出されませんでした。 setOnItemClickListener（）メソッドを削除し、代わりにlistFragmentのonListItemClickメソッドが呼び出されました。
CheckBoxはデフォルトでフォーカス可能です。これは、リストアイテムのクリックがCheckBoxの切り替えとして解釈され、onListItemClick（…）メソッドに到達しないことを意味します。このListViewの内部の癖は、リストアイテムのレイアウト（CheckBoxやボタンなど）に表示されるフォーカス可能なウィジェットを非フォーカスにして、リストアイテムのクリックが期待どおりに機能することを確認する必要があることを意味します。 CheckBoxは情報を報告するだけで、アプリケーションロジックに関連付けられていないため、簡単な解決策があります。 CheckBoxをフォーカス不可として定義できます。
the Big Nerd Ranch Guide bookから
リストアイテムのすべての子ビューでプロパティAndroid：focusableをfalseに設定すると、クリックハンドラーが発生しないことがわかりました。次の設定は、ListFragmentで機能します。
navigation_fragment.xml：
<ListView xmlns:Android="http://schemas.Android.com/apk/res/Android"
Android:id="@+id/navigationEntries"
Android:layout_height="wrap_content"
Android:layout_width="fill_parent" />
navigation_entry.xml：
<LinearLayout xmlns:Android="http://schemas.Android.com/apk/res/Android"
Android:id="@+id/navigationEntry"
Android:orientation="vertical"
Android:layout_height="wrap_content"
Android:layout_width="fill_parent"
<!-- enhance this -->
<ImageView
Android:layout_width="fill_parent"
Android:layout_height="1dp"
Android:background="@Android:color/darker_gray" />
私は自分自身の同様の問題を理解しようと何時間も費やしました。チェックボックスと2つのテキストビュー、および単純なカスタムアダプターを含むかなりシンプルな複合テキストビューがありました。提案されたさまざまなことを試した後、テキストビューが1つになるまでビューを取り出しました。それでもうまくいきませんでした。何をしたか知っていますか？使用していたカスタムビューのxmlレイアウトファイルを削除し、1つずつ作成し直しました。同じ名前で同じものを作りました。何らかの理由でそのファイルが気に入らなかっただけです。さあ、行きます。 Eclipseの奇妙な不具合の1つだと思います。