Is there a better option? Due to this, I get a lot of NullPointerExceptions while debugging? Is this a design flaw or is this something I have to deal with? Is there any way to improve on these?Ģ) I have learned a convention of assigning a null value to all objects if they are empty/not present (If a square does not contain a piece, then null is assigned). Is there any way I can clean it up ( divide it up maybe)? Something that bugs me is the implementation of isValidCastling() function and move() function. ![]() I have got some specific questions about the implementation that I would love to have answeredġ) The BoardManager.java class seems to be pretty large. The GUI is not included here but if you would like to look at it then here is my repository. I thought it would be hard to review my code without the basic classes. I hope I have not included a lot of code here. Idioms, conventions, anything that comes to your mind. How the same implementation can be better written.
0 Comments
Leave a Reply. |