Merge pull request #783 from sir-gon/develop

[BUGFIX] SonarQube: Prefer class field declaration over `this` assign…

Author: sir-gon

src/hackerrank/implementation/countApplesAndOranges.js

@@ -18,9 +18,7 @@ function countApplesAndOranges(s, t, a, b, apples, oranges) {
     }
   }
 
-  const result = [];
-  result.push(cApples);
-  result.push(cOranges);
+  const result = [cApples, cOranges];
 
   return result.join('\n');
 }

src/hackerrank/interview_preparation_kit/dynamic_programming/max_array_sum.js

@@ -7,7 +7,7 @@ const bigIntMax = (...args) =>
   args.reduce((m, e) => (e > m ? e : m), BigInt(0));
 
 function maxSubsetSum(arr) {
-  const arrCopy = arr.map((x) => BigInt(x));
+  const arrCopy = arr.map(BigInt);
 
   if (arrCopy.length === 0) {
     return 0;

src/hackerrank/interview_preparation_kit/sort/ctci_bubble_sort.js

@@ -5,12 +5,11 @@
 const SEPARATOR = '\n';
 
 class SortableGroup {
-  group;
+  count = 0;
 
-  count;
+  group;
 
   constructor(group) {
-    this.count = 0;
     this.group = group;
   }