《Java数据结构和算法（第二版）》第2章编程作业2.6中noDup方法错误分析及纠正方法

原创于 2008-08-04 17:42:00 发布 · 1.5k 阅读

CC 4.0 BY-SA版权

文章标签：

7 篇文章

订阅专栏

博客围绕数据结构和算法，运用Java进行编程相关内容。虽无具体内容，但可知涉及信息技术领域的数据结构、算法知识及Java编程实现。

11 条评论

GenRong 2009.08.06
参考了各位的做法后，得到我的解法 public void noDup() { int i,j,count = 0; for (i = 0; i < nElems; i ) { for (j = i 1; j < nElems; j ) { if (i != j && a[j] != -1 && a[i] == a[j]) { a[j] = -1; count ; } } } long[] arr = new long[nElems - count]; for (i = 0, j = 0; i < nElems; i ) { if (a[i] != -1) { arr[j ] = a[i]; } } a = arr; nElems -= count; }

acm456 2009.02.11
public void noDup() { for(int i=0;i<nElems;i++) { for(int j =i+1;j<nElems;j++) { if(a[i]==a[j]) { a[j] = -1; } } } for(int i=nElems-1; i>=0; i--) { if(a[i] == -1) delete(-1); } } 上面的我也写错了，改正写，最后那个循环貌似是脱裤子放屁
- 半点闲回复acm456 2009.02.24
  “最后那个循环貌似是脱裤子放屁”，在我的代码中最后那个循环是绝对必要的，请注意作业要求“然后将所有的null删除,当然还要缩小数组的大小。” 你的代码中并没有达到上述作业要求，且每次都调用delete()方法效率是非常慢的，数组中元素越多所带来的效率问题将成倍增长。

acm456 2009.02.11
楼上的，你这个算法不能删除最后一个-1 public void noDup() { for(int i=0;i<nElems;i++) { for(int j =i+1;j<nElems;j++) { if(a[i]==a[j]) { a[j] = -1; } } } for(int i=nElems-1; i>=0; i--) { if(a[i] == -1) delete(-1); } for(int i=0; i<nElems; i++) { if(a[i]==-1) { a[i] = a[i+1]; nElems--; } } } 这么写应该可以解决你的问题

ad198622 2008.10.23
我的思路是这样的：假如有十个数据，而本题中为nElems个数据，对于第一个数据，只需要跟后面的九个比，第二个数据只要跟后面的八个比较，第三个数据只要跟后面的七个比较，依此类推，就不会把自身给干掉（也就是设成-1）了。接着，再用highArray.java中已经写好的delete函数，把含(-1)的删了就可以了，代码如下： for(int i=0;i<nElems;i++){ for(int j =i+1;j<nElems;j++){ if(a[i]==a[j]){ a[j] = -1; } } } for(int i=0;i<nElems;i++){ if(a[i]==-1){ delete(a[i]); } } }

kangeroo2008 2008.09.11
还漏了一个判断条件： public void noDup(){ int i; int j; for (i = 0; i < nElems; i++) for (j = 0; j < nElems; j++){ if (i == j){ continue; } else if (a[i] == a[j]){ if (i == (nElems - 1)){ nElems--; } a[j] = a[--nElems]; } } }
- 半点闲回复kangeroo2008 2008.09.11
  有意义吗？？？

kangeroo2008 2008.09.10
这样缩进应该容易看一点： public void noDup() { int i,j; for (i = 0; i < nElems; i++) for (j = 0; j < nElems; j++) { if (i == j) { continue; } else if (a[i] == a[j]) { a[j] = a[--nElems]; } } }

kangeroo2008 2008.09.10
public void noDup(){ int i,j; for(i=0;i<nElems;i++){ for(j=0;j<nElems;j++){ if(i == j) continue; else{ if(a[i] == a[j]){ a[j] = a[--nElems]; } } } } }

tornadoyang 2008.08.27
public void noDup() { int j; for(j=0;j<nElems;j++) for(int k=0;k<nElems;k++) if(j!=k) if(a[j]==a[k]) { for(int i=j;i<nElems;i++) a[i]=a[i+1]; nElems--; //delete(a[j]); } } 我的代码是这样的思路是数组的每一项跟每一项进行比较，但自身不比较请楼主提出一下高见
- 半点闲回复tornadoyang 2008.08.27
  1、“if(j!=k)自身不比较”，我的代码中不存在自身与自身比较的问题。请看下面的代码 for(int j=0; j<i; j++)//注意j<i时下面代码才得到执行。举例你可以思考一下，我就不详细说明了：第0趟：i=0,j=0。j<i?条件不成立循环结束。第1趟：i=1,j=0。j<i?条件成立，执行内循环中的语句。 j++, j=1 i=1,j=1。j<i?条件不成立，循环结束。第2趟：i=2,j=0。j<i?.......依次类推。每完成一次内部循环后就右移一位，因此算法就不再处理比较过后的数据了。 2、for(int i=j;i<nElems;i++) a[i]=a[i+1]; j后面的数据都向前移动一个位置，但这样就导致了很长的O(N2)的时间级，起码在有很多重复数据项的情况下是这样的，所以我暂不采用。 3、nElems--; //delete(a[j]); 书中要求“当然还要缩小数组的大小”，上述语句只是说明了数组中目前有多少数据项。