Ergebnisse auf einem Label anzeigen lassen ?

[kaper28]

Hallo Freunde,

Ich versuche das Ergebniss in dieser Aplication in einem Label( ausgabe heißt das Label ) anzuzeigen aber klappt irgend wie nicht .

mfg
kaper28

package rechner;

import java.awt.FlowLayout;
import java.awt.GridLayout;
import java.awt.Label;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JTextArea;
import javax.swing.JTextField;

public class BMI_Gui extends JFrame {

	private JLabel lgröße, lgewicht, lbmi, lbmiliste;
	private JTextField größe, gewicht, bmi;
	private JButton berechnen, beenden;
	private JTextArea liste;
	private double gross, gew, bmi1;
	private JLabel ausgabe;
	
	
	public String berechnen(){
		gew = Double.parseDouble(gewicht.getText());
	    gross = Double.parseDouble(größe.getText());
		bmi1 = gew * gross;
		return null;
		
	}
	class BMIlistener implements ActionListener {

		public void actionPerformed(ActionEvent e) {

			
			if (e.getActionCommand().equals("Beenden"))
				System.exit(0);

			

			if (e.getActionCommand().equals("Berechnen"))
				

			ausgabe.setText(berechnen());
		}
	
		}

	

	//	der Konstruktor
	public BMI_Gui(String titel) {
		super(titel);

		JPanel panel = new JPanel();
		lgröße = new JLabel("Größe");
		lgewicht = new JLabel("Gewicht");
		lbmi = new JLabel("BMI");
		lbmiliste = new JLabel("BMI LISTE");
		ausgabe =  new JLabel("");
	
		größe = new JTextField();
		gewicht = new JTextField();
		bmi = new JTextField();
		

		berechnen = new JButton("Berechnen");
		beenden = new JButton("Beenden");

		liste = new JTextArea();

		add(panel);
		add(lgröße);
		add(lgewicht);
		add(lbmi);
		add(ausgabe);
		add(lbmiliste);
		
		add(größe);
		add(gewicht);
		add(bmi);
		add(berechnen);
		add(beenden);
		add(liste);

		setLayout(null);
		lgröße.setBounds(10, 35, 110, 25);
		größe.setBounds(65, 35, 110, 25);

		lgewicht.setBounds(10, 85, 110, 25);
		gewicht.setBounds(65, 85, 110, 25);

		lbmi.setBounds(10, 135, 110, 25);
		bmi.setBounds(65, 135, 110, 25);
		ausgabe.setBounds(65,185,110,25);

		lbmiliste.setBounds(201, 15, 110, 25);
		liste.setBounds(200, 35, 110, 125);

		
		
		berechnen.setBounds(30, 215, 110, 25);
		beenden.setBounds(200, 215, 110, 25);

		BMIlistener listener = new BMIlistener();
		beenden.addActionListener(listener);
		berechnen.addActionListener(listener);
		setSize(350, 300);
		setVisible(true);


	
	}




}

 

[Gast]

du gibst in zeile 32 immer null zurück,dort muss der Strig zurüeckgegeben werden.

 

[kaper28]

du gibst in zeile 32 immer null zurück,dort muss der Strig zurüeckgegeben werden.

stimmt habe ich , sorry

ausgabe.setText(berechnen1());

nur ich bekommen kein ergebniss auf dem bildschrim

 

[Gast]

also bei mir funktioniert es,wenn ich in Zeile 32
return String.valueOf(bmi1);
schreibe

 

[Schandro]

guck mit System.out.printlns ob
ausgabe.setText(berechnen());
überhaupt aufgerufen wird und guck dir an, ob
berechnen();
nicht zufällig
""
returnt.

 

[kaper28]

guck mit System.out.printlns ob
ausgabe.setText(berechnen());
überhaupt aufgerufen wird und guck dir an, ob
berechnen();
nicht zufällig
""
returnt.

doch mit system.out funktionert es kein problem , die mwthode wir aufgerufen

 

[Schandro]

und guck dir an, ob
berechnen();
nicht zufällig
""
returnt.

Hast du das auch schon gecheckt?

 

[kaper28]

liebe freunde es hat funktioniert ,mein fehler war die null züück zu rufen

public String berechnen(){ 
      gew = Double.parseDouble(gewicht.getText()); 
       gross = Double.parseDouble(größe.getText()); 
      bmi1 = gew * gross; 
      return null; 
       
   }

das muß

 return String.valueOf(bmi1);

so stimmt es danke schön .....

 

[licht400]

Habe auch ein Problem mit setText...
wie ich im normalfall damit umgehen muss ist mir bekannt.
das problem ist, wo ich ein tag dran saß, um es herauszufinden liegt daran,
das ich ein jtabbedPanel erstelle und das ganze erzeugt dynamische panels..und darauf produziere ich ein labe und ein button, mit dem ich label.setText machen möcht. das ganze funktioniert nur auf dem letzten tab. Das verstehe ich gerade nicht.


Hier der code

import java.awt.BorderLayout;
import java.awt.Container;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Image;
import java.awt.Insets;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.BorderFactory;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTabbedPane;


public class Main extends JDialog {

private JTabbedPane tabPanel;
private JLabel pageDisplay;

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        JDialog app = new Main();
        //app.setUndecorated(true);
	app.setSize(1024, 768);
        app.setTitle("^^");
        //app.setResizable(false);
        //app.setAlwaysOnTop(true);
	app.setVisible(true);
    }
    
    public Main() {
       
        erzeugePanels();
 
    }
    private void erzeugePanels(){
        tabPanel = new JTabbedPane();
        this.getContentPane().setLayout(null);
        tabPanel.setSize(1024,768);
	this.getContentPane().add(tabPanel);
        
        JPanel mainPanel = new JPanel();
        mainPanel.setLayout(new BorderLayout());
       //Panel Gruppenauswahl
		tabPanel.addTab("Gruppenauswahl",createGroupPanel());
		// Panels Linkauswahl
		for (int i = 0;i<4;i++) {
			tabPanel.addTab("Gruppe"+i+1,createLinkPanel(i));}
    }
    
    private JPanel createGroupPanel() {
		// Panel mit Hintergrundbild laden
		JPanel mainPanel = new JPanel();
		mainPanel.setLayout(new BorderLayout());
		
		return mainPanel;
	}
    
    private JPanel createLinkPanel(int id){
        
        JPanel mainPanel = new JPanel();
	mainPanel.setLayout(new BorderLayout());
       
       
	JPanel cmdPanel = new JPanel();
	//cmdPanel.setBorder(BorderFactory.createLoweredBevelBorder());
	cmdPanel.setOpaque(false);
        cmdPanel.setLayout(new BorderLayout());
        cmdPanel.setPreferredSize(new Dimension(300,75));//185
        mainPanel.add(cmdPanel,BorderLayout.WEST);
                
        JPanel zusatzPanel = new JPanel();
        zusatzPanel.setOpaque(false);
        Container con = zusatzPanel;
        con.setLayout(new GridBagLayout());
        zusatzPanel.setPreferredSize(new Dimension(500,75));
        // zusatzPanel.setBorder(BorderFactory.createLoweredBevelBorder());
        mainPanel.add(zusatzPanel, BorderLayout.EAST);

        
        //zurück
        JButton buttonPrevious = new JButton("previous");   
        buttonPrevious.setPreferredSize(new Dimension(100, 60));
        GridBagConstraints gridBagConstraints = new GridBagConstraints();
        gridBagConstraints.insets = new Insets(2,20, 2, 20);
        gridBagConstraints.anchor = GridBagConstraints.WEST;
        gridBagConstraints.gridx = 0;
        gridBagConstraints.gridy = 0;
        gridBagConstraints.gridheight = 2;
        zusatzPanel.add(buttonPrevious, gridBagConstraints); 
        buttonPrevious.addActionListener(new ActionListener() {
                public void actionPerformed(ActionEvent e) {
                    System.out.println("previous gedrückt true");
                    pageDisplay.setText("passiert nix");
                    System.out.println("getText(): "+pageDisplay.getText());
                }

            
        });

        // Seitenanzahl für die Folienseiten
        pageDisplay = new JLabel();
        pageDisplay.setPreferredSize(new Dimension(100, 50));
        // gridBagConstraints = new GridBagConstraints();
        //gridBagConstraints.insets = new Insets(2, 2, 2, 2);
        gridBagConstraints.gridx = 1;
        gridBagConstraints.gridy = 0;
        gridBagConstraints.gridheight = 2;
        zusatzPanel.add(pageDisplay, gridBagConstraints); 
        pageDisplay.setText(" / ");
        pageDisplay.setBorder(BorderFactory.createLoweredBevelBorder());
        //pageDisplay.setBorder(BorderFactory.createEtchedBorder());
        
    return mainPanel;
    }
    
    
}

[/code]

 

[Verjigorm]

pageDisplay ist bei dir eine Klassenvariable, die du mehrmals benutzt als Label.
Damit überschreibst du in jedem Tab die Referenz auf das Label.
Die letzte gültige Referenz ist dann halt im letzten Tab.

änder das mal so ab:
(wobei du dem Label am besten nen anderen namen gibst!)

        // Seitenanzahl für die Folienseiten
        final JLabel pageDisplay = new JLabel(); //<-------------
        pageDisplay.setPreferredSize(new Dimension(100, 50));
        // gridBagConstraints = new GridBagConstraints();
        //gridBagConstraints.insets = new Insets(2, 2, 2, 2);
        gridBagConstraints.gridx = 1;
        gridBagConstraints.gridy = 0;
        gridBagConstraints.gridheight = 2;
        zusatzPanel.add(pageDisplay, gridBagConstraints);
        pageDisplay.setText(" / ");
        pageDisplay.setBorder(BorderFactory.createLoweredBevelBorder());
        //pageDisplay.setBorder(BorderFactory.createEtchedBorder());
        
        buttonPrevious.addActionListener(new ActionListener() {
                public void actionPerformed(ActionEvent e) {
                    System.out.println("previous gedrückt true");
                    pageDisplay.setText("passiert nix");
                    System.out.println("getText(): "+pageDisplay.getText());
                }

           
        });

 

[licht400]

ok das funktioniet soweit.

möcht die variabel pageDisplay als variable deklarieren, sprich öffentlich für die klasse.
da ich eine getter methode benötige für diese ...

aber wenn ich oben stehen final JLabel pageDisplay;

und später in der createLinkPanekl methode pageDisplay = new JLabel(); sagt er mir...cannot assign a value to final variable pagedisplay..

wie bekomme ich die öffentlich für die klasse?

 

[Verjigorm]

Du kannst nicht EINE Variable deklarieren für 4 Labels

Du musst dir schon alle 4 Labels als Klassenvariablen deklarieren.
Oder ein Array/Liste von labels oder sowas.

Oder genauer beschreiben, was du machen möchtest

 

[licht400]

ja verstehe das..aber denoch müsste ich von einer anderen klasse auf diese labels zu greifen..

und wenn ich..ich benötige jetz nur ein label

wenn ich das mit

final JLabe labelTest;

deklarieren und später dann in der methode

labelTest = new JLabel() mache..sagt er mir cannot assign a value to final variable labelTest!!

ich brauch das label öffentlich für die klasse und final ) ...

weil ich ein getter methode habe..

public JLabel getLabel(){
return label; <----aber das jlabel nicht öffentlich is für die klasse, ist doof !
}

 

[Verjigorm]

wenn du später "von aussen" auf die Labels zugreifen möchtest, dann musst du dir die Referenz zu diesen Labels halt irgendwo speichern.

Momentan geht die Referenz quasi verloren, weil die Labels in der Methode deklariert werden.
Ausser dem Actionlistener hat später keiner mehr Zugriff auf das Label.

Wie gesagt, entweder alles als Klassenvariable anlegen oder die Referenz zu den Labels in einem Array/Liste speichern.

 

[Verjigorm]

Ach ja, generell sei noch zu erwähnen, dass du mal die defaultCloseOperation setzen solltest.
Momentan wird das Fenster nämlich nur unsichtbar und das Programm nicht beendet.

 

[licht400]

ich hatte nur das nötigste drin, was das problem darstellt

mhh zur klassenvariable, das klappt so nicht

public final JLabel label;

und später in der methode..

label = new Label();
und wie bekomm ich die öffentlich als klassenvariabel? *bahnhof gerade*

 

[Verjigorm]

final wegmachen oder

public final JLabel label = new Label();

wobei du das label nicht public machen solltest, sondern getter/setter benutzen

 

[licht400]

ja das hab ich schon probiert, da zeigt er mir das label nicht mehr an, wenn ich JLabel so erstelle.#

private final JLabel label = new JLabel();


und in der methode createLinkPanel steht dann drin

label.setText("irgendwas")...
<----und das bleibt leer, bzw wird kein label angezeigt.


..das mit der getter methode war klar, hatte mich verschrieben mit public

public JLabel getLabel(){
return label,
}

 

[Verjigorm]

Ich würds in eine Liste packen, auf die du dann zugreifen kannst

und als Zugriffsfunktion sowas: (fehlt nochn paar Überprüfungen oder sowas..)

    public void setText(int id, String text)
    {
    	labels.get(id).setText(text);
    }

import java.awt.BorderLayout;
import java.awt.Container;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Image;
import java.awt.Insets;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
import java.util.List;

import javax.swing.BorderFactory;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTabbedPane;


public class Main extends JDialog {

private JTabbedPane tabPanel;
//private JLabel pageDisplay;
private final List<JLabel> labels = new ArrayList<JLabel>();

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        JDialog app = new Main();
        //app.setUndecorated(true);
   app.setSize(1024, 768);
        app.setTitle("^^");
        //app.setResizable(false);
        //app.setAlwaysOnTop(true);
   app.setVisible(true);
    }
   
    public Main() {
       
        erzeugePanels();

    }
    private void erzeugePanels(){
        tabPanel = new JTabbedPane();
        this.getContentPane().setLayout(null);
        tabPanel.setSize(1024,768);
   this.getContentPane().add(tabPanel);
       
        JPanel mainPanel = new JPanel();
        mainPanel.setLayout(new BorderLayout());
       //Panel Gruppenauswahl
      tabPanel.addTab("Gruppenauswahl",createGroupPanel());
      // Panels Linkauswahl
      for (int i = 0;i<4;i++) {
         tabPanel.addTab("Gruppe"+i+1,createLinkPanel(i));}
    }
   
    private JPanel createGroupPanel() {
      // Panel mit Hintergrundbild laden
      JPanel mainPanel = new JPanel();
      mainPanel.setLayout(new BorderLayout());
      
      return mainPanel;
   }
   
    private JPanel createLinkPanel(int id){
       
        JPanel mainPanel = new JPanel();
   mainPanel.setLayout(new BorderLayout());
       
       
   JPanel cmdPanel = new JPanel();
   //cmdPanel.setBorder(BorderFactory.createLoweredBevelBorder());
   cmdPanel.setOpaque(false);
        cmdPanel.setLayout(new BorderLayout());
        cmdPanel.setPreferredSize(new Dimension(300,75));//185
        mainPanel.add(cmdPanel,BorderLayout.WEST);
               
        JPanel zusatzPanel = new JPanel();
        zusatzPanel.setOpaque(false);
        Container con = zusatzPanel;
        con.setLayout(new GridBagLayout());
        zusatzPanel.setPreferredSize(new Dimension(500,75));
        // zusatzPanel.setBorder(BorderFactory.createLoweredBevelBorder());
        mainPanel.add(zusatzPanel, BorderLayout.EAST);
        
        //zurück
        JButton buttonPrevious = new JButton("previous");   
        buttonPrevious.setPreferredSize(new Dimension(100, 60));
        GridBagConstraints gridBagConstraints = new GridBagConstraints();
        gridBagConstraints.insets = new Insets(2,20, 2, 20);
        gridBagConstraints.anchor = GridBagConstraints.WEST;
        gridBagConstraints.gridx = 0;
        gridBagConstraints.gridy = 0;
        gridBagConstraints.gridheight = 2;
        zusatzPanel.add(buttonPrevious, gridBagConstraints);
        
        // Seitenanzahl für die Folienseiten
        final JLabel pageDisplay = new JLabel();
        labels.add(pageDisplay);
        pageDisplay.setPreferredSize(new Dimension(100, 50));
        // gridBagConstraints = new GridBagConstraints();
        //gridBagConstraints.insets = new Insets(2, 2, 2, 2);
        gridBagConstraints.gridx = 1;
        gridBagConstraints.gridy = 0;
        gridBagConstraints.gridheight = 2;
        zusatzPanel.add(pageDisplay, gridBagConstraints);
        pageDisplay.setText(" / ");
        pageDisplay.setBorder(BorderFactory.createLoweredBevelBorder());
        //pageDisplay.setBorder(BorderFactory.createEtchedBorder());
        
        buttonPrevious.addActionListener(new ActionListener() {
                public void actionPerformed(ActionEvent e) {
                    System.out.println("previous gedrückt true");
                    pageDisplay.setText("passiert nix");
                    System.out.println("getText(): "+pageDisplay.getText());
                }

           
        });
       
    return mainPanel;
    }
   
   
}

 

[licht400]

Danke, ich verstehe das.
Aber wozu brauch ich eine Liste, wenn ich jetz nur ein Label nutze??

hab jetz des öfteren gelesen..

deklariere einfach

private final JLabel label;

in irgendeiner methode sollte dann stehen

label = new JLabel(); <<---aber hier kommt cannot assign a value to find variable label..was für ein wert wird gemeint. der wird doch zu gewiesen oder nicht!?
....

 

[licht400]

ok, habe es verstanden und mit der Liste gemacht