Получение NPE (Отредактировано)

Question

Я все еще получаю NPE, но теперь это из моего onCreate(). Когда я удаляю кнопки, все работает нормально.

Вот XML (каждая таблица строк из отдельных XMLs):

<TableRow 
     android:id="@+id/tableRow1" 
     android:layout_width="wrap_content" 
     android:layout_height="wrap_content" 
     android:layout_marginTop="50dp" > 

     <Button 
      android:id="@+id/doneP1" 
      android:layout_width="50dp" 
      android:layout_height="wrap_content" 
      android:text="Done" /> 

     <Button 
      android:id="@+id/resetP1" 
      android:layout_width="wrap_content" 
      android:layout_height="wrap_content" 
      android:text="Reset" /> 

    </TableRow> 

<TableRow 
      android:id="@+id/tableRow1" 
      android:layout_width="wrap_content" 
      android:layout_height="wrap_content" 
      android:layout_marginTop="50dp" > 

      <Button 
       android:id="@+id/doneP2" 
       android:layout_width="50dp" 
       android:layout_height="wrap_content" 
       android:text="Done" /> 

      <Button 
       android:id="@+id/resetP2" 
       android:layout_width="wrap_content" 
       android:layout_height="wrap_content" 
       android:text="Reset" /> 

     </TableRow> 

Вот код:

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

     /*doneP1 = (Button) this.findViewById(R.id.doneP1); 
     doneP2 = (Button) this.findViewById(R.id.doneP2); 
     resetP1 = (Button) this.findViewById(R.id.resetP1); 
     resetP2 = (Button) this.findViewById(R.id.resetP2); 

     checkTurn(); //starts the game 

     //buttons 
     doneP1.setOnClickListener(this); 
     doneP2.setOnClickListener(this); 
     resetP1.setOnClickListener(this); 
     resetP2.setOnClickListener(this);*/ //commented out 
    } 

и вне OnCreate

@Override 
public void onClick(View v) { 
     switch(v.getId()) { 
     case R.id.doneP1: 
      checkTurn(); 
      break; 
     case R.id.doneP2: 
      checkTurn(); 
      break; 
     case R.id.resetP1: 
      break; 
     case R.id.resetP2: 
      break; 
     } 
} 

здесь является checkturn

public void checkTurn() 
    { 
     if(turn == 1) 
     { 
      changeLayout(turn); 
      turn = 2; 
     } 
     else 
     { 
      changeLayout(turn); 
      turn = 1; 
     } 
    } 

вот changelayout (EDIT: новый changeLayout, я просто инициализируется каждый кнопки внутри метода, но до сих пор никакого эффекта, я делаю это правильно?)

public void changeLayout(int turn) 
{ 
    FragmentManager fm  = getSupportFragmentManager(); 
    Fragment  fragment = fm.findFragmentById(R.id.fragment_container); 
    FragmentTransaction ft; 

    if(turn == 1) //if its player 1's turn 
    { 
     doneP1 = (Button) this.findViewById(R.id.doneP1); 
     resetP1 = (Button) this.findViewById(R.id.resetP1); 
     doneP1.setOnClickListener(this); 
     resetP1.setOnClickListener(this); 
     if(fragment == null) //checks wether the current framelayout contains a fragment or not 
     { 
      ft = fm.beginTransaction(); 
      ft.add(R.id.fragment_container, new PlayerTurn1()); 
      ft.commit(); 
      //setPlayer1(); 
     } 
     else 
     { 
      ft = fm.beginTransaction(); 
      ft.replace(R.id.fragment_container, new PlayerTurn1()); 
      ft.commit(); 

     } 
    } 
    else 
    { 
     doneP2 = (Button) this.findViewById(R.id.doneP2); 
     resetP2 = (Button) this.findViewById(R.id.resetP2); 
     doneP2.setOnClickListener(this); 
     resetP2.setOnClickListener(this); 
     if(fragment == null) 
     { 
      ft = fm.beginTransaction();; 
      ft.add(R.id.fragment_container, new PlayerTurn2()); 
      ft.commit(); 
      //setPlayer2(); 
     } 
     else 
     { 
      ft = fm.beginTransaction();; 
      ft.replace(R.id.fragment_container, new PlayerTurn2()); 
      ft.commit(); 
     } 
    } 
} 

и вот LogCat

03-24 07: 51: 55.954: E/AndroidRuntime (5092): вызвано: java.lang.NullPointerException 03-24 07: 51: 55.954: E/AndroidRuntime (5092): at As2.packageTK.TicTacToeGame .onCreate (TicTacToeGame.java:88)

линия 88 является:

doneP1.setOnClickListener(this); 

Я работал над этим в течение около 4 часов, и я до сих пор не может решить простую задачу. У меня голова пускает слюни, и я действительно устал, я хочу исправить это, прежде чем я пойду спать, поэтому я буду благодарен за быстрый ответ. Я очистил и перестроил свою проблему ... ничего. Кажется, все хорошо, но нет ... NPE всегда имеет бюст в.

Answer 1

Если вы получаете NPE на

doneP1.setOnClickListener(this); 

, что означает, что doneP1 равна нулю. Вы проверили только печать/loggin doneP1, doneP2, resetP1 и resetP2 (до линии 88, конечно). Например:

Log.d(TAG, "doneP1 = "+doneP1); 
Log.d(TAG, "doneP2 = "+doneP2); 
... 

Я предполагаю, что все эти компоненты равны нулю. Это означает, что вы, вероятно, загрузите неправильный макет. Или, может быть, вы загрузили правильную с самого начала, но изменить его на что-то еще в

public void changeLayout(int turn) 

EDIT:

Ваших TableRows находятся в разных раскладках?

"Here are the XML(each table rows are from separate XMLs):" 

Тогда, конечно, компоненты P1 будут нулевыми при загрузке макета P2 и наоборот.

EDIT2:

Загрузите ссылки в changeLayout.Так как у вас одновременно есть две кнопки на экране (сброс и выполнение), вам понадобятся только две ссылки (т. Е. Вам не нужны как donep1, так и donep2). А затем в макете изменения вы измените ссылку на правильную кнопку. Таким образом, вам нужна логика, чей ход был в onClick кнопок.

Answer 2

Как это выглядит, вы получаете те представления в onCreate:

doneP1 = (Button) this.findViewById(R.id.doneP1); 
    doneP2 = (Button) this.findViewById(R.id.doneP2); 
    resetP1 = (Button) this.findViewById(R.id.resetP1); 
    resetP2 = (Button) this.findViewById(R.id.resetP2); 

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

public void checkTurn() 
{ 
    if(turn == 1) 
    { 
     changeLayout(turn); 
     turn = 2; 
    } 
    else 
    { 
     changeLayout(turn); 
     turn = 1; 
    } 
} 

и запустив:

ft.replace(R.id.fragment_container, new PlayerTurn1()); 

Вы в основном теряете ссылку на эти виды. потому что их уже нет. они были заменены представлениями другого фрагмента.

EDIT:

alternatice бы установить onClick свойство в файле макета:

Установите "android:onClick="onDoneButtonOnClick"

и в своей деятельности создать метод:

public void onDoneButtonOnClick (View v) 
{ 
     // do here what every you want with this button. 
} 

В этом случае вам придется удалить реализацию onClickListener от вашей деятельности.