未知的StackOverFlow错误,可能与类对象ArrayList和数据存储有关

这是我得到的错误代码:

Exception in thread "main" java.lang.StackOverflowError at JavaKeywords.(JavaKeywords.java:5) at LAddClass.(LAddClass.java:8) at LStore.(LStore.java:9) at LAddClass.(LAddClass.java:9) at LStore.(LStore.java:9) 

以下是上述三个类别。 很抱歉发布了几乎整个程序的代码,但我真的不知道错误在哪里。 下面突出显示错误代码所包含的三行。

这是一个UML-toJava转换器的程序。 用户必须输入类名和类类型(公共/私有),并将变量存储在虚拟类中。 我已经创建了一个类对象的ArrayList来存储用户的输入,String className和boolean isPrivate。 稍后,必须在GUI窗口中打印类名和类型,以便用户复制文本。

我猜错误与将值( String className, String classMethod, boolean isPrivate )存储到ArrayList无关,后者是LStore类对象的arrayList。 我之前有一个Null Pointer Exception错误,它应该与arraylist有关,在我改变了一些代码和类名后,我得到了这个新的stackOverFlow错误。

LAddClass class用于使用checkName()并将bolean isPrivate转换为字符串供以后使用

 public class LAddClass{ private String className; private String methodName; private boolean isPrivate; JavaKeywords keyObject = new JavaKeywords(); LStore stObject = new LStore(className, methodName,isPrivate);//<<<<<<<<<<<<<<<<<<<<<<<<<<< related to the error public String getPublic(){ String s; if (stObject.getIsPrivate() == true) s = " private"; else s = "public"; return s; } public void setPublic(){ } public boolean checkName(String name){ boolean check = true; for (int i=0; i<=stObject.getListSize(); i++){ if (keyObject.containsKeyword(name) || name.equals(stObject.getClassName())){ boolean o = false; check = o; }// end if }// end for return check; }// end checkName }//end class 

LStore class ,它是用于将变量存储到ArrayList

 import java.util.*; public class LStore { public static ArrayList classes = new ArrayList(); public boolean isPrivate; public String className; public String methodName; LAddClass classObject = new LAddClass(); //<<<<<<<<<<<<<<<<<<<<<<<<<<< related to the error public LStore(String name, String method, boolean isP){ this.className = name; this.isPrivate = isP; this.methodName = method; classes.add(this); } public String getClassName(){ return className; } public String getMethodName(){ return methodName; } public boolean getIsPrivate(){ return isPrivate; } public int getListSize(){ return classes.size(); } public String getJavaCode(){ String javaCode = (classObject.getPublic() + " " + className +"{\n"+"\t"+methodName+"\t\n}"+"}"); return javaCode; } } 

这里是GUI类GuiAddClass供用户创建新类。 我认为它可能包含一些错误,所以我会把它作为参考。

 import javax.swing.*; import java.awt.*; import java.awt.event.*; public class GuiAddClass extends JFrame{ LAddClass classObject = new LAddClass(); private JRadioButton publicButton, privateButton; private JLabel clazz; private JTextField inputClassName; private JLabel mothod; private JTextField inputMethodName; private JLabel note; private JButton confirmButton; private JButton cancelButton; public GuiAddClass(){ super("Create class"); setLayout(new FlowLayout()); publicButton = new JRadioButton("public", false); privateButton = new JRadioButton("private", true); clazz = new JLabel("Class Name: "); inputClassName = new JTextField("ExampleClass",10); mothod = new JLabel("Method Name*: "); inputMethodName = new JTextField("doSomething()",10); note = new JLabel("*All methods are public void in default. You may only create one method for a class."); confirmButton = new JButton("Confirm"); cancelButton = new JButton("Cancel"); add(publicButton); add(privateButton); add(clazz); add(inputClassName); add(mothod); add(inputMethodName); add(note); add(confirmButton); add(cancelButton); ButtonGroup group = new ButtonGroup(); group.add(publicButton); group.add(privateButton); Handler handler = new Handler(); NewHandler newhandler = new NewHandler(); confirmButton.addActionListener(handler); cancelButton.addActionListener(newhandler); }// end constructor AddClass() private class Handler implements ActionListener{ public void actionPerformed(ActionEvent event){ String cName = inputClassName.getText(); String mName = inputMethodName.getText(); boolean isP = true; if (classObject.checkName(cName) == false){ JOptionPane.showMessageDialog(null, "Class name invalid. " + "\nEntered name should not contain java keywords or equal to other existing names. " + "\nPlease try again."); } else if (classObject.checkName(cName) == true) { JOptionPane.showMessageDialog(null, "Class saved."); cName = inputClassName.getText(); mName = inputMethodName.getText(); if (event.getSource() == publicButton) { isP = false; } else if (event.getSource() == privateButton) { isP = true; } new LStore(cName, mName, isP); } }// end actionPerformed() }// end Handler class private class NewHandler implements ActionListener{ public void actionPerformed(ActionEvent event){ setVisible(false); } } }// end AddClass 

JavaKeywords class最初来自如何检查类名是否有效? ,一个类,用于检查className是否等于java保留关键字的方法。 它也可能与错误有关,因为这个类似乎经常被检测为错误

 import java.util.ArrayList; import java.util.Arrays; import java.util.HashSet; /** * JavaKeywords is a Utility class related to the reserved keywords * * @author MrLore from https://stackoverflow.com/questions/13979172/how-to-check-if-the-class-name-is-valid */ public class JavaKeywords { private static final HashSet keywords = new HashSet(Arrays.asList(new String[]{ //There are 50 keywords, and 3 literals; true, false and null. "abstract", "assert", "boolean", "break", "byte", "case", "catch", "char", "class", "const", "continue", "default", "do", "double", "else", "enum", "extends", "false", "final", "finally", "float", "for", "goto", "if", "implements", "import", "instanceof", "int", "interface", "long", "native", "new", "null", "package", "private", "protected", "public", "return", "short", "static", "strictfp", "super", "switch", "synchronized", "this", "throw", "throws", "transient", "true", "try", "void", "volatile", "while" , "string", "int" })); public static boolean isKeyword(String toCheck){ return getKeywords().contains(toCheck); }//End isKeyword() public static String[] getAsArray(){ return getKeywords().toArray(new String[getKeywords().size()]); }//End getAsArray() public static ArrayList getAsArrayList(){ return new ArrayList(getKeywords()); }//End getAsArrayList() public static HashSet getAsHashSet(){ return getKeywords(); }//End getAsHashSet() public static HashSet getKeywords() { return keywords; }//End getKeywords public boolean containsKeyword(String toCheck){ toCheck = toCheck.toLowerCase(); //<<<<<<<<<<<<<<<<<<<<<<< this line had been detected as error of null-pointer-exception for(String keyword : keywords){ if(toCheck.equals(keyword) || toCheck.endsWith("." + keyword) || toCheck.startsWith(keyword + ".") || toCheck.contains("." + keyword + ".")){ return true; }//End if }//End for return false; }//End containsKeyword() }//End JavaKeywords 

这就是所有代码! 我的其他类只是GUI类,与数据存储无关,所以我想我不应该发布它们。 感谢您阅读我的问题,如果您有任何想法,请帮助:(

您可以通过阅读stacktrace找到问题:

  • 创建LJKeywords实例时LJKeywords (通过LJKeywords. (LJKeywords.java:10)消息)。
  • 在创建LAddClass实例时(通过LAddClass. (LAddClass.java:8)消息)会发生这种情况。
  • 在创建LStore类的实例时(通过LStore. (LStore.java:9)消息)会发生这种情况。

然后,通过检查LAddClassLStore类构造函数和字段,我发现:

 public class LAddClass{ LJKeywords keyObject = new LJKeywords(); LStore stObject = new LStore(className, methodName,isPrivate); //... } public class LStore { LAddClass classObject = new LAddClass(); //... } 

那么,这是做什么的? 当你创建一个LStore对象引用时,它将在内部创建一个LAddClass对象引用,它在内部将创建一个新的LStore对象引用,它在内部将创建一个LAddClass对象引用,它在内部……好了,现在你看到它的LAddClass ,这里有一个无限的对象实例化循环。

怎么办才能解决这个问题? 删除此无限循环后,您可以通过单独创建LAddClassLStore ,并在客户端类中相互关联。 例如:

 class Client { void foo() { LAddClass lAddClass = new LAddClass(); LStore lStore = new LStore(); lStore.setLAddClass(lAddClass); lAddClass.setLStore(lStore); //... } } 

另外,我建议将类的名称更改为更重要的名称。 例如,使用Storage而不是LAddClass ,而不是LAddClass使用UMLClass (或更合适的名称)。

问题是,为了创建LStore实例,您需要创建一个LAddClass – 并且为了创建LAddClass实例,您需要创建一个LStore实例。 所以一个构造函数正在有效地调用另一个,它正在调用第一个,等等LJKeywords类在这里是一个红色的鲱鱼,我认为 – 这只是在创建一个新的LAddClass实例时调用LJKeywords构造函数,而这恰好是打破骆驼背部的稻草。

从根本上说,您需要弄清楚您希望LStoreLAddClass之间的关系如何工作。 他们无法像这样无条件地相互建构。

我还强烈建议你从所有这些类名中删除L前缀。