您好,我有一个关于 Batcher 奇偶合并排序的问题。我有以下代码:
public class Batcher {
public static void batchsort(int a[], int l, int r) {
int n = r-l+1;
for (int p=1; p<n; p+=p)
for (int k=p; k>0; k/=2)
for (int j=k%p; j+k<n; j+=(k+k))
for (int i=0; i<n-j-k; i++)
if ((j+i)/(p+p) == (j+i+k)/(p+p))
exch(a, l+j+i, l+j+i+k);
}
public static void main(String[] args) {
int a[] = new int[] {2, 4, 3, 4, 6, 5, 3};
batchsort(a, 0, a.length - 1);
for (int i=0; i<a.length; i++) {
System.out.println(a[i]);
}
}
public static void exch(int a[], int i, int j) {
int t = a[i];
a[i] = a[j];
a[j] = t;
}
}
我会提供一些关于代码功能的注释:
它分为由变量
p
索引的阶段,最后一个阶段是当 p==n
是批处理程序时 奇偶合并 下一个阶段是当 p==n/2
与第一阶段和所有阶段进行奇偶合并时穿过 n/2 的比较器消除了倒数第三个阶段,即当 p==n/4
与前两个阶段奇偶合并时,所有穿过 n/4 的任意倍数的比较器被消除,依此类推。
结果是:
3
3
4
4
5
2
6
我错过了什么?
怎么了?
我不知道算法,但你需要在切换之前比较批量排序中的值,例如
if ((j + i) / (p + p) == (j + i + k) / (p + p))
{
if (a[l + j + i] > a[l + j + i + k])
{
exch(a, l + j + i, l + j + i + k);
}
}
或者如果您使用临时变量作为组合索引,可能会更具可读性,例如
if ((j + i) / (p + p) == (j + i + k) / (p + p))
{
int i1 = l + j + i;
int i2 = l + j + i + k;
if (a[i1] > a[i2])
{
exch(a, i1, i2);
}
}
但是上面的评论者是对的——这确实写得不太可读。您应该尝试确保相等的大括号具有相同的缩进,嵌套的 for() 比其包含的 for 具有更多的缩进,等等,并且您也应该选择更具描述性的变量名称。
'* this assumes array() is globally available
'* Batcher Odd-even mergesort is limited to power of 2 size arrays. it will
'* malfunction in any other case. coded from c source by codeguy (qb64.net)
SUB batcherOddEvenMergeSort (Start&, Finish&)
IF (Finish& > 1) THEN
m& = (Finish&) \ 2
batcherOddEvenMergeSort Start&, m&
batcherOddEvenMergeSort Start& + m&, m&
batcheroddEvenMerge Start&, Finish&, 1
END IF
END SUB
SUB batcheroddEvenMerge (Start&, Finish&, r&)
m& = r& * 2
IF m& > 0 AND m& < Finish& THEN
batcheroddEvenMerge Start&, Finish&, m&
batcheroddEvenMerge Start& + r&, Finish&, m&
i& = Start& + r&
DO
IF i& + m& > Start& + Finish& THEN
EXIT DO
ELSE
IF array(i&) > array(i& + r&) THEN
swap array(i&), array(i& + r&)
END IF
i& = i& + m&
END IF
LOOP
ELSE
IF array(Start&) > array(Start& + r&) THEN
swap array(Start&), array(Start& + r&)
END IF
END IF
END SUB
'* this is adapted from c code and works correctly
这是一个固定的子程序。
void sort(int n)
{
for (int p = 1; p < n; p += p)
for (int k = p; k > 0; k /= 2)
for (int j = k % p; j + k < n; j += k + k)
//for (int i = 0; i < n - (j + k); i++) // wrong line
for (int i = 0; i < k; i++)
if ((i + j)/(p + p) == (i + j + k)/(p + p))
printf("%2i cmp %2i\n", i + j, i + j + k);
}
//This is a fixed subroutine
void batchsort(int a[], int l, int r) {
int n = r - l + 1;
for (int p = 1; p < n; p += p)
for (int k = p; k > 0; k /= 2)
for (int j = k % p; j + k < n; j += (k + k))
for (int i = 0; i < n - j - k; i++)
if ((j + i) % (p + p) < p && a[l + j + i] > a[l + j + i + k])
exch(a, l + j + i, l + j + i + k);
}