getKontostand Fehler

[requid]

Hallo,
kann mir Jemand sagen warum von der Variable kontostand nichts abgezogen wird
wenn ich die Methode Kontostand.setKontostand() aufrufe oder der alte Kontostand
beim aufrufen von Kontostand.getKontostand() angezeigt wird?

CODE

package com.########.###########;

public class Kontostand {

	private static int kontostand = 100;

	public static void setKontostand(int euro) {
		kontostand = kontostand + euro;
		
	}
	
	public static int getKontostand() {
		return(kontostand);
		
	}

}

Danke im Vorraus! Bin gerade echt am verzweifeln... Alle Probleme hab ich selber
lösen können, aber dann warscheinlich nur so eine Kleinigkeit

 

[NurTee]

Ich bin mir nicht sicher, aber da scheinen mir zuviele statics zu sein...wie sieht die superclass aus, ich meine damit die main args?

 

[requid]

Ich bin mir nicht sicher, aber da scheinen mir zuviele statics zu sein...wie sieht die superclass aus, ich meine damit die main args?

package com.########.###########;

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

import javax.swing.JFrame;

public class Geldautomat implements ActionListener{
public static JFrame fenster;

	public static void main(String[] args) {
		fenster = new JFrame("Geldautomat");
		fenster.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		fenster.setVisible(true);
		fenster.setSize(1000, 500);
		fenster.setLocationRelativeTo(null);
		fenster.getContentPane().setBackground(Color.WHITE);
		fenster.setLayout(null);
		
		new GUI("start");

	}

	@Override
	public void actionPerformed(ActionEvent e) {
		if(e.getSource() == GUI.auszahlung){
            new GUI("auszahlung");
		
		}
		if(e.getSource() == GUI.f){
            System.out.println("auszahlung 50");
            Kontostand.setKontostand(50);
            new GUI("ende");
		
		}
		if(e.getSource() == GUI.h){
            System.out.println("auszahlung 100");
            Kontostand.setKontostand(-100);
		
		}
		if(e.getSource() == GUI.zh){
            System.out.println("auszahlung 200");
            Kontostand.setKontostand(-200);
		
		}
		if(e.getSource() == GUI.fh){
            System.out.println("auszahlung 500");
            Kontostand.setKontostand(-500);
		
		}
		
		
	}

}

Die ist noch nicht ganz fertig! Alos z.B. da ist nur einmal new GUI oder anstatt -50 nur 50...
alles zu testzwecken

 

[SoWow]

Ich bin mir nicht genau sicher was du da machst. Aber hast du dich schonmal damit auseinander gesetzt was static bedeutet? Hast du die zweite Klasse selbst geschrieben? Wenn ja: Wo wird das Objekt "Kontostand" erzeugt? Die Geldautomat-Klasse lässt ein Objekt Namens "Kontostand" auf sich eine Methode anwenden namens setKontostand zum Beispiel. Das heißt dass du einen Konstruktor in der Geldautomat-Klasse aufrufen musst, der sich in der Kontostand-Klasse befindet. Wenn du das gemacht hast, musst du das static noch weg lassen denn static heißt so viel wie: Variablen und Methoden die nicht bei einer Objektkonstruktion zum Objekt gehören, sondern nur einmal pro Klasse existieren.

 

[requid]

Ich bin mir nicht genau sicher was du da machst. Aber hast du dich schonmal damit auseinander gesetzt was static bedeutet? Hast du die zweite Klasse selbst geschrieben? Wenn ja: Wo wird das Objekt "Kontostand" erzeugt? Die Geldautomat-Klasse lässt ein Objekt Namens "Kontostand" auf sich eine Methode anwenden namens setKontostand zum Beispiel. Das heißt dass du einen Konstruktor in der Geldautomat-Klasse aufrufen musst, der sich in der Kontostand-Klasse befindet. Wenn du das gemacht hast, musst du das static noch weg lassen denn static heißt so viel wie: Variablen und Methoden die nicht bei einer Objektkonstruktion zum Objekt gehören, sondern nur einmal pro Klasse existieren.

D.h ich soll die set und get Methoden in einen Konstruktor verschieben?? und in der Superclass ein Objekt aus Konto erzeugen??
Und wo soll ich das static weg machen? Hab' ja viele??? :L

Ach Ja ich hab die Klasse selbst geschrieben!

 

[Harry Kane]

D.h ich soll die set und get Methoden in einen Konstruktor verschieben?? und in der Superclass ein Objekt aus Konto erzeugen??

Nein. Du sollst dir für deine Kontostand-Klasse überhaupt erstmal einen Konstruktor definieren, der es dir erlaubt, den Kontostand festzulegen. Übrigens finde ich die Methode "getKontostand" in der Klasse "Kontostand" komisch. Vorschlag: nenne deine Klasse "Konto" und gebe der eine Methode "getGuthaben" oder so.
Und du sollst nicht in der Superclass eine Instanz von Konto erzeugen, sondern in der Klassem die die public static void main(String[] args) methode enthält. Das ist KEINE Superclass!

Und wo soll ich das static weg machen? Hab' ja viele??? :L

Praktisch überall, nur nicht bei der public static void main(String [] args).
Der Sinn der Aurufe von

new GUI("ende");

usw. in der actionPerformed()-Methode erschliesst sich mir auch nicht. Was soll das? Was für eine Klasse ist "Gui" überhaupt? Wahrscheinlich erweitert sie JFrame, wer weiss das schon. Kurioserweise greifst du in der actionperformed() auch auf statische Variablen der Gui-Klasse zu. Alles in allem ein ziemliches Durcheinander! :noe:
Zurück zu deiner eigentlichen Frage bzgl. des nicht veränderten Kontostands: Werden nur die Werte nicht geändert, oder bekommst du noch nicht einmal eine Ausgabe auf der Konsole? Ich tippe mal auf letzteres und vermute, daß du auf den ganzen Objekten

GUI.f

,

GUI.zh

etc. (ich vermute es handelt sich dabei um JButtons) keinen ActionListener registriert hast.