antoinejt / js-tic-tac-toe Goto Github PK

• Put source files into a src/ folder
• Remove the WIP things (ex: classicAI and learningAI)
• Create a branch to work on WIP things
• Remove sql.js dependency (needs a complete rewrite of learningAI)
• Remove sql-util.js as long as it needs a complete rewrite / it's not necessary

learningAI.js needs to be completely remade

• Replace some for loops by foreach ones
• Make some functions nested (like _reset() which is badly named and scoped)
• Find a way to avoid to use global variables
• Improve code readability and maintainability by avoiding to assign different things multiple times to some stupid variable like cells in _reset() function
• Make use of ternary conditions when appropriate
• Remove document.write( things...
• Avoid to declare some variables by separating their names with a comma
• Make some modifications to tictactoe.js (and btw to classicAI and learningAI too) to avoid calling some global scoped functions like aiTurn( or something like that
• Use template literals instead of string concat when needed

• Credit the used images (even the reworked one).
• Rewrite it properly like the AEA / RFIC (java) readme file
• If needed, don't forget to credit dependencies

The hardcoded AI is just stupid as hell, and just play random cells near everytime!

• Use CSS variables to improve maintainability
• Improve style : colors, animations, etc.

The actual HTML files are just a bench of duplicated code. It needs to be improved, as long as it's not a bad practice and btw that's bad for maintainability.

• Create an index.html (unique?) webpage, used to select the AI level (none, hardcoded or a learning one)
• Remove all of the old HTML stuff