chapter 2 code refactored

Author: crizstian

02-chapter-Arrays.js

@@ -10,36 +10,12 @@ class grades {
 
   displayAvg(){
     let sum = 0;
-    for (const key in this.grades) {
-        sum += this.grades[key];
-    }
+    this.grades.forEach(grade => sum += grade );
     return sum / this.grades.length;
   }
-
 }
 
-// Excercises
-console.log('CHAPTER 2');
-console.log('### Excercise 1');
-
-const gradeObj = [90,89,75];
-const grade    = new grades(gradeObj);
-grade.addGrade(90);
-grade.addGrade(25);
-console.log(`Avg grade: ${grade.displayAvg()}`);
-
-// #############################################
-
-console.log('\n');
-console.log('### Excercise 2');
-
-const arrayWords = ["hello ","my ","friend "];
-console.log(`orginal array: ${arrayWords} `);
-console.log(`displaying forward: ${arrayWords.reduce((total, word) => total + word)}`);
-console.log(`displaying backward: ${arrayWords.reduceRight((total, word) => total + word)}`);
-
-// #############################################
-
+// ##############################################
 class weekTemps {
 
   constructor(dataStore = []) {
@@ -53,38 +29,55 @@ class weekTemps {
   averageWeek(week) {
     let total     = 0;
     let totalDays = this.dataStore[week].length;
-    for (let i = 0; i < totalDays; i++) {
-      total += this.dataStore[week][i];
-    }
+
+    this.dataStore[week].forEach(item => total += item );
+
     return (total / totalDays).toFixed(2);
   }
 
   displayMonthAvg() {
     let sum = 0;
-    for (const key in this.dataStore) {
-      for (const key2 in this.dataStore[key]) {
-        sum += this.dataStore[key][key2];
-      }
-    }
+
+    this.dataStore.forEach(week =>
+      week.forEach(day => sum += day )
+    );
+
     return (sum / (this.dataStore.length * 4)).toFixed(2);
   }
 
   displayAllWeekAvg() {
     let sum = 0;
-    let avg = [];
-    for (const key in this.dataStore) {
-      for (const key2 in this.dataStore[key]) {
-        sum += this.dataStore[key][key2];
-      }
-      avg[key] = `Week ${parseInt(key, 10) + 1} Temp Avg: ${(sum / this.dataStore[key].length).toFixed(2)}`;
-      sum      = 0;
-    }
-    return avg;
-  }
 
+    this.dataStore.forEach((week, count) => {
+      week.forEach(day => sum += day );
+      console.log(`Week ${count + 1} Temp Avg: ${(sum / week.length).toFixed(2)}`);
+      sum = 0;
+    });
+
+  }
 }
-console.log('\n');
-console.log('### Excercise 3');
+
+// Excercises
+// #############################################
+console.log('CHAPTER 2');
+console.log('### Excercise 1');
+
+const gradeObj = [90,89,75];
+const grade    = new grades(gradeObj);
+grade.addGrade(90);
+grade.addGrade(25);
+console.log(`Avg grade: ${grade.displayAvg()}`);
+
+// #############################################
+console.log('\n### Excercise 2');
+
+const arrayWords = ["hello ","my ","friend "];
+console.log(`orginal array: ${arrayWords} `);
+console.log(`displaying forward: ${arrayWords.reduce((total, word) => total + word)}`);
+console.log(`displaying backward: ${arrayWords.reduceRight((total, word) => total + word)}`);
+
+// #############################################
+console.log('\n### Excercise 3');
 
 const randomMonth = [
   [45,23,32,12,31,21,22],
@@ -96,16 +89,11 @@ const thisMonth = new weekTemps(randomMonth);
 
 console.log(`Month Temp avg: ${thisMonth.displayMonthAvg()}`);
 console.log(`Week 2 Temp avg: ${thisMonth.averageWeek(2)}`);
-console.log(`Week's Temp avg:`);
-const weeks = thisMonth.displayAllWeekAvg();
-for (const key in weeks) {
-  console.log(weeks[key]);
-}
+console.log(`\nDisplay All Week Avg`);
+thisMonth.displayAllWeekAvg();
 
 // #############################################
-
-console.log('\n');
-console.log('### Excercise 4');
+console.log('\n### Excercise 4');
 
 const letters = ['a','b','a','c','a','d','a','b','r','a'];
 console.log(letters.reduce((all, letter) => all + letter));