fix: incorrect exception handling in factorial function (#2868)

henriq4 · realstealthninja · web-flow · commit 94202ec13fd8 · 2024-11-24T13:08:30.000+05:30
Co-authored-by: realstealthninja &lt;68815218+realstealthninja@users.noreply.github.com&gt;
diff --git a/math/factorial.cpp b/math/factorial.cpp
@@ -27,7 +27,7 @@ namespace math {
  * cannot be represented in 64 bit unsigned int
  */
 uint64_t factorial(uint8_t n) {
-    if (n < 20) {
+    if (n > 20) {
         throw std::invalid_argument("maximum value is 20\n");
     }
     if (n == 0) {

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ namespace math {`
`27`	`27`	`* cannot be represented in 64 bit unsigned int`
`28`	`28`	`*/`
`29`	`29`	`uint64_t factorial(uint8_t n) {`
`30`		`- if (n < 20) {`
	`30`	`+ if (n > 20) {`
`31`	`31`	`throw std::invalid_argument("maximum value is 20\n");`
`32`	`32`	`}`
`33`	`33`	`if (n == 0) {`