0

I am able to do this:

  def largestAt(fun: (Int) => Int, inputs: Seq[Int]):Int = {
    inputs.reduceLeft(math.max(_,_))
  }

But, when I am trying to do this: 

def largestAt(fun: (Int) => Int, inputs: Seq[Int]):Int = {
   inputs.reduceLeft(math.max(fun(_),fun(_)))
}

I am getting below mentioned compilation error:

overloaded method value max with alternatives:
[error]   (x: Double,y: Double)Double <and>
[error]   (x: Float,y: Float)Float <and>
[error]   (x: Long,y: Long)Long <and>
[error]   (x: Int,y: Int)Int
[error]  cannot be applied to (Int => Int, Int => Int)
[error]     inputs.reduceLeft(math.max(fun(_),fun(_)))

I just want to pass the result of the fun to math.max call. How can I achieve this.

Thanks in advance...

Mario Galic
  • 47,285
  • 6
  • 56
  • 98
Abhay Kumar
  • 522
  • 3
  • 9

1 Answers1

3

Anonymous function placeholder parameter syntax in the expression 

inputs.reduceLeft(math.max(fun(_),fun(_)))

expands to something like

inputs.reduceLeft(math.max(a => fun(a), b => fun(b)))

and max does not take a pair of functions as input. Instead, you are likely after something like

inputs.reduceLeft((a, b) => math.max(fun(a), fun(b)))

On the other hand the following usage of underscore works

inputs.reduceLeft(math.max(_, _))

because it expands to

inputs.reduceLeft((a, b) => math.max(a, b))

Placeholder syntax should probably be used judiciously, and the key to using it correctly is understanding the scope of underscore:

If the underscore is inside an expression delimited by () or {}, the innermost such delimiter that contains the underscore will be used;

Mario Galic
  • 47,285
  • 6
  • 56
  • 98
  • The trick is to stop using it for anything that is not a simple expression _(like `foo.map(_.x)` or `bar.reduce(_ + _)`)_. If I would get a dollar everytime someone have a problem because they used the `_` syntax... – Luis Miguel Mejía Suárez May 14 '20 at 12:50