//New method public void EnabledButtons() { this.btnPosition2.setEnabled(false); this.btnPosition1.setEnabled(false); this.btnPosition3.setEnabled(false); this.btnPosition6.setEnabled(false); this.btnPosition5.setEnabled(false); this.btnPosition4.setEnabled(false); this.btnPosition7.setEnabled(false); this.btnPosition8.setEnabled(false); this.btnPosition9.

setEnabled(false); } ______________________________________________________________________ // New method public void joueurActif() { if (jugador == "X") { jugador = "0"; } else { jugador = "X"; } } ______________________________________________________________________

//Methodo refactorizado public boolean ganador() { if ((btnPosition1.getText() == btnPosition2.getText() ) && (btnPosition2.getText() == btnPosition3.getText()) && (btnPosition3.getText() != "")) { EnabledButtons(); JOptionPane.showMessageDialog(null,"JUGADOR :" + jugador + " a ganado"); joueurActif(); return true; } if ((btnPosition4.getText() == btnPosition5.getText() ) && (btnPosition5.getText() == btnPosition6.getText()) && (btnPosition6.getText() != "")) { EnabledButtons(); JOptionPane.showMessageDialog(null,"JUGADOR :" + jugador + " a ganado"); joueurActif(); return true; } if ((btnPosition7.getText() == btnPosition8.getText() ) && (btnPosition8.getText() == btnPosition9.getText()) && (btnPosition9.getText() != ""))

{ EnabledButtons(); JOptionPane.showMessageDialog(null,"JUGADOR :" + jugador + " a ganado"); joueurActif(); return true; } if ((btnPosition1.getText() == btnPosition4.getText() ) && (btnPosition4.getText() == btnPosition7.getText()) && (btnPosition7.getText() != "")) { EnabledButtons(); JOptionPane.showMessageDialog(null,"JUGADOR :" + jugador + " a ganado"); joueurActif(); return true; } if ((btnPosition2.getText() == btnPosition5.getText() ) && (btnPosition5.getText() == btnPosition8.getText()) && (btnPosition8.getText() != "")) { EnabledButtons(); JOptionPane.showMessageDialog(null,"JUGADOR :" + jugador + " a ganado"); joueurActif(); return true; } if ((btnPosition3.getText() == btnPosition6.getText() ) && (btnPosition6.getText() == btnPosition9.getText()) && (btnPosition9.getText() != "")) { EnabledButtons(); JOptionPane.showMessageDialog(null,"JUGADOR :" + jugador + " a ganado"); joueurActif(); return true; } if ((btnPosition1.getText() == btnPosition5.getText() ) && (btnPosition5.getText() == btnPosition9.getText()) && (btnPosition9.getText() != "")) { EnabledButtons(); JOptionPane.showMessageDialog(null,"JUGADOR :" + jugador + " a ganado"); joueurActif(); return true; } if ((btnPosition3.getText() == btnPosition5.getText() ) && (btnPosition5.getText() == btnPosition7.getText()) && (btnPosition7.getText() != "")) { EnabledButtons(); JOptionPane.showMessageDialog(null,"JUGADOR :" + jugador + " a ganado"); joueurActif(); return true; }

return false; } _____________________________________________________________________ //Methodo refactorizado private void btnPosition1ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_btnPosition1ActionPerformed btnPosition1.setText( jugador); if(!ganador()); { joueurActif(); btnPosition1.setEnabled( false); JOptionPane.showMessageDialog(null,"Es el turno del jugador : " + jugador); } }//GEN-LAST:event_btnPosition1ActionPerformed _____________________________________________________________________ //Methodo refactorizado private void btnPosition2ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_btnPosition2ActionPerformed btnPosition2.setText(jugador); if(!ganador()); { joueurActif(); btnPosition2.setEnabled(false); JOptionPane.showMessageDialog(null,"Es el turno del jugador : " + jugador); } }//GEN-LAST:event_btnPosition2ActionPerformed _____________________________________________________________________ //Methodo refactorizado private void btnPosition3ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_btnPosition3ActionPerformed btnPosition3.setText(jugador); if(!ganador()); { joueurActif(); btnPosition3.setEnabled(false); JOptionPane.showMessageDialog(null,"Es el turno del jugador : " + jugador); } }//GEN-LAST:event_btnPosition3ActionPerformed _____________________________________________________________________ //Methodo refactorizado private void btnPosition4ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_btnPosition4ActionPerformed btnPosition4.setText(jugador);

if(!ganador()); { joueurActif(); btnPosition4.setEnabled(false); JOptionPane.showMessageDialog(null,"Es el turno del jugador : " + jugador); } }//GEN-LAST:event_btnPosition4ActionPerformed _____________________________________________________________________ //Methodo refactorizado private void btnPosition5ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_btnPosition5ActionPerformed btnPosition5.setText(jugador); if(!ganador()); { joueurActif(); btnPosition5.setEnabled(false); JOptionPane.showMessageDialog(null,"Es el turno del jugador : " + jugador); } }//GEN-LAST:event_btnPosition5ActionPerformed _____________________________________________________________________ //Methodo refactorizado private void btnPosition6ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_btnPosition6ActionPerformed btnPosition6.setText(jugador); if(!ganador()); { joueurActif(); btnPosition6.setEnabled(false); JOptionPane.showMessageDialog(null,"Es el turno del jugador : " + jugador); } }//GEN-LAST:event_btnPosition6ActionPerformed _____________________________________________________________________ //Methodo refactorizado private void btnPosition7ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_btnPosition7ActionPerformed btnPosition7.setText(jugador); if(!ganador()); { joueurActif(); btnPosition7.setEnabled(false); JOptionPane.showMessageDialog(null,"Es el turno del jugador : " + jugador); } }//GEN-LAST:event_btnPosition7ActionPerformed _____________________________________________________________________ //Methodo refactorizado

private void btnPosition8ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_btnPosition8ActionPerformed btnPosition8.setText(jugador); if(!ganador()); { joueurActif(); btnPosition8.setEnabled(false); JOptionPane.showMessageDialog(null,"Es el turno del jugador : " + jugador); } }//GEN-LAST:event_btnPosition8ActionPerformed _____________________________________________________________________ //Methodo refactorizado private void btnPosition9ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_btnPosition9ActionPerformed btnPosition9.setText(jugador); if(!ganador()); { joueurActif(); btnPosition9.setEnabled(false); JOptionPane.showMessageDialog(null,"Es el turno del jugador : " + jugador); } }//GEN-LAST:event_btnPosition9ActionPerformed

Check style

Diagrama de clases

Metrics