为什么字符串添加需要这么长时间才能构建？

Question

我在UIlabel 中添加文本，以及它的性能成本（我使用了构建时间分析器using this link）。如何优化这段代码？

for value in model?.offerings ?? [] {
    offeringsLabel.text = offeringsLabel.text! + " | " +  (value.name ?? "") + "," +  (value.type ?? "")//this addition cost to performance
}

我也试过[ array].joined，但这并没有什么不同

Answer 1

您可以使用Array.reduce首先获取完整字符串，而不是在每次迭代中将文本分配给UILabel并在下一次再次读取它

let fullString = (model?.offerings ?? []).reduce("", { string, value in
    string + " | " +  (value.name ?? "") + "," +  (value.type ?? "")
}
offeringsLabel.text = fullString

重复设置text 会影响性能，例如，它会触发动态大小标签的大小重新计算

Answer 2

您可以尝试附加功能，例如：

let valueName = value.name ?? ""
offeringsLabel.text?.append(valueName) 

Answer 3

我的建议是首先在Offering 对象中添加一个属性description 以正确处理name 和value（您的解决方案始终在name 和value 之间放置一个逗号，无论name有没有价值）

var description : String? {
    let desc = [name, value].compactMap{$0}.joined(separator:",")
    return desc.isEmpty ? nil : desc
}

而不是循环使用compactMap 和joined

offeringsLabel.text = model?.offerings?.compactMap { $0.description }.joined(separator:" | ") ?? ""

Answer 4

你应该在这里使用临时变量。 运算符?? 在复杂表达式中使用可能会显着增加编译时间

所以您可以使用以下代码更新您的代码（是的，它不短，但我们应该帮助编译器）

let offerings = model?.offerings ?? []
var offeringsText = ""
for value in offerings {
    let name = value.name ?? ""
    let type = value.type ?? ""
    let valueText = " | " +  name + "," +  type
    let offeringsText = offeringsText + valueText
}
offeringsLabel.text = offeringsText

希望对您有所帮助！

Answer 5

首先，对于基本问题。为什么慢？根据我的经验，链式+ 是导致大量编译时性能问题的最常见原因。这是因为+ 有很多重载（我在 4.2 的 stdlib 中计算了 103 个）。 Swift 不仅仅需要证明+ 可以按照你的意愿在(String, String) 上使用。它必须证明没有其他可能的重载组合同样有效（如果有，它将是模棱两可的并且是错误的）。这不仅包括简单的情况，如 (Int, Int)。它还包括适用于 (String, String) 的复杂的基于协议的重载，但不如 (String, String) 精确，例如：

static func + <Other>(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 为 nil 时，你为什么还要运行这段代码？为什么model 是可选的？您是否可以在数据中将其设为非可选，或者您是否应该在此方法的前面使用guard let model = model else { return }？

根据我的经验，导致 Swift 过于复杂的常见原因是过度使用 Optional。 Optional 非常重要和强大，但你不应该使用它，除非你真正的意思是“这里没有任何价值是合法的，并且与'空'不同。”