#1-4 Task. Roman Tarasenko #845
Conversation
| @@ -22,7 +22,7 @@ | |||
| * '', 'bb' => 'bb' | |||
| */ | |||
| function concatenateStrings(value1, value2) { | |||
| throw new Error('Not implemented'); | |||
| return value1+value2; | |||
There was a problem hiding this comment.
Please, try to follow these rules https://javascript.info/coding-style
spaces between operators
| @@ -73,7 +76,7 @@ function getAverage(value1, value2) { | |||
| * (-5,0) (10,-10) => 18.027756377319946 | |||
| */ | |||
| function getDistanceBetweenPoints(x1, y1, x2, y2) { | |||
| throw new Error('Not implemented'); | |||
| return Math.sqrt(((x2-x1)*(x2-x1)) +((y2-y1)*(y2-y1))); | |||
There was a problem hiding this comment.
Try to improve using Math.hypot
| @@ -160,7 +164,7 @@ function parseNumberFromString(value) { | |||
| * 1,2,3 => 3.741657386773941 | |||
| */ | |||
| function getParallelipidedDiagonal(a,b,c) { | |||
| throw new Error('Not implemented'); | |||
| return Math.sqrt((a*a + b*b) + c*c) | |||
There was a problem hiding this comment.
Try to improve using Math.hypot
| @@ -23,7 +23,11 @@ | |||
| * [0, 1, 2, 3, 4, 5], 5 => 5 | |||
| */ | |||
| function findElement(arr, value) { | |||
| throw new Error('Not implemented'); | |||
| if (arr.find((el) => el === value)) { | |||
| return arr.indexOf(value); | |||
There was a problem hiding this comment.
Just this line will work as a solution
| let arrPos = []; | ||
| arr.map((el)=>{ | ||
| if(el > 0){ | ||
| arrPos.push(el) | ||
| } | ||
| }); | ||
| return arrPos; |
There was a problem hiding this comment.
Please, use Array.prototype.filter method here
| function removeFalsyValues(arr){ | ||
| let newArr = []; | ||
| arr.map((el) =>{ | ||
| if (Boolean(el) !== false){ |
There was a problem hiding this comment.
Redundant condition. Just if (el) { will work
Please, see https://javascript.info/logical-operators
| let newArr = arr.reverse(); | ||
| if (arr.length > 3) { | ||
| newArr.length = 3; | ||
| } | ||
| return newArr; |
There was a problem hiding this comment.
If test cases had unsorted array, it woudn't work.
To prevent errors in case tests will reworked, please add sort method in place of reverse
| let newArr = []; | ||
| if (arr.length){ | ||
| arr.map((el) =>{ | ||
| if (Boolean(el) === false){ |
There was a problem hiding this comment.
if (!el) will work
Please, see https://javascript.info/logical-operators
| let newArr = []; | ||
| arr.map((el) =>{ | ||
| let i = newArr.find(item => item === el); | ||
| if (i === undefined){ |
There was a problem hiding this comment.
!newArr.includes(el) will work. And remove line with find
|
And try to resolve conflicting files. It's very significant in real project |
https://travis-ci.org/github/RomanTarasenko/js-assignments/builds/714484766