Added more notes
Dev Ghai

Dev Ghai commited on 2013-12-17 15:12:48
Showing 1 changed files, with 4 additions and 0 deletions.

... ...
@@ -1,3 +1,7 @@
1 1
 There is no need to have BoggleList:CurateList function. The word should be just ignored and there should be no need to create a new file. The idea was to store a curated list for subsequent runs to reduce load times. To do this, actually the full tree of dictionary should be persisted and deserialzed in case it matches the rules for which we need to construct the tree.
2 2
 
3 3
 Maintaining 'Q' and 'U' together is deeply embedded in the whole program. This takes away some flexibility that comes with defining your own ruleset. Hence the idea should be to remove the constraint while adding it to the tree while loading the list. This would make it even more efficient because instead of String, we would be able to use 'readonly char' type that is much lighter for all operations and memory usage.
4
+
5
+Curated list will corrupt the data if boardlength is different but the min word length is passed the same. This is because max word size, or boardlength^2 + 1.
6
+
7
+For each thread, store the words in local copy to the threaf and at the end of all the threads, collect them in one list. This way you would be able to remove all the locks.
4 8
\ No newline at end of file
5 9