2D-Grafik 2 Probleme beim zeichnen

[NiXPhoE]

Hallo

ich hab im folgenden Code 2 Probleme.
1. Es wird nicht auf den grünen Bereich gemalt?
2. Wenn ich die Anwendung schliese, dann wird der Text beim zweiten öffnen immer zwei mal gezeichnet?

ShockCfg.java
[JAVA=42]
import javax.swing.JFrame;

public class ShockCfg
{
public static void main(final String args[])
{
RickenbackerMain rick = new RickenbackerMain();
rick.setVisible(true);
rick.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
}
}
[/code]

RickenbackerMain.java
[JAVA=42]
import java.awt.*;
import java.awt.geom.*;

import javax.swing.*;

public class RickenbackerMain extends JFrame
{
private static final long serialVersionUID = 1L;
int x = 138;
int y = 159;
private GeneralPath gp;
private Font f;
public Toolkit tk;
public Image theICON;

public RickenbackerMain()
{
super("ShockConfig 3.03.01");
this.setBackground(Color.black);
this.setSize(640, 480);
}

public void paint(Graphics g)
{
Graphics2D g2 = (Graphics2D)g;
g2.setColor(Color.green);
f = new Font("Luicidan", Font.PLAIN, 10);
this.setFont(f);
tk = this.getToolkit();
theICON = tk.getImage("image.jpg");
this.setIconImage(theICON);

g2.drawString("Joystick", x, y);
gp = new GeneralPath();
gp.moveTo(x-5, y-5);
gp.lineTo(x-10, y-5);
gp.lineTo(x-10, y+35);
gp.lineTo(x+135, y+35);
gp.lineTo(x+135, y-5);
gp.lineTo(x+47, y-5);
g2.draw(gp);
g2.drawRect(x+2, y+11, 8, 8);
g2.drawString("MainJoy", x+16, y+20);

g2.drawRect(x-20, y-20, 425, 190);
g2.fillRect(x-20, y+170, 425, 30);
g2.setColor(Color.black);
g2.drawRect(x-20, y+170, 50, 14);
g2.drawString("MainPanRes", x-15, y+170);

}
}
[/code]

 

[njans]

[STRIKE]Erstmal solltest du nicht die Paint methode, sondern die paintComponent Methode überschreiben!
Paint überschreibt man bei AWT, nicht bei SWING.[/STRIKE]
Übersehen, dass du ja direkt von JFrame erbst, der hat keine paintComponent.

Du solltest aber mal dafür sorgen, dass du nicht ständig dein Bild überlagerst:
super.paint(g); mal als erstes in deine paint Methode.

Generell würde ich dir empfehlen nicht den Frame zu beerben, sondern ein JPanel und auf dem zu zeichnen. Da wirst du dann weniger Problem mit bekommen.

 

[NiXPhoE]

ok, danke nochmals!

 

[njans]

import java.awt.*;
import java.awt.geom.*;
import javax.swing.*; 
 
public class RickenbackerMain extends JPanel
{
	public static void main(String[] args)
	{
		JFrame frame = new JFrame();
		frame.add(new RickenbackerMain());
		frame.setSize(640, 480);
		frame.setVisible(true);
	}
	
    private static final long serialVersionUID = 1L;
    int x = 138; 
    int y = 159;
    private GeneralPath gp;
    private Font f;
    public Toolkit tk;
    public Image theICON;
    
    public RickenbackerMain()
    {
        //super("ShockConfig 3.03.01");
        this.setBackground(Color.black); 
        this.setSize(640, 480);
    }
    
    public void paintComponent(Graphics g) 
    {
    	super.paintComponent(g);
    	
        Graphics2D g2 = (Graphics2D)g;
        g2.setColor(Color.green);
        f = new Font("Luicidan", Font.PLAIN, 10);
        this.setFont(f);
        tk = this.getToolkit();
        theICON = tk.getImage("image.jpg");
        //this.setIconImage(theICON);
               
       g2.drawString("Joystick", x, y);
       gp = new GeneralPath();
       gp.moveTo(x-5, y-5);
       gp.lineTo(x-10, y-5);
       gp.lineTo(x-10, y+35);
       gp.lineTo(x+135, y+35);
       gp.lineTo(x+135, y-5);
       gp.lineTo(x+47, y-5);
       g2.draw(gp);
       g2.drawRect(x+2, y+11, 8, 8);
       g2.drawString("MainJoy", x+16, y+20);
       
       g2.drawRect(x-20, y-20, 425, 190);
       g2.fillRect(x-20, y+170, 425, 30);
       g2.setColor(Color.black);
       g2.drawRect(x-20, y+170, 50, 14); 
       g2.drawString("MainPanRes", x-15, y+170);     
    }
}

So würde das dann aussehen. Allerdings würde ich auch davon abraten so viel in der paintComponent zu machen sondern vieles auszulagern: Man sollte dort wirklich nur zeichnen. Das ist nur ein ein Ausschnitt aus deinem Programm, ich wollte es jedoch mal erwähnen.

 

[NiXPhoE]

also das geht bei mir nicht, weil ich einen JFrame brauche bei dem ich auch das Icon ändern kann. also die Ornung die mir vorschwebt ist:

ShockCfg für die einzelnen Aplikationen.
Rickenbacker für das JFrame.
RickenbackerMain, für die Oberfläche also den Panel.

 

[njans]

Kannst du doch machen: Warum machst du es nicht generell etwas schöner und kapselst die ganezn Methoden des Jframes einmal ab:

public class MyGraphicalComponentOrWhatever
private JFrame myUsedFrame;
private JPanel myCustomPanel;

public MyGraphicalComponentOrWhatever()
{
  myUsedFrame = new JFrame();
  //.. hier eben den ganzen Krams mit dem Frame machen
  //... und das JPanel erzeugen und es hinzufügen
}

public void setVisible(boolean aFlag)
{
  frame.setVisible(aFlag);;
}
}

So hast du dann durch die Klasse MyGraphicalComponentOrWhatever (Name steht noch zur Auswahl ) eine Wrapper Klasse, die gekonnt alle unnötigen Funktionen nach außen abschirmt und an der du eben implementieren kannst, was du willst. So hast du eben nicht das Problem, dass du den Frame in der main erstellst, sondern nur das MyGraphicalComponentOrWhatever Objekt. Ebenfalls kannst du dann eben Frame und panel verwalten, wie du möchtest.

Edit:
Mir ist nicht ganz klar, wie du dir das vorstellst. Allerdings sollte man zuerst einmal klar stellen: Was soll das Programm denn am Ende machen?

 

[NiXPhoE]

jetzt hab ich wie schon vorher ein graues Frame?

 

[njans]

A) Wäre durchaus sinnvoll meine editierte Frage aus dem vorpost zu beantworten.
B) Ich kann keinen Sourcecode erraten, daher musst du mir zeigen, was du momentan hast

 

[NiXPhoE]

import java.awt.*;
import javax.swing.*;

public class Rickenbacker
{
	@SuppressWarnings("unused")
	private static final long serialVersionUID = 1L;
	private JFrame rick;
	public Toolkit tk;
	public Image theICON;
	
	public Rickenbacker(SetRes setres)
	{
		rick = new JFrame("ShockCfg 3.03.01");
		rick.setSize(640, 480);
   		rick.setVisible(true);
		rick.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		tk = rick.getToolkit();
		theICON = tk.getImage("image.jpg");
		rick.setIconImage(theICON);
	}
}

import java.awt.*;
import java.awt.geom.*;

import javax.swing.*; 
 
public class RickenbackerMain extends JPanel
{
	private static final long serialVersionUID = 1L;
	int x = 138; 
    int y = 159;
    private GeneralPath gp;
    private Font f;
    private JPanel ricky;
    
    public RickenbackerMain()
    {
    	ricky = new JPanel();
    	ricky.setBackground(Color.black);
    }
 
    public void paintComponent(Graphics g) 
    {
    	super.paintComponent(g);
    	Graphics2D g2 = (Graphics2D)g; 
    	SetRes setres = new SetRes();
    	g2.setColor(Color.green);
    	f = new Font("Luicidan", Font.PLAIN, 10);
    	this.setFont(f);
    	       
       g2.drawString(setres.getTxt().getString("MainPaneJoy"), x, y);
       gp = new GeneralPath();
       gp.moveTo(x-5, y-5);
       gp.lineTo(x-10, y-5);
       gp.lineTo(x-10, y+35);
       gp.lineTo(x+135, y+35);
       gp.lineTo(x+135, y-5);
       gp.lineTo(x+47, y-5);
       g2.draw(gp);
       g2.drawRect(x+2, y+11, 8, 8);
       g2.drawString(setres.getTxt().getString("MainJoy"), x+16, y+20);
       
       g2.drawRect(x-20, y-20, 425, 190);
       g2.fillRect(x-20, y+170, 425, 30);
       g2.setColor(Color.black);
       g2.drawRect(x-20, y+170, 50, 14); 
       g2.drawString(setres.getTxt().getString("MainPanRes"), x-15, y+170);

 

[Marco13]

Nur überflogen

    //private JPanel ricky; // Weg, die Klasse IST schon ein JPanel
    
    public RickenbackerMain()
    {
        //ricky = new JPanel();
        //ricky.setBackground(Color.black);
        this.setBackground(Color.BLACK);
    ...

 

[njans]

public void paintComponent(Graphics g) 
    {
        super.paintComponent(g);

        // Füllen des Hintergrundes mit der default color
        g.fillRect(0,0,this.getWidth(),this.getHeight());


        Graphics2D g2 = (Graphics2D)g; 
        SetRes setres = new SetRes(); // Soll das unm.maui.server.SetRes sein?
        g2.setColor(Color.green);

        // Geht kürzer
        // f = new Font("Luicidan", Font.PLAIN, 10);
        //this.setFont(f);
       g2.setFont(new Font("Luicidan", Font.PLAIN, 10));
        // Idealier weise speicherst du den Font anstatt ihn jedes mal neu zu erstellen               

       g2.drawString(setres.getTxt().getString("MainPaneJoy"), x, y);
       // Den path lieber nicht hier jedesmal erstellen, sondern da, wo es sinnvoll ist
       gp = new GeneralPath();
       gp.moveTo(x-5, y-5);
       gp.lineTo(x-10, y-5);
       gp.lineTo(x-10, y+35);
       gp.lineTo(x+135, y+35);
       gp.lineTo(x+135, y-5);
       gp.lineTo(x+47, y-5);
       g2.draw(gp);
       g2.drawRect(x+2, y+11, 8, 8);
       g2.drawString(setres.getTxt().getString("MainJoy"), x+16, y+20);
       
       g2.drawRect(x-20, y-20, 425, 190);
       g2.fillRect(x-20, y+170, 425, 30);
       g2.setColor(Color.black);
       g2.drawRect(x-20, y+170, 50, 14); 
       g2.drawString(setres.getTxt().getString("MainPanRes"), x-15, y+170);

 

[NiXPhoE]

also hier noch mal der Code, weil ich habe immer noch ein graues JFrame.

ShockCfg:

import java.awt.Color;
import javax.swing.UIManager;
public class ShockCfg
public static void main(final String args[])
	{
		UIManager.put("Button.focus",new javax.swing.plaf.ColorUIResource(Color.GREEN));
		UIManager.put("CheckBox.focus",new javax.swing.plaf.ColorUIResource(Color.GREEN));
		UIManager.put("RadioButton.focus",new javax.swing.plaf.ColorUIResource(Color.GREEN));
		
		SetRes setres = new SetRes();
new Rickenbacker(setres);
}

import java.awt.*;
import javax.swing.*;

public class Rickenbacker extends JFrame
{
	@SuppressWarnings("unused")
	private static final long serialVersionUID = 1L;
	private JFrame rick;
	public Toolkit tk;
	public Image theICON;
	
	public Rickenbacker(SetRes setres)
	{
		super("ShockCfg 3.03.01");
		this.setSize(640, 480);
   		this.setVisible(true);
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		tk = this.getToolkit();
		theICON = tk.getImage("image.jpg");
		this.setIconImage(theICON);
	}
}

import java.awt.*;
import java.awt.geom.*;
import javax.swing.*; 
 
public class RickenbackerMain extends JPanel
{
	private static final long serialVersionUID = 1L;
	int x = 138; 
    int y = 159;
    private GeneralPath gp;
        
    public RickenbackerMain()
    {
    	 this.setBackground(Color.BLACK);
    }

    public void paintComponent(Graphics g) 
    {
    	super.paintComponent(g);
    	
    	// Füllen des Hintergrundes mit der default color
        g.fillRect(0,0,this.getWidth(),this.getHeight());
        
    	Graphics2D g2 = (Graphics2D)g; 
    	SetRes setres = new SetRes();
    	g2.setColor(Color.GREEN);
    	
    	g2.setFont(new Font("Luicidan", Font.PLAIN, 10));
    	
       g2.drawString(setres.getTxt().getString("MainPaneJoy"), x, y);
       gp = new GeneralPath();
       gp.moveTo(x-5, y-5);
       gp.lineTo(x-10, y-5);
       gp.lineTo(x-10, y+35);
       gp.lineTo(x+135, y+35);
       gp.lineTo(x+135, y-5);
       gp.lineTo(x+47, y-5);
       //g2.draw(gp);
       g2.drawRect(x+2, y+11, 8, 8);
       g2.drawString(setres.getTxt().getString("MainJoy"), x+16, y+20);
       
       g2.drawString(setres.getTxt().getString("MainPaneRog"), x+155, y);
       gp.moveTo(x+150, y-5);
       gp.lineTo(x+145, y-5);
       gp.lineTo(x+145, y+35);
       gp.lineTo(x+265, y+35);
       gp.lineTo(x+265, y-5);
       gp.lineTo(x+224, y-5);
       //g2.draw(gp);
       g2.drawRect(x+156, y+12, 8, 8);
       g2.drawString(setres.getTxt().getString("MainRog"), x+169, y+20);

       g2.drawRect(x-20, y-20, 425, 190);
       g2.fillRect(x-20, y+170, 425, 30);
       g2.setColor(Color.BLACK);
       g2.drawRect(x-20, y+170, 50, 14); 
       g2.drawString(setres.getTxt().getString("MainPanRes"), x-15, y+170);
              
    }
}

 

[jgh]

dir fehlt ein

	add(new RickenbackerMain());

in deiner Klasse [c]Rickenbacker[/c] im Konstruktor.

Und allgemein sollten Variablennamen sprechend sein...also wenn schon Rickenbacker, dann Rickenbacker(Main)Panel, RickenbackerFrame usw

 

[njans]

Du könntest dein JPanel auch als innere Klasse für deinen JFrame erstellen. Das würde ich jedoch eher nicht machen. Den JFrame zu beerben scheint mir momentan sehr sinnlos, da du ihm ja keine Eigenschaften verpasst, die du nicht auch an einem normalen JFrame setzen könntest.

Zu Wissen, was du damit machen wolltest würde da sicherlich noch helfen.

Nebenbei:

private JFrame rick;

In deinem JFrame bringt dir doch wenig, du hast doch schon ein JFrame Objekt.

 

[NiXPhoE]

Das hier:

// Füllen des Hintergrundes mit der default color
        //g.fillRect(0,0,this.getWidth(),this.getHeight());

ist überflüssig, damit wird der Hintergrung grau.

Jetzt wird nicht mehr doppelt geschrieben.
Fragt sich nur noch, warum nicht auf den grünen Rectangle geschrieben wird?

g2.drawRect(x-20, y-20, 425, 190);
       g2.fillRect(x-20, y+170, 425, 30);
       g2.setColor(Color.BLACK);
       g2.drawRect(x-20, y+170, 50, 14); 
       g2.drawString(setres.getTxt().getString("MainPanRes"), x-15, y+170);

 

[njans]

Das liegt daran, dasss der String über das Kästchen geschrieben wird. drawString hat als startpunkt nicht die linke, obere Ecke sondern die linke untere Ecke. Verschiebe den String weiter nach unten.

P.S.

// So setzt man den Color damit der Background, so in diesem Fall, weiß wird.
g.setColor(Color.WHITE);
g.fillRect(0,0,this.getWidth(),this.getHeight());

 

[NiXPhoE]

Danke, so geht das!

Ich setze den Hintergrund mit:

this.setBackground(Color.BLACK);

 

[njans]

Damit setzt du nur die Farbe. Im Aufruf super.paintComponent(g) wird dann die mit der gesetzten Farbe der Hintergrund gefüllt. Das kannst du eben auch manuell machen. Bietet sich eben dann an, wenn man von einem Button o.ä. erbt und nicht jedes mal den Button mit dem Super Aufruf zeichnen will, sondern eben nur das, was man selbst macht.

Alle Probleme also nun beseitigt ?

 

[NiXPhoE]

Ja, es geht auf jeden Fall nur um den Hintergrund, aber danke werd mir den Tip merken.