Skip to content

London | ITP-May-2025 | Sisay Mehari | Module-Data-Flows | Sprint 1- feature/destructuring #226

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
sisaymehari wants to merge 2 commits into from
Closed

London | ITP-May-2025 | Sisay Mehari | Module-Data-Flows | Sprint 1- feature/destructuring #226

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2ea2b47
Implement using object destructuring, Gryffindor student filter, tea…
sisaymehari Jul 20, 2025
a28710a
Filter and log only Gryffindor students from hogwarts array
sisaymehari Jul 28, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Sprint-1/destructuring/exercise-1/exercise.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ const personOne = {

// Update the parameter to this function to make it work.
// Don't change anything else.
function introduceYourself(___________________________) {
function introduceYourself({ name, age, favouriteFood }) {
console.log(
`Hello, my name is ${name}. I am ${age} years old and my favourite food is ${favouriteFood}.`
);
Expand Down
19 changes: 19 additions & 0 deletions Sprint-1/destructuring/exercise-2/exercise.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,3 +70,22 @@ let hogwarts = [
occupation: "Teacher",
},
];
function getGryffindorStudents(hogwarts) {
for (const { firstName, lastName, house } of hogwarts) {
if (house === "Gryffindor") {
Copy link

@LonMcGregor LonMcGregor Jul 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remember to be consistent with indentation. Here your for block is missing indentation.

Copy link
Author

@sisaymehari sisaymehari Jul 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right, thank you.

Copy link

@LonMcGregor LonMcGregor Jul 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The function is called getGryffindorStudents - you are correctly filtering on house here, is there anything else to filter?

Copy link
Author

@sisaymehari sisaymehari Jul 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added another filter.

console.log(`${firstName} ${lastName}`);
}
}
}
getGryffindorStudents(hogwarts);


function getTeachersWithPets(hogwarts) {
for (const { firstName, lastName, occupation, pet } of hogwarts) {
if (occupation === "Teacher" && pet) {
console.log(`${firstName} ${lastName}`);
}
}
}

getTeachersWithPets(hogwarts);
11 changes: 11 additions & 0 deletions Sprint-1/destructuring/exercise-3/exercise.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,14 @@ let order = [
{ itemName: "Hot Coffee", quantity: 2, unitPricePence: 100 },
{ itemName: "Hash Brown", quantity: 4, unitPricePence: 40 },
];
console.log("QTY ITEM TOTAL");
let overallTotal = 0;
for (const{ itemName, quantity, unitPricePence } of order) {
const total = quantity * unitPricePence;
overallTotal += total;
console.log(
`${quantity.toString().padStart(2)} ${itemName.padEnd(30)} £${(total / 100).toFixed(2)}`
);
}
console.log(`\nTotal: £${(overallTotal/100).toFixed(2)}`);