Getter/Setter NullPointer Exception

[ZxSpectrum]

Hallo,

ich habe ein kleines Problem bei meinem Programm und stehe völlig auf dem Schlauch...

Ich habe drei Klassen:
- Player
- PlayerPanelSingle
- Game

Nun sieht es so aus, dass ich in PlayerPanelSingle einen JTextField habe, aus den ich im Game einen String rausholen möchte und diesen String anschließend im Game für den Konstruktor von Player verwenden. Wenn ich dies in PlayerPanelSingle mache funktioniert es ideal, sobald ich das ganze zu der Klasse Game verlagern will, bekomme ich sofort eine NullPointer Exception. Kann mir vielleicht jemand von euch einen Tipp geben, wo mein Denkfehler liegt?

Dies ist das relevante von der Klasse "Player"

private String name;
private char symbol;
private boolean current;

public String getName(){
        return name;
    }//getName

    public void setName(String insertName){
        name = insertName;
    }//setName

So hole ich die Variable direkt im PlayerPanelSingle raus, hier funktioniert´s noch

Player player = new Player();
setTmp(playerName.getText()); //Text aus dem JTextField wird zwischen gespeichert
player.setName(getTmp()); // Und anschließend in den SetName von Player eingeschrieben
System.out.println(player.getName()); //It works!

Komischerweise, wenn ich das ganze im Game so ausführe, kriege ich die NullPointer:

Player player = new Player();
pps.setTmp(pps.playerName.getText());
player.setName(pps.getTmp());
System.out.println(player.getName());

Kann mir jemand einen Hinweis geben, wie man das lösen kann?
Vielen Dank im Voraus!

 

[mrBrown]

Das Problem werden nicht die Getter/Setter sein. Ich würde drauf tippen, das pps oder pps.playerName null sind, dafür müsste man aber mehr Code sehen, und wissen, in welcher Zeile genau der Fehler auftritt

 

[ZxSpectrum]

Hm... Ich habe zumindest festgestellt, dass es schon bei der Übergabe von PlayerPanelSingle zu Game Mekker gibt.

Das ist meine PlayerPanelSingle Klasse:

  private JTextField nameField = new JTextField();
   private String playerName;
   private String tmp;
   
   public String getPlayerName() {
     return playerName;
   }

   public void setPlayerName(String tmp) {
     this.playerName = tmp;
   }

   public PlayerPanelSingle() {
     this.setTitle("Spieler Panel");
     this.setSize(350, 150);
     initComponents();
     setWindowToMiddle();
     this.setResizable(false);
     this.setVisible(true);
   }//Dialog

   public void initComponents(){

     this.getContentPane().setLayout(new GridBagLayout());
     GridBagConstraints c = new GridBagConstraints();

     c.fill   = GridBagConstraints.HORIZONTAL;
     c.gridx   = 0;
     c.insets= new Insets(0, 0, 0, 5);
     this.getContentPane().add(playerLabel,c);

     c.gridx = 1;
     c.ipadx = 40;
     nameField.setText("Spieler1");
     this.getContentPane().add(this.nameField,c);

     c.gridy = 2;
     checkFirst.setSelected(true);
     this.getContentPane().add(checkFirst, c);

     c.gridy = 3;
     save.addActionListener(this);
     this.getContentPane().add(save,c);
   }//initComponents

   //
   public void actionPerformed (ActionEvent ae){
     if(ae.getSource() == this.save){
       
       tmp = nameField.getText();//Text aus dem JTextField zwischenspeichern
       setPlayerName(tmp);//rein in den setter
       System.out.println(getPlayerName());//it wörks!
       
     }//save
   }//ActionPerformed

 

[mrBrown]

In der liegt der Fehler wohl auch nicht...zeig doch mal Game

 

[ZxSpectrum]

Ok, hier ist die Game.
Eine komische Sache fällt mir schon auf, dass bei der Übergabe vom Symbol kann ich diese Variable sauber reinsetzen und rausholen, das aber nicht über diesen JTextField geht...
Zeile 59-61

package spielOOP;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.AbstractAction;
import javax.swing.Action;

public class Game{

   Board b = new Board();
   Options o = new Options();
   PlayerPanelSingle pps;
   Player player;
   public String symbol;
//   private String name;

   Game(){//Konstruktor
    
     //An dieser Stelle wird das betätigen des Buttons "START" empfangen
     ListenToActions();
   }

   public void ListenToActions(){
     Action newGame = new StartGameAction( this );
     b.setReset.addActionListener(newGame);

     Action saveMode = new SaveMode(this);
     pps.save.addActionListener(saveMode);

     Action play = new PlayAction(this);
     for (int i = 0; i < b.gameField.length; i++) {
       for (int j = 0; j < b.gameField.length; j++) {
         b.gameField[i][j].addActionListener(play);
       }
     }//for
   }//Listen to Actions

   //MUSS NOCH BEARBEITET WERDEN
   public void generate(){
     for (int i = 0; i < b.gameField.length; i++) {
       for (int j = 0; j < b.gameField.length; j++) {
         this.b.gameField[i][j].setEnabled(true);
         this.b.gameField[i][j].setText(" ");
       }//for j
     }//for i
   }

   public void generatePlayer(){//Hier werden die Spieler erstellt    
     if(pps.checkFirst.isSelected()){
      
       Player player = new Player();
      
       String insertName = pps.getPlayerName();
       player.setName(insertName);
       System.out.println(player.getName());
      
//       player.setName("Spieler1");
       player.setSymbol("X");
       symbol = player.getSymbol();
       player.setCurrent(true);
       System.out.println(player.getName() + " " + player.getSymbol() + " " + player.getCurrent());
          
     }else {
       Player player = new Player();
       player.setName("Spieler1");
       player.setSymbol("O");
       symbol = player.getSymbol();
       player.setCurrent(false);
//       System.out.println(player.getName() + " " + player.getSymbol() + " " + player.getCurrent());
     }
   }//play

   public void play(){
     Player player = new Player();
     if(b.gameField[0][0].getModel().isArmed()){
       b.gameField[0][0].setText(symbol);
     }else if(b.gameField[0][1].getModel().isArmed()){
       b.gameField[0][1].setText(symbol);
     }else if(b.gameField[0][2].getModel().isArmed()){
       b.gameField[0][2].setText(symbol);
     }else if(b.gameField[1][0].getModel().isArmed()){
       b.gameField[1][0].setText(symbol);
     }else if(b.gameField[1][1].getModel().isArmed()){
       b.gameField[1][1].setText(symbol);
     }else if(b.gameField[1][2].getModel().isArmed()){
       b.gameField[1][2].setText(symbol);
     }else if(b.gameField[2][0].getModel().isArmed()){
       b.gameField[2][0].setText(symbol);
     }else if(b.gameField[2][1].getModel().isArmed()){
       b.gameField[2][1].setText(symbol);
     }else if(b.gameField[2][2].getModel().isArmed()){
       b.gameField[2][2].setText(symbol);
     }
   }//play

   //############################################## ABSTRACT ACTIONS ################################################  
   class StartGameAction extends AbstractAction{

     private Game game;
     private Board board;

     public StartGameAction( Game game){
       this.game = game;
     }

     public void actionPerformed( ActionEvent e ) {
       game.generate();
     }
   }//Start Game Action

   public class SaveMode extends AbstractAction {

     private Game game;

     public SaveMode( Game game){
       this.game = game;
     }

     public void actionPerformed( ActionEvent e ) {
       game.generatePlayer();
     }
   }//Save Mode

   public class PlayAction extends AbstractAction {

     private Game game;

     public PlayAction( Game game){
       this.game = game;
     }

     public void actionPerformed( ActionEvent e ) {
       game.play();
     }
   }//Play Action

}//Game

 

[Cromewell]

Du initialisierst pps ja auch nicht, soweit ich das sehe?!

 

[mrBrown]

Initialisierst du irgendwo pps?