Creating a Chess AI. Is this bad programming?

Creating the human interface.

So I need to know when the user finally clicks somewhere to move a piece.

So the GUI class has this interface that detects mouse clicks and I just made the snippet executed at every click point to an array in the main program (pointer initiated in constructor) and change coordinates. Then I have a very small snippet in main program that loops until a change in coordinates is made.

Is the bad/good programming?

And different suggestions?

You shouldn't be writing a GUI. You should be writing your chess engine to use the Chess Engine Communication Protocol, then using one of the many free chess GUIs that already exist.