Hacker News Reader

Comment by jryan49 | Hacker News Reader

Is it just me or is

  .where(p -> p.getCategory == 2 || p.getCategory == 5 && isProductUnderDiscount(p))

and

  if (product.getCategory() == 2 || product.getCategory() == 5) {
			boolean yes = isProductUnderDiscount(product);

not functionally equivalent due to && precedence?

&& precedence mans that the && arguments are bound first, so:

    .where(p -> p.getCategory == 2 || p.getCategory == 5 && isProductUnderDiscount(p))

is equivalent to:

    if (product.getCategory() == 2 || ( product.getCategory() == 5 && isProductUnderDiscount(product) );

The extra set of parentheses are not needed due to precedence, but have been included for clarification.

Which is not functionally the same as OPs second example which would be equivalent to

    (product.getCategory() == 2 || product.getCategory() == 5) && isProductUnderDiscount(product)