Рисовать с помощью Swing Java

Вот некоторые вещи, которые вы не делаете в правильном смысле.

1. Во-первых, вы устанавливаете видимое свойство JFrame на видимость намного раньше, чем фактически добавили к нему все компоненты, и заставьте его реализовать свой размер.
2. Во-вторых, вы используете Absolute Layout, которого следует избегать в большинстве ситуаций.
3. В-третьих, вы явно создаете объект Graphics, который следует избегать, и вместо этого используйте тот, который предоставляется методом paintComponent (... ) по умолчанию

Взгляните на этот пример:

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

public class DrawingCircleExample {

    private JPanel drawingBoard;
    private JButton button;

    private static final int GAP = 5;

    private void displayGUI () {
        JFrame frame = new JFrame ( "" );
        frame.setDefaultCloseOperation ( JFrame.DISPOSE_ON_CLOSE );

        JPanel contentPane = new JPanel ();
        contentPane.setLayout ( new BorderLayout ( GAP, GAP ) );
        drawingBoard = new DrawingBoard ();
        contentPane.add ( drawingBoard, BorderLayout.CENTER );

        button = new JButton ( "Draw" );
        button.addActionListener ( new ActionListener () {
            @Override
            public void actionPerformed ( ActionEvent ae ) {
                 DrawingBoard board = ( DrawingBoard ) drawingBoard;
                 board.setState ();
            }
        } );
        contentPane.add ( button, BorderLayout.PAGE_END );

        frame.setContentPane ( contentPane );
        frame.pack ();
        frame.setLocationByPlatform ( true );
        frame.setVisible ( true );
    }

    public static void main ( String [] args ) {
        Runnable runnable = new Runnable () {
            @Override
            public void run () {
                new DrawingCircleExample ().displayGUI ();
            }
        };
        EventQueue.invokeLater ( runnable );
    }
}

class DrawingBoard extends JPanel {

    private static final int TOTAL_RECTANGLES = 5;
    private static final int WIDTH = 400;
    private static final int HEIGHT = 300;
    private static final int RADIUS = 50;
    private static final int X = 50;
    private static final int Y = 50;
    private int counter;
    private int moveXBy;
    private boolean isActive;
    private int count;

    public DrawingBoard () {
        setOpaque ( true );
        counter = 0;
        count = 0;
        isActive = false;
        moveXBy = ( RADIUS + ( counter ) * RADIUS );
    }

    public boolean setState () {        
        isActive = true;
        System.out.println ( "Outside MoveXBy: " + moveXBy );
        ++counter;
        counter %= TOTAL_RECTANGLES;
        int x = ( RADIUS + ( counter ) * RADIUS );
        if ( moveXBy != x ) {
            System.out.println ( "Inside First MoveXBy: " + moveXBy );
            repaint ( moveXBy, RADIUS, X, Y );
            moveXBy = x;
            System.out.println ( "Inside Second MoveXBy: " + moveXBy );
            repaint ( moveXBy, RADIUS, X, Y );          
        }       

        return isActive;
    }

    @Override
    public Dimension getPreferredSize () {
        return new Dimension ( WIDTH, HEIGHT );
    }

    @Override
    protected void paintComponent ( Graphics g ) {
        super.paintComponent ( g );
        g.drawRect ( 50, RADIUS, X, Y );
        g.drawRect ( 100, RADIUS, X, Y );
        g.drawRect ( 150, RADIUS, X, Y );
        g.drawRect ( 200, RADIUS, X, Y );
        g.drawRect ( 250, RADIUS, X, Y );

        g.setColor ( Color.RED );
        g.fillOval ( moveXBy, RADIUS, X, Y ) ;
    }
}

EDIT относительно комментария:

Для всех графических пользовательских интерфейсов требуется какой-то основной фрейм приложения для отображения. В Swing это пример javax.swing.JFrame. Поэтому наш первый шаг - создать экземпляр этого класса и убедиться, что все работает так, как ожидалось. Обратите внимание, что при программировании в Swing ваш код создания GUI должен быть помещен в Thread Dispatch Thread (EDT), больше информации о Concurrency в Swing. Это предотвратит потенциальные условия гонки, которые могут привести к тупиковой ситуации.

DrawingBoard также переопределяет getPreferredSize, который возвращает желаемую ширину и высоту панели (в этом случае 400 - это width, 300 - высота). Из-за этого классу DrawingCircleExample больше не нужно указывать размер кадра в пикселях. Он просто добавляет панель в фрейм, а затем вызывает пакет.

Метод paintComponent - это место, где выполняется ваша обычная картина. Этот метод определяется javax.swing.JComponent и затем переопределяется вашими подклассами для обеспечения их пользовательского поведения. Его единственный параметр, объект java.awt.Graphics, предоставляет ряд методов для рисования 2D-фигур и получения информации о среде графических приложений. В большинстве случаев объект, который фактически получен этим методом, будет экземпляром java.awt.Graphics2D (подкласс Graphics), который обеспечивает поддержку сложного 2D-рендеринга.

Большинство стандартных компонентов Swing имеют внешний вид, реализованный отдельными объектами "UI Delegate". Вызов super.paintComponent(g) передает графический контекст делегату компонента компонента, который рисует фон панели.

Чтобы наша пользовательская картина была максимально эффективной, мы будем отслеживать координаты X (переменная moveXBy в нашем случае) и перерисовывать только те области экрана, которые изменились. Это рекомендуемая передовая практика, которая будет поддерживать ваше приложение как можно эффективнее.

Вызов метода repaint. Этот метод определяется java.awt.Component и является механизмом, который позволяет вам программно перерисовывать поверхность любого данного компонента. Он имеет версию no-arg (которая реплицирует весь компонент) и версию multi-arg (которая реплицирует только указанную область.) Эта область также известна как клип. Вызов многопользовательской версии repaint требует немного дополнительных усилий, но гарантирует, что ваш код рисования не будет тратить циклы, перерисовывая области экрана, которые не изменились.

Поскольку мы ручная настройка клипа, наш setState метод вызывает repaint метод не один раз, а дважды. Первое обращение указывает Swing на перерисовку области компонента, где ранее был овал (унаследованное поведение использует делегат интерфейса пользователя, чтобы заполнить эту область текущим цветом фона.) Второй вызов создает область компонента, где в настоящее время овал, Важно отметить, что хотя мы дважды вызывали repaint в одном и том же обработчике событий, Swing достаточно умен, чтобы воспринимать эту информацию и перерисовывать все разделы экрана в одной операции с краской. Другими словами, Swing не будет перекрашивать компонент дважды подряд, даже если это то, что делает код.

Дополнительную информацию см. В разделе " Выполнение пользовательской живописи"