Eclipse Fehler besser zu verstehen!!!

LamaDingo

Neues Mitglied
Servus,

folgendes Programm habe ich erstellt:
Man hat fünf Spieler, die man erstellt und damit Namen eingibt. Dann wählt man die Rundenanzahl aus mit den Buttons. Nach dem man einen der Buttons betätigt hat, öffnet sich eine weiteres Fenster, wo sich die eingetragenen Spieler befinden und man ihr Werte sieht.

Doch Eclipse wirft einen Fehler aus mit dem ich nicht anfangen kann. Dies passiert wenn man die Rundenanzahl Buttons betätigt. Das zweite Fenster öffnet sich nicht.

Also hätte ich zwei Fragen: wie kann man die Fehler bei Eclipse besser verstehen.
Weil:
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at oberflaeche.MainWindow$6.actionPerformed(MainWindow.java:294)

etc…. verstehe ich kaum und ich weiß nicht was ich damit anfangen soll.

Und vielleicht können Sie meinen code mal ansehen und sagen, was ich da so verkehrt mache….

Danke schön 😉

MainWindow Klasse beinhaltet alle Einträge und Auswahl der Rundenanzahl.

Java:
package oberflaeche;
        public class MainWindow extends JFrame {
	public ShooterGame gameWindow;
	protected Player Player1 = new Player();
	protected Player Player2 = new Player();
	protected Player Player3 = new Player();
	protected Player Player4 = new Player();
	protected Player Player5 = new Player();
	private int counterTextfield = 1;
	private boolean buttonPressed = false;
	private int rownumber;
	private boolean optical_view = false;
	private JPanel contentPane;
	private JTextField txtPlayer1;
	private JTextField txtPlayer2;
	private JTextField txtPlayer3;
	private JTextField txtPlayer4;
	private JTextField txtPlayer5;
	private JButton threeRow;
	private JLabel lblChooseTheNumber;
	private JButton fiveRows;
	private JButton sevenRows;
	private JButton nineRows;
	private JButton btnCreate;

	/**
	 * Launch the application.
	 */
	public static void main(String[] args) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					MainWindow frame = new MainWindow();
					frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	}
	

	/**
	 * Create the frame.
	 */
	public MainWindow() {
		setTitle("Shooter");
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 318, 353);
		contentPane = new JPanel();
		contentPane.setBackground(new Color(0, 153, 51));
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		setContentPane(contentPane);
		GridBagLayout gbl_contentPane = new GridBagLayout();
		gbl_contentPane.columnWidths = new int[]{0, 0, 0, 0};
		gbl_contentPane.rowHeights = new int[]{0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0};
		gbl_contentPane.columnWeights = new double[]{0.0, 1.0, 0.0, Double.MIN_VALUE};
		gbl_contentPane.rowWeights = new double[]{0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, Double.MIN_VALUE};
		contentPane.setLayout(gbl_contentPane);
		
		JLabel lblChooseByClicking = new JLabel("Max 5 players");
		lblChooseByClicking.setHorizontalAlignment(SwingConstants.CENTER);
		lblChooseByClicking.setForeground(new Color(0, 0, 51));
		lblChooseByClicking.setFont(new Font("Comic Sans MS", Font.BOLD, 13));
		GridBagConstraints gbc_lblChooseByClicking = new GridBagConstraints();
		gbc_lblChooseByClicking.insets = new Insets(0, 0, 5, 5);
		gbc_lblChooseByClicking.gridx = 1;
		gbc_lblChooseByClicking.gridy = 0;
		contentPane.add(lblChooseByClicking, gbc_lblChooseByClicking);
		
		txtPlayer1 = new JTextField();
		GridBagConstraints gbc_txtPlayer1 = new GridBagConstraints();
		gbc_txtPlayer1.insets = new Insets(0, 0, 5, 5);
		gbc_txtPlayer1.fill = GridBagConstraints.HORIZONTAL;
		gbc_txtPlayer1.gridx = 1;
		gbc_txtPlayer1.gridy = 1;
		contentPane.add(txtPlayer1, gbc_txtPlayer1);
		txtPlayer1.setColumns(10);
		
		
		txtPlayer2 = new JTextField();
		GridBagConstraints gbc_txtPlayer2 = new GridBagConstraints();
		gbc_txtPlayer2.insets = new Insets(0, 0, 5, 5);
		gbc_txtPlayer2.fill = GridBagConstraints.HORIZONTAL;
		gbc_txtPlayer2.gridx = 1;
		gbc_txtPlayer2.gridy = 2;
		contentPane.add(txtPlayer2, gbc_txtPlayer2);
		txtPlayer2.setColumns(10);
		txtPlayer2.setVisible(false);
		
		txtPlayer3 = new JTextField();
		GridBagConstraints gbc_txtPlayer3 = new GridBagConstraints();
		gbc_txtPlayer3.insets = new Insets(0, 0, 5, 5);
		gbc_txtPlayer3.fill = GridBagConstraints.HORIZONTAL;
		gbc_txtPlayer3.gridx = 1;
		gbc_txtPlayer3.gridy = 3;
		contentPane.add(txtPlayer3, gbc_txtPlayer3);
		txtPlayer3.setColumns(10);
		txtPlayer3.setVisible(false);
		
		txtPlayer4 = new JTextField();
		GridBagConstraints gbc_txtPlayer4 = new GridBagConstraints();
		gbc_txtPlayer4.insets = new Insets(0, 0, 5, 5);
		gbc_txtPlayer4.fill = GridBagConstraints.HORIZONTAL;
		gbc_txtPlayer4.gridx = 1;
		gbc_txtPlayer4.gridy = 4;
		contentPane.add(txtPlayer4, gbc_txtPlayer4);
		txtPlayer4.setColumns(10);
		txtPlayer4.setVisible(false);
		
		txtPlayer5 = new JTextField();
		GridBagConstraints gbc_txtPlayer5 = new GridBagConstraints();
		gbc_txtPlayer5.insets = new Insets(0, 0, 5, 5);
		gbc_txtPlayer5.fill = GridBagConstraints.HORIZONTAL;
		gbc_txtPlayer5.gridx = 1;
		gbc_txtPlayer5.gridy = 5;
		contentPane.add(txtPlayer5, gbc_txtPlayer5);
		txtPlayer5.setColumns(10);
		txtPlayer5.setVisible(false);
		
		//anfangs die Felder unsichtbar.
		
		final JButton plus = new JButton("+");
		plus.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
				
				if (buttonPressed == true)
				{
					counterTextfield++;
					buttonPressed = false;
					if (counterTextfield == 2)
					{
						txtPlayer2.setVisible(true);
						contentPane.revalidate();
					}
					if (counterTextfield == 3)
					{
						txtPlayer3.setVisible(true);
						contentPane.revalidate();
					}
					if (counterTextfield == 4)
					{
						txtPlayer4.setVisible(true);
						contentPane.revalidate();
					}
					if (counterTextfield == 5)
					{
						txtPlayer5.setVisible(true);
						contentPane.revalidate();
					}
					if (counterTextfield >= 5)
					{
						plus.setVisible(false);
						contentPane.revalidate();
					}
				}
				
			}
		});
		
		plus.setFont(new Font("Comic Sans MS", Font.PLAIN, 11));
		GridBagConstraints gbc_plus = new GridBagConstraints();
		gbc_plus.fill = GridBagConstraints.HORIZONTAL;
		gbc_plus.insets = new Insets(0, 0, 5, 0);
		gbc_plus.gridx = 2;
		gbc_plus.gridy = 1;
		contentPane.add(plus, gbc_plus);
		
		btnCreate = new JButton("Create");
		btnCreate.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
				
				buttonPressed=true;
				
				if (counterTextfield == 1)
				{

					Player1.setName(txtPlayer1.getText());
					Player1.setExist(true);
				}
				if (counterTextfield == 2)
				{
					Player2.setName(txtPlayer2.getText());
					Player2.setExist(true);
				}
				if (counterTextfield == 3)
				{
					Player3.setName(txtPlayer3.getText());
					Player3.setExist(true);
				}
				if (counterTextfield == 4)
				{
					Player4.setName(txtPlayer4.getText());
					Player4.setExist(true);
				}
				if (counterTextfield == 5)
				{
					Player5.setName(txtPlayer5.getText());
					Player5.setExist(true);
				}
				
				if (counterTextfield >= 5)
				{
					btnCreate.setVisible(false);
					contentPane.revalidate();
				}
					
			}
		});
		btnCreate.setFont(new Font("Comic Sans MS", Font.PLAIN, 11));
		GridBagConstraints gbc_btnCreate = new GridBagConstraints();
		gbc_btnCreate.fill = GridBagConstraints.HORIZONTAL;
		gbc_btnCreate.insets = new Insets(0, 0, 5, 0);
		gbc_btnCreate.gridx = 2;
		gbc_btnCreate.gridy = 3;
		contentPane.add(btnCreate, gbc_btnCreate);
		
		JButton btnDone = new JButton("Done");
		btnDone.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
				
				if(buttonPressed == true || counterTextfield > 1)
				{
				btnCreate.setVisible(false);
				plus.setVisible(false);
				contentPane.revalidate();
				}
			}
		});
		btnDone.setFont(new Font("Comic Sans MS", Font.PLAIN, 11));
		GridBagConstraints gbc_btnDone = new GridBagConstraints();
		gbc_btnDone.fill = GridBagConstraints.HORIZONTAL;
		gbc_btnDone.insets = new Insets(0, 0, 5, 0);
		gbc_btnDone.gridx = 2;
		gbc_btnDone.gridy = 5;
		contentPane.add(btnDone, gbc_btnDone);
		
		lblChooseTheNumber = new JLabel("Choose the number of rows:");
		lblChooseTheNumber.setForeground(new Color(0, 0, 51));
		lblChooseTheNumber.setFont(new Font("Comic Sans MS", Font.BOLD, 13));
		GridBagConstraints gbc_lblChooseTheNumber = new GridBagConstraints();
		gbc_lblChooseTheNumber.insets = new Insets(0, 0, 5, 5);
		gbc_lblChooseTheNumber.gridx = 1;
		gbc_lblChooseTheNumber.gridy = 6;
		contentPane.add(lblChooseTheNumber, gbc_lblChooseTheNumber);
		
		threeRow = new JButton("3 Rows");
		threeRow.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
				gameWindow.setVisible(true);
				rownumber = 3;
				
			}
		});
		threeRow.setFont(new Font("Comic Sans MS", Font.PLAIN, 11));
		GridBagConstraints gbc_threeRow = new GridBagConstraints();
		gbc_threeRow.anchor = GridBagConstraints.WEST;
		gbc_threeRow.insets = new Insets(0, 0, 5, 5);
		gbc_threeRow.gridx = 1;
		gbc_threeRow.gridy = 7;
		contentPane.add(threeRow, gbc_threeRow);
		
		fiveRows = new JButton("5 Rows");
		fiveRows.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				gameWindow.setVisible(true);
				rownumber = 5;
			}
		});
		fiveRows.setFont(new Font("Comic Sans MS", Font.PLAIN, 11));
		GridBagConstraints gbc_fiveRows = new GridBagConstraints();
		gbc_fiveRows.anchor = GridBagConstraints.EAST;
		gbc_fiveRows.insets = new Insets(0, 0, 5, 5);
		gbc_fiveRows.gridx = 1;
		gbc_fiveRows.gridy = 8;
		contentPane.add(fiveRows, gbc_fiveRows);
		
		sevenRows = new JButton("7 Rows");
		sevenRows.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				gameWindow.setVisible(true);
				rownumber = 7;
			}
		});
		sevenRows.setFont(new Font("Comic Sans MS", Font.PLAIN, 11));
		GridBagConstraints gbc_sevenRows = new GridBagConstraints();
		gbc_sevenRows.anchor = GridBagConstraints.WEST;
		gbc_sevenRows.insets = new Insets(0, 0, 5, 5);
		gbc_sevenRows.gridx = 1;
		gbc_sevenRows.gridy = 9;
		contentPane.add(sevenRows, gbc_sevenRows);
		
		nineRows = new JButton("9 Rows");
		nineRows.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				rownumber = 9;
				gameWindow.setVisible(true);
			}
		});
		nineRows.setFont(new Font("Comic Sans MS", Font.PLAIN, 11));
		GridBagConstraints gbc_nineRows = new GridBagConstraints();
		gbc_nineRows.anchor = GridBagConstraints.EAST;
		gbc_nineRows.insets = new Insets(0, 0, 0, 5);
		gbc_nineRows.gridx = 1;
		gbc_nineRows.gridy = 10;
		contentPane.add(nineRows, gbc_nineRows);
	}

	protected int getCounterTextfield()
	{
		return counterTextfield;
	}
	
	protected int getRowNumber()
	{
		return rownumber;
	}
}

ShooterGame beinhaltet die Einträge und gibt diese aus.

Java:
package oberflaeche;

import java.util.*;

public class ShooterGame extends JFrame {
	public MainWindow mwindow = new MainWindow();
	private int personCounter = 1;
	private int rowsToPlay = 0;
	private int rowsToCheck = 0;
	private int playerList = mwindow.getCounterTextfield();
	String showNumber = "";
	private Random rn = new Random();
	private int randomNumber  = 0;
	private int buttonCounter = 0;
	private JPanel contentPane;
	private int threeHitsCounter = 0;

	/**
	 * Launch the application.
	 */
	public static void main(String[] args) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					ShooterGame frame = new ShooterGame();
					frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	}

	/**
	 * Create the frame.
	 */
	public ShooterGame() {
		rowsToPlay = mwindow.getRowNumber();
		
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 284, 426);
		contentPane = new JPanel();
		contentPane.setBackground(Color.RED);
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		setContentPane(contentPane);
		contentPane.setLayout(null);
		
		JLabel Playerlist = new JLabel("Playerlist:");
		Playerlist.setFont(new Font("Comic Sans MS", Font.BOLD | Font.ITALIC, 14));
		Playerlist.setBounds(24, 11, 145, 28);
		contentPane.add(Playerlist);
		
		JLabel Player1_ = new JLabel(mwindow.Player1.getName());
		Player1_.setFont(new Font("Comic Sans MS", Font.BOLD, 12));
		Player1_.setBounds(24, 43, 145, 28);
		contentPane.add(Player1_);		
		
		JLabel Player2_ = new JLabel(mwindow.Player2.getName());
		Player2_.setFont(new Font("Comic Sans MS", Font.BOLD, 12));
		Player2_.setBounds(24, 82, 145, 28);
		contentPane.add(Player2_);
		if(mwindow.Player2.getExist() == true)
		{
			Player2_.setVisible(true);
		}
		else
		{
			Player2_.setVisible(false);
		}
		
		JLabel Player3_ = new JLabel(mwindow.Player3.getName());
		Player3_.setFont(new Font("Comic Sans MS", Font.BOLD, 12));
		Player3_.setBounds(24, 121, 145, 28);
		contentPane.add(Player3_);
		
		JLabel Player4_ = new JLabel(mwindow.Player4.getName());
		Player4_.setFont(new Font("Comic Sans MS", Font.BOLD, 12));
		Player4_.setBounds(24, 160, 145, 28);
		contentPane.add(Player4_);
		
		JLabel Player5_ = new JLabel(mwindow.Player5.getName());
		Player5_.setFont(new Font("Comic Sans MS", Font.BOLD, 12));
		Player5_.setBounds(24, 199, 145, 28);
		contentPane.add(Player5_);
		
		JLabel lblScore = new JLabel("Score:");
		lblScore.setFont(new Font("Comic Sans MS", Font.BOLD | Font.ITALIC, 14));
		lblScore.setBounds(176, 11, 82, 28);
		contentPane.add(lblScore);
		
		final JLabel ScorePlayer1 = new JLabel(Integer.toString(mwindow.Player1.getHit()));
		ScorePlayer1.setFont(new Font("Comic Sans MS", Font.BOLD, 12));
		ScorePlayer1.setBounds(179, 50, 68, 14);
		contentPane.add(ScorePlayer1);
		
		final JLabel ScorePlayer2 = new JLabel(Integer.toString(mwindow.Player2.getHit()));
		ScorePlayer2.setFont(new Font("Comic Sans MS", Font.BOLD, 12));
		ScorePlayer2.setBounds(179, 89, 68, 14);
		contentPane.add(ScorePlayer2);
		
		final JLabel ScorePlayer3 = new JLabel(Integer.toString(mwindow.Player3.getHit()));
		ScorePlayer3.setFont(new Font("Comic Sans MS", Font.BOLD, 12));
		ScorePlayer3.setBounds(179, 128, 68, 14);
		contentPane.add(ScorePlayer3);
		
		final JLabel ScorePlayer4 = new JLabel(Integer.toString(mwindow.Player4.getHit()));
		ScorePlayer4.setFont(new Font("Comic Sans MS", Font.BOLD, 12));
		ScorePlayer4.setBounds(179, 167, 68, 14);
		contentPane.add(ScorePlayer4);
		
		final JLabel ScorePlayer5 = new JLabel(Integer.toString(mwindow.Player5.getHit()));
		ScorePlayer5.setFont(new Font("Comic Sans MS", Font.BOLD, 12));
		ScorePlayer5.setBounds(179, 206, 68, 14);
		contentPane.add(ScorePlayer5);
		
		numbersgenerator();
		if(randomNumber <= 20)
		{
			showNumber = Integer.toString(randomNumber);
		}
		else
		{
			showNumber = "BULLSEYE";
		}
		
		JLabel ThrowNumber = new JLabel(showNumber);
		ThrowNumber.setFont(new Font("Monotype Corsiva", Font.BOLD, 21));
		ThrowNumber.setHorizontalAlignment(SwingConstants.CENTER);
		ThrowNumber.setBounds(24, 238, 223, 70);
		contentPane.add(ThrowNumber);
		
		JButton btnSingle = new JButton("Single");
		btnSingle.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
				if (rowsToCheck <= rowsToPlay )	
				{
				if (personCounter == 1)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player1.setHit(mwindow.Player1.getHit()+1);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter ++;
						rowsToCheck ++;
					}
				}
				
				if (personCounter == 2)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player2.setHit(mwindow.Player2.getHit()+1);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter ++;
					}
				}
				
				if (personCounter == 3)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player3.setHit(mwindow.Player3.getHit()+1);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter ++;
					}
				}
				
				if (personCounter == 4)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player4.setHit(mwindow.Player4.getHit()+1);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter ++;
					}
				}
				
				if (personCounter == 5)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player5.setHit(mwindow.Player5.getHit()+1);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter = 0 ;
					}
				}
			}
			}
		});
		btnSingle.setBounds(10, 319, 75, 23);
		contentPane.add(btnSingle);
		
		JButton btnDouble = new JButton("Double");
		btnDouble.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				if (rowsToCheck <= rowsToPlay )
				{
				if (personCounter == 1)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player1.setHit(mwindow.Player1.getHit()+2);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter ++;
						rowsToCheck ++;
					}
				}
				
				if (personCounter == 2)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player2.setHit(mwindow.Player2.getHit()+2);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter ++;
					}
				}
				
				if (personCounter == 3)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player3.setHit(mwindow.Player3.getHit()+2);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter ++;
					}
				}
				
				if (personCounter == 4)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player4.setHit(mwindow.Player4.getHit()+2);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter ++;
					}
				}
				
				if (personCounter == 5)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player5.setHit(mwindow.Player5.getHit()+2);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter = 0 ;
					}
				}
			}
			}
		});
		btnDouble.setBounds(94, 319, 75, 23);
		contentPane.add(btnDouble);
		
		JButton btnTriple = new JButton("Triple");
		btnTriple.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
			if (rowsToCheck <= rowsToPlay )	
			{
				if (personCounter == 1)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player1.setHit(mwindow.Player1.getHit()+3);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter ++;
						rowsToCheck ++;
					}
				}
				
				if (personCounter == 2)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player2.setHit(mwindow.Player2.getHit()+3);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter ++;
					}
				}
				
				if (personCounter == 3)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player3.setHit(mwindow.Player3.getHit()+3);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter ++;
					}
				}
				
				if (personCounter == 4)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player4.setHit(mwindow.Player4.getHit()+3);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter ++;
					}
				}
				
				if (personCounter == 5)
				{
					if(threeHitsCounter < 2)
					{
						mwindow.Player5.setHit(mwindow.Player5.getHit()+3);
						threeHitsCounter++;
					}
					if (threeHitsCounter ==3)
					{
						personCounter = 0 ;
					}
				}
			}
		}
		});
		btnTriple.setBounds(179, 319, 79, 23);
		contentPane.add(btnTriple);
		
		JButton btnNoHit = new JButton("No Hit!");
		btnNoHit.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
			if (rowsToCheck <= rowsToPlay )
			{
				if (personCounter == 1)
				{
					ScorePlayer1.setText(Integer.toString(mwindow.Player1.getHit()));
					threeHitsCounter = +3;
					rowsToCheck ++;
				}
				
				if (personCounter == 2)
				{
					ScorePlayer2.setText(Integer.toString(mwindow.Player2.getHit()));
					threeHitsCounter = +3;
				}

				if (personCounter == 3)
				{
					ScorePlayer3.setText(Integer.toString(mwindow.Player3.getHit()));
					threeHitsCounter = +3;
				}
				
				if (personCounter == 4)
				{
					ScorePlayer4.setText(Integer.toString(mwindow.Player4.getHit()));
					threeHitsCounter = +3;
				}
				
				if (personCounter == 5)
				{
					ScorePlayer5.setText(Integer.toString(mwindow.Player5.getHit()));
					threeHitsCounter = +3;
				}
			}
		}
		});
		btnNoHit.setBounds(94, 353, 75, 23);
		contentPane.add(btnNoHit);
		
	}

	public int numbersgenerator()
	{
		randomNumber  =  rn.nextInt()%21+1;
		if(randomNumber<0)
		{
			randomNumber = randomNumber * (-1);
		}
		
			return  randomNumber;
	}
}

Einzeln funktionieren die beiden Fenster, doch MainWindow kann ShooterGame nicht aufrufen:
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at oberflaeche.MainWindow$6.actionPerformed(MainWindow.java:294)....

Sorry für den langen Code und falls ich doch was falsch verfasst habe.

Vielen Dank für euere Hilfe, ich komme ohne euren Support nicht weiter.

Gruß LamaDingo
 
In den ganzen Zeilen, die mit at beginnen steht der sog. StackTrace. In ihm ist die Aufrufhierarchie und die Zeile, in der die Exception geworfen wurde angezeigt. Such mal in Zeile MainWindow 294. Lass dir mal die Zustände der Variablen ausgeben, debugg mal mit dem Debugger.
 
Servus Gucky,

danke für die Antwort.
Ich verstehe nicht ganz, welche Varialen du meinst?
Diese vielleicht:

protected Player Player1 = new Player();
protected Player Player2 = new Player();
protected Player Player3 = new Player();
protected Player Player4 = new Player();
protected Player Player5 = new Player();
private int counterTextfield = 1;
private boolean buttonPressed = false;
private int rownumber;
private boolean optical_view = false;
private JPanel contentPane;
private JTextField txtPlayer1;
private JTextField txtPlayer2;
private JTextField txtPlayer3;
private JTextField txtPlayer4;
private JTextField txtPlayer5;

Ich weiß wo der Fehler passiert, es sind auch 4 Fehler, da vier Buttons. Zeile 294 verweist auf:
gameWindow.setVisible(true);
Und das bringt mir nichts, da ich hier das andere Fensteraufrufen will, was aber nicht klappt.

Debugger verläuft bei mir gleich wie Run, bzw. was sollte da anders sein?

Vielen Dank für deine Hilfe

Gruß LamaDingo 😉
 
In deinem geposteten Code verweist Zeile 294 zwar nicht auf gameWindow.setVisible(true);, aber dafür Zeile 297. Offenbar hast du die Variable gameWindow in der Klasse MainWindow nicht initialisiert.
Ich finde es gelinde gsagt merkwürdig, daß die Klasse MainWindow eine Instanzvariable vom Typ ShooterGame hat und die Klasse ShooterGame eine Instanzvariable vom Typ MainWindow. Wenn diese Instanzvariablen beim Erzeugen der Objekte auch noch neu angelegt statt übergeben werden, ist der Stackoverflowe quasi vorprogrammiert.
 
@TE: es ist nicht zweckmässig, die weitere Diskussion als PN laufen zu lassen.
Wenn Du Variablen aus deiner MainWindow-Klasse an deine ShooterGame Klasse übertragen willst, verwende einen setter. Es bringt absolut gar nüscht, in einer Instanz von MainWindow Parameter zu setzen und dann zu versuchen, sie aus einer anderen Instanz zu lesen.
Zeile 38 in deiner Shootergame Klasse (rowsToPlay = mwindow.getRowNumber()😉 macht wahrscheinlich nicht, was du erwartest, weil sie Daten aus einer MainWindow-Instanz holt, die nie sichtbar gemacht wurde.

Was mir beim Drüberfliegen noch aufgefallen ist: Die Logik in den anonymen ActionListenern, die du an die JButtons btnSingle, btnDouble und btnTriple gehängt hast, scheinen alle was ähnliches zu machen. Du solltest den gemeinsamen Teil der Logik in eine separate Klasse auslagern. Damit könntest du bestimmt ein paar der momentan 250 Zeilen sparen für die ActionListener-Logik sparen.
Außerdem ist in den ActionListenern die Logik in den Blöcken, die mit if (personCounter == anfangen, weitestgehend identisch. Für personCounter wird rowsToPLay incrementiert, für die anderen personCounter nicht. Ausserdem wird abhängig vom personCounter ein anderes ScorePLayer-Textfeld mit Inhalt gefüllt und dazu der HitCount aus einem Player-Objekt beschafft.
Für den ActionListener am plus-JButton in der Klasse MainWindow gilt ähnliches. Statt der Blöcke:
Java:
if (counterTextfield == 1){
    txtPlayer1.setVisible(true);
    contentPane.revalidate();
}
if (counterTextfield == 2){
    txtPlayer2.setVisible(true);
    contentPane.revalidate();
}
//usw....
tuts es nach Anlegen von ein par Arrays auch
Java:
    txtPlayer[countertextField-1].setVisible(true);
    contentPane.revalidate();
}
Und aus
Java:
if (counterTextfield == 1){
    Player1.setName(txtPlayer1.getText());
    Player1.setExist(true);
}
if (counterTextfield == 2){
    Player2.setName(txtPlayer1.getText());
    Player2.setExist(true);
}
//usw....
wird
Java:
    players[countertextField-1].setName(txtPlayer[countertextField-1].getText());
    players[countertextField-1].setExist(true);
}


Ich denke du solltest noch ein paar Basics lernen, und dann dein Design überarbeiten.
 
Zuletzt bearbeitet:

Neue Themen


Zurück
Oben