Wird die Variable nicht übergeben ?

[Zidinjo]

Hallo liebe Freunde,

bin grade dabei wieder irgendwas zu programmieren. Nun habe ich wieder mal ein Problem und verstehe dieses nicht. Warum wird meine Variable public int arrayReihnfolge nicht der Klasse Bilder übertragen ? Beim der Klasse Bilder hat die Variable reinfolge immer den wert null, obwohl ich die Variable arrayReihnfolge beim klicken eines Buttons um 1 erhöhe. Diese Variable wird per abstract Methode specialShape zur Klasse Übertragen aber der Wert der dort ankommt ist immer null.

package de.marvin.oopLabor5;

import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Point;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;

import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JPanel;

public abstract class Geruest extends JPanel implements ActionListener, MouseListener
{
	private static final long serialVersionUID = 1L;
	
	protected int rahmenStartx = 5;
	protected int rahmenStarty = 5;
	protected int rahmenBreite = 10;
	protected int rahmenHoehe = 10;
	private JButton[] but;
	private double bildGroesse = 1.1;
	protected int arrayGroesse;
	public int arrayReinfolge;
	
	private final ImageIcon bild1 = new ImageIcon("pics/luppeWeg.png");
	private final ImageIcon bild2 = new ImageIcon("pics/luppeRein.png");
	private final ImageIcon bild3 = new ImageIcon("pics/zuruck.png");
	private final ImageIcon bild4 = new ImageIcon("pics/back.png");
	private final ImageIcon bild5 = new ImageIcon("pics/forward.png");
	
	
	
	abstract void specialShape(Graphics g, int x, int y, int breite, int hoehe, int reinfolge);

	protected void buttonErstellen(int anzahl)
	{
		JButton[] buttons = new JButton[anzahl];
		for(int i = 0; i < buttons.length; i++)
		{
			switch(i)
			{
			case 0:
				buttons[i] = new JButton();
				buttons[i].setIcon(bild1);
				this.add(buttons[i]);
				buttons[i].addActionListener(this);
				break;
			case 1:
				buttons[i] = new JButton();
				buttons[i].setIcon(bild2);
				this.add(buttons[i]);
				buttons[i].addActionListener(this);
				break;
			case 2:
				buttons[i] = new JButton();
				buttons[i].setIcon(bild3);
				this.add(buttons[i]);
				buttons[i].addActionListener(this);
				break;
			case 3:
				buttons[i] = new JButton();
				buttons[i].setIcon(bild4);
				this.add(buttons[i]);
				buttons[i].addActionListener(this);
				break;
			case 4:
				buttons[i] = new JButton();
				buttons[i].setIcon(bild5);
				this.add(buttons[i]);
				buttons[i].addActionListener(this);
				break;
			default:
				buttons[i] = new JButton("empty");
				this.add(buttons[i]);
				buttons[i].addActionListener(this);
				break;
			}
		}
		but = buttons.clone();
	}

	@Override
	public void paint(Graphics g) 
	{
		super.paint(g);
		Dimension dBorder = this.getSize();
		Dimension dFill = new Dimension((int)(dBorder.width/bildGroesse),(int)(dBorder.height/bildGroesse));
		Point p0Fill = new Point((dBorder.width - dFill.width) /2,(dBorder.height - dFill.height)/2);
		
		specialShape(g, p0Fill.x, p0Fill.y, dFill.width, dFill.height,arrayReinfolge);
	}
	

	@Override
	public void actionPerformed(ActionEvent e) 
	{
		if(e.getSource() == but[0])
		{
			System.out.println("hi");
		}
		else if(e.getSource() == but[1])
		{
			
		}
		else if(e.getSource() == but[2])
		{
			this.bildGroesse = 1;
		}
		else if(e.getSource() == but[3])
		{
			
		}
		else if(e.getSource() == but[4])
		{
			arrayReinfolge++;
			repaint();
		}
	}

	@Override
	public void mouseClicked(MouseEvent arg0) {
		// TODO Auto-generated method stub
		
	}

	@Override
	public void mouseEntered(MouseEvent arg0) {
		// TODO Auto-generated method stub
		
	}

	@Override
	public void mouseExited(MouseEvent arg0) {
		// TODO Auto-generated method stub
		
	}

	@Override
	public void mousePressed(MouseEvent arg0) {
		// TODO Auto-generated method stub
		
	}

	@Override
	public void mouseReleased(MouseEvent arg0) {
		// TODO Auto-generated method stub
		
	}
}

Klasse 2 Bilder

package de.marvin.oopLabor5;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Image;
import java.awt.Toolkit;
import java.io.File;

public class Bilder extends Geruest
{
	private static final long serialVersionUID = 1L;
	private File[] bilder;
	

	@Override
	void specialShape(Graphics g, int x, int y, int breite, int hoehe, int reinfolge) 
	{
		File myDirectory = new File ("pics/verzeichnis");
		bilder = myDirectory.listFiles();
		Image[] verzeichnisse = new Image[bilder.length];
		arrayGroesse = bilder.length;
		
		for(int i = 0; i < bilder.length;i++)
		{
			verzeichnisse[i] = Toolkit.getDefaultToolkit().getImage(bilder[i].toString());
		}
		g.drawImage(verzeichnisse[reinfolge],x,y,breite,hoehe,this);
		g.setColor(Color.BLUE);
		g.drawRect(x-this.rahmenStartx, y-this.rahmenStarty, breite+this.rahmenBreite, hoehe+this.rahmenHoehe);
	}

}

 

[VfL_Freak]

Moin,

Du hast die Variable zwar deklariert, aber NICHT initialisiert !!
Ergo bekommst Du da, wo Du sie das erste Mal nutzt eine NPE !!

Gruß Klaus

 

[Zidinjo]

Erstmal danke Klaus. Ich habe natürlich wieder ein wenig rumprobiert. Bei meinen anderen Klassen kann ich per Methode getReihnfolge die Variable aufrufen nur genau in der Klasse nicht. Habe auch schon alles gelöscht und nur System.out.println(getReihnfolge()) in der Klasse gehabt und dies klappt auch nicht Ich weiß nicht woran das liegen kann. Bei alle anderen Klassen klappt das aber nur nicht in dieser. Kann mir jemand helfen ich versteh das nicht.

package de.marvin.oopLabor5;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Image;
import java.awt.Toolkit;
import java.io.File;

public class Bilder extends Geruest
{
	private static final long serialVersionUID = 1L;
	private File[] bilder;
	private final File myDirectory = new File ("pics/verzeichnis");
	
	private Image[] bilderFuellen()
	{
		bilder = myDirectory.listFiles();
		Image[] verzeichnisse = new Image[bilder.length];
		
		for(int i = 0; i < bilder.length;i++)
		{
			verzeichnisse[i] = Toolkit.getDefaultToolkit().getImage(bilder[i].toString());
		}
		
		return verzeichnisse;
	}

	@Override
	void specialShape(Graphics g, int x, int y, int breite, int hoehe) 
	{
		g.drawImage(bilderFuellen()[this.getReinfolge()],x,y,breite,hoehe,this);
		g.setColor(Color.BLUE);
		g.drawRect(x-this.rahmenStartx, y-this.rahmenStarty, breite+this.rahmenBreite, hoehe+this.rahmenHoehe);
	}

}

Main

package de.marvin.oopLabor5;

public class Main {

	public static void main(String[] args) 
	{
		new MyFrame();
	}
}

MyFrame Klasse

package de.marvin.oopLabor5;

import java.awt.BorderLayout;

import javax.swing.JFrame;

public class MyFrame 
{
	
	private int fGroesse = 400;
	private String WindowName = "Bildvorschau";

	MyFrame()
	{
		JFrame f = new JFrame();
		
		f.setSize(fGroesse, fGroesse);
		f.setTitle(WindowName);
		f.setLayout(new BorderLayout());
		f.setLocationRelativeTo(null);
		
		f.add(new Buttons(), BorderLayout.SOUTH);
		f.add(new Bilder(), BorderLayout.CENTER);
		
		
		f.setVisible(true);
	}
}

Gerüstbau // Vorlage

package de.marvin.oopLabor5;

import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Point;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;

import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JPanel;

public abstract class Geruest extends JPanel implements ActionListener, MouseListener
{
	private static final long serialVersionUID = 1L;
	
	protected int rahmenStartx = 5;
	protected int rahmenStarty = 5;
	protected int rahmenBreite = 10;
	protected int rahmenHoehe = 10;
	private JButton[] but;
	private double bildGroesse = 1.1;
	public int arrayReihnfolge = 0;
	
	private final ImageIcon bild1 = new ImageIcon("pics/luppeWeg.png");
	private final ImageIcon bild2 = new ImageIcon("pics/luppeRein.png");
	private final ImageIcon bild3 = new ImageIcon("pics/zuruck.png");
	private final ImageIcon bild4 = new ImageIcon("pics/back.png");
	private final ImageIcon bild5 = new ImageIcon("pics/forward.png");
	
	abstract void specialShape(Graphics g, int x, int y, int breite, int hoehe);

	protected void buttonErstellen(int anzahl)
	{
		JButton[] buttons = new JButton[anzahl];
		for(int i = 0; i < buttons.length; i++)
		{
			switch(i)
			{
			case 0:
				buttons[i] = new JButton();
				buttons[i].setIcon(bild1);
				this.add(buttons[i]);
				buttons[i].addActionListener(this);
				break;
			case 1:
				buttons[i] = new JButton();
				buttons[i].setIcon(bild2);
				this.add(buttons[i]);
				buttons[i].addActionListener(this);
				break;
			case 2:
				buttons[i] = new JButton();
				buttons[i].setIcon(bild3);
				this.add(buttons[i]);
				buttons[i].addActionListener(this);
				break;
			case 3:
				buttons[i] = new JButton();
				buttons[i].setIcon(bild4);
				this.add(buttons[i]);
				buttons[i].addActionListener(this);
				break;
			case 4:
				buttons[i] = new JButton();
				buttons[i].setIcon(bild5);
				this.add(buttons[i]);
				buttons[i].addActionListener(this);
				break;
			default:
				buttons[i] = new JButton("empty");
				this.add(buttons[i]);
				buttons[i].addActionListener(this);
				break;
			}
		}
		but = buttons.clone();
	}

	@Override
	public void paint(Graphics g) 
	{
		super.paint(g);
		Dimension dBorder = this.getSize();
		Dimension dFill = new Dimension((int)(dBorder.width/bildGroesse),(int)(dBorder.height/bildGroesse));
		Point p0Fill = new Point((dBorder.width - dFill.width) /2,(dBorder.height - dFill.height)/2);
		
		specialShape(g, p0Fill.x, p0Fill.y, dFill.width, dFill.height);
	}
	

	@Override
	public void actionPerformed(ActionEvent e) 
	{
		if(e.getSource() == but[0])
		{
			System.out.println("hi");
		}
		else if(e.getSource() == but[1])
		{
			
		}
		else if(e.getSource() == but[2])
		{
			this.bildGroesse = 1;
		}
		else if(e.getSource() == but[3])
		{
			
		}
		else if(e.getSource() == but[4])
		{
			this.arrayReihnfolge++;
			repaint();
		}
	}
	
	public int getReinfolge()
	{
		return this.arrayReihnfolge;
	}

	@Override
	public void mouseClicked(MouseEvent arg0) {
		// TODO Auto-generated method stub
		
	}

	@Override
	public void mouseEntered(MouseEvent arg0) {
		// TODO Auto-generated method stub
		
	}

	@Override
	public void mouseExited(MouseEvent arg0) {
		// TODO Auto-generated method stub
		
	}

	@Override
	public void mousePressed(MouseEvent arg0) {
		// TODO Auto-generated method stub
		
	}

	@Override
	public void mouseReleased(MouseEvent arg0) {
		// TODO Auto-generated method stub
		
	}
}

Buttons Klasse

package de.marvin.oopLabor5;

import java.awt.Graphics;

public class Buttons extends Geruest
{
	private static final long serialVersionUID = 1L;
	private int ButtonAnzahl = 5;
	
	Buttons()
	{
		this.buttonErstellen(ButtonAnzahl);
	}

	@Override
	void specialShape(Graphics g, int x, int y, int breite, int hoehe) 
	{
	}
}

und die Klasse wo nichts klappt

package de.marvin.oopLabor5;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Image;
import java.awt.Toolkit;
import java.io.File;

public class Bilder extends Geruest
{
	private static final long serialVersionUID = 1L;
	private File[] bilder;
	private final File myDirectory = new File ("pics/verzeichnis");
	
	private Image[] bilderFuellen()
	{
		bilder = myDirectory.listFiles();
		Image[] verzeichnisse = new Image[bilder.length];
		
		for(int i = 0; i < bilder.length;i++)
		{
			verzeichnisse[i] = Toolkit.getDefaultToolkit().getImage(bilder[i].toString());
		}
		
		return verzeichnisse;
	}

	@Override
	void specialShape(Graphics g, int x, int y, int breite, int hoehe) 
	{
		g.drawImage(bilderFuellen()[this.getReinfolge()],x,y,breite,hoehe,this);
		g.setColor(Color.BLUE);
		g.drawRect(x-this.rahmenStartx, y-this.rahmenStarty, breite+this.rahmenBreite, hoehe+this.rahmenHoehe);
	}

}

 

[VfL_Freak]

Moin,

hmm, in deinem neuen Code sehe ich die Variable gar nicht mehr!
WO tritt denn die NPE genau auf?? Poste den StackTrace !!

Habe dann nochmal über den ersten Post geschaut

Warum wird meine Variable public int arrayReihnfolge nicht der Klasse Bilder übertragen ? Beim der Klasse Bilder hat die Variable reinfolge immer den wert null, obwohl ich die Variable arrayReihnfolge beim klicken eines Buttons um 1 erhöhe. Diese Variable wird per abstract Methode specialShape zur Klasse Übertragen aber der Wert der dort ankommt ist immer null.

Meine Antwort war da vlt. nicht ganz richtig!
Du inkrementierst die Variable "arrayReihnfolge", nur wann passiert dass?
VOR oder NACH dem Aufruf von "specialShape"?

Wie auch immer, es wäre sicher eione gute Idee :bae: sie auch zu initialisieren, wie ich schon schrieb:

public int arrayReinfolge = 0;     //  or whatever ...

Gruß Klaus

 

[Zidinjo]

Habe eigentlich keine Fehlermeldung. Nur wenn ich auf ein Button klicke actionPerformed ( dann wird hier die variable um 1 erhöht) und mit repaint die paint methode angesprochen und dann sollte eig die klassenvariable mit den neuen wert versehen sein und die methode mit getReihnfolge den neuen wert der Klasse Bilder übergeben. Aber dies klappt nicht. Also wenn ich auf den Button klicke passiert nichts. Normalerweise sollte er ein Bild weiter gehen, da das Programm eine kleine Bildvorschau ist. Die Variable wird vor dem specialshape mit 0 initialisiert.

 

[VfL_Freak]

Moin,

Die Variable wird vor dem specialshape mit 0 initialisiert.

Ok, die Stelle habe ich jetzt doch gefunden :toll:

und mit repaint die paint methode angesprochen

Bist Du da sicher ???? :noe:

:meld: BTW: es heißt übrigens Reihenfolge

Gruß Klaus

 

[Dompteur]

Du hast da eine seltsame Ableitungshierarchie. Sowohl "Bilder" als auch "Buttons" sind von "Geruest" abgeleitet.
"arrayReihnfolge" liegt in der Klasse "Geruest".
Da du im Programm sowohl ein Objekt der Klasse "Bilder" und eines der Klasse "Buttons" erstellst, hast du in jeder davon eine Instanzvariable "arrayReihnfolge".
Ich tippe darauf, dass du da durcheinander kommst. Du veränderst die Variable in einer Instanz und greifst auf den Wert in der anderen Instanz zu.

Bevor du da weitersuchst, solltest du deine Ableitungshierarchie in Ordnung bringen.

 

[Zidinjo]

Moin Leute,

der Fehler war letztendlich, dass die Klasse Bilder keine Buttons initialisiert hatte und deshalb, wenn man auf den Button geklickt hat nichts passierte. Die andern Klassen hatten die Buttons zum Component hinzugefügt und dort war wohl auch eine initialisierung der JButtons.

 

[Zidinjo]

Hallo liebe Leute,

ich habe noch eine kurze Frage. Kann mir jemand den Fehler erklären und wie man diesen lösen könnte. Ich versuche über ein Button die Variable zahl zu erhöhen. Die Variable wird per getZahl Methode der anderen Klasse übergeben. Dort kommt immer 0 raus. Warum ist das so?

Frame Klasse

import java.awt.BorderLayout;

import javax.swing.JFrame;

public class Frame 
{
	
	Frame()
	{
		JFrame f = new JFrame();
		f.setSize(400, 400);
		f.setLayout(new BorderLayout());
		f.setLocationRelativeTo(null);
		
		f.add(new Button(), BorderLayout.CENTER);
		f.add(new Sehen(), BorderLayout.SOUTH);
		
		f.setVisible(true);
	}
}

Button Klasse

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JPanel;


public class Button extends JPanel implements ActionListener
{
	private int zahl = 0;
	
	Button()
	{
		JButton testBut = new JButton("test");
		this.add(testBut);
		testBut.addActionListener(this);
	}
	
	protected int getZahl()
	{
		return this.zahl;
	}

	@Override
	public void actionPerformed(ActionEvent e) 
	{
		zahl++;
		
	}
}

Klasse wo die Zahl ausgegeben wird

import java.awt.Graphics;

import javax.swing.JPanel;


public class Sehen extends JPanel 
{
	
	
	Sehen()
	{
		Button but = new Button();
		System.out.println(but.getZahl());
	}

}

 

[VfL_Freak]

Moin,

Button but = new Button();
System.out.println(but.getZahl());

Du deklarierst hier zwar alles, auf nur durch den Aufruf des Konstruktors wird kein Event ausgelöst !!
Und ein späterer Buttonklick würde zwar vielleicht hochzählen, aber dann wird halt nix mehr ausgegeben!

Zudem würde ich im "actionPerfomed" auch die ActionEvents auswerten! Dafür ist die ganze Logik doch gedacht !
Und gib mal dort NACH dem Inkrementieren einen entsprechenden Text aus, das siehst Du besser, WANN das passiert !!

Gruß Klaus

 

[Zidinjo]

Verstanden, was meinst du mit dem ActionEvent auswerten ? Wie ist das denn möglich, dass ich mit einer Klasse kommuniziere und ein wert übergebe und dies mit dem Actionlistener.

 

[VfL_Freak]

Moin,

Verstanden, was meinst du mit dem ActionEvent auswerten ? Wie ist das denn möglich, dass ich mit einer Klasse kommuniziere und ein wert übergebe und dies mit dem Actionlistener.

ActionEvent und ActionListener

Gruß Klaus

 

[Zidinjo]

Ja, vielen Dank für die Seite Habe einiges gelernt aber trotzdem steht da nicht wirklich wie ich zwischen den einzelen Klassen variablen übergebe.

 

[VfL_Freak]

Moin,

... aber trotzdem steht da nicht wirklich wie ich zwischen den einzelen Klassen variablen übergebe.

ääh, was willst Du genau von wo nach wo übergeben?
Bau Dir Getter und Setter in die einzelnen Klassen, damit kannst ggf. problemlos Werte abfragen !
[informatikunterricht] - Getter und Setter in Java (auf informatikZentrale.de)

Gruß Klaus

 

[Zidinjo]

Ich bedanke mich erstmal für deine Mühe. Mein Ziel ist es, dass ich von einer Klasse, wenn ich dort ein Button klicke eine variable erhöhe und diesen wert soll zu einer Klasse übermittelt werden, die die paint methode hat und dort die variable eingesetzt wird. Grob habe ich es auch mit getter und setter methoden gemacht. Da kommt immer Null raus. In der Paint methode macht die Methode repaint() ein sinn, dann ist das eine endloschleife. Also müsste bei der Button klasse wenn ich ein Button klicke die Variable um 1 erhöht die mit der getter methode übergeben und zeitgleich irgendwo dort die repaint methode plazieren, da sonst die paint methode ja nicht nochmal durchläuft.

 

[VfL_Freak]

ohne Code können wir nur rumraten ....

 

[Zidinjo]

So hier nochmal im Code. Ich weiß, dass repaint() methode dort nicht hingehören. Aber wie gesagt, ich bin überfragt.

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JPanel;


public class Button extends JPanel implements ActionListener
{
	private int zahl = 0;
	JButton testBut = new JButton("hi");
	
	Button()
	{
		this.add(testBut);
		testBut.addActionListener(this);
	}
	
	protected int getZahl()
	{
		return this.zahl;
	}
	
	protected void setZahl(int zahl)
	{
		this.zahl = zahl;
	}

	@Override
	public void actionPerformed(ActionEvent e) 
	{
		if(e.getSource() == testBut)
		{
			zahl++;
			setZahl(zahl);
			repaint();
		}
		
	}
}

Hier ist die zweite, ich weiß, dass die paint methode zum "malen" ist aber ich will nachher was zeichnen und da muss beim klicken eine variable übergeben werden.

import java.awt.Graphics;

import javax.swing.JPanel;


public class Sehen extends JPanel 
{
	Button but = new Button();
	@Override
	protected void paintComponent(Graphics g) 
	{
		super.paintComponent(g);
		System.out.println(but.getZahl());
		
		
	}

}