Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Use logical operator as combine closure in reduce

I am trying to reduce an array of Bools by applying the logical operator OR (||) using the following code, however I get an error:

func reduceBools(values: [Bool]) -> Bool {     return values.reduce(false, combine: ||) } 

Ambiguous reference to member '||'

Analogously for integers the code works like a charm.

func reduceInts(values: [Int]) -> Int {     return values.reduce(0, combine: +) } 

I was able to make it work by adding a || function (code below) or using a { $0 || $1 } closure but I dislike these approaches and I would prefer simply passing the operator.

func ||(lhs: Bool, rhs: Bool) -> Bool {     return lhs || rhs } 

The same thing happens for the logical AND (&&) operator.

How can I make it work without using the hack above?

like image 389
fpg1503 Avatar asked Jan 09 '16 22:01

fpg1503


People also ask

Which operators can be used to combine conditions?

You can use the AND and OR operators to combine two or more conditions into a compound condition. AND, OR, and a third operator, NOT, are logical operators. Logical operators, or Boolean operators, are operators designed to work with truth values: true, false, and unknown.


2 Answers

As an alternative, you could use the following approach

// || func reduceBoolsOr(values: [Bool]) -> Bool {     return values.contains(true) }  // && func reduceBoolsAnd(values: [Bool]) -> Bool {     return !values.contains(false) } 

Note that .reduce comes with an overhead. If the end result is the importance of your question (rather than enquiring above the unexpected behaviour of || and && operators in this context), then perhaps the pragmatic approach above can be of help, even if it doesn't really reduce the array, however producing the same result due to the simple nature of the boolean type.

like image 81
dfrib Avatar answered Oct 04 '22 00:10

dfrib


Swift 4.2+ / Xcode 10.0+

In modern versions of Swift there is allSatisfy function, which checks all elements for satisfying some rule.


In OP's case:

values.allSatisfy { $0 } 
like image 32
えるまる Avatar answered Oct 04 '22 01:10

えるまる