3 回答
TA贡献1804条经验 获得超8个赞
您必须添加return coins;at 和 ofchange方法,但您可以保留它的方式。返回和更改数组是一种代码味道,因为该方法既对对象进行操作(修改它)并返回结果。
为了使其工作,您可以denomination按如下方式定义您的方法:
public static List<Integer> denominations(int amount) {
List<Integer> result = new ArrayList<Integer>();
change(amount, result, 0);
return result;
}
编辑:
该列表是空的,因为它唯一改变的地方是这里:
coins.add(DENOMINATIONS[pos]);
change(remaining - DENOMINATIONS[pos], coins, pos);
coins.remove(coins.size() - 1);
添加和删除元素的位置。是你写的让它空了:)
编辑2:
我建议传递第二个对象,这将是您要返回的列表的副本,并且不会被修改。
TA贡献1811条经验 获得超5个赞
您似乎认为 java 是通过引用传递的,这是不正确的。Java 方法是按值传递的。
我已经更新了你的代码:
改变方法:
private static List<Integer> change(int remaining, List<Integer> coins, int pos) { // Updated method return type;
if (pos < 0 || pos >= DENOMINATIONS.length) { // check if position is invalid
return new ArrayList<>(); // return an empty list
}
if (remaining == DENOMINATIONS[pos]) { // check if remaining is equal to denominations[pos]
coins.add(DENOMINATIONS[pos]); // add the denominations to the coins result
return coins; // return the result
} else if (remaining > DENOMINATIONS[pos]) { // check if remaining is greater than denominations[pos]
coins.add(DENOMINATIONS[pos]);// add the possible denominations to the coins result
remaining = remaining - DENOMINATIONS[pos]; // calculate the new remaining
if (remaining >= DENOMINATIONS[pos]) { // check if remaining is greater than or equal to denominations[pos]
return change(remaining, coins, pos); // stick to this position
} else {
return change(remaining, coins, pos + 1); // increment pos to go to the lower denominations
}
} else if (remaining < DENOMINATIONS[pos]) { // check if remaining is lesser than denominations[pos]
if (coins.isEmpty()) { // if coins is empty then go to the next denomination
return change(remaining, coins, pos + 1);
} else {
coins.remove(coins.size() - 1); // remove the previous denomination
return change(remaining + DENOMINATIONS[pos - 1], coins, pos); // go back to the previous remaining and // try this DENOMINATIONS[pos]
}
}
return coins;
}
面额法:
public static List<Integer> denominations(int amount) {
return change(amount, new ArrayList<Integer>(), 0);
}
输入:13
输出:[5, 5, 3]
TA贡献1821条经验 获得超4个赞
change应该返回布尔值,表示是否已找到答案。
所以change身体看起来像这样:
if (remaining == 0) {
return true;
}
...
if (change(...)) return true;
...
return false; // It's last line of body
添加回答
举报