作者:网络小爬虫 | 来源:互联网 | 2022-11-27 17:28
我正在添加文本UIlabel
,以及它的性能成本(我使用此链接使用了构建时间分析器).我该如何优化此代码?
for value in model?.offerings ?? [] {
offeringsLabel.text = offeringsLabel.text! + " | " + (value.name ?? "") + "," + (value.type ?? "")//this addition cost to performance
}
我也尝试了,[ array].joined
但这没有任何差异
1> Rob Napier..:
首先,针对潜在的问题.它为什么慢?+
在我的经验中,链接是造成大量编译时性能问题的最常见原因.这是因为+
有很多重载(我在4.2的stdlib中算了103).Swift不仅要证明+
可以(String, String)
按照您的意愿使用它.它必须证明没有其他可能的过载组合同样有效(如果有的话,它是模糊的和错误的).这不仅仅包括像(Int,Int)这样的简单情况.它还包括适用于(String,String)的复杂的基于协议的重载,但不像(String,String)那样精确:
static func + (lhs: String, rhs: Other) -> String
where Other : RangeReplaceableCollection, Character == Other.Element
这需要很长时间,因为当你有一堆+
链接在一起时,它是组合爆炸性的.每个子表达式都必须根据它可能返回的所有内容重新考虑.
怎么解决?首先,我不会for
在这种情况下使用循环逐个构建一个字符串.只需将map
每个Offering
字符串连接到所需的字符串,然后将它们连接在一起
offeringsLabel.text = model?.offerings
.map { "\($0.name ?? ""),\($0.type ?? "")" }
.joined(separator: " | ")
这不仅会更快地编译,而且IMO会更清楚你正在做什么.它也不需要!
,这很好.(还有其他方法可以解决这个问题,但这是一个很好的副作用).
也就是说,这也表明你的模型中存在一个可能的问题.这是一个单独的问题,但我仍然会认真对待它.任何时候你有代码看起来像:
optionalString ?? ""
你需要问一下,这真的是可选的吗?你真的对待一个名字与一个空名称不同吗?如果没有,我相信name
并且type
应该只是String
而不是String?
,然后这变得更加简单.
offeringsLabel.text = model?.offerings
.map { "\($0.name),\($0.type)" }
.joined(separator: " | ")
此代码与您的代码略有不同.如果model
是nil
,你的代码留下offeringsLabel
孤单.我的代码清除了文本.这提出了一个更深层次的问题:为什么你在运行这个代码时model
是否为零?为什么是model
可选的?你能否在数据中使它成为非可选项,或者你是否应该guard let model = model else { return }
在此方法中使用它?
根据我的经验,过度复杂的Swift的常见原因是过度使用Optional.可选是非常重要和强大的,但你不应该使用它,除非你真的意味着"在这里没有任何价值是合法的,而不仅仅是'空'."