我正在编写一个代码来生成数组元素的排列。我写了两种不同类型的交换函数,一种使用临时存储工作正常,而另一种不使用任何临时存储的交换函数不生成输出。为什么会发生这种情况?
以下代码可以正常使用
#include<iostream>
#include<cstdio>
using namespace std;
int tt=0;
void swap1 (int v[], int i, int j) {
int t;
t = v[i];
v[i] = v[j];
v[j] = t;
}
void permute(int arr[],int n,int index)
{
if(index==n)
{
for(int i=0;i<n;i++) printf ("%c", arr[i]) ;
printf("\n");
tt++;
}
else
for(int j=index; j < n ;j++)
{
swap1(arr,index,j);
permute(arr,n,index+1);
swap1(arr,j,index);
}
}
int main()
{
int arr[]={'a','b','c','d'};
permute(arr,4,0);
cout<<endl;
printf("%d\n",tt);
getchar();
}
output:
acbd
acdb
adcb
adbc
bacd
badc
bcad
bcda
bdca
bdac
cbad
cbda
cabd
cadb
cdab
cdba
dbca
dbac
dcba
dcab
dacb
dabc
24
虽然以下代码不输出排列:
#include<iostream>
#include<cstdio>
using namespace std;
int tt=0;
void swap(int v[],int i,int j)
{
v[i]= v[i] + v[j];
v[j]= v[i] - v[j];
v[i]= v[i] - v[j];
}
void permute(int arr[],int n,int index)
{
if(index==n)
{
for(int i=0;i<n;i++) printf ("%c", arr[i]) ;
printf("\n");
tt++;
}
else
for(int j=index; j < n ;j++)
{
swap(arr,index,j);
permute(arr,n,index+1);
swap(arr,j,index);
}
}
int main()
{
int arr[]={'a','b','c','d'};
permute(arr,4,0);
cout<<endl;
printf("%d\n",tt);
getchar();
}
output:
24
答案 0 :(得分:7)
请相信你所信仰的神灵(或其他方面),不要像你的第二个交换功能(或可怕的XOR交换技巧)那样使用丑陋的黑客。
与典型的临时变量解决方案相比,它们完全不必要且通常更慢。它们还会导致您的代码在The Daily WTF这样的网站上出现,而且几代程序员(必须保持这种残骸)会在未来几个世纪中诅咒您的名字: - )
并且,无论如何,如果您“交换”的两个元素是相同的,那么它们将无法工作,因为您即刻:
v[i]= v[i] + v[j];
其中i
和j
的值相同,您丢失了其他步骤所需的信息。
您可以按如下方式查看此操作。假设您有两个不同的变量,a = 20
和b = 30
。完成您的步骤:
a = a + b; // a <- (20 + 30) = 50, b still = 30.
b = a - b; // b <- (50 - 30) = 20, a still = 50.
a = a - b; // a <- (50 - 20) = 30, b still = 20.
并且它们被交换了,尽管你可能想要查看溢出和编码方案的边缘情况,这些方案不是两个补码(无论如何,在C中,不确定C ++是否允许补码或符号幅度)
但是让我们看看当a
和b
是相同的变量(不是相同的值,而是实际相同的)时会发生什么>变量,就像参考一样)。所以他们“两个”的价值都是20,你可以预期他们在交换后仍然是20,但让我们来看看:
a = a + b; // a <- (20 + 20) = 40, AND b = 40 as well.
b = a - b; // b <- (40 - 40) = 0, AND a = 0 as well.
a = a - b; // a <- ( 0 - 0) = 0, AND b = 0 as well.
答案 1 :(得分:2)
void swap(int v[],int i,int j)
{
v[i]= v[i] + v[j];
v[j]= v[i] - v[j];
v[i]= v[i] - v[j];
}
当i == j
(然后将v [i]设置为0)时,不起作用,这是循环中发生的事情
for (int j = index; j < n; ++j) {
swap(arr, index, j);