Improper and incomplete implementation of CycleDetection.js
10kartik opened this issue · comments
Kartik K commented
Implementation of CycleDetection.js seems inappropriate.
- Weird function name
main()
and its invocation can cause the program to crash. - Tests are missing.
- Not exported, Hence not resuable.
- Can be optimized further (Optional)
Lars Müller commented
The algorithm is complete. It has poor code quality though, as you have pointed out. Feel free to fix it.