4

我的代码在数组上运行算法,并将结果存储在 ArrayList 中。问题是我无法访问 ArrayList 的内容以进行后续处理。尽管我的实际代码长达数千行,但我已经解决了这个问题,并在下面的短代码段中重新创建了这个问题。您可以学习下面的三个类并在您的 IDE 中运行它们而无需更改以自己重现问题。如您所见,它在 makeArrayList.java 中填充了 ArrayList,但是 ArrayList 的内容随后在 getArrayList.java 中不可见。

谁能告诉我如何修复下面的代码,以便 ArrayList 的内容在 getArrayList.java 和 myGUI.java 中变得可见/可用?

下面是三个类的代码:

myGUI.java 的代码是:

package arrayListPractice;

import java.awt.Dimension;
import javax.swing.JFrame;

public class myGUI extends JFrame {
public myGUI() {
    super("test GUI");
    this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    this.setPreferredSize(new Dimension(300, 200));
    getArrayList getArrList = new getArrayList();
    getArrList.getPeaks();
    this.pack();}

public static void main(String args[]) {
    myGUI myFrame = new myGUI();
    myFrame.setVisible(true);}}

getArrayList.java 的代码是:

package arrayListPractice;
import java.util.*;

public class getArrayList {
public static ArrayList<Integer> PeakList;
int myLength = 3500;
double[] myArray=new double[myLength];

public ArrayList<Integer> getPeaks(){
    for(int h=0;h<myLength;h++){myArray[h]=Math.sqrt((double)h);}
    PeakList = new makeArrayList(myArray,myLength);
    System.out.println("in getArrayList.getPeaks, PeakList.size() is: "+PeakList.size());
    return PeakList;}}

makeArrayList.java 的代码是:

package arrayListPractice;

import java.util.*;

public class makeArrayList extends ArrayList<Integer> {
ArrayList<Integer> myArrayList= new ArrayList<Integer>();

public makeArrayList(double[] myArray, int arrayLength) {
    // NOTE: My actual code does many transformations to myArray.  The resulting myArrayList 
    // contains only 1/1000 of the points in myArray.  This code is just simplified for debugging.
    for(int i=0;i<arrayLength;i++){myArrayList.add((int)Math.pow(myArray[i],2));}
    System.out.println("in makeArrayList, PeakList.size() is: "+myArrayList.size());}}
4

1 回答 1

4

您在同一个类中混淆和组合继承和组合:

class makeArrayList extends ArrayList<Integer> {
   ArrayList<Integer> myArrayList = new ArrayList<Integer>();

   public makeArrayList(double[] myArray, int arrayLength) {
      // NOTE: My actual code does many transformations to myArray. The
      // resulting myArrayList
      // contains only 1/1000 of the points in myArray. This code is just
      // simplified for debugging.
      for (int i = 0; i < arrayLength; i++) {
         myArrayList.add((int) Math.pow(myArray[i], 2));
      }
      System.out.println("in makeArrayList, PeakList.size() is: "
            + myArrayList.size());
   }
}

请注意,此类都包含一个 ArrayList扩展了 ArrayList 并且您试图使这两个 ArrayLists 可互换,但它们不是。

一些建议:

  • 这个类不需要扩展 ArrayList,所以去掉扩展,而是通过使用组合来简化和澄清。
  • 除非您有充分的理由这样做,否则请避免使用 static 任何东西。这不是您的主要问题的一部分,而是您的示例代码的问题。
  • 为了其他人阅读您的代码并帮助您或为您评分,不要害怕使用空格来使您的代码更具可读性。页面房地产并不昂贵。还要阅读并使用 Java 命名约定,包括将类名的首字母大写。这将使其他人(我们!)更容易阅读和理解您的代码。

例如,

import java.util.ArrayList;

public class MyNonGUI2  {

   public static void main(String args[]) {
      GetArrayList2 getArrList = new GetArrayList2();
      getArrList.getPeaks();
   }
}

class GetArrayList2 {
   public ArrayList<Integer> PeakList;
   int myLength = 3500;
   double[] myArray = new double[myLength];

   public ArrayList<Integer> getPeaks() {
      for (int h = 0; h < myLength; h++) {
         myArray[h] = Math.sqrt((double) h);
      }
      PeakList = new MakeArrayList2(myArray, myLength).getArrayList();
      System.out.println("in GetArrayList2.getPeaks, PeakList.size() is: "
            + PeakList.size());
      return PeakList;
   }
}

class MakeArrayList2 {
   ArrayList<Integer> myArrayList = new ArrayList<Integer>();

   public MakeArrayList2(double[] myArray, int arrayLength) {
      for (int i = 0; i < arrayLength; i++) {
         myArrayList.add((int) Math.pow(myArray[i], 2));
      }
      System.out.println("in MakeArrayList2, PeakList.size() is: "
            + myArrayList.size());
   }

   public int size() {
      return myArrayList.size();
   }

   public ArrayList<Integer> getArrayList() {
      return new ArrayList<Integer>(myArrayList);
   }
}
于 2011-09-09T02:05:30.397 回答