When I button click, I want to move in a range where I set up

Question

This code is intended to move the image shown in the screen shot.

What I am trying to do is when I click the button, it is supposed to move like

the knight in chess. It does not move at all showing the following error.

What is the problem here? and I uploaded my code.

Thank you for your help and consider

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




public class BoardClass extends JFrame{


    private Container contents;

    private JButton[][] squares = new JButton[8][8];
    private Color colorBlack = Color.BLACK;
    private Color colorOrange = Color.ORANGE;

    private int sheeprow=7;
    private int sheepcol=0;

    private int woolfrow1=0;
    private int woolfcol1=1;
    private int woolfrow2=0;
    private int woolfcol2=3;
    private int woolfrow3=0;
    private int woolfcol3=5;
    private int woolfrow4=0;
    private int woolfcol4=7;

    private ImageIcon sheep = new ImageIcon("sheep.PNG");
    private ImageIcon wolf = new ImageIcon("wolf.PNG");

    public BoardClass()
    {
        super("Sheep and wolf game"); 

        contents = getContentPane();
        contents.setLayout(new GridLayout(8, 8));

        ButtonHandler buttonhandler = new ButtonHandler();

        for(int i=0;i<8;i++)
        {
            for(int j=0;j<8;j++)
            {


                squares[i][j] = new JButton();
                if((i+j)%2!=0) // it allows to show black blocks
                {
                    squares[i][j].setBackground(colorBlack);
                }
                else
                {
                    squares[i][j].setBackgroun(colorOrange);
                }
                contents.add(squares[i][j]);
                squares[i][j].addActionListener(buttonhandler);
            }
        }
        squares[sheeprow][sheepcol].setIcon(sheep);
        /*squares[woolfrow1][woolfcol1].setIcon(wolf);
        squares[woolfrow2][woolfcol2].setIcon(wolf);
        squares[woolfrow3][woolfcol3].setIcon(wolf);
        squares[woolfrow4][woolfcol4].setIcon(wolf);*/

        setSize(500,500);
        setResizable(false);
        setLocationRelativeTo(null); //centers window
        setVisible(true);

    }


    private boolean isValidMove(int i, int j)
    {
        int rowDelta = Math.abs(i-sheeprow);
        int colDelta = Math.abs(j-sheepcol);

        if((rowDelta==1) && (colDelta==2))
        {
            return true;
        }
        if((colDelta==1)&&(rowDelta==2))
        {
            return true;
        }
        return false;
    }

    private void processClick(int i, int j)
    {
        if(isValidMove(i,j)==false)
        {
            return;
        }
        squares[sheeprow][sheepcol].setIcon(null);
        squares[i][j].setIcon(sheep);
        sheeprow = i;
        sheepcol = j;
    }

private class ButtonHandler implements ActionListener
{
    public void actionPerformed(ActionEvent e) 
    {
        Object source = e.getSource();

        for(int i=0;i<8;i++)
        {
            for(int j=0;i<8;j++)
            {

                if(source==squares[i][j])-- this part showing some error
                {
                    processClick(i,j);
                    return;
                }
            }
        }

    }
}

    public static void main(String[] args) {
        // TODO Auto-generated method stub

        new BoardClass();
    }
}

You'll want to post [mcve] code as code-formatted text *with* your question. — DontKnowMuchBut Getting Better, Nov 26 '17 at 22:00

score 0 · Accepted Answer · answered Nov 26 '17 at 22:25

0

In actionPerformed you need to change the line

for(int j=0;i<8;j++)

to

for(int j=0;j<8;j++)

answered Nov 26 '17 at 22:25

yacc

2,915
4
19
33

When I button click, I want to move in a range where I set up

1 Answers1