Swing Fensterinhalt aktualisieren

[Ballu]

Hallo,

ich bin noch rellativ neu in java und habe eine Frage bezüglich der GUI. Ich habe eine GUI erstellt und jetzt möchte ich den Fensterinhalt mit neuen Variableninhalt neu laden:

erstmal die beiden Klassen:

package testGUI;

import javax.swing.*;
import java.awt.Event;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Random;
import java.util.HashMap;
import java.util.Map;

public class Question extends JFrame implements ActionListener {
	// Fragen und Antworten
	Map<Integer, String> question = new HashMap<Integer, String>();
	Map<Integer, String> answer = new HashMap<Integer, String>();

	Random r = new Random();
	int nummerfrage = r.nextInt(26) + 1;
	int punkte = 0;
	int anzahlfrage = 1;
	String fragegui = "Hier sollte die Frage stehen ...";

	public void frageneinlesen() {
           //30 Fragen mit answer.put(laufendeNummer,"Frage");
	}

	public void antworteneinlesen() {

           //30 Antworten mit answer.put(laufendeNummer,"Antwort");
	}

	public void frage_finden() {
		for (Map.Entry<Integer, String> entry : answer.entrySet()) {
			if (entry.getKey() == nummerfrage) {
				fragegui = entry.getValue();
			}
		}
	}

	JPanel cp;
	JLabel anzahlfragen = new JLabel("Frage: " + anzahlfrage + "/8");
	JLabel anzahlpunkte = new JLabel("Punkte: " + punkte + "/8");
	JLabel frageLabel = new JLabel(fragegui);
	JLabel antwortLabel = new JLabel("Antwort:");
	JTextField antwortfeld = new JTextField();
	JButton continueButton = new JButton("Weiter");
	JButton exitButton = new JButton("Beenden");

	public void initGUI() {
		JPanel p = (JPanel) this.getContentPane();

		p.setLayout(null);

		frageLabel.setBounds(20, 5, 550, 30);
		p.add(frageLabel);

		antwortLabel.setBounds(20, 50, 110, 30);
		p.add(antwortLabel);

		antwortfeld.setBounds(90, 50, 400, 30);
		p.add(antwortfeld);

		anzahlfragen.setBounds(20, 75, 100, 50);
		p.add(anzahlfragen);

		anzahlpunkte.setBounds(150, 75, 100, 50);
		p.add(anzahlpunkte);

		continueButton.setBounds(20, 130, 110, 20);
		p.add(continueButton);

		exitButton.setBounds(450, 130, 110, 20);
		p.add(exitButton);

		continueButton.addActionListener(this);
		exitButton.addActionListener(this);
	}

	public Question() {
		int random = 1;
		frageneinlesen();
		antworteneinlesen();
		frage_finden();
		this.setTitle("Test der Allgemeinbildung");
		this.setSize(600, 200);
		initGUI();
	}
	@Override
	public void actionPerformed(ActionEvent event) {
		Object source = event.getSource();
		if (source == continueButton) {
			String antwort = "";
			for (Map.Entry<Integer, String> entry : answer.entrySet()) {
				if (entry.getKey() == nummerfrage) {
					antwort = entry.getValue();
				}
				if (antwortfeld.getText().contains(antwort))
					punkte++;

			}
		}
		if (source == exitButton) {
			// beenden
			System.exit(0);
		}
	}

}

package testGUI;

import testGUI.Question;

public class Run_Question {

	public static void main(String[] args) {
		//erste Idee/Test mit festen Größen
		int anzahlfrage = 9;
		for (int i = 5; i < 10; i++){
			Question fragebogen = new Question();
			fragebogen.setLocation(400, 300);
			fragebogen.setVisible(true);
		}
	}

}

Jetzt hab ich 1 Problem und eine Frage: Das Problem:

	public Question() {
		int random = 1;
		frageneinlesen();
		antworteneinlesen();
		frage_finden();
		this.setTitle("Test der Allgemeinbildung");
		this.setSize(600, 200);
		initGUI();
	}

Bei der Ausgabe der GUI wird die Variable fragegui nicht überschieben.
Die Frage: Ich möchte grundsätzlich, dass ich mit der Random Klasse eine zufallszahl im Bereich von 1 und 22 erzeuge. Diese Variable soll die Keyvalue der ersten Frage sein, danach sollen die 7 folgenden Fragen die insgesammt 8 Fragen des Quizes sein. Und jetzt ist nur die Frage, wie ich das mache, dass der Fensterinhalt mit den neuen Werten neu geladen wird.

 

[Gast2]

Du musst ja nicht bei jeder neuen Frage die komplette GUI erneuern. Es reicht doch wenn du einfach die Texte der JLabels änderst oder nicht?

 

[Ballu]

Ja sicher würde das reichen, aber müsste ich den Teil:

    JPanel cp;
    JLabel anzahlfragen = new JLabel("Frage: " + anzahlfrage + "/8");
    JLabel anzahlpunkte = new JLabel("Punkte: " + punkte + "/8");
    JLabel frageLabel = new JLabel(fragegui);
    JLabel antwortLabel = new JLabel("Antwort:");
    JTextField antwortfeld = new JTextField();
    JButton continueButton = new JButton("Weiter");
    JButton exitButton = new JButton("Beenden");

nicht in ein eigene Methode packen und Variablen mit Übergeben?

 

[bERt0r]

Nein, die Texte von Komponenten kannst du mit setText(String text) zur Laufzeit ändern. Ganz ohne new. Dazu ist es natürlich notwendig, dass du eine Referenz auf den Komponent hältst, also am besten die Komponenten als Klassenvariable deklarieren.

 

[Ballu]

Ok ich hab es jetzt so geändert:

		int nummerfrage = 0;
		int punkte = 0;
		int anzahlfrage = 0;
		String fragegui = "Hier sollte die Frage stehen -.-";
		String fixedword_Frage ="Frage: ";
		String fixedword_Punkte ="Punkte: ";
		String fixedword_Antwort ="Antwort: ";
		String fixedword_Weiter ="Weiter";
		String fixedword_Beenden ="Beenden";
		
		public static void GUILabels(int punkte,int anzahlfrage, String fixedword_Frage,String fixedword_Punkte, String fixedword_Antwort,String fixedword_Weiter,String fixedword_Beenden,String fragegui){
		JPanel cp;
		JLabel anzahlfragen = new JLabel(fixedword_Frage + anzahlfrage + "/8");
		JLabel anzahlpunkte = new JLabel(fixedword_Punkte + punkte + "/8");
		JLabel frageLabel = new JLabel(fragegui);
		JLabel antwortLabel = new JLabel(fixedword_Antwort);
		JTextField antwortfeld = new JTextField();
		JButton continueButton = new JButton(fixedword_Weiter);
		JButton exitButton = new JButton(fixedword_Beenden);
		}

allerdings sagt jetzt public void initGUI(), dass er die ganzen Labels und Co nicht findet, wenn ich es zu public void initGUI(GUILabels()) ändere sagt er missplaced contructors. ...

 

[Paddelpirat]

In deinem letzten Post sind die Labels etc ja auch nur noch lokal für diese eine Methode definiert.

Du solltest sie wieder aus der Methode holen, wie in deinem ersten Post und dann sowas in der Art machen:

public static void setGUILabels(int punkte,int anzahlfrage, String fixedword_Frage,String fixedword_Punkte, String fragegui){
        
        anzahlfragen.setText(fixedword_Frage + anzahlfrage + "/8");
        anzahlpunkte.setText(fixedword_Punkte + punkte + "/8");
        frageLabel.setText(fragegui);
        }

Und dann rufst du die Methode

setGUILabels

auf, wenn du die Texte ändern möchtest.

Schau dir am besten auch nochmal Namenskonventionen in Java an. Der obige Code ist nicht wirklich schön ;-)

 

[Ballu]

Bist du sicher mit dem static? Weil ich bekomme die Fehlermeldung, dass er keine Statische Methode für ein nicht Statisches Feld erzeugen kann, ohne static gehts.

 

[jgh]

 

[Ballu]

Der aktuelle Stand:

package quiz;

import javax.swing.*;

import java.awt.Event;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Random;
import java.util.HashMap;
import java.util.Map;

public class Quiz extends JFrame implements ActionListener {
	// Fragen und Antworten
		Map<Integer, String> question = new HashMap<Integer, String>();
		Map<Integer, String> answer = new HashMap<Integer, String>();
		
		public int setStart(){
		Random r = new Random();
		return r.nextInt(21)+1;
		}
		
		int nummerfrage = 0;
		int punkte = 0;
		int anzahlfrage = 1;
		String fragegui = "Hier sollte die Frage stehen -.-";
		String fixedword_Frage ="Frage: ";
		String fixedword_Punkte ="Punkte: ";
		String fixedword_Antwort ="Antwort: ";
		String fixedword_Weiter ="Weiter";
		String fixedword_Beenden ="Beenden";
		
		JPanel cp;
		JLabel anzahlfragen = new JLabel(fixedword_Frage + anzahlfrage + "/8");
		JLabel anzahlpunkte = new JLabel(fixedword_Punkte + punkte + "/8");
		JLabel frageLabel = new JLabel(fragegui);
		JLabel antwortLabel = new JLabel(fixedword_Antwort);
		JTextField antwortfeld = new JTextField();
		JButton continueButton = new JButton(fixedword_Weiter);
		JButton exitButton = new JButton(fixedword_Beenden);
		
		public void setGUILabels(int punkte,int anzahlfrage, String fixedword_Frage,String fixedword_Punkte, String fragegui){
        anzahlfragen.setText(fixedword_Frage + anzahlfrage + "/8");
        anzahlpunkte.setText(fixedword_Punkte + punkte + "/8");
        frageLabel.setText(fragegui);
		}
		public void frageneinlesen() {
			question.put(1,"Die vier großen Energieunternehmen in Deutschland sind E.on,E, Vattenfall und ...");
			question.put(2, "Was der DAX für Frankfurt, ist der FTSE für ...");	
                //usw		
		}

		public void antworteneinlesen() {
			answer.put(1, "enbw");
			answer.put(2, "athen");
                         //usw
		}

		public String frage_finden(int randomstart) {
			for (Map.Entry<Integer, String> entry : question.entrySet()) {
				if (entry.getKey() == randomstart) {
					fragegui = entry.getValue();
				}
			}
			return fragegui;
		}


		public void initGUI() {
			JPanel p = (JPanel) this.getContentPane();

			p.setLayout(null);

			frageLabel.setBounds(20, 5, 550, 30);
			p.add(frageLabel);

			antwortLabel.setBounds(20, 50, 110, 30);
			p.add(antwortLabel);

			antwortfeld.setBounds(90, 50, 400, 30);
			p.add(antwortfeld);

			anzahlfragen.setBounds(20, 75, 100, 50);
			p.add(anzahlfragen);

			anzahlpunkte.setBounds(150, 75, 100, 50);
			p.add(anzahlpunkte);

			continueButton.setBounds(20, 130, 110, 20);
			p.add(continueButton);

			exitButton.setBounds(450, 130, 110, 20);
			p.add(exitButton);

			continueButton.addActionListener(this);
			exitButton.addActionListener(this);
		}

		public Quiz(int nr_der_frage) {
			frageneinlesen();
			antworteneinlesen();
			frage_finden(nr_der_frage);
			setGUILabels(punkte,anzahlfrage, fixedword_Frage, fixedword_Punkte,fragegui);
			this.setTitle("Test der Allgemeinbildung");
			this.setSize(600, 200);
			initGUI();
		}

		@Override
		public void actionPerformed(ActionEvent event) {
			Object source = event.getSource();
			if (source == continueButton) {
				String antwort = "";
				for (Map.Entry<Integer, String> entry : answer.entrySet()) {
					if (entry.getKey() == nummerfrage) {
						antwort = entry.getValue();
					}
					if (antwortfeld.getText().toLowerCase().contains(antwort))
						punkte++;

				}
			}
			if (source == exitButton) {
				// beenden
				System.exit(0);
			}
		}

	}

package quiz;

import java.util.Random;

import beleg3.Quiz;

public class Run_Quiz {

	public static void main(String[] args) throws InterruptedException {
		Random r = new Random();
		int randomstart = r.nextInt(21)+1;
            
			Quiz fragebogen = new Quiz(randomstart);
            fragebogen.setLocation(400, 300);
            fragebogen.setVisible(true);
        }
		
	}

Das Resultat: Es wird die richtige Frage in bezug auf die Zufallszahl ausgegeben und alles richtig angezeigt.

 

[Paddelpirat]

Sorry, das

static

kam noch von copy & paste aus deinem Post. Deine etwas komische Methode

GUILabels

war auch

static

. Aber du hast es ja hin bekommen. ;-)

 

[Ballu]

Jop

noch ein Nachtrag der mir aufgefallen ist:

 public Quiz(int nr_der_frage) {
            nummerfrage = nr_der_frage;
            frageneinlesen();
....
}

damit auch die richtige Nummer der aktuelle Frage immer wieder überschrieben wird.
Jetzt ist nur noch dir Frage offen, nach dem aktualisieren der GUI mit den neuen Werten ...

 

[Paddelpirat]

Mal auf die schnelle, so in etwa:

    public void actionPerformed(ActionEvent event) {
        Object source = event.getSource();
        if (source == continueButton) {
            String antwort = "";
            for (Map.Entry<Integer, String> entry : answer.entrySet()) {
                if (entry.getKey() == nummerfrage) {
                    antwort = entry.getValue();
                }
                if (antwortfeld.getText().toLowerCase().contains(antwort)) {
                    punkte++;
                }

            }
            anzahlfrage++;
            setGUILabels(punkte, anzahlfrage, fixedword_Frage, fixedword_Punkte, fragegui);
            
        }
        if (source == exitButton) {
            // beenden
            System.exit(0);
        }
    }

Funktioniert natürlich noch nicht alles so wie du es sicherlich haben willst. Aber durch aufrufen der Methode

setGUILabels

in diesem Abschnitt kannst du deine Texte aktualisieren.