Dessine un Rectangle dans JFrame ne fonctionne pas

J'ai ce bout de code qui est censé dessiner un rectangle sur un JFrame mais quand je lance le programme sur Eclipse, il ouvre juste le cadre, mais ne permet pas de tracer le cercle sur elle.

Voici le code:

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

public class Infout {
    Infout(){
        JFrame frame = new JFrame();

        frame.setSize(300, 400);
        frame.setTitle("An Empty Frame");
        frame.setDefaultCloseOperation(3);

        frame.setVisible(true);
    }

    public static void main(String[] args) {
         //TODO Auto-generated method stub
         Infout m = new Infout();
         m.paint(null); 
    }

    public void paint(Graphics g) 
    {
         g.drawRect(5, 5, 105, 105);
    }
}

Quelqu'un peut-il me dire pourquoi il ne fonctionne pas correctement?

Lorsque vous appelez la méthode de peinture
[Voir ce lien] stackoverflow.com/questions/9713432/...
+1 Pour l'affichage de votre question avec un valide SSCCE.

OriginalL'auteur AbSaintDane | 2014-03-14

  1. 2

    Je crois que ce que vous cherchez à faire est d'appeler frame.repaint(); à la place. Ce n'est toujours pas résoudre votre problème, cependant, parce que votre paint() méthode n'est pas réellement de la substitution de la JFrame's paint() méthode parce que votre classe ne s'étend pas JFrame, c'est juste la création d'un JFrame dans le constructeur.

    Ainsi, vous pouvez soit faire à la dernière minute remplacer et déplacer votre méthode de peinture dans le fait que (comme par polypiel de réponse), ou (personnellement, je pense que plus d'éloquence), vous pouvez avoir votre classe étendre JFrame, comme;

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

public class Infout extends JFrame{
    Infout(){
        setSize(300, 400);
        setTitle("An Empty Frame");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setVisible(true);
    }

    public static void main(String[] args) {
         Infout m = new Infout();
         m.repaint(); 
    }

    @Override
    public void paint(Graphics g) 
    {
         g.drawRect(5, 5, 105, 105);
    }

}

    Comme d'autres l'ont souligné cependant paintComponent() est un meilleur choix de de remplacer, et vous devriez vous rappeler de faire un appel à super() au début de la nouvelle méthode redéfinie. Vous devrez alors créer un nouveau JPanel à mettre à l'intérieur de votre JFrame parce que JFrame ne dispose pas d'un paintComponent() méthode pour remplacer la.

    Pour ce faire, vous pouvez supprimer votre paint() méthode de votre classe entièrement et ajoutez la dernière minute remplacer dans votre constructeur:

        setLayout(new BorderLayout());
    add(new JPanel(){
        @Override
        public void paintComponent(Graphics g){
            super.paintComponent(g);
            g.drawRect(5, 5, 105, 105);
        }
    }, BorderLayout.CENTER);

    Pour l'extensibilité et de la bonne orientée objet design cependant, vous pourriez être mieux dans le long terme de la définition de votre propre JPanel sous-classe et en remplaçant la paintComponent(Graphics) méthode. Désolé pour les indécis, j'espère que cette aide.

    La peinture Custom doivent être exécutés à paintComponent() de JComponent avec l'appel de super() méthode.
    Un problème à la fois, ils coopérative doit comprendre ce qu'ils ont fait de mal et pourquoi leur code actuel ne fonctionne pas la première. Je vais modifier ma réponse à suggérer que paintComponent() est un meilleur choix de de remplacer.
    Si l'OP n'est pas demander à ce sujet, vous avez besoin de lui suggérer d'utiliser de meilleur moyen pour éviter les problèmes dans l'avenir.
    Ajouter un exemple avec paintComponent et j'ai' ll vous donner des voix.
    jusqu'votes;)

    OriginalL'auteur Rudi Kershaw

  2. 2

    Vous avez réussi à briser complètement la peinture de la chaîne en appelant le:

    m.paint(null);

    Il devrait être: 

    m.repaint();

    Comme un résultat d'un appel à repaint(), le paint(Graphics) méthode sera appelée automatiquement, mais avec un valide objet graphique.

    Noter qu'il est généralement préférable de remplacer la paintComponent(Graphics) méthode de JPanel qui est ajouté à une JFrame (ou une fenêtre, ou une applet ou une boîte de dialogue..).

    La chose à faire lors de la substitution ou l'autre méthode, c'est d'appeler la méthode super avant de faire quelque chose d'autre.

    OriginalL'auteur Andrew Thompson

  3. 1

    Le doit remplacer la méthode de peinture de la JFrame. Une méthode de peinture dans Infout est inutile.

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

public class Infout {
    Infout(){
        JFrame frame = new JFrame() {
          @Override public void paint(Graphics g) {
            g.drawRect(5, 5, 105, 105);
          }
        };

        frame.setSize(300, 400);
        frame.setTitle("An Empty Frame");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        frame.setVisible(true);
    }

    public static void main(String[] args) {
         //TODO Auto-generated method stub
         Infout m = new Infout();
         //You shouldn't need to call repaint
    }
}
    "frame.setDefaultCloseOperation(3);" Ne pas utiliser des numéros de magie quand il y a des constantes définies.
    C'est pourquoi le @Override annotation est utile
    La peinture Custom doivent être exécutés à paintComponent() de JComponent avec l'appel de super() méthode.

    OriginalL'auteur polypiel

  4. 0

    Ne pas appeler à la peinture à l'externe et à l'appel de la super.de la peinture dans votre paint() de la méthode.

    public void paint(Graphics g)
{
 super.paint(g);
//....

}

    et supprimer m.peinture(null).

    La peinture Custom doivent être exécutés à paintComponent() de JComponent avec l'appel de super() méthode.

    OriginalL'auteur Shriram