アラートが表示されない

Question

###前提・実現したいこと
今、簡単な割り勘アプリ(合計金額/人数を結果ラベルに表示)を作っています。
そのアプリの機能の一つとして、金額あるいは人数のどちらかが空白だとアラートが表示される
###発生している問題・エラーメッセージ

アラートが表示されない。

###該当のソースコード

import UIKit

//delegateの宣言
class ViewController: UIViewController, UITextFieldDelegate {

  @IBOutlet weak var resultLabel: UILabel!
  @IBOutlet weak var priceTextField: UITextField!
  @IBOutlet weak var numberTextField: UITextField!
 



  override func viewDidLoad() {
    super.viewDidLoad()

    //delegateの所在をかく
    priceTextField.delegate = self
    numberTextField.delegate = self
    
    //各TextFieldに入力できるのは数字だけ
    self.priceTextField.keyboardType = UIKeyboardType.numberPad
    self.numberTextField.keyboardType = UIKeyboardType.numberPad
  }
  
  
  

  override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
     }
  
    

  
  //ボタンを押した時
  @IBAction func resultButton(_ sender: Any) {
    
    
    
    if priceTextField.text == nil || numberTextField.text == nil {
      
      //UIAlertControllerインスタンスの作成
      let alertController = UIAlertController(title: "エラー", message: "合計金額又は、人数が未入力です", preferredStyle: .alert)
      
      //追加するアクションの設定
      let ryoukaiAction = UIAlertAction(title: "了解", style: .default, handler: nil)
      
      //設定したアラートアクションをUIAlertControllerに追加する
      alertController.addAction(ryoukaiAction)
      //アラートを表示
      present(alertController, animated: true, completion: nil)
      
      
    } else {
  

      guard let pricetext = priceTextField?.text, let price = Int(pricetext) else {
              return
      }
      
      guard let numbertext = numberTextField?.text, let number = Int(numbertext) else {
              return
            }
      

      
      let result:Double = Double(price/number)
      
      resultLabel.text = String(result)
     
      
    }
  
  }

 
  
  
  //delegateメソッド
  override func touchesBegan(_ touches: Set<UITouch>, with event: UIEvent?) {
    
    //キーボード以外のところをタップするとキーボードを閉じる
    if priceTextField.isFirstResponder {
      
      priceTextField.resignFirstResponder()
      
    }
    
    //キーボード以外のところをタップするとキーボードを閉じる
    if numberTextField.isFirstResponder{
      numberTextField.resignFirstResponder()
    }
    
  }
  
  
  
}

###補足情報(言語/FW/ツール等のバージョンなど)
swift3.0
xcode8.2.1
macOS Sierra(バージョン 10.12)

Answer 1

if priceTextField.text == nil || numberTextField.text == nil {
でブレークポイントを設定し、値を確認してください。
nilではなく""(長さ0の文字列）になっていませんか？

if priceTextField.text?.characters.count == 0 || numberTextField.text?.characters.count == 0 {
}

のようにテキスト長で判定すれば良いと思います。

Comments

[fuzzball]

.textがnilのときにif文の中に入りません。
Objective-Cの場合はこれでOKなのでラクなのですが。

[sha]

UITextFiledがIBoutletだしこれで良いと思いました。
同様の質問のようにされるのが良いと思います。

[A2wwM]

ありがとうございました。
同様の質問も参考にさせていただきました！

[A2wwM]

ちなみに、ブレークポイントを置いて値を確認してみました。
ブレークポイントはあまり使ったことがなかったのでお伺いしたいのですが
ブレークポイントを設定してみたところ、numbertext(値を設定していない)の横に "" と書かれていました。
これが、shaさんのおっしゃっている「長さ0の文字列を持っている」ということなのでしょうか？

[sha]

ブレークポイントをどこに置いたのかがわかりませんが
http://qiita.com/Kta-M/items/b8f6f74b3288611e7536
を参考にUITextFieldのtextプロパティ値を直接確認すると良いと思います。
条件の位置にブレークポイントを設定するとnumbertextは長さ0の文字列も持っていない状態です。
ブレークポイントの位置によりますが、numbertext(値を設定していない)の横に
「 = (String)""」 と書かれていたのであれば「長さ0の文字列を持っている」状態です。

[A2wwM]

丁寧に教えてくださってありがとうございます。
こちらのリンク先も見させて頂きます！

Answer 2

同様の質問

https://teratail.com/questions/63231

Comments

[A2wwM]

ありがとうございます。拝見させていただきました。
たぶん私と作っているものが同じですね！
自分とは違う書き方なので参考になります。

Answer 3

if priceTextField.text == nil || numberTextField.text == nil {

このif文の中には入っているでしょうか？
カラ文字列のときにもアラートを表示させたいのであればカラ文字列チェックを行って下さい。

Comments

[fuzzball]

ぉぅ‥

[fromageblanc]

w

[A2wwM]

そもそもnil(値がない)とカラ文字列で違いがあるのですね。。。とりあえず調べてみます。

[A2wwM]

ありがとうございます。

Answer 4

ここじゃない？

    //if priceTextField.text == nil || numberTextField.text == nil {

    if priceTextField.text == "" || numberTextField.text == "" {

Comments

[fuzzball]

これだとnilのときに入らないですね。

[fromageblanc]

たしかに！

[fuzzball]

if (priceTextField.text ?? "").isEmpty || (numberTextField.text ?? "").isEmpty {
これで。

[fromageblanc]

おーgoodです！ 実行されないコードがあるならまずは条件のところをチェックしてほしいですね

[A2wwM]

ありがとうございます！
if (priceTextField.text ?? "").isEmpty || (numberTextField.text ?? "").isEmpty {
で解決できました。
ただ一つわからないことがありまして、nilの時に入らないとはどういう意味何ですか？(すみません

[fromageblanc]

nilの時に入らないというのは、私の回答に対するレスですね。空欄("")しかみてないよということです。ただ今回の場合はストボーでインスタンスされる際、textプロパティは空欄でイニシャライズされるようですのでnil判定は実質必要ないのですが、暗黙保証のオプショナル（ビックリマーク付き）で定義しているので、このような場合はなるべくnil判定はおこなうようにしたほうが良いと思います。

[A2wwM]

なるほど、確かにみてみると空欄("")の時のみですね。
ところで、priceTextField.text ?? ""はNil合体演算子ですか？
そうであれば、fromageblancさんがおっしゃるnil判定も行われているということです...よね。(意味のわからないことを言ってたらすみませんっ)

[fuzzball]

>>ストボーでインスタンスされる際
いつの間にか、コードで生成しても""がデフォルトになったようですね。

[A2wwM]

ベストアンサーは決定しましたが、もしよろしければ最後に上記の質問にお答え頂けますと幸いです。

[fromageblanc]

「はい」が回答になります

[A2wwM]

ありがとうございます。