Swing Methode add([Component]) funzt nich

[Runtime]

Moin,
Ich habe eine Klasse namens Taskbar, die von JPanel erbt und eine Klasse namens StartMenu.
Jatzt möchte ich ein StartMenu darauf tun. Bei allen anderen Komponenten funktioniert es, aber
bei der Klasse StartMenu nicht.???:L

Taskbar:

package javaworld.components;

import java.awt.Color;
import java.awt.Component;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.Point;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.util.Vector;
import javax.swing.JPanel;

public class Taskbar extends JPanel implements MouseListener {

    public Taskbar(StartMenu startMenu, Component parent){
        this.startMenu = startMenu;startMenu.setVisible(true);
        this.parent = parent;
        setSize(parent.getWidth(), 40);
        Point location = parent.getLocation();
        int height = parent.getHeight();
        height -= getHeight();
        setLocation(location.x, height);
        iconCount = 0;
        startMenu.setLocation(20, 0);
        addMouseListener(this);
        components = new Vector();
        add(startMenu);
    }

    @Override
    public void paint(Graphics gr){
        super.paint(gr);
        Graphics2D g = (Graphics2D) gr;
        g.setColor(getBackground());
        g.fillRect(0, 0, getWidth(), getHeight());
        g.setColor(Color.black);
        g.drawLine(0, 0, getWidth(), 0);
        g.setColor(getLighterColor(getBackground()));
        g.drawLine(1, 1, getWidth(), 1);
        trayIcons = new Vector();
        repaintComponents();
    }
    
    @Override
    public Component add(Component component){
        super.add(component);
        components.add(component);
        return component;
    }

    private void repaintComponents(){
        for(int i = 0; i < components.size(); i++){
            components.get(i).repaint();
        }
    }

    private Color getLighterColor(Color color){
        int red = color.getRed();
        int green = color.getGreen();
        int blue = color.getBlue();
        red += 70;
        green += 70;
        blue += 70;
        if(red > 255) red = 255;
        if(green > 255) green = 255;
        if(blue > 255) blue = 255;
        return new Color(red, green, blue);
    }

    public TrayIcon add(TrayIcon trayIcon){
        trayIcon.add(trayIcon);
        return trayIcon;
    }

    public void mouseClicked(MouseEvent e) {
        System.out.println("[" + e.getX() + ", " + e.getY() + "]");
    }

    public void mousePressed(MouseEvent e) {}

    public void mouseReleased(MouseEvent e) {}

    public void mouseEntered(MouseEvent e) {}

    public void mouseExited(MouseEvent e) {}

    private StartMenu startMenu;
    private Component parent;
    private int iconCount;
    private Vector<Component> components;
    private Vector<TrayIcon> trayIcons;

}

StartMenu:

package javaworld.components;

import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.image.BufferedImage;
import java.util.Vector;
import javax.swing.JComponent;

public class StartMenu extends JComponent implements MouseListener {

    public StartMenu(){
        items = new Vector();
        setSize(40, 40);
    }

    public Dimension getPopupSize(){
        return popupSize;
    }

    public void setPopupSize(int width, int height){
        popupSize.setSize(width, height);
    }

    public void addStartMenuItem(StartMenuItem item){
        items.add(item);
    }

    public void setIcon(BufferedImage icon){
        this.icon = icon;
    }

    @Override
    public void paint(Graphics g){
        super.paint(g);
        g.setColor(Color.black);
        g.fillRect(0, 0, 100, 100);
    }

    public void mouseClicked(MouseEvent e) {}

    public void mousePressed(MouseEvent e) {}

    public void mouseReleased(MouseEvent e) {
        popup.show((StartMenuItem[])items.toArray());
    }

    public void mouseEntered(MouseEvent e) {}

    public void mouseExited(MouseEvent e) {}

    private StartMenuPopup popup;
    private Dimension popupSize;
    private Vector<StartMenuItem> items;
    private BufferedImage icon;

}

Ich wär sehr froh, wenn mir Jemand helfen könnte.

 

[hdi]

Schmeiss deinen Component-Vektor weg und lass das Overriden der add-Methode. repaint() brauchst du nicht per Hand aufzurufen, ausserdem repainted eine Komponente immer implizit all ihre Kinderkomponenten.

das setVisible(true) von StartMenu muss ausserdem weg im Konstruktor der TaskBar. Dann schmeiss deine paint-Methode raus, diese ganzen Dinge sollst du nicht überschreiben! Wenn du nen schwarzen Hintergrund haben willst o.ä. dann nutze startMenu.setBackground(Color.BLACK) und ähnliche Methoden.

Merke: paint() oder add() usw zu überschreiben ist - wenn man Anfänger ist - IMMER falsch! Und auch ansonsten sollte man einen SEHR guten Grund haben, den man meistens nicht hat.

 

[Runtime]

Das mit dem add() und setColor(Color.black) hab ich nur gemacht, weil ich probiert habe den Fehler zu lösen. Und was soll ich sonst machen ausser paint() zu überschreiben? Und übrigens programmiere ich schon seit einem Jahr.

 

[Marco13]

Als Swing-Anfänger ( ) sollte man bei einer JComponent nur paintComponent überschreiben.

 

[hdi]

Aber auch nur dann wenn man etwas malen möchte. Und dazu gehören nicht Komponenten! Die brauchst du nur adden per add() und fertig! kein paint(), kein repaint() oder sonst was, das passiert alles automatisch. Und solche Dinge wie Hintergrundfarbe oder Rahmen usw sollte man auch nicht malen da es ja dafür die Methoden für Components gibt (setBackground/setBorder/usw)

paint()/paintComponent() sollte man nur verwenden wenn man jetzt irgendwie spezielle Dinge Zeichnen möchte. zB bei Spielen die Texturen oder sowas.

 

[Runtime]

Wenn wir gleich dabei sind, was ist der Unterschied zwischen paintComponent() und paint()?

 

[L-ectron-X]

http://www.java-forum.org/java-basi...6-unterschied-zwischen-2-grafik-methoden.html

 

[Runtime]

Gibt es das auch in Deutsch (bin noch Schüler)?