-
-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
503 abalone #184
503 abalone #184
Conversation
Dev to master SG 153 othello pass replay fix
Dev -> Master: SG 154 Backammon Undos
f0d6c4d
to
c69a7d3
Compare
@@ -61,16 +61,15 @@ object Board { | |||
|
|||
} | |||
|
|||
// TODO Abalone Set | |||
// an Hexagon of width 5 fits in a square of width 9 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might be good to have some extra parameter that indicates this isn't a regular 9x9
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure what you suggested for that one, can you give me an example ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At the moment, all of our boards are regular square/rectangular. So when we say a board is of say Width 9 and Height 2, this means that all of the rows are of width 9.
This will remain true for a hexagonal board of the shape that is needed for the game Hex.
However, this won't be true for Abalone as each rows width is variable. So I was suggesting that we add an extra parameter (a boolean) to indicate this difference in shape. Not sure what to call it, but it would be false for all current boards and true for the Abalone board size.
I can't think of an obvious place where this new parameter must be used, but it could well be useful
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let me know if you expect me to do it differently and if I should already consider applying the change to other Board files
Can't we just define something like "if this variable exists then the Board does not have a regular square size" ?
Would avoid modifying other Board.scala files 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if this variable exists then the Board does not have a regular square size
Yes exactly this.
I would make it a parameter of the BoardSize class. You could default it true as an input.
This would probably mean you need to edit the constructor of all BoardSize classes in other game logics (and wrapper layer) but thats fine. As its a default parameter it shouldn't require any other code changes.
also starts the work on valid moves generation
feat: FEN notation
No description provided.