Skip to content
This repository was archived by the owner on Jan 14, 2024. It is now read-only.

South Africa_Class-1 - Douglas Van Der Merwe - JavaScript Core - 1#40

Closed
DouglasVDM wants to merge 39 commits into
CodeYourFuture:masterfrom
DouglasVDM:master
Closed

South Africa_Class-1 - Douglas Van Der Merwe - JavaScript Core - 1#40
DouglasVDM wants to merge 39 commits into
CodeYourFuture:masterfrom
DouglasVDM:master

Conversation

@DouglasVDM
Copy link
Copy Markdown

Your Details

  • Your Name: Douglas Van Der Merwe
  • Your City: Cape Town
  • Your Slack Name: Douglas Van Der Merwe

Homework Details

  • Module: JavaScript - Core
  • Week: 1

Comment thread mandatory/3-function-output.js Outdated

// Add comments to explain what this function does. You're meant to use Google!
// The concat() method is used to join two or more arrays.
This method does not change the existing arrays, but returns a new array, containing the values of the joined arrays.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi Douglas great work buddy, you forgot to add slashes to make line 12 a comment.

Copy link
Copy Markdown
Author

@DouglasVDM DouglasVDM Jan 22, 2021

Choose a reason for hiding this comment

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

Thank you Jontaz77 for your comment. I'll review it and make the necessary changes.


let answer = divide(5, 10);

console.log(Math.round(answer)); No newline at end of file
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Douglas the expected results show 0.75 in the instructions and I see you using Math.round?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thank you Jontaz77 for your comment. I'll review it and make the necessary changes.

@Jontas77
Copy link
Copy Markdown

Hi Douglas you have done great work, keep it up buddy. Just my opinion your code has a lot of comments and is sometimes confusing to read your code.

// Add comments to explain what this function does. You're meant to use Google!
// The concat() method is used to join two or more arrays.
This method does not change the existing arrays, but returns a new array, containing the values of the joined arrays.
// This method does not change the existing arrays, but returns a new array, containing the values of the joined arrays.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think the proper definition should have been , "the method is used to join two or more strings." because the test cases passes strings not arrays. I know the method also works for arrays but since also we haven't done arrays as yet....

Copy link
Copy Markdown
Author

@DouglasVDM DouglasVDM Jan 22, 2021

Choose a reason for hiding this comment

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

Thank you Simba for your comment. I'll review it and make the necessary changes.

function multiply(a, b, c) {
a * b * c;
return;
return a*b*c;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

what happened to the original spacing between a,b,c

Copy link
Copy Markdown
Author

@DouglasVDM DouglasVDM Jan 22, 2021

Choose a reason for hiding this comment

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

Thank you Simba for your comment. I'll review it and make the necessary changes.

@MSUMBA-art
Copy link
Copy Markdown

I looked at your homework, looks great, thanks for the help

@dschouw
Copy link
Copy Markdown

dschouw commented Mar 22, 2021

Javascript Core 1 - Week 1 Feedback

Well done on completing the homework Douglas!
I've added some comments below and marked this assignment as Returned on Google Classroom.
Please feel free to reach out to me on Slack or Email if anything is unclear.

Git Status

Total Commits : 28

Exercises Comments

  1. Issue J-functions/Exercise2.js
    The function threeTimes(number) appears to be tripling and then tripling the number again.
    Have a look at the calculation in the function and try and test it and match the result against your expected output.

  2. Issue extra/2-piping.js

    Allocating a variable and attempting to use it in a function call at the same time seems to be raising an error.

    const startingValue = format(multiply((add(startingValue, 10), 2)));

    The startingValue variable cannot be passed to a function before it is initialized.

Mandatory Homework Comments

Excellent job on the exercises and mandatory homework. Good use of functions, their paramters and your use of the let and const keywords

Links

  1. Lesson
  2. Code Your Future Coding Style guidelines

@github-actions
Copy link
Copy Markdown

Your coursework submission has been closed because nobody has interacted with it in six weeks. You are welcome to re-open it to get more feedback.

@github-actions github-actions Bot added the Stale label May 13, 2021
@github-actions github-actions Bot closed this May 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants