Ad
  • Default User Avatar

    It should be mentioned in the description that you may need to include the same zero multiple times in the result. I assume the task was just to find rational zeros, without caring for their multiplicities.

  • Default User Avatar

    Just getting started, but I did not see a mention of what degree the polynomials would be. I assume only deg <= 2 by the test cases?

  • Default User Avatar

    Added random tests with more complicated rational roots and a more complex multiplicity test (the random tests also include examples with complex multiplicities)

  • Custom User Avatar
    • Needs random tests
    • Needs polynomials that returns more complicated rational roots (e.g 1/7, or something like 50/1001)
    • Needs more sophisticated tests on multiplicities (e.g polynomial with roots [0,0,1])