For-Schleife Mismatch Fehlertyp in Scala

Question

ich lerne Scala, aber ich bin nicht in der Lage die for-Schleife in diesem Code Trog gehen

def adjacentElementsProduct(inputArray: Array[Int]): Int = { 

    var maxSoFar = 0 
    var maxHere = 0 

    //for (i <- 0:Int to (inputArray.length-1))  <- error 
    for (i <- 0 to inputArray.length-1)   //<- error 
    { 
      if(maxHere * inputArray(i) > 0) 
       maxHere *= inputArray(i) 
      else 
       maxHere = 0 

      if(maxHere > maxSoFar) 
       maxSoFar = maxHere 

      maxSoFar 
    } 

} 

Compiler Ergebnisse:

(ohne: Int)

file.scala on line 6: error: type mismatch; 
found : Unit 
required: Int 
     for (i <- 0 to inputArray.length-1) 
      ^

(mit: Int)

file.scala on line 6: error: identifier expected but integer literal found. 
    for (i <- 0:Int to (inputArray.length - 1)) 
              ^
file.scala on line 19: error: ')' expected but '}' found. 
} 
^ 

Was ist los? Wie kann ich es lösen?

Vielen Dank, Davide

Answer 1

adjacentElementsProduct Bedürfnisse ein Int zurückzukehren, müssen Sie setzen, was Sie am Ende des Verfahrens zurückkehren wollen (außerhalb der for-Schleife).

Sie können auch for (i <- 0 to inputArray.length-1) zu for (i <- inputArray) vereinfachen und inputArray(i) durch i ersetzen.

Answer 2

der Rückgabewert maxSoFar sollte die aus for-Schleife {}, wie folgt aus:

def adjacentElementsProduct(inputArray: Array[Int]): Int = { 

    var maxSoFar = 0 
    var maxHere = 0 

    //for (i <- 0:Int to (inputArray.length-1))  <- error 
    for (i <- 0 to inputArray.length-1)   //<- error 
    { 
      if(maxHere * inputArray(i) > 0) 
       maxHere *= inputArray(i) 
      else 
       maxHere = 0 

      if(maxHere > maxSoFar) 
       maxSoFar = maxHere 


    } 
    maxSoFar 

} 

Answer 3

Wenn das maximale Produkt von benachbarten nicht-Null ganzen Zahlen zu tun, was Sie versuchen, hier wird zurückkehren ein mehr ist Scala-esque Ansatz.

def adjacentElementsProduct(inputArray: Array[Int]): Int = 
    inputArray.scanLeft(0)((a,b) => if (a==0) b else a*b).max 

---

Erklärung (etwas vereinfacht)

Betrachten Sie es auf diese Weise: myArray.scanLeft(init)(op) wird ein neues Array bauen, wie etwas aussieht:

Array(init 
    , op(init,    myArray(0)) 
    , op(previousOpResult, myArray(1)) 
    , op(previousOpResult, myArray(2)) 
    . . . 
    )