Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitc5a2566

Browse files
bug: abs returns 0 on an empty array (TheAlgorithms#1473)
* bug: update edge case for empty array* bug: add edge case for empty arrays* feat: add test case for empty array---------Co-authored-by: Ridge Kimani <ridgekimani@gmail.com>
1 parent52858f8 commitc5a2566

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

‎Maths/Abs.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,11 @@
1212
constabs=(num)=>{
1313
constvalidNumber=+num// converted to number, also can use - Number(num)
1414

15-
if(Number.isNaN(validNumber)){
15+
if(Number.isNaN(validNumber)||typeofnum==='object'){
1616
thrownewTypeError('Argument is NaN - Not a Number')
1717
}
1818

19-
returnvalidNumber<0 ?-validNumber :validNumber// if number is lessthen zero mean negative then it converted to positive. i.e -> n = -2 = -(-2) = 2
19+
returnvalidNumber<0 ?-validNumber :validNumber// if number is lessthan zero mean negative then it converted to positive. i.e -> n = -2 = -(-2) = 2
2020
}
2121

2222
export{abs}

‎Maths/test/Abs.test.js‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ describe('Testing abs function', () => {
55
expect(()=>abs('234a')).toThrow()
66
expect(()=>abs({})).toThrow()
77
expect(()=>abs([12,-32,-60])).toThrow()
8+
expect(()=>abs([])).toThrow()// coerces to 0
89
})
910

1011
it('Testing for number of string type',()=>{

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp