LaVOZs

The World’s Largest Online Community for Developers

'; scala - Block can not contain declarations - LavOzs.Com

This question already has an answer here:

You could make your code work with few fixes:

import scala.util.control.Breaks._ //we need this import to allow breaks since Scala doesn't support them out-of-box

val dysfunctional = udf((ids: Array[Long], values: Array[Double], amount: Double, complain_id: Long) => {
    var fines: String = "" //you need to initalize var
    var amountSum = amount //assign amount to var to allow to reassigment

    breakable {
      ids.indices foreach { i =>
        {
          val (id, value) = (ids(i), values(i))
          val penalty = if (value > amount) amount else value
          amountSum = amountSum - penalty
          fines = fines + (amount, id, complain_id, penalty)
          if (amount <= 0)
            break
        }
      }
    }
    fines
  })

This would work, but many people will frown upon it since it's very non-functional approach and Scala encourages writing functional code. You might try to change it to something like this:


val moreFunctional = (ids: Array[Long], values: Array[Double], amount: Double, complain_id: Long) => {

    val (_, fines) = (ids, values)
      .zipped // zip values and ids to single array of tuples
      .toStream //change it to stream to allow lazy computation
      .scanLeft((amount, "")) { //we pass tuple of amount and empty string as our initial state to scanLeft
        case ((amount, fines), (id, value)) => //second argument of scanLeft is function which receives previous state and currently processed element of array
          val penalty = if (value > amount) amount else value
          (amount, fines + (amount, id, complain_id, penalty).toString()) //here we passs next state for next iteration of scanLeft
      }
      .takeWhile { //we proceed with computations as long and amount is above zero
        case (amount, _) => amount > 0
      }
      .last //we're only interested in last state produced by scan left

    fines
  }