Sichtbarkeit von Buttons un Frame

[Flo TGI Rastatt]

Hoi ich bin wie eig jeder der hier Postet^^ ein eig fast totale Noob ws Java angeht naja wir maches halt in de Schul un jaetzt hab ich mal was ausprobiert un joa des funzt halt net so ganz wie ich mir des vorgestellt hab( trotz Java ist auch eine Insel) un joa unsere Lehrer sin etwas unkompetent des wegge isch bei dene eig nachfrage zwecklos^^ nun zu meim Prob ich hab folgendes geschrieben un hab des prob das ich wenn ich des auf ruf nur der hintergrund zu sehe ist und erst wenn ich das Fenster vergrößer bzw verkleiner ich die schrift sehe :roll: un joa zudem gibts au keine Buttons wär super wenn ihr mir hefen könntet un lasst euch net von dem Text erschrecken mir ist nix besseres eingefalle

import java.awt.*;
import java.awt.event.*;

public class Window extends Frame
{
	Frame fr;
  
	public Window()
	{		
	Frame fr =new Frame();
	fr.setSize(650,200);
	fr.setVisible(true);
	fr.setBackground(Color.PINK);
	fr.setTitle("Die große Frage^^");
	fr.setLayout( new FlowLayout(10,88,20));
	Label lb =new Label("Willst du mit mir gehen??");
	Font f=new Font("Arial",Font.PLAIN,36);
	lb.setFont(f);
	fr.add(lb);
	buttons();
	}
	public void buttons()
	{
		Button bt=new Button("Ja");
		Button bt1=new Button("Nein");	
		Button bt2=new Button("Vielleicht");		
		fr.add(bt);
		fr.add(bt1);	
		fr.add(bt2);
		
		 ActionListener al = new ActionListener() {
		      public void actionPerformed( ActionEvent e ) {
		      	Frame fr =new Frame();
		    	fr.setSize(650,200);
		    	fr.setVisible(true);
		    	fr.setBackground(Color.BLUE);
		    	fr.setTitle("Antwort");
		      		
		      }
		    };
		bt.addActionListener( al );
		
		 ActionListener al1 = new ActionListener() {
		      public void actionPerformed( ActionEvent e ) {
		      	Frame fr =new Frame();
		    	fr.setSize(650,200);
		    	fr.setVisible(true);
		    	fr.setBackground(Color.RED);
		    	fr.setTitle("Antwort");
		      }
		    };
		bt1.addActionListener( al1 );
		
		 ActionListener al2 = new ActionListener() {
		      public void actionPerformed( ActionEvent e ) {
		      	Frame fr =new Frame();
		    	fr.setSize(650,200);
		    	fr.setVisible(true);
		    	fr.setBackground(Color.YELLOW);
		    	fr.setTitle("Antwort");
		      }
		    };
		bt2.addActionListener( al2 );

			
		
			addWindowListener(new WindowAdapter(){
	        public void windowClosing(WindowEvent e){
	            System.exit(0);
	        }
	    });
		
		
	
	
	
	}	
  public static void main( String[] args )
  {
    new Window();
    
  }
}

Gruß Flo un danke für eure Antworten

 

[Wildcard]

Mach das setVisible am ende des Konstruktors.

 

[Guest]

Jo danke hat schon funktioniert danke abba wie mach ich des jetzt mit den Buttons da is auch noch en Fehlder drinne den ich nicht find weil die werden au noch nicht angezeigt
danke Gruß Flo

 

[Guest]

Jo danke hat schon funktioniert danke abba wie mach ich des jetzt mit den Buttons da is auch noch en Fehlder drinne den ich nicht find weil die werden au noch nicht angezeigt
danke Gruß Flo

Du Flo, du kommst nicht zufällig aus der Nähe von Bayreuth oder?

 

[marble]

Erster Fehler:

Du erzeugst im Konstruktor oben ein neues Frame das die member variable Frame fr überschreibt weils den gleichen Namen Trägt wie im Konstruktor. Lösche das "Frame" in "Frame fr" im Konstruktor.


Zweiter Fehler:

In deinen ActionListenern überschreibst du jedesmal das Frame (genau wie im Konstruktor). Hier kannst du jeweils die komplette Zeile (also "Frame fr = new Frame()") löschen, da das Frame bereits im Konstruktor initialisiert wurde und so nur überschrieben wird.


Dritter Fehler:

du musst den WindowListener deinem Frame fr hinzufügen nicht "dir" selbst (statt this.addWindowListener... einfach fr.addWindowListener). Schließlich ist das das "Fenster" das angezeigt wird.


Vierter Fehler:

fr.setVisible sollte am Ende kommen... nachdem du alles hinzugefügt hast.


So... neben der Tatsache, dass das Programm total schlecht geschrieben ist (anonyme klassen hättest du hier wunderbar unterbringen können), du deine Klasse von Frame erben lässt ohne davon gebrauch zu machen, weiß ich gar nicht wie es sein kann, dass man so wenig wissen kann und sich schon an awt ranwagt... aber sei's drum - als ich gesehen habe was du da programmiert hast bin ich fast vom stuhl gefallen.... klassiker... hier der korriegierte code, damit du auch mal siehst was du (beinahe) gemacht hast. ich habe allerdings wirklich nur die compile fehler ausgemerzt. dran feilen würde ich an deiner stelle noch. Ich hab die Klasse in Test umbenannt - leg dir ne neue an damit du schön vergleichen kannst, was da falsch war. Formatiert und benannt wars ja vorher schon beschissen ;-).. viel spass:

import java.awt.*;
import java.awt.event.*;

public class Test extends Frame
{
   Frame fr;
 
   public Test()
   {      
   fr =new Frame();
   fr.setSize(650,200);
   fr.setBackground(Color.PINK);
   fr.setTitle("Die große Frage^^");
   fr.setLayout( new FlowLayout(10,88,20));
   Label lb =new Label("Willst du mit mir gehen??");
   Font f=new Font("Arial",Font.PLAIN,36);
   lb.setFont(f);
   fr.add(lb);
   buttons();
   }
   public void buttons()
   {
      Button bt=new Button("Ja");
      Button bt1=new Button("Nein");   
      Button bt2=new Button("Vielleicht");      
      fr.add(bt);
      fr.add(bt1);   
      fr.add(bt2);
      
       ActionListener al = new ActionListener() {
            public void actionPerformed( ActionEvent e ) {
        
             fr.setSize(650,200);
             fr.setVisible(true);
             fr.setBackground(Color.BLUE);
             fr.setTitle("Antwort");
                  
            }
          };
      bt.addActionListener( al );
      
       ActionListener al1 = new ActionListener() {
            public void actionPerformed( ActionEvent e ) {
        
             fr.setSize(650,200);
             fr.setVisible(true);
             fr.setBackground(Color.RED);
             fr.setTitle("Antwort");
            }
          };
      bt1.addActionListener( al1 );
      
       ActionListener al2 = new ActionListener() {
            public void actionPerformed( ActionEvent e ) {
               
             fr.setSize(650,200);
             fr.setVisible(true);
             fr.setBackground(Color.YELLOW);
             fr.setTitle("Antwort");
            }
          };
      bt2.addActionListener( al2 );

         
      
         fr.addWindowListener(new WindowAdapter(){
           public void windowClosing(WindowEvent e){
               System.exit(0);
           }
       });
      
      

         fr.setVisible(true);
   
   
   }   
  public static void main( String[] args )
  {
    new Test();
   
  }
}

 

[Guest]

Erster Fehler:

Du erzeugst im Konstruktor oben ein neues Frame das die member variable Frame fr überschreibt weils den gleichen Namen Trägt wie im Konstruktor. Lösche das "Frame" in "Frame fr" im Konstruktor.


Zweiter Fehler:

In deinen ActionListenern überschreibst du jedesmal das Frame (genau wie im Konstruktor). Hier kannst du jeweils die komplette Zeile (also "Frame fr = new Frame()") löschen, da das Frame bereits im Konstruktor initialisiert wurde und so nur überschrieben wird.


Dritter Fehler:

du musst den WindowListener deinem Frame fr hinzufügen nicht "dir" selbst (statt this.addWindowListener... einfach fr.addWindowListener). Schließlich ist das das "Fenster" das angezeigt wird.


Vierter Fehler:

fr.setVisible sollte am Ende kommen... nachdem du alles hinzugefügt hast.


So... neben der Tatsache, dass das Programm total schlecht geschrieben ist (anonyme klassen hättest du hier wunderbar unterbringen können), du deine Klasse von Frame erben lässt ohne davon gebrauch zu machen, weiß ich gar nicht wie es sein kann, dass man so wenig wissen kann und sich schon an awt ranwagt... aber sei's drum - als ich gesehen habe was du da programmiert hast bin ich fast vom stuhl gefallen.... klassiker... hier der korriegierte code, damit du auch mal siehst was du (beinahe) gemacht hast. ich habe allerdings wirklich nur die compile fehler ausgemerzt. dran feilen würde ich an deiner stelle noch. Ich hab die Klasse in Test umbenannt - leg dir ne neue an damit du schön vergleichen kannst, was da falsch war. Formatiert und benannt wars ja vorher schon beschissen ;-).. viel spass:

import java.awt.*;
import java.awt.event.*;

public class Test extends Frame
{
   Frame fr;
 
   public Test()
   {      
   fr =new Frame();
   fr.setSize(650,200);
   fr.setBackground(Color.PINK);
   fr.setTitle("Die große Frage^^");
   fr.setLayout( new FlowLayout(10,88,20));
   Label lb =new Label("Willst du mit mir gehen??");
   Font f=new Font("Arial",Font.PLAIN,36);
   lb.setFont(f);
   fr.add(lb);
   buttons();
   }
   public void buttons()
   {
      Button bt=new Button("Ja");
      Button bt1=new Button("Nein");   
      Button bt2=new Button("Vielleicht");      
      fr.add(bt);
      fr.add(bt1);   
      fr.add(bt2);
      
       ActionListener al = new ActionListener() {
            public void actionPerformed( ActionEvent e ) {
        
             fr.setSize(650,200);
             fr.setVisible(true);
             fr.setBackground(Color.BLUE);
             fr.setTitle("Antwort");
                  
            }
          };
      bt.addActionListener( al );
      
       ActionListener al1 = new ActionListener() {
            public void actionPerformed( ActionEvent e ) {
        
             fr.setSize(650,200);
             fr.setVisible(true);
             fr.setBackground(Color.RED);
             fr.setTitle("Antwort");
            }
          };
      bt1.addActionListener( al1 );
      
       ActionListener al2 = new ActionListener() {
            public void actionPerformed( ActionEvent e ) {
               
             fr.setSize(650,200);
             fr.setVisible(true);
             fr.setBackground(Color.YELLOW);
             fr.setTitle("Antwort");
            }
          };
      bt2.addActionListener( al2 );

         
      
         fr.addWindowListener(new WindowAdapter(){
           public void windowClosing(WindowEvent e){
               System.exit(0);
           }
       });
      
      

         fr.setVisible(true);
   
   
   }   
  public static void main( String[] args )
  {
    new Test();
   
  }
}

Also mal abgesehen davon, dass du die Fehler alle erkannt hast (marlbe) finde ich, dass man Anfängern durchaus etwas netter gegenüber treten könnte. Mag ja sein, dass er hier wirklich schlecht programmiert hat, nur denke ich, dass solche Ausdrücke wie beschissen hier VOLLKOMMEN UNANGEBRACHT sind.

 

[Guest]

package test;

import java.awt.*;
import java.awt.event.*;

public class Window extends Frame {
   // Label hier oben bekannt machen, dass unten Farbe mit gesetzt werden kann
   Label lb;

   public Window() {
      // Hier muss kein Frame angelegt werden, da du von Frame erbst 
      super();
      // Den WindowListener kann man auch schon hier hinzufügen
      addWindowListener(new WindowAdapter() {
         public void windowClosing(WindowEvent e) {
            System.exit(0);
         }
      });
      // fr.set.... ist hier nicht mehr nötig, weil die Methode von Frame
      // aus so aufgerufen werden können (Vererbung)
      setSize(650, 200);
      setBackground(Color.PINK);
      setTitle("Die große Frage^^");
      // Ein normales Flowlayout reicht hier aus
      setLayout(new FlowLayout());
      lb = new Label("Willst du mit mir gehen??");
      Font f = new Font("Arial", Font.PLAIN, 36);
      lb.setFont(f);
      add(lb);
      buttons();
      setVisible(true);
   }

   public void buttons() {
      Button bt = new Button("Ja");
      Button bt1 = new Button("Nein");
      Button bt2 = new Button("Vielleicht");
      add(bt);
      add(bt1);
      add(bt2);
      
      bt.addActionListener(new ActionListener() {
         public void actionPerformed(ActionEvent e) {
            lb.setBackground(Color.BLUE);
            setBackground(Color.BLUE);
            setTitle("Antwort");
         }
      });
      
      bt1.addActionListener(new ActionListener() {
         public void actionPerformed(ActionEvent e) {
            lb.setBackground(Color.RED);
            setBackground(Color.RED);
            setTitle("Antwort");
         }
      });
      
      bt2.addActionListener(new ActionListener() {
         public void actionPerformed(ActionEvent e) {
            lb.setBackground(Color.YELLOW);
            setBackground(Color.YELLOW);
            setTitle("Antwort");
         }
      });
   }

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

 

[Flo TGI Rastatt]

Jo also erst mal danke das ihr euch die Mühe gemacht habt mein wenn ihr es sagt schlecht geschriebenes Programm auseinander genommen habt un es mir berichtigt habt ich weis ja dass ich noch Entwicklungspotential hab un ich arbeit ja au drann besser zu werde also danke werd des was ihr mal gesagt habt zu herze nehme un versuh es des nexte mal besser zu mache danke
Gruß Florian

p.s: Nein ich kom nicht aus Bayreuth sondern aus Rastatt in de nähe von Karlsruhe :wink: