Question

你能否帮我在C＃代码中发现错误来计算数组中的反转？计数不正确。例如，在数组{3,8,6,1}中它应该是4，但它是3.但是，它适用于数组{4,3,2,1}并显示6。程序读取的文本文件（1111.txt）中的每个字符串都包含1个数组元素。谢谢。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.IO;

namespace C
{
class Program
{
public static long k1 = 0;
static long Marge_sort(int[] A, int p, int r)
{
int q = p;
if (r <= p) return 0;
q = (p + r) / 2;
C.Program.k1 = Marge_sort(A, p, q);
C.Program.k1 += Marge_sort(A, q + 1, r);
C.Program.k1 += Marge(A, p, q, r);
return C.Program.k1; 
}
static long Marge(int[] A, int p, int q, int r)
{
long k1 = 0;

int n1 = q - p+1 ;
int[] L = new int[n1+1];
int n2 = r - q;
int i;
int[] R = new int[n2+1];
for(i=0;i<n1;i++)
{
L[i] = 0;
L[i] = A[p + i ];
}
L[n1] = int.MaxValue;
for (i = 0; i < n2; i++)
{
R[i] = 0;
R[i] = A[q + i+1];
}
R[n2] = int.MaxValue;
int j = 0; i = 0;
for(int k=p;k<=r;k++)
{
if (L[i] <= R[j])
{
A[k] = L[i];
i++;
}
else
{
A[k] = R[j];
j++;
k1 = q - i+1 ;
}

}
return k1;

}
static void Main(string[] args)
{
long k1 = 0;
int leght = 0, el = 0;
int[] Ch = new int[leght];
StreamReader s = File.OpenText(@"D:\1111.txt");//IntegerArray
string read = null;
while ((read = s.ReadLine()) != null)
{
//Console.WriteLine(read);
leght++;
Array.Resize(ref Ch, leght);
Ch[el] = Convert.ToInt32(read);
el++;
}
s.Close();
k1=Marge_sort(Ch, 0, leght-1 );

Console.WriteLine("\n"+k1.ToString());
Console.ReadLine();

}

}
}

Answer 1

对于数组{3,6,8,1}，正确的答案是3：

{3,6,8,1} - ＆gt; {3,6,1,8} - ＆gt; {3,1,6,8} - ＆gt; {1,3,6,8}

Answer 2

接近Marge

末尾的行

k1 = q - i+1 ;

应该是

k1 += n1 - i;

首先，您要保持L元素R必须在前面移动的元素的总数。因此，运营商应+=而不是=。

其次q测量原始数组A中的位置。您正在计算小数组L中的许多元素，因此您应该从n1中减去索引，即该数组的大小（不包括保护值）。如果A很大，你就会开始看到疯狂的结果：当你合并只有少数元素的小子阵列，但非常接近A的顶部时，你会看到巨大的值q，与L的大小无关。

当您确定代码正常工作时，您应该转到codereview.stackexchange.com。它们只能帮助处理代码，但它们会为您提供更多关于如何以更清晰，更易读的方式编写内容的指针。你的代码充满了小错误，比如不必要的命名空间，未使用的初始化等等（但我认为这是让它在担心这些之前正常工作的正确方法）。

数组中的反转次数不正确

2 个答案: