Checking textfield for nil

[herm57]

Totally can not figure out how to protect against a textfield that is nil, always crashing on dollarsTextField with Unexpectedly found nil while unwrapping an Optional value

Here’s what I have, any help would be great, thank you

import UIKit

class ViewController: UIViewController {

@IBOutlet var dollarsTextField: UITextField!
@IBOutlet var halvesTextField: UITextField!
@IBOutlet var quartersTextField: UITextField!
@IBOutlet var dimesTextField: UITextField!
@IBOutlet var nickelsTextField: UITextField!
@IBOutlet var penniesTextField: UITextField!

@IBOutlet var dollarLabel: UILabel!
@IBOutlet var halveLabel: UILabel!
@IBOutlet var quarterLabel: UILabel!
@IBOutlet var dimeLabel: UILabel!
@IBOutlet var nickelLabel: UILabel!
@IBOutlet var pennyLabel: UILabel!

@IBOutlet var valueLabel: UILabel!
@IBOutlet weak var coinLabel: UILabel!

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

@IBAction func calculate(_ sender: Any) {
    
    let a = Float(dollarsTextField.text!)!
    let b = Float(halvesTextField.text!)! * 0.50
    let c = Float(quartersTextField.text!)! * 0.25
    let d = Float(dimesTextField.text!)! * 0.10
    let e = Float(nickelsTextField.text!)! * 0.05
    let f = Float(penniesTextField.text!)! * 0.01
    
    let g = Float(dollarsTextField.text!)!
    let h = Float(halvesTextField.text!)!
    let i = Float(quartersTextField.text!)!
    let j = Float(dimesTextField.text!)!
    let k = Float(nickelsTextField.text!)!
    let l = Float(penniesTextField.text!)!
   
    dollarLabel.text = String(format:"%0.2f", a)
    halveLabel.text = String(format:"%0.2f", b)
    quarterLabel.text = String(format:"%0.2f", c)
    dimeLabel.text = String(format:"%0.2f", d)
    nickelLabel.text = String(format:"%0.2f", e)
    pennyLabel.text = String(format:"%0.2f", f)

    valueLabel.text = String(format: "%0.2f", a+b+c+d+e+f)
    coinLabel.text = String(format: "%0.0f", g+h+i+j+k+l)
    
    // Ive tried this and some other code, and Im at wits end!
    
    //        if dollarsTextField != nil {
    //            let output = String(format: "%0.2f", a)
    //            dollarLabel.text = "\(output)"
    //        } else {
    //            dollarLabel.text = "0"
    //            dollarsTextField.text = "0"
    //        }
    
            if dollarsTextField == nil {
                //dollarLabel.text = "0"
                dollarsTextField.text = "0"
            } else {
                let output = String(format: "%0.0f", g)
                dollarsTextField.text = "\(output)"
            }
    
}

}

[robertomachorro]

@herm57

UITextField!

Because ** dollarsTextField** is of type UITextField!, the assumption is that it is bound via a StoryBoard binding and should never be null. In your case it is. I would recommend checking in StoryBoard the binding and make sure it’s linked with the right variable.

[pavelar]

IMHO you are using ! way too much but regardless. if you look at your examples of code you tried, you missed checking the text value of dollarsTextField, maybe you were staring at your code too long.

So the following should work

if dollarsTextField.text != nil {

[herm57]

Hi pavelar, I agree with you on all of the ! marks. Ive tried to minimize
them with no success, I changed the first line in the if statement but Im still crashing on dollarsTextField with Unexpectedly found nil while unwrapping an Optional value

And roberto I believe that the binding is ok.

Are optionals always this difficult?

Thank you guys, herm

[pavelar]

IMHO, optionals are great and should force the developer to take the time to think through how to handle.

In your case you are making an assumption that a value exists, as we know users will not always do what they are supposed to do. Each of your UITextFields that are needed in a calculation should be verified before proceeding, probably in a separate func so you can show them a some message that the field is required if they leave it blank.

Anyways hope that is helpful, this is normal for forms.