Swiftで素数判定アプリを作っているのですが...

Question

import UIKit

class ViewController: UIViewController {

    @IBOutlet var Label1: UILabel!
    @IBOutlet weak var Text1: UITextField!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view, typically from a nib.
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }

    @IBAction func Button1(sender: AnyObject) {
        let n = Text1.text
        if Int(n!) < 2 {
        Label1.text = "素数ではありません"
        }
        if Int(n!) == 2{
        Label1.text = "素数です"
        }
        if Int(n!)! % 2 == 0{
        Label1.text = "素数ではありません"
        }
        var i = 0
        for i == 3; i <= Int(n!)! / i; i += 2{
        if Int(n!)! % i == 0{
        Label1.text = "素数ではありません"
        }
        }
        Label1.text = "素数です"
        }

        
    }

このようにしてみましたがエラーになります。
エラー回避の方法と、またこれが素数を判定するものとして正しいのかということ自体がわからないので教えていただけるとありがたいです。

Answer 1

おそらく、素数判定の最後のfor文で、Int(n!)で引っかかってエラーがでていると思われます。

こんな感じでどうでしょうか。

一度、if let 文を使って、textFieldのtextがあるかどうかと、文字列がIntに変換できるかを確認してから、
素数判定を走らせているので、判定文の中で、アンラップする必要もないです。

環境がわかりませんが、Swift1.2以上なら大丈夫かと思います。
素数判定は色々ありますが、daisuke-hanさんの書いてあるものそのまま(試し割り法)にしてあります

    func isPrimeNumber(num: Int) -> Bool {
        if num < 2 {
            return false
        } else if num == 2 {
            return true
        }
        
        if num % 2 == 0 {
            return false
        }
        
        for var i = 3;  i * i <= num; i += 2 {
            if num % i == 0 {
                return false
            }
        }
        return true
    }
    
    @IBAction func Button1(sender: AnyObject) {
        if let text = Text1.text, number = Int(text) {
            Label1.text = isPrimeNumber(number) ? "素数です。" : "素数ではありません。"
        } else {
            Label1.text = "判定できませんでした。"
        }
    }

Comments

[yuba]

```
i < num / i
```
のところ
```
i <= num / i
```
じゃないと、例えばnumが9のときにループを抜けちゃってtrueを返しちゃうんじゃないですかね。
さらに言うと、かけ算と割り算の負荷を考えると
```
i*i <= num
```
の方がより良いと思います。

[sgr_ksmt]

ご指摘ありがとうございます。
回答の方、修正させていただきます^^/

[daisuke-han]

ありがとうございました。
完璧でした。これまで知らなかった if let についても学ぶ機会になり大変良かったです。
yuba さんもご指摘ありがとうございました。