1

テキストフィールドが空の時にボタンイベントを発動するとエラーが出てしまいます。

テキストフィールドに何も打ち込まれてない時、小数点以外の文字を打ち込んだ時の処理を教えてください。

追記

テキストフィールドが何も打ち込まれていないとき、違う文字が打ち込まれたときはx1及びx2に0を代入したいです。

internal func onClickMyButton(sender: UIButton){


        let x1 = Float(myTextField.text!)
        let x2 = Float(myTextField2.text!)

        y = x1! * x2!

        score.text = String(format: "%.f回", y)
    }
2
  • 自分の投稿を見返してみて、失敗したなと思ったら編集で改善する事が出来ます。ソースコードの載せ方など覚えて見て下さい。また、質問が回答によって解決したらその回答を採用して下さい。自己解決した場合は自分で回答を作成する事も推奨されています。
    – mjy
    Commented 2016年4月15日 6:43
  • コードの修正他ありがとうございます。ついでに言うと"swift"と言うのはタグで示すことができるので普通はタイトルには含めません。さて、質問内容の方ですが、「テキストフィールドに何も打ち込まれてない時、小数点以外の文字を打ち込んだ時」にどんな処理をするかは、あなたがどんな動作をさせたいか、によって変わってきます。「今のアプリは練習用」と割り切って、適当な値(例えば0とか)で代替して、知らんふりして処理を続けると言うのもありでしょうし、ユーザにエラー表示をして再度同じボタンが押されるまで待つ、と言うのもありだと思います。どうしたいのか、まずは方針を決めてください。
    – OOPer
    Commented 2016年4月15日 8:25

1 件の回答 1

1

Swiftの!をforced unwrappingとして使うのは、Swiftに対して「もし値がnilならアプリをクラッシュさせてください」と言っていることになります。Swiftのコードサジェスチョンでは、Optionalではいけない場所にOptionalになる可能性のある式を使うと、!を付け加えるのを第一候補に提示してきますが、あまり鵜呑みにせずに、本当に「nilならクラッシュ」があなたの望む動作なのか確認してください。

さて、あなたのコードには4箇所!があります。UITextFieldtextプロパティはOptionalつまりnilになる可能性があります。またFloat.init(_:)StringFloatに変換する場合もnilになる可能性(数として解釈できない場合)があります。

Optionalなデータをnon-Optionalとして使う方法は、危険な!を使う以外にも大きく2種類あります。

(1) Optional bindingと呼ばれるif-letを使う方法

値がnilでないことを確かめてから変数(letだから定数といった方が良いかもですが)に値を代入してくれます。もしnilだったら、普通のif文と同じようにelseを使ってその時の処理を書けます。

    let x1: Float
    if let
        text = myTextField.text, //`text`には、`myTextField.text`が`nil`でない時だけ値が入る
        value = Float(text) //`value`には、`Float(text)`が`nil`でない時だけ値が入る
    {
        x1 = value //`value`は`Float(text)`が`nil`でなかった時の値が入っている
    } else {
        //この部分は、`myTextField.text`か`Float(text)`のどちらかが`nil`だった時に実行される
        x1 = 0
    }

(x2についてもほとんど同じコードを書くだけ。)

(2) 値がnilだった時のデフォルト値を与える??演算子を使う

a ?? bと書くとaの値がnilでない場合は、その値をそのまま、nilだった場合はbの値を返す式になります。bの部分を「aに値がない場合のデフォルト値」として使うことができます。こちらを使うと例えば、こんな書き方ができます。

    let x1 = Float(myTextField.text ?? "0") ?? 0.0

(?? "0"の方がmyTextField.textnilだった時のデフォルト値、?? 0.0の方がFloat(...)nilだった時のデフォルト値、と言うことになります。これもx2についてはほとんど同じことを書くだけなのですぐにわかると思います。)

どちらの方が良いかは、ご自身が将来今のアプリをどの程度まで発展させるのか、くらいまで考えてご決定ください。

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。