Overriding Object.toString
is a good approach.
However, your current implementation makes a major mistake by creating a new Move
object (see below).
To call the routine (once you fix it), do exactly what you're already doing:
jcb.engineMove(move.toString());
If toString()
should only be used for debugging (as mre says) you could implement another method named getText
that does the same thing.
IMPORTANT NOTE:
You should not be creating a new Move
object inside its toString
method.
This is a very bad idea (as mentioned by others).
Your toString
method should simply build a string and return it.
与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…