AWT Graphics initialisieren

[realodds]

Hallo,
ich habe ein kleines Problem in einem größeren Projekt:

//Klasse Drawing (JComponent)
public void paint(Graphics g) {
Graphics2D g2 = (Graphics2D) g;
String str1 = "";
g2.drawString(str1,0,90);
}
public void setStr1(String str1) {
this.str1 = str1;
}

soweit verstehe ich es noch, g wird automatisch initialisiert.

//Klasse Graphic
JFrame frame = new JFrame("");
frame.setDefaultCloseOperation(JFrame.EXIR_ON_CLOSE);
frame.setSize(500,500);
Drawing d = new Drawing();
frame.add(d);
//bei einem ereignis wird str1 (aus Drawing) verändert und soll erneut gezeichnet werden
//nicht das ereignis
if (1<3){
d.setStr1("Hallo");
//paint(Graphics g) soll erneut durchlaufen, um den string zu zeichnen
/*
Graphics g;
d.paint(g);
//Das klappt natülich nicht, da g null ist
*/
//wenn ich es mit dem probierte:
d.paint(d.getGraphics);
//klappte es auch nicht, da es null ist "java.lang.NullPointerException"
}

Welche Graphics muss ich benutzen, wie kann ich sie initialisieren oder gehe ich da komplett falsch heran?
Danke im Voraus

 

[mihe7]

gehe ich da komplett falsch heran?

Ja. Erstens überschreibt man in Swig nicht paint sondern paintComponent, zweitens reicht ein einfaches d.repaint();

 

[realodds]

Ja. Erstens überschreibt man in Swig nicht paint sondern paintComponent, zweitens reicht ein einfaches d.repaint();

Also ich habe jetzt, nach deinem Vorschlag, es geändert und d.repaint(); benutzt, allerdings klappt das auch nicht. Es ist kein Problem bezüglich der Graphics mehr, aber trotzdem habe ich im Debug Modus überprüft und bei d.repaint(); geht es nicht in die andere klasse hinein, um den string zu zeichnen.

 

[LimDul]

Wie sieht deine Methode paintComponent aus? So wie da oben paint? Da erzeugst du ein neues String Objekt was du zeichnest und zeichnest nicht das Feld.

 

[realodds]

Ja wie da oben, oben ist es halt vereinfacht. Aber bei repaint(); wird gar nicht erst in die andere methode reingegangen bzw. es wird auch nicht gezeichnet

 

[kneitzel]

Also bei repaint geht er nicht direkt rein in die andere Methode. Repaint signalisiert nur, dass die Komponente erneut gemalt werden soll. Das erfolgt dann kurze Zeit später sobald der UI Thread dazu kommt.

Zeig doch bitte etwas mehr von Deinem Code und was wichtig ist:
a) Deine Komponente überschreibt paintComponent korrekt.
b) Du blockierst nicht den UI Thread.

 

[realodds]

Es ist nicht viel mehr als das dort oben:

public class Drawing extends JComponent{
String str1 = "";
//noch weitere Strings, die aber denke ich nicht wichtig sind
public paintComponent(Graphics g){
Graphics2D g2 = (Graphics2D) g;
g2.drawString(str1,5,90);
g2.drawRect(0,0,100,100);
}
}

public class Graphic extends JFrame implements MouseListener{
public void gui(){
setDeafultCloseOperation(JFrame.EXIT_ON_CLOSE);
setSize(800,800);
setUndecorated(true);
Drawing d = new Drawing();
add(d);
}
public void mousePressed(MouseEvent e){
Rectangle r1 = new Rectangle(0,0,100,100);
if (r1.contains(e.getPoint())){
d.setStr1("Pressed here");
}
}
}

Und dann ist das Problem, dass bei repaint(); str1 immer noch als "" gezeichnet wird, da es nur nochmal gezeichnet wird und nicht durchläuft

 

[kneitzel]

Also da sieht erst einmal alles soweit ok aus und so wie ich Dich jetzt verstehe ist das Problem auch eher so, dass str1 nicht umgesetzt wird?

Soll das in dem MouseListener erfolgen? Hast Du denn denn hinzu gefügt?

Und in gui() erzeugst Du eine Drawing Instanze und speichert diese in einer lokalen Variable d. Wenn Du auch eine Instanzvariable d hast, die Du in mousePressed nutzen willst, dann enthält die natürlich nicht die Drawing Instanz, die du in gui() erzeugt hast....

 

[realodds]

Ja, stimmt das habe ich alles in Program drin, hier hab ichs vergessen ^^". Das Programm geht in den MousePressed rein und die bedingung usw. klappt alles. str1 wird zu "Pressed here" umgeändert und dies soll dann gezeichnet werden.

 

[kneitzel]

Also mit den limitierten Ausschnitten wird dir wohl keiner wirklich helfen können fürchte ich. Bau ein minimales, komplettes Beispiel mit der Problematik, das du ganz einstellst und dann können wir gerne im Detail schauen ...

 

[mihe7]

Ja, stimmt das habe ich alles in Program drin, hier hab ichs vergessen ^^". Das Programm geht in den MousePressed rein und die bedingung usw. klappt alles. str1 wird zu "Pressed here" umgeändert und dies soll dann gezeichnet werden.

Nochmal: Deinem d.setStr1 muss ein repaint() des Drawing-Objekts folgen. Am Besten machst Du das in der Methode setStr1.

 

[Blender3D]

Hallo,
ich habe ein kleines Problem in einem größeren Projekt:

Ich habe Dir ein kleines Beispiel gemacht.
Links klicken --> neue Position des Textes
Rechts klicken --> neuen Text eingeben

import javax.swing.JComponent;

@SuppressWarnings("serial")
public class Drawing extends JComponent {
    private int x = 0;
    private int y = 0;
    private String text = null;

    public String getText() {
        return text;
    }
    @Override
    protected void paintComponent(Graphics g) {
        super.paintComponent(g);
        if (text != null)
            g.drawString(text, x, y);
    }
    public void setTextLocation(int x, int y) {
        this.x = x;
        this.y = y;
    }
    public void setText(String text) {
        this.text = text;
    }
}

import java.awt.Dimension;
import java.awt.Point;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import javax.swing.JFrame;
import javax.swing.JOptionPane;
public class start {
    public static void main(String[] args) {
        JFrame frame = new JFrame("Drawing click left or right on frame ");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        Drawing drawing = new Drawing();
        drawing.setText("test");
        drawing.setTextLocation(0, 90);
        frame.add(drawing);
        frame.setPreferredSize(new Dimension(500, 500));
        frame.addMouseListener(new MouseAdapter() {
            @Override
            public void mouseClicked(MouseEvent e) {
                if (e.getButton() == MouseEvent.BUTTON3) {
                    String text = JOptionPane.showInputDialog(drawing, "Text eingeben", drawing.getText());
                    if (text != null)
                        drawing.setText(text);
                }
                if (e.getButton() == MouseEvent.BUTTON1) {
                    Point pos = e.getPoint();
                    drawing.setTextLocation(pos.x, pos.y);
                }
                drawing.repaint();
            }
        });
        frame.pack();
        frame.setVisible(true);
    }
}

 

[realodds]

Also ein Beispiel von mir:

public class Graphic implements MouseListener{
public static void main(String[] args){
JFrame window = new JFrame();
window.setSize(300,300);
window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
Drawing drw = new Drawing();
window.add(drw);
window.addMouseListener(this)
    public void mousePressed(MouseEvent e) {
    Drawing drw = new Drawing();
    drw.setStr("Clicked");
    //drw.repaint();
    }
    
}
}

public class Drawing extends JComponent {
String str = "";
    public void setStr1(String str1) {
        this.str1 = str1;
    }
public void paintComponent (Graphics g){
g.drawLine(0,90,300,90);
g.drawString(str,140,90);
}
}

weil ich das nicht so wirklich verstehe bzw auf meins umsetzen kann ^^". Trotzdem Danke.
Was müsste ich hier konkret ändern? Danke im Voraus und für all eure Antworten ^^

 

[mihe7]

public class Graphic implements MouseListener {
    private Drawing drawing;

    public void run() {
        drawing = new Drawing();
        drawing.addMouseListener(this)

        JFrame window = new JFrame();
        window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        window.add(drawing);
        window.setSize(300,300);
        window.setVisible(true);
    }

    public void mousePressed(MouseEvent e) {
        drawing.setStr("Clicked");
    }

    public static void main(String[] args) {
        SwingUtilities.invokeLater(() -> new Graphic().run());
    }
}

public class Drawing extends JComponent {
    private String str = "";

    public void setStr1(String str1) {
        this.str1 = str1;
        repaint();
    }

    public void paintComponent (Graphics g){
        g.drawLine(0,90,300,90);
        g.drawString(str,140,90);
    }
}

 

[LimDul]

Also ein Beispiel von mir:

public class Graphic implements MouseListener{
public static void main(String[] args){
JFrame window = new JFrame();
window.setSize(300,300);
window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
Drawing drw = new Drawing();
window.add(drw);
window.addMouseListener(this)
    public void mousePressed(MouseEvent e) {
    Drawing drw = new Drawing();
    drw.setStr("Clicked");
    //drw.repaint();
    }
    
}
}

Die Zeile
Drawing drw = new Drawing();
in mousePressed ist kompletter Humbug. Du erzeugt ein neues Drawing Objekt und änderst da den string. Das ist schön, ändert aber nix an dem angezeigten Drawing Objekt. Die Zeile muss komplett raus. Du willst ja das Drawing Objekt ändern, was du in der main Methode erzeugt hast.

 

[realodds]

Ahhhh vielen, vielen, vielen Dank an euch alle jetzt hat es geklappt!