Lights out internal structural design#141
Conversation
Syncing Main to MetroCS Repository
…esign Added stub methods in LightsOutGame.java
jody
left a comment
There was a problem hiding this comment.
See the inline comments.
Also remove build artifact ConsoleGameHub.jar from this PR.
| public LightsOutBoard generateSolvable(final int rows, | ||
| final int cols, final Random rng); |
There was a problem hiding this comment.
I expect this to be a compilation error. Please provide documentation of an error-free build.
There was a problem hiding this comment.
I see in your comment that you identified "checkstyle" errors here. I think that's likely to be a result of this not being correct Java, which is a precondition for meaningful usage of checkstyle.
Looking at the code, it appears that you're tyring to define an abstract class called BoardGenerator, and an abstract method within that class called generateSolvable. But there's no abstract keyword on either.
Or it could be that generateSolvable was meant to be a method stub. In that case there needs to be a body defined that at least returns the proper data type.
Closes: #89
Issues: