Welcome to ShenZhenJia Knowledge Sharing Community for programmer and developer-Open, Learning and Share
menu search
person
Welcome To Ask or Share your Answers For Others

Categories

I'm getting the error "missing return in function expected to return 'Bool'" but I can't figure out why.

func isPrime(_ number: Int) -> Bool {
   for primeDivisors in 2..<number {
       if number % primeDivisors == 0 {
           return true
       } else {
           return false
       }
   }
}

isPrime(13)
See Question&Answers more detail:os

与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…
thumb_up_alt 0 like thumb_down_alt 0 dislike
174 views
Welcome To Ask or Share your Answers For Others

1 Answer

Your prime checker is incorrect, because the very first iteration of the loop returns a value. You have to go through the entire loop before deciding that the number is prime.

func isPrime(_ number: Int) -> Bool {
    for primeDivisors in 2..<number {
        if number % primeDivisors == 0 {
            return false
        }
    }
    return true
}

Note that this code is inefficient, because it continues checking divisibility more times than it is necessary: you can stop checking upon reaching square root of number.


与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…
thumb_up_alt 0 like thumb_down_alt 0 dislike
Welcome to ShenZhenJia Knowledge Sharing Community for programmer and developer-Open, Learning and Share
...