textview onclicklistenerが動作しません

Question

私はRelativeLayoutに手作りのCreatet TextView配列を持っています。すべては、以外完璧示しています コードは動作しませonclicklistener：

private RelativeLayout mRLMain; 
private TextView [] mTV; 
private OnClickListener mTextOnClickListener; 

protected void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    mRLMain = (RelativeLayout) findViewById(R.id.rlMainGame); 
    mTV = new TextView[20]; 

    mSqueareLayout = new RelativeLayout.LayoutParams(mSquareWidth,mSquareWidth); 

    for (i = 0; i < 20; i++) { 
     mTV [i] = new TextView(MainActivity.this); 
     mTV [i].setBackgroundColor(Color.RED); 
     mTV [i].setTextColor(Color.WHITE); 
     mTV [i].setX(i+100); 
     mTV [i].setId(i); 

     mTV [i].setOnClickListener(mTextOnClickListener); 

     mRLMain.addView(mTV [i], mSqueareLayout); 
    } 

    mTextOnClickListener = new OnClickListener() { 
     @Override 
     public void onClick(View view) { Log.d("XXX","CLICK"); } 
    }; 
} 

は

mTV [i].setFocusable(false); 
mTV [i].setFocusableInTouchMode(false); 
mTV [i].setClickable(true); 
mTV [i].setDuplicateParentStateEnabled(false); 

、何もしてみません： - エラー - ノーログの

Answer 1

stting前に、あなたのclickListenerを初期化。したがって、注文を変更することで問題を解決することができます。

for (i = 0; i < 20; i++) { 
    mTV [i] = new TextView(MainActivity.this); 
    mTV [i].setBackgroundColor(Color.RED); 
    mTV [i].setTextColor(Color.WHITE); 
    mTV [i].setX(i+100); 
    mTV [i].setId(i); 

    // Set clickListener 
    mTV [i].setOnClickListener(new OnClickListener() { 
     @Override 
     public void onClick(View view) { 
      Log.d("XXX","CLICK"); 
      Toast.makeText(MainActivity.this, "Position "+ i, Toast.LENGTH_LONG).show(); 
     }); 

    mRLMain.addView(mTV [i], mSqueareLayout); 
} 

Answer 2

ClickListenerは、いくつかの複雑なロジックを有することになる場合、（例えば、内部クラス）別のクラスにそれを抽出するためにも優れています。

// same top part 
for (int i = 0; i < 20; i++){ 
      mTV [i] = new TextView(MainActivity.this); 
      mTV [i].setBackgroundColor(Color.RED); 
      mTV [i].setTextColor(Color.WHITE); 
      mTV [i].setX(i+100); 
      mTV [i].setId(i); 

      // main change here 
      mTV [i].setOnClickListener(new CustomClickListener(i)); 

      mRLMain.addView(mTV [i], mSqueareLayout); 
     } 
    } 

static class CustomClickListener implements View.OnClickListener { 

    private int viewId; 

    // another option is to pass a reference to a view here 
    // private View view; 
    // in this case, you should create appropriate constructor 

    public CustomClickListener(int viewId) { 
     this.viewId = viewId; 
    } 

    @Override 
    public void onClick(View view) { 
     Log.d("XXX","Click on view " + viewId); 
     // some logic on this view 
    } 
}