Skip to content

Commit 86856f6

Browse files
committed
Merge pull request #119 from NovaHe/fix/263
fix 263: remove redundant compute
2 parents 1eb4bf6 + 594af3f commit 86856f6

File tree

2 files changed

+11
-6
lines changed

2 files changed

+11
-6
lines changed

leetcode/0263.Ugly-Number/263. Ugly Number.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
11
package leetcode
22

33
func isUgly(num int) bool {
4-
for i := 2; i < 6 && num > 0; i++ {
5-
for num%i == 0 {
6-
num /= i
4+
if num > 0 {
5+
for _, i := range []int{2, 3, 5} {
6+
for num%i == 0 {
7+
num /= i
8+
}
79
}
810
}
911
return num == 1

website/content/ChapterFour/0200~0299/0263.Ugly-Number.md

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,14 +58,17 @@ Explanation: 14 is not ugly since it includes another prime factor 7.
5858
package leetcode
5959

6060
func isUgly(num int) bool {
61-
for i := 2; i < 6 && num > 0; i++ {
62-
for num%i == 0 {
63-
num /= i
61+
if num > 0 {
62+
for _, i := range []int{2, 3, 5} {
63+
for num%i == 0 {
64+
num /= i
65+
}
6466
}
6567
}
6668
return num == 1
6769
}
6870

71+
6972
```
7073

7174

0 commit comments

Comments
 (0)