unwrapping optionals about swift-style-guide HOT 7 CLOSED

You first example isn't quite right. The first example does not need the if statement:

someOptional?.someFunc()

or

if let s = someOptional {
  s.someFunc()
}

My vote is to use the first form when you only perform a single operation on the optional, but to use the second form if you need to perform multiple operations. For example:

if let s = someOptional {
  s.someFunc()
  s.someOtherFunc()
}

from swift-style-guide.

Yeah, I wasn't clear with my examples. I'm really only interested in the multiline examples -- things where you want to use an if statement.

I agree for single line things we should do the ? inline.
--------- Original Message --------- Subject: Re: [swift-style-guide] unwrapping optionals (#18)
From: "ColinEberhardt" [email protected]
Date: 6/13/14 1:16 am
To: "raywenderlich/swift-style-guide" [email protected]
Cc: "elephantronic" [email protected]

You first example isn't quite right. The first example does not need the if statement:
someOptional?.someFunc() or
if let s = someOptional { s.someFunc() } My vote is to use the first form when you only perform a single operation on the optional, but to use the second form if you need to perform multiple operations. For example:
if let s = someOptional { s.someFunc() s.someOtherFunc() } -
Reply to this email directly or view it on GitHub.

from swift-style-guide.

I like the second style myself since it avoids the unwrapping within a nice familiar let. But it does depend on naming the optional properly to make it read nicely.

from swift-style-guide.

I think it should be like this:
Use optional binding: if let s = so { } when you only need to unwrap once and have multiple operations to perform on so

Use optional chaining: so?.moreOptionals?.function() if you need to unwrap more than once or only have one operation to perform on so

if so { } else { } should only be used if you just need to check for so's existence and thus it doesn't make sense to bind it to another variable

from swift-style-guide.

Agree completely with @jackwu95

from swift-style-guide.

I agree also with @jackwu95

from swift-style-guide.

Agreed also, let's close this one out.

from swift-style-guide.