[swift-evolution] removeFirst, optional equiviliant

[Max Moiseev]

Ahhh, right.

I believe the thinking here is that since this is an avoidable error, it should be handled in the client code with an `if !array.isEmpty { … }`), leaving errors to really exceptional and unexpected conditions.
Using optional here will serve the same purpose, IMHO, but instead of preventing the condition, one would have to react to the consequences later. Moreover the type will now be Optional<Element> and it would also be really tempting to write something like `array.removeFirst()!` and have the same trapping behavior.

Dave, Dmitri, please correct me if I’m wrong.

max

> On Jan 6, 2016, at 10:34 AM, James Campbell <james at supmenow.com> wrote:
> 
> What I mean't is it would be great is if it was a native swift error :) so we could use try? syntax.
> 
> On Wed, Jan 6, 2016 at 6:32 PM, Max Moiseev <moiseev at apple.com <mailto:moiseev at apple.com>> wrote:
> Hi James,
> 
> I believe this code <https://github.com/apple/swift/blob/master/stdlib/public/core/RangeReplaceableCollectionType.swift#L235> already handles empty array scenario by failing if the precondition is not met.
> Or do you have something else in mind?
> 
> max
> 
>> On Jan 6, 2016, at 9:36 AM, James Campbell via swift-evolution <swift-evolution at swift.org <mailto:swift-evolution at swift.org>> wrote:
>> 
>> If you call removeFirst and the array is empty it would be great if it was optional so it could return nil or at least it threw an error so you could handle that case.
>> 
>> -- 
>>  Wizard
>> james at supmenow.com <mailto:james at supmenow.com>
>> +44 7523 279 698 <tel:%2B44%207523%20279%20698> _______________________________________________
>> swift-evolution mailing list
>> swift-evolution at swift.org <mailto:swift-evolution at swift.org>
>> https://lists.swift.org/mailman/listinfo/swift-evolution <https://lists.swift.org/mailman/listinfo/swift-evolution>
> 
> 
> 
> 
> -- 
>  Wizard
> james at supmenow.com <mailto:james at supmenow.com>
> +44 7523 279 698

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.swift.org/pipermail/swift-evolution/attachments/20160106/f94dbe4d/attachment.html>