Skip to content

Conversation

@tobyev
Copy link
Contributor

@tobyev tobyev commented Aug 19, 2025

I completed my Lesson 06 homework by implementing the ExpressionCalculator class in expression_calculator.ts and ensuring it correctly handles addition, multiplication, division, exponentiation (pow), and the overall calculation formula. I verified my code works by running the interactive program in lesson6.ts and confirmed all tests pass.

Problem area(s)

  • Ensuring ExpressionCalculator matched the Calculator interface expected by the tests
  • Getting npm commands to run from the correct subfolder with the package.json
  • Understanding the correct use of the pow method versus Math.pow in the calculation

Lessons learned:

  • Always check which folder contains package.json before running npm commands
  • Make sure your class fully implements the interface the tests expect
  • Use the interactive prompts to verify your calculation logic before running automated tests

Copy link
Contributor

@anthonydmays anthonydmays left a comment

Choose a reason for hiding this comment

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

🎓 Automated Grading Report

Student: Tobais Evans
Date: 8/19/2025

Feedback

Tobais Evans has successfully met all the functional requirements for the assignment. The calculate method in expression_calculator.ts has been implemented correctly, performing a series of arithmetic operations as expected. The student configured the .env.test file with the correct group letter 'A', ensuring compliance with the assignment's requirements. Technically, the code is well-structured and adheres to TypeScript standards, with clear method separation for each arithmetic operation, enhancing readability and maintainability. All GitHub checks have passed, indicating that the code is free of syntax errors and follows the project's style guidelines. However, there is no attempt to tackle the stretch assignment, so the stretch score remains at 0. Overall, excellent work on the core assignment!


This is an automated preliminary review. Please review and adjust before finalizing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants