[BUGFIX] sonarqube: Prefer Math.max() to simplify ternary expressions.

Ternary expressions should be replaced with "Math.min()" or "Math.max()" for simple comparisons typescript:S7766

Author: Gonzalo Diaz

src/hackerrank/interview_preparation_kit/dynamic_programming/max_array_sum.ts

@@ -4,7 +4,11 @@
  */
 
 const bigIntMax = (...args: bigint[]): bigint =>
-  args.reduce((m, e) => (e > m ? e : m), BigInt(0));
+  args.reduce((m, e) => {
+    const _e = BigInt(e);
+    const _m = BigInt(m);
+    return _e > _m ? _e : _m;
+  }, BigInt(0));
 
 function maxSubsetSum(arr: number[]): number {
   const arrCopy: bigint[] = arr.map((x: number): bigint => BigInt(x));

src/projecteuler/problem0005.ts

@@ -33,7 +33,7 @@ function _replaceMaximum(
   const element = _group.get(_element);
 
   if (element) {
-    _group.set(_element, count > element ? count : element);
+    _group.set(_element, Math.max(count, element));
   } else {
     _group.set(_element, count);
   }

src/projecteuler/problem0018.ts

@@ -32,7 +32,7 @@ function problem0018(_triangle: number[][]): number {
   console.debug(`leafs: ${leafs.toString()}`);
 
   const __START_FROM__ = 0;
-  const max = leafs.reduce((a, b) => (a > b ? a : b), __START_FROM__);
+  const max = leafs.reduce((a, b) => Math.max(a, b), __START_FROM__);
 
   return max;
 }