我不确定这个编码有什么问题？ [关闭]

Question

关闭。 这个问题需要调试细节。 它目前不接受答案。

---

想改善这个问题吗？ 更新问题，使其成为 Stack Overflow的主题。

5年前关闭。

改进这个问题

下面是我对冒泡排序的实现。

它没有给我任何错误。 但是我仍然没有得到预期的输出？

package working_5_hours_coding_feels_good;

public class Working_5_hours_coding_feels_good {

    private static final int N = 3;
    public static void main(String[] args) {
        int i,j,temp ;
        int []A=new int[N];
        A[0]=12;
        A[1]=53;
        A[2]=2;
        System.out.println("this is in real form No sorting");
        for(i=0;i<N;i++)
        {
        System.out.print(A[i]+"  ");
        }
          for(j=0;j<N;j++)
            {
              for(i=0;i<N;i++)
                {
                      if (A[i]>A[i+1])
                        {
                         temp=A[i];
                          A[i]=A[i+1];
                          A[i+1]=temp;
                          }

                }

            }
        System.out.println("this is in bubble sorting procedure");
        for(i=0;i<N;i++)
        {
        System.out.println(A[i]+"  ");
        }  
    }

}

Answer 1

你有一个 java.lang.ArrayIndexOutOfBoundsException。 在这部分代码中，您有这个循环：

for (i = 0; i < N; i++) {
            if (A[i] > A[i + 1]) {
                temp = A[i];
                A[i] = A[i + 1];
                A[i + 1] = temp;
            }
 }

您正在迭代直到 N，并尝试到达 N+1 行 (A[i+1]) 之后的行，这会导致错误。

如果您以这种方式迭代，您将不再有错误：

for (i = 0; i < N - 1; i++) {

Answer 2

更改内部 for 循环 - 您现在使用 limit until N 并使用数组索引为 [i+1] 因此在某个时间点您点击 N+1 这会引发 ArrayIndexOutOfBoundsException。

将该条件更改为 ->

for (i=0; i<N-1;i++) {
            if (A[i] > A[i + 1]) {
                temp = A[i];
                A[i] = A[i + 1];
                A[i + 1] = temp;
            }
 }

现在它应该可以工作了。