What about mapUnwrapped ?

- Jay Zhao

> On 6 Dec 2016, at 12:48, David Sweeris <[email protected]> wrote:
> 
> 
>> On Dec 5, 2016, at 7:46 PM, Jay Zhao via swift-evolution 
>> <[email protected] <mailto:[email protected]>> wrote:
>> 
>> Hi there,
>> 
>> Code explains everything:
>> 
>>      
>> 
>> /// Maybe a bad API naming in Swift? See below:
>> 
>>         /// array1 can be array of any object that have a `count` method
>>         let array1 = [[String]]()
>>         let array2 :[String]? = nil
>>         
>>         
>>         // I believe the confusion between `array.map` and 
>> `optionalArray.map` is really bad.
>>         // Because when you read code like this, you can not tell which is 
>> which:
>>         _ = array1.map({$0.count})
>>         _ = array2.map({$0.count})
>>         
>>         // It can be clearer:
>>         // 1, we pass `self.element` into the closure
>>         _ = array1.map({$0.count})
>>         // 2, we pass self directly into the closure
>>         _ = array2.mapMe({$0.count})
>> 
>> 
>> The mapFlat method is also problematic.
> 
> It might be possible to convince me to support renaming map for optionals, 
> but not to mapMe. As it is, you’ll have the same problem with anything having 
> a map function where the element(s) also have a map function. Sets of arrays, 
> arrays of dictionaries, etc. 
> 
> - Dave Sweeris

_______________________________________________
swift-evolution mailing list
[email protected]
https://lists.swift.org/mailman/listinfo/swift-evolution

Reply via email to