Fix: largestPrimeFactor function, Add: largestPrimeFactor(8) test (#38756)
* Fix `largestPrimeFactor(8)` result * Improve `largestPrimeFactor` performance * Add `largestPrimeFactor(8)` test
This commit is contained in:
committed by
GitHub
parent
7126bacf34
commit
8a858b470c
@ -34,6 +34,8 @@ tests:
|
||||
testString: assert.strictEqual(largestPrimeFactor(5), 5);
|
||||
- text: <code>largestPrimeFactor(7)</code> should return 7.
|
||||
testString: assert.strictEqual(largestPrimeFactor(7), 7);
|
||||
- text: <code>largestPrimeFactor(8)</code> should return 2.
|
||||
testString: assert.strictEqual(largestPrimeFactor(8), 2);
|
||||
- text: <code>largestPrimeFactor(13195)</code> should return 29.
|
||||
testString: assert.strictEqual(largestPrimeFactor(13195), 29);
|
||||
- text: <code>largestPrimeFactor(600851475143)</code> should return 6857.
|
||||
@ -68,14 +70,18 @@ largestPrimeFactor(13195);
|
||||
|
||||
|
||||
```js
|
||||
const largestPrimeFactor = (number)=>{
|
||||
const largestPrimeFactor = (number) => {
|
||||
let largestFactor = number;
|
||||
for(let i = 2;i<largestFactor;i++){
|
||||
if(!(largestFactor%i)){
|
||||
largestFactor = largestFactor/i;
|
||||
largestPrimeFactor(largestFactor);
|
||||
|
||||
for (let i = 2; i <= Math.sqrt(largestFactor); i++) {
|
||||
if (!(largestFactor % i)) {
|
||||
let factor = largestFactor / i;
|
||||
let candidate = largestPrimeFactor(factor);
|
||||
|
||||
return i > candidate ? i : candidate;
|
||||
}
|
||||
}
|
||||
|
||||
return largestFactor;
|
||||
}
|
||||
```
|
||||
|
Reference in New Issue
Block a user