Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Swift expression was too complex to be solved in reasonable time

I'm having an error when compiling a project in Xcode, it says:

Expression was too complex to be solved in reasonable time; consider breaking up the expression into distinct sub-expressions

here's the code:

static func random(min: CGFloat, max: CGFloat) -> CGFloat {
    return CGFloat(Float(arc4random()/0xFFFFFFFF) * (max - min) + min)
}
like image 736
lagro23 Avatar asked Nov 19 '15 03:11

lagro23


1 Answers

Why not reduce the complexity for the compiler by breaking the expression down into two sub-expressions?

static func random(min: CGFloat, max: CGFloat) -> CGFloat {
    let rand = CGFloat(arc4random()/0xFFFFFFFF)
    return (rand * (max - min) + min)
}

You can also use UINT32_MAX (or the more "Swifty" UInt32.max or .max) in place of 0xFFFFFFFF to improve readability. If I recall, 0xFFFFFFFF is the hex value of the max value of an unsigned 32-bit Integer as defined in the <stdint.h> header.

#define UINT32_MAX 0xffffffff  /* 4294967295U */
like image 131
JAL Avatar answered Oct 22 '22 08:10

JAL