Finalized week3 JS assignment

konjit · konjit · commit 501c4a9c64dd · 2024-12-05T12:29:37.000+01:00
diff --git a/Week3/prep-exercises/1-hyf-program/1-find-mentors.js b/Week3/prep-exercises/1-hyf-program/1-find-mentors.js
@@ -8,7 +8,7 @@ import { modules, students, mentors, classes } from "./hyf.js";
  *  ['John', 'Mary']
  */
 
-// I tried it first with forEach function and thought ahh I could use reduce
+// I tried it first with forEach and thought ahh i can use reduce
 // const possibleMentorsForModule = (moduleName) => {
 //   let mentorForModule = [];
 //   mentors.forEach(mentor => {
@@ -19,7 +19,7 @@ import { modules, students, mentors, classes } from "./hyf.js";
 //   return mentorForModule;
 // };
 
-const possibleMentorsForModule = (moduleName) => {
+ const possibleMentorsForModule = (moduleName) => {
   return mentors.reduce((acc, mentor) => {
     if (mentor.canTeach.includes(moduleName)) {
       acc.push(mentor.name);
@@ -28,6 +28,15 @@ const possibleMentorsForModule = (moduleName) => {
   }, []);
 
 };
+
+export const mentorForModule = (moduleName) => {
+  return mentors.reduce((acc, mentor) => {
+    if (mentor.nowTeaching === moduleName) {
+      acc.push(mentor);
+    }
+    return acc;
+  }, []);
+};
 // You can uncomment out this line to try your function
 // console.log(possibleMentorsForModule("using-apis"));
 
@@ -39,7 +48,7 @@ const possibleMentorsForModule = (moduleName) => {
  */
 const findMentorForModule = (moduleName) => {
   const result = possibleMentorsForModule(moduleName);
-  if(!result.length) return `No mentor found for module ${moduleName}`;
+  if( !result ) return `No mentor found for module ${moduleName}`;
 
   if(result.length === 1){
     return result[0];
diff --git a/Week3/prep-exercises/1-hyf-program/2-class-list.js b/Week3/prep-exercises/1-hyf-program/2-class-list.js
@@ -1,5 +1,5 @@
 import { modules, students, mentors, classes } from "./hyf.js";
-
+import { mentorForModule } from "./1-find-mentors.js";
 /**
  * We would like to have a list of everyone that is currently participating in a class.
  * This means the students, but also the mentors that are currently teaching the class.
@@ -11,8 +11,22 @@ import { modules, students, mentors, classes } from "./hyf.js";
  *
  *  [{ name: 'John', role: 'student' }, { name: 'Mary', role: 'mentor' }]
  */
+
+
+const filterStudentsByClassName = className => {
+  return students.filter(student => {
+    return student.class === className && student.graduated === false})
+    .map(({name}) => ({name, role: 'student'}));
+};
+
 const getPeopleOfClass = (className) => {
-  // TODO complete this function
+  const participatingStudents = filterStudentsByClassName(className);
+
+  const specificClass = classes.find(({name}) => name === className);
+  const participatingMentors = mentorForModule(specificClass.currentModule)
+                          .map(({name}) => ({name, role:'mentor'}));
+  
+  return [...participatingStudents, ...participatingMentors];
 };
 // You can uncomment out this line to try your function
 // console.log(getPeopleOfClass('class34'));
@@ -30,7 +44,14 @@ const getPeopleOfClass = (className) => {
  *  }
  */
 const getActiveClasses = () => {
-  // TODO complete this function
+  const activeClasses = classes
+       .filter(({active}) => { return active === true })
+       .map(({name}) => ({name}));
+
+  return activeClasses.reduce((acc, activeClass) => {
+    acc[activeClass.name] = getPeopleOfClass(activeClass.name); 
+    return acc;
+  }, {});
 };
 // You can uncomment out this line to try your function
 // console.log(getActiveClasses());
