web-dev-qa-db-fra.com

Modification de la monnaie Algorithme In scala à l'aide de la récursion

J'essaie de programmer le problème de changement de monnaie dans Scala à l'aide de la récursion. Le code que j'ai écrit est comme suit.

def countChange(money: Int, coins: List[Int]): Int = {
  def ways(change: List[Int], size: Int, capacity: Int): Int = {
    if(capacity == 0) 1
    if((capacity < 0) || (size <= 0)) 0

    //println and readLine to check and control each recursive call.

    println("calling ways(",change, change.length-1, capacity,") + ways(",change,   change.length, capacity - change(change.length - 1),")")
    readLine()
    //

    ways(change, change.length-1, capacity) + ways(change, change.length, capacity - change(change.length - 1))
  }
  ways(coins, coins.length, money)
}

En exécutant le code, il ne se termine pas et continue d'appeler le premier appel récursif. Où est-ce que je vais mal?

16
Muavia

Simplement indiquant qu'une valeur ne marque pas Scala renvoyez-la; vous avez besoin d'un retour explicite, ou il doit être le dernier élément indiqué. Ainsi:

if (capacity == 0) return 1

ou

if (capacity == 0) 1
else if (...)
else { ... }
7
Rex Kerr

Voici ma mise en œuvre: je l'ai testé et ça marche bien

def countChange(money: Int, coins: List[Int]): Int = {

    def count(capacity: Int, changes: List[Int]): Int = {
                if(capacity == 0) 
                  1
                else if(capacity < 0) 
                  0
                else if(changes.isEmpty && capacity>=1 )
                  0
                else
                        count(capacity, changes.tail) + count(capacity - changes.head, changes)
    }

    count(money, coins.sortWith(_.compareTo(_) < 0))
}
19
mysteriousscent

Juste une autre solution

def countChange(amount: Int, coins: List[Int]): Int = coins match {
  case _ if amount == 0 => 1
  case h :: t if amount > 0 => countChange(amount - h, h :: t) + countChange(amount, t)
  case _ => 0
}
7
Carlos Caldas

Hé, je pensais juste qu'il serait préférable de voir non seulement le montant mais aussi la liste d'entre eux, alors mettez-le sur l'exemple ci-dessus comme:

def moneyChanges(money: Int, coins: List[Int]) : Option[List[Seq[Int]]]= {
  var listOfChange=List[Seq[Int]]()
  def changeMoney(capacity: Int, changes: List[Int], listOfCoins: Option[Seq[Int]]): Int = {
    if (capacity == 0) {
      listOfChange = listOfCoins.get :: listOfChange
      1
    } else if (capacity < 0)
      0
    else if (changes.isEmpty && capacity >= 1)
      0
    else {
      changeMoney(capacity, changes.tail, listOfCoins) +
      changeMoney(capacity - changes.head, changes, 
      Some(changes.head +: listOfCoins.getOrElse(Seq())))
    }
  }

  changeMoney(money, coins.sortWith(_.compareTo(_) < 0), None)
  Some(listOfChange)
}
2
Suat KARAKUSOGLU

voici une approche DP pour réduire beaucoup de ré-calcul dans une approche récursive

object DP {
  implicit val possibleCoins = List(1, 5, 10, 25, 100)
  import collection.mutable.Map

  def countChange(amount: Int)(implicit possibleCoins: List[Int]) = {
    val min = Map((1 to amount).map (_->Int.MaxValue): _*)
    min(0) = 0
    for {
      i <- 1 to amount
      coin <- possibleCoins
      if coin <= i && min(i - coin) + 1 < min(i)
    } min(i) = min(i-coin) + 1
    min(amount)
  }

  def main(args: Array[String]) = println(countChange(97))
}

voir DP de débutant à avancé pour l'algorithme

0
Leonmax

Au-dessous du code est similaire à celui de l'exemple ci-dessus, sauf que j'utilise le cas de correspondance au lieu de si

def countChange(money: Int, coins: List[Int]): Int = {
    def change(m: Int, coinList: List[Int], count: Int): Int =
      m match {
        case _ if m < 0 => count
        case _ if coinList.isEmpty => {
          m match {
            case 0 => count + 1
            case _ => count
          }
        }
        case _ => change(m, coinList.tail, count) + change(m - coinList.head, coinList, count)
      }
    change(money, coins, 0)
  }
0
Prasad