Wieso java.lang.StackOverflowError (minimales programm)

[Fred1]

Hi!

Ich steh gerade voll an =(

Ich hab hier mal schnell ein kleines beispiel geschrieben das den selben fehler produziert wie mein eigentliches programm...

Beschreibung: Ich hab 2 Klassen (graphics1, test). In jeder Klasse erzeuge ich ein Objekt um auf die Attribute , Methoden der anderen klasse zugreifen zu können. Allerdings bekomm ich folgende Fehlermeldung beim ausführen:

Exception in thread "AWT-EventQueue-0" java.lang.StackOverflowError
at sun.awt.windows.WToolkit.getScreenInsets(Native Method)
at sun.awt.windows.WToolkit.getScreenInsets(Unknown Source)
at java.awt.Window.init(Unknown Source)
at java.awt.Window.<init>(Unknown Source)
at java.awt.Frame.<init>(Unknown Source)
at java.awt.Frame.<init>(Unknown Source)
at javax.swing.JFrame.<init>(Unknown Source)
at test.graphics1.<init>(graphics1.java:29)
at test.test.<init>(test.java:5)...........
.....

graphics1 Klasse:

package test;
import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;

import javax.swing.WindowConstants;
import javax.swing.SwingUtilities;

public class graphics1 extends javax.swing.JFrame {
	private JButton jButton1;
	
	test t1 = new test();

	public static void main(String[] args) {
		SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				graphics1 inst = new graphics1();
				inst.setLocationRelativeTo(null);
				inst.setVisible(true);
			}
		});
	}
	
	public graphics1() {
		super();
		initGUI();
	}
	
	private void initGUI() {
		try {
			BorderLayout thisLayout = new BorderLayout();
			setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
			getContentPane().setLayout(thisLayout);
			{
				jButton1 = new JButton();
				getContentPane().add(jButton1, BorderLayout.CENTER);
				jButton1.setText("jButton1");
				jButton1.addActionListener(new ActionListener() {
					public void actionPerformed(ActionEvent evt) {
						jButton1ActionPerformed(evt);
					}
				});
			}
			pack();
			setSize(400, 300);
		} catch (Exception e) {
			e.printStackTrace();
		}
	}
	
	private void jButton1ActionPerformed(ActionEvent evt) {
		System.out.println("jButton1.actionPerformed, event="+evt);
		t1.test();
}

}

test Klasse:

package test;
import src.Graphics;

public class test {
	graphics1 g1= new graphics1();	
	

	
	void test(){
		
		g1.setTitle("hallo");
	}
	
}

Ich hoffe ihr könnt mir helfen! =)

MFG

 

[Ark]

Das ist hart: Von Namenskonventionen und Schnittstellen keine Ahnung, aber erst mal ne GUI bauen! :shock:

Aber zum Thema: Du hast es geschafft, eine gegenseitige Rekursion zu basteln: ein neues graphics1 macht ein neues test, und ein neues test macht ein neues graphics1, und das neue graphics1 macht ein neues test, und dieses neue test macht ein neues graphics1 ...

Ark

 

[Fred1]

erstmal danke! =)

Alles klar, nur wie kann ich das Problem den jetzt umgehn?

 

[Ark]

Nimm Zeile 13 aus graphics1 raus.

Und beschäftige dich erst einmal mit den Grundlagen. Es sieht so aus, als könntest du das gebrauchen.

Ark

EDIT: Nee, nimm lieber Zeile 5 aus test raus ... oder nicht? Junge, was ist das für ein Code?! Da wird man ja irre. >_>

Ark

 

[Fred1]

Werd ich machen!

@Problem:

Wenn ich zeile 13 rausnehme dann kann ich aber nichtmehr auf die methode test zugreifen.

 

[Fred1]

Wenn ich Zeile 5 Rausnehm dann kann ich ja nichtmehr auf die attribute/methoden von Graphics zugreifen.

 

[Dominik82]

Wozu überhaupt eine Klasse test?
Die bringt keinen Nutzen für das Programm, einfach die Methode test in die Klasse graphics1 packen und entsprechend anpassen, dann gibts keine Probleme.
Eine Testklasse macht hier meiner Meinung nach nur Sinn, wenn du dort die main Methode implementierst und von dort dann graphics1 aufrufst...

 

[Ark]

Dein Code ist echt zum Fürchten.

Ich versuche mal eben, was Anständiges daraus zu machen. Das Ergebnis wird hier reineditiert werden.

Ark

EDIT:

package test;

import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;

import javax.swing.WindowConstants;

public class Fenster extends javax.swing.JFrame implements ActionListener{

	private JButton jButton1;

	public Fenster(){
		setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
		
		getContentPane().setLayout(new BorderLayout());
		jButton1=new JButton();
		jButton1.setText("jButton1");
		jButton1.addActionListener(this);
		getContentPane().add(jButton1,BorderLayout.CENTER);
		
		pack();
		setSize(400,300);
	}

	public void actionPerformed(ActionEvent evt){
		System.out.println("jButton1.actionPerformed, event="+evt);
		setTitle("hallo");
	}
}

package test;

import javax.swing.*;

public class Test{

	private static Fenster g1;

	public static void main(String[] args){
		g1=new Fenster();
		g1.setLocationRelativeTo(null);
		g1.setVisible(true);
	}
}

Die Struktur hat sich völlig verändert, wie man sieht.

Ark

 

[Dominik82]

package test;
import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;

import javax.swing.WindowConstants;
import javax.swing.SwingUtilities;

public class graphics1 extends javax.swing.JFrame {
   private JButton jButton1;

   public static void main(String[] args) {
      SwingUtilities.invokeLater(new Runnable() {
         public void run() {
            graphics1 inst = new graphics1();
            inst.setLocationRelativeTo(null);
            inst.setVisible(true);
         }
      });
   }
   
   public graphics1() {
      super();
      initGUI();
   }
   
   private void initGUI() {
      try {
         BorderLayout thisLayout = new BorderLayout();
         setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
         getContentPane().setLayout(thisLayout);
         {
            jButton1 = new JButton();
            getContentPane().add(jButton1, BorderLayout.CENTER);
            jButton1.setText("jButton1");
            jButton1.addActionListener(new ActionListener() {
               public void actionPerformed(ActionEvent evt) {
                  jButton1ActionPerformed(evt);
               }
            });
         }
         pack();
         setSize(400, 300);
      } catch (Exception e) {
         e.printStackTrace();
      }
   }
   
   void test(){
	      
	      this.setTitle("hallo");
	   }
   
   private void jButton1ActionPerformed(ActionEvent evt) {
      System.out.println("jButton1.actionPerformed, event="+evt);
      test();
}

}

 

[Fred1]

Die Test Klasse ist eigentlich nur dazu da, damit ich die methoden Attribute etc dorthinein schreibe um dann in der Graphics klasse darauf zugreifen zu können.

So muss ich nicht die ganzen Methoden etc in die Graphics Klasse schreiben.

 

[Dominik82]

Also ich würde das dann folgendermaßen machen :

Klasse zur Auslagerung der Methoden :

package test;

public class Outsourced {
	
	private javax.swing.JFrame frame;
	
	public Outsourced(javax.swing.JFrame frame) {
		this.frame = frame;
	}
	
	 void test(){
	      
	      frame.setTitle("hallo");
	   }
}

graphics Klasse

package test;
import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;

import javax.swing.WindowConstants;
import javax.swing.SwingUtilities;

public class graphics1 extends javax.swing.JFrame {
   private JButton jButton1;

   public static void main(String[] args) {
      SwingUtilities.invokeLater(new Runnable() {
         public void run() {
            graphics1 inst = new graphics1();
            inst.setLocationRelativeTo(null);
            inst.setVisible(true);
         }
      });
   }
   
   public graphics1() {
      super();
      initGUI();
   }
   
   private void initGUI() {
      try {
         BorderLayout thisLayout = new BorderLayout();
         setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
         getContentPane().setLayout(thisLayout);
         {
            jButton1 = new JButton();
            getContentPane().add(jButton1, BorderLayout.CENTER);
            jButton1.setText("jButton1");
            jButton1.addActionListener(new ActionListener() {
               public void actionPerformed(ActionEvent evt) {
                  jButton1ActionPerformed(evt);
               }
            });
         }
         pack();
         setSize(400, 300);
      } catch (Exception e) {
         e.printStackTrace();
      }
   }
   
   
   private void jButton1ActionPerformed(ActionEvent evt) {
	  Outsourced o = new Outsourced(this);
      System.out.println("jButton1.actionPerformed, event="+evt);
      o.test();
}

}

Gruß Dominik

 

[Ark]

Man beachte mein obiges EDIT. Danke. ^^

Ark