Java 为什么我的递归解决方案打印重复项?
问题是: 编写一个静态方法makeChange,该方法使用递归回溯来找到所有方法,使用便士(1美分)、镍币(5美分)、十美分(10美分)和二十五美分(25美分)对给定金额的货币进行更改。例如,当零钱换37美分时,你可以使用:1个25美分、1个10美分和2个便士;3个一角硬币和7个便士;或其他组合 您的方法应该接受一个参数:要进行更改的美分数。您的方法的输出应该是每种类型硬币的所有组合的序列,每行一个,加起来就是这个数量。例如,如果客户端代码包含以下调用: System.out.println(“p N D Q”)Java 为什么我的递归解决方案打印重复项?,java,recursion,Java,Recursion,问题是: 编写一个静态方法makeChange,该方法使用递归回溯来找到所有方法,使用便士(1美分)、镍币(5美分)、十美分(10美分)和二十五美分(25美分)对给定金额的货币进行更改。例如,当零钱换37美分时,你可以使用:1个25美分、1个10美分和2个便士;3个一角硬币和7个便士;或其他组合 您的方法应该接受一个参数:要进行更改的美分数。您的方法的输出应该是每种类型硬币的所有组合的序列,每行一个,加起来就是这个数量。例如,如果客户端代码包含以下调用: System.out.println(“
System.out.println(“-------------”)
权宜之计(28)
生成的总输出应如下所示: pndq
------------
[3,0,0,1]
[3,1,2,0]
[3,3,1,0]
[3,5,0,0]
[8,0,2,0]
[8,2,1,0]
[8,4,0,0]
[13,1,1,0]
[13,3,0,0]
[18,0,1,0]
[18,2,0,0]
[23,1,0,0]
[28,0,0,0]
我的解决方案是保留一个与四个面额对应的整数列表。另一种方法处理列表中“硬币”的总价值。递归方法增加每个面额的“硬币”数量,直到总和等于给定值,然后打印。不幸的是,我的解决方案打印出了每种可能的硬币组合的许多副本。为什么?
public static void makeChange(int n) {
ArrayList<Integer> combos = new ArrayList<Integer>();
combos.add(0);
combos.add(0);
combos.add(0);
combos.add(0);
System.out.println(" P N D Q");
System.out.println("------------");
makeChange(n, combos);
}
public static void makeChange(int n, ArrayList<Integer> combos) {
int sum = getSum(combos);
if (sum == n) {
System.out.println(combos.toString());
} else {
for (int i = 0; i < combos.size(); i++) {
combos.set(i, combos.get(i) + 1);
if (getSum(combos) <= n) {
makeChange(n, combos);
}
combos.set(i, combos.get(i) - 1);
}
}
}
public static int getSum(ArrayList<Integer> combos) {
int sum = combos.get(0);
sum += 5 * combos.get(1);
sum += 10 * combos.get(2);
sum += 25 * combos.get(3);
return sum;
}
publicstaticvoidmakechange(intn){
ArrayList组合=新建ArrayList();
组合。添加(0);
组合。添加(0);
组合。添加(0);
组合。添加(0);
System.out.println(“P N D Q”);
System.out.println(“-------------”);
makeChange(n,组合);
}
公共静态void makeChange(int n,ArrayList组合){
int sum=getSum(组合);
如果(总和=n){
System.out.println(combos.toString());
}否则{
对于(int i=0;i 如果(getSum(combos)您的for
循环每次调用递归函数时都将i
变量重置为零,这会导致重复输出
您需要将此变量作为输入参数传入:
public static void makeChange(int n) {
List<Integer> combos = new ArrayList<Integer>(); // you should declare as List, not ArrayList
combos.add(0);
combos.add(0);
combos.add(0);
combos.add(0);
System.out.println(" P N D Q");
System.out.println("------------");
makeChange(n, 0, combos);
}
public static void makeChange(int n, int i, List<Integer> combos) {
int sum = getSum(combos);
if (sum == n) {
System.out.println(combos.toString());
} else {
while (i < combos.size()) {
combos.set(i, combos.get(i) + 1);
if (getSum(combos) <= n) {
makeChange(n, i, combos);
}
combos.set(i, combos.get(i) - 1);
++i;
}
}
}
对于makeChange(6)
,它输出:
P N D Q
------------
[6, 0, 0, 0]
[1, 1, 0, 0]
请记住,p
编辑
要反转输出,可以先将输出写入列表,然后在处理完成后输出列表:
public static void makeChange(int n) {
List<Integer> combos = new ArrayList<Integer>();
List<String> output = new ArrayList<String>();
combos.add(0);
combos.add(0);
combos.add(0);
combos.add(0);
System.out.println(" P N D Q");
System.out.println("------------");
makeChange(n, 0, combos, output);
for(String s: output) {
System.out.println(s);
}
}
public static void makeChange(int n, int i, List<Integer> combos, List<String> output) {
int sum = getSum(combos);
if (sum == n) {
output.add(0, combos.toString());
} else {
while (i < combos.size()) {
combos.set(i, combos.get(i) + 1);
if (getSum(combos) <= n) {
makeChange(n, i, combos, output);
}
combos.set(i, combos.get(i) - 1);
++i;
}
}
}
如果您也显示了输出,这会有所帮助。您在代码中的什么时候检查组合是否已被记录?我看不到,但可能只是我的眼睛。从该代码中获得重复输出是有意义的。我不确定您是否可以避免它。您可以跟踪以前的解决方案,而不是打印出来,但是在这个递归代码中,这将是非常混乱的。不是一次投票,不是一次接受,不是一个答案?但是可能有一种完全不同的方法使用递归,不会产生重复。谢谢。但是我如何才能反转打印顺序,使其与预期输出匹配?如果您只是反转遍历顺序,但使用相同的方法程序,它将不起作用。已更新函数以反转输出
public static void makeChange(int n) {
List<Integer> combos = new ArrayList<Integer>();
List<String> output = new ArrayList<String>();
combos.add(0);
combos.add(0);
combos.add(0);
combos.add(0);
System.out.println(" P N D Q");
System.out.println("------------");
makeChange(n, 0, combos, output);
for(String s: output) {
System.out.println(s);
}
}
public static void makeChange(int n, int i, List<Integer> combos, List<String> output) {
int sum = getSum(combos);
if (sum == n) {
output.add(0, combos.toString());
} else {
while (i < combos.size()) {
combos.set(i, combos.get(i) + 1);
if (getSum(combos) <= n) {
makeChange(n, i, combos, output);
}
combos.set(i, combos.get(i) - 1);
++i;
}
}
}
P N D Q
------------
[3, 0, 0, 1]
[3, 1, 2, 0]
[3, 3, 1, 0]
[3, 5, 0, 0]
[8, 0, 2, 0]
[8, 2, 1, 0]
[8, 4, 0, 0]
[13, 1, 1, 0]
[13, 3, 0, 0]
[18, 0, 1, 0]
[18, 2, 0, 0]
[23, 1, 0, 0]
[28, 0, 0, 0]