setter funktionieren nicht

[Cathamenia]

Hallo!

Entweder bin ich total blöd oder hier stimmt etwas nicht. Ich habe ein Programm in dem es 2 Klassen gibt. Die eine Klasse ist für die GUI zuständig, die andere berechnet etwas. Wenn in der Klasse die etwas berechnet wird ein bestimmter Wert rauskommt, soll in der GUI zb ein Label geändert werden. Ich mach das mit folgender Funktion:

void setList(){
        System.out.println("ich werde aufgerufen");
        label1.setText("Test");
    }

Nun weiß ich nicht warum, aber das Funktioniert nicht. Das System.out.println() was ich zum testen eingebaut habe wird aufgerufen, aber am label wird nix geändert.

:?: :?: :?:

 

[SebiB90]

ruf mal validate() bzw repaint() auf
das label muss nämlich erst neu gezeichnet werden

 

[Cathamenia]

Das brachte keine Auswirkung. Ich habe die setList() mal aus der selben klasse per Button-Click aufgerufen und da wird beides gemacht, aber wenn es es aus der anderen Klasse aufrufe, wird das label nicht verändert.

 

[Beni]

Du bist sicher, dass es das richtige Label ist. Manchmal hat man eine falsche Instanz (und wenn man zwei Labels hat, kann man an Nr.1 lange herumschrauben, Nr.2 wird sich nicht veränderun...)

 

[Cathamenia]

also wie gesagt, ich habe das ausgiebig gestestet und das label kann auch aus der klasse selber verändert werden.


Das hier:

private void startActionPerformed(java.awt.event.ActionEvent evt) {                                      
        if (start.getText().equals("Start")){
            start.setText("Stop");
            setList();
        } else {
            start.setText("Start");
        }
    }

liegt in der selben Klasse wie das label und da wird es auch geändert, aber wenn ich setList() aus einer anderen klasse aufrufe, wird nur in die Konsole geprintet.

 

[SebiB90]

wieso heißt das label im 1. code label1 und im 2. code start?
das ist doch der fehler oder nicht

edit:ups das ist ja der button
edit2:zeig mal den code des fensters und der klasse aus derdu setList() aufrufst

 

[Cathamenia]

Das Graphische habe ich von NetBeans machen lassen, weil das eigentlich nur ein kleines Programm sein sollte und ich keine Lust hatte das selber zu machen. Die Klasse sieht folgendermaßen aus:

import java.util.Timer;
import javax.swing.JLabel;
import javax.swing.UIManager;

/**
 *
 * @author  Torsten
 */
public class MainFrame extends javax.swing.JFrame {
    
    /** Creates new form MainFrame */
    public MainFrame() {
        initComponents();
    }
      
    
    public  boolean check = false;
    
    public void lauf() {
	Timer timer = new Timer();
	// nach 1 Sek gehts los und dann alle 10 Sekunden
	timer.schedule(new Task(), 1, 10000);
    }
    
    
    /** This method is called from within the constructor to
     * initialize the form.
     * WARNING: Do NOT modify this code. The content of this method is
     * always regenerated by the Form Editor.
     */
    // <editor-fold defaultstate="collapsed" desc=" Generated Code ">                          
    private void initComponents() {
        toolBar = new javax.swing.JToolBar();
        start = new javax.swing.JButton();
        label1 = new java.awt.Label();
        urlList = new java.awt.List();
        jMenuBar1 = new javax.swing.JMenuBar();
        menu = new javax.swing.JMenu();

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);
        toolBar.setEnabled(false);

        start.setText("Start");
        start.setPreferredSize(new java.awt.Dimension(65, 23));
        start.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                startActionPerformed(evt);
            }
        });

        label1.setText("label1");

        menu.setText("Datei");
        jMenuBar1.add(menu);

        setJMenuBar(jMenuBar1);

        org.jdesktop.layout.GroupLayout layout = new org.jdesktop.layout.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
            .add(toolBar, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, 352, Short.MAX_VALUE)
            .add(org.jdesktop.layout.GroupLayout.TRAILING, layout.createSequentialGroup()
                .addContainerGap()
                .add(label1, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)
                .add(229, 229, 229)
                .add(start, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)
                .add(10, 10, 10))
            .add(layout.createSequentialGroup()
                .addContainerGap()
                .add(urlList, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, 332, Short.MAX_VALUE)
                .addContainerGap())
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
            .add(layout.createSequentialGroup()
                .add(toolBar, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, 25, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(org.jdesktop.layout.LayoutStyle.RELATED)
                .add(urlList, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, 301, Short.MAX_VALUE)
                .addPreferredGap(org.jdesktop.layout.LayoutStyle.RELATED)
                .add(layout.createParallelGroup(org.jdesktop.layout.GroupLayout.TRAILING)
                    .add(start, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)
                    .add(label1, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE))
                .addContainerGap())
        );
        pack();
    }// </editor-fold>                        

    private void startActionPerformed(java.awt.event.ActionEvent evt) {                                      
        if (start.getText().equals("Start")){
            start.setText("Stop");
            lauf();
            //label1.setText("test");
            //setList();
        } else {
            start.setText("Start");
        }
    }                                     
    
   
    /**
     * @param args the command line arguments
     */
    public static void main(String args[]) {

        try {
            UIManager.setLookAndFeel(
            UIManager.getSystemLookAndFeelClassName() );
        } catch( Exception e ) { 
            e.printStackTrace(); 
        }
        
        java.awt.EventQueue.invokeLater(new Runnable() {
            public void run() {
                new MainFrame().setVisible(true);
            }
        });
    }
    
    public void setList(){
        label1.setText("Test");
    }
    
   //static javax.swing.JLabel jLabel1;
    // Variables declaration - do not modify                     
    private javax.swing.JMenuBar jMenuBar1;
    private java.awt.Label label1;
    private javax.swing.JMenu menu;
    private javax.swing.JButton start;
    private javax.swing.JToolBar toolBar;
    private java.awt.List urlList;
    // End of variables declaration                   
    
}

 

[Beni]

Was ist denn "Task" (von "new Task()" in Zeile 23)? Dadrin geht doch die Action ab, also poste es auch :wink:

 

[Cathamenia]

Stimmt, dann sende ich das auch nochmal...

Zur erklärung des Programms, damit es evtl leichter zu verstehen ist:

Ich will wissen, ob bestimmte artikel im Coke-Store erhältlich sind oder nicht, denn sie sind schnell vergriffen, also lasse ich mein Programm dannach gucken indem ich den quelltext auslese und dann darauf reagieren kann.

package coke;

import java.applet.Applet;
import java.applet.AudioClip;
import java.io.BufferedReader;
import java.io.File;
import java.io.IOException;
import java.io.InputStreamReader;
import java.io.Reader;
import java.net.URL;
import java.util.TimerTask;

class Task extends TimerTask{

	static boolean status = false;
	static int i = 1;
        MainFrame frame = new MainFrame();

	public void run() {
		// boolean validUrl = false;
		URL url;

		try {
			// URL oeffnen und Reader anlegen
			url = new URL("https://store.cokefridge.de/customer/home.php?cat=250");
			Reader inStream = new InputStreamReader(url.openStream());
			BufferedReader inReader = new BufferedReader(inStream);

			// Inhalt der URL zeilenweise auslesen und ausgeben
			boolean stelle = false;
			int counter = 8;
			for (String s; (s = inReader.readLine()) != null;) {
				if (s.equals("<input type=hidden name=productid value=\"16227\">")) {
					stelle = true;
					// System.out.println("bin da");
				}
				if (stelle) {
					counter--;
				}

				if (counter == 0) {
					if (s.equals("[b]Schon vergriffen![/b]")) {
						status = false;
					} else {
						status = true;
					}
				}
				
			}

                        
			if (status) {
                            frame.check = true;
                            frame.setList();
			} else {
                            frame.check = true;
                            frame.setList();
                        }
			
			i++;

			inReader.close();
			inStream.close();
		}

		// Fehlerbehandlung, wenn Reader fehler melden
		catch (IOException e) {
			System.out.println("Verbindungsfehler");
			System.exit(0);
		}
	}
	
}

 

[Beni]

Der Fehler ist Zeile 18 in Task: du erstellst ein neues MainFrame. Das MainFrame das im Task bekannt ist, und das "echte" MainFrame haben nichts miteinander zu tun.

Versuch mal folgends: übergib im Konstruktor von Task das MainFrame.

private MainFrame frame = null;

public Task( MainFrame frame ){
  this.frame = frame;
}

 

[Cathamenia]

Vielen Dank! so hat es funktioniert. Da wär ich wohl nicht drauf gekommen. Also danke nochmal!