error: 'else' without 'if'

[MBP9004]

I've been trying to work this one out most common problem seems to be semi colon or overuse of curly bracket but i'm not too sure whats wrong with mine.

import java.awt.*;
import javax.swing.*;
import java.awt.event.*;

class MyFrame extends JFrame
{ 
	JButton[][] b = new JButton[3][3];
	int x, y; // clicked button
	
	private void findClickedButton(JButton[][] b, JButton b0)
	{	// sets outer x, y to the indices of where 'b0' is in 'b'
		for (int i=0; i<3; i++)
			for (int j=0; j<3; j++)
				if (b0 == b[i][j])
				{	
					x = i;   
					y = j;   
					return;
				}
	}
									
	public MyFrame()
	{
		setTitle("Test Event Handling 3");
		setSize(300, 300);
		setLocation(100, 100);
		
		Container c = getContentPane();		
		c.setLayout(new GridLayout(3, 3));
			int count = 0;		
		for (int i=0; i<3; i++)
			for (int j=0; j<3; j++)
			{
				b[i][j] = new JButton();
				c.add(b[i][j]);
				b[i][j].addActionListener(
					new ActionListener()
					{	public void actionPerformed(ActionEvent e)
						{
							if(count %2)
							{
							JButton b0 = (JButton) e.getSource();
							findClickedButton (b, b0); // coords placed in x, y
							b[x][y].setText("X");
								else 
									b[x][y].setText("O");
							}
						}
					});
			}
		
		setDefaultCloseOperation (JFrame.EXIT_ON_CLOSE);
		setVisible(true);

	} // MyFrame constructor

} // MyFrame

[JosAH]

							if(count %2)
							{
							JButton b0 = (JButton) e.getSource();
							findClickedButton (b, b0); // coords placed in x, y
							b[x][y].setText("X");
								else 
									b[x][y].setText("O");
							}

This part doesn't look healthy ...

kind regards,

Jos

[MBP9004]

That's the bit that's confusing me, i'm not really sure what i'm doing wrong.

[PhHein]

The else statement is within the if statement.

[MBP9004]

Got it working :)

if(count %2 == 0)
							{
							JButton b0 = (JButton) e.getSource();
							findClickedButton (b, b0); // coords placed in x, y
							b[x][y].setText("X");
							}
							else
							{
							JButton b1 = (JButton) e.getSource();
							findClickedButton (b, b1); // coords placed in x, y
							b[x][y].setText("O");
							}
							count = count + 1;

[wsaryada]

		for (int i=0; i<3; i++)
			for (int j=0; j<3; j++)
				if (b0 == b[i][j])
				{	
					x = i;   
					y = j;   
					return;
				}

I believe that this is a bad style for writing a code. It is better that we use code block for things like loops, if statements, etc, so that we clearly see the scope of our code. Writing code like this can easily introduce unwanted bugs in the program.