Swift:什么时候不适合使用函数式方法
发布于2015年7月24日
原文链接
昨天,我在代码库中发现了我所认为的极具Swift风格的代码。
var minionImages = [UIImage]()
for i in 1...7 {
if let minionImage = UIImage(named: "minionIcon-\(i)") {
minionImages.append(minionImage)
}
}
这让我立刻想到这是使用函数式解决方案来进行重构的绝佳机会。哦吼吼!我可以做函数式编程了!!!是的,我非常激动。
那么,明显的解决方案是在这儿使用map:
let minionImagesMapped = (1...7)
.map { UIImage(named: "minionIcon-\($0)") }
我对自己感到很满意。这段代码只有一行AND函数。但这样做无法满足之前的一个必要条件-这个数组得是UIImages [UIImage]
数组,但是这段代码返回的是可选类型 UIImages [UIImage?]
数组。
所以我google了下,然后为了得到非可选类型UIImages数组的可接受的函数式解决方案是这样的:
let minionImagesFiltered = (1...7)
.map { UIImage(named: "minionIcon-\($0)") }
.filter { $0 != nil}
.map { $0! }
这样的确有用,但是我并不开森。这段代码又长又拙笨,而且我讨厌不得不强迫解包可选类型,尽管这样做是安全的。然而,主要的原因还是这样看起来并不像原生for循环那样简单有效。从以下代码可以看的很清楚:
可能我这样做是错的,但是我最后选择了for循环。从中得到的教训:有时候在Swift中使用for循环是正确的解决方案。当然我可能在这里遗漏了一些东西,很高兴能在评论中听到你们的想法!
升级##
由于有些童鞋在下面的评论中以及Twitter上指出,可以在这里使用Swift2.0的flatMap
作为函数式解决方案:
let minionImagesFlattened = (1...7).flatMap { UIImage(named: "minionIcon-\($0)") }
在这里,我不得不诚实的说,每次我看到有关flatMap
的内容时,我并不能完全理解它做了什么或是怎么做的。它并不像map
以及filter
那样给我的感觉自然。我不懂它在这儿是怎么做的,所以我不得不多查一下。
这让我再次得出结论,我无法确认这对于代码可读性是最好的方式。如果那些对Swift和函数式编程了解很少的人看我的代码的话,他们能理解发生了神马吗?同时,我可以确定的是,每个人只需看一眼就能够理解for循环。