Начинающий Graphics2D Java: repaint ()

Объекты Graphics недолговечны, нестабильны, и вы не должны использовать их таким образом. Вместо установки g2 или любой другой Graphics поле создайте поле Color, скажем, с именем Color circleColor = ...;, и измените это. В методе protected void paintComponent(Graphics g) вызовите g.setColor(circleColor);, и это должно сработать.

Удалите эти поля, так как они опасны:

// static Graphics2D g2;
// static Graphics2D g3; 

Также ваш код показывает чрезмерное использование модификатора static, и я рискну рекомендовать, чтобы ни одно из ваших полей не было статическим, за исключением константы:

static final int NUM

Метод repaint () в конечном итоге вызовет метод paintComponent () вашей панели. Swing передаст объект Graphics, который будет использоваться в рисовании.

В коде рисования вы всегда жестко указываете цвет как КРАСНЫЙ. Не делай этого.

Вместо этого вам нужно определить переменную в своем классе панели. Допустим, «circleColor» (то есть «circleColor» заменяет вашу переменную «c», потому что имена переменных должны быть более информативными, а не просто одиночным символом).

Затем в вашем коде ActionListener вы делаете:

//g2.setColor(Color.green);
circleColor = Color.green;

Это в методе paintCompnent (), который вы делаете:

//c = Color.red;
g.setColor(circleColor);

Также:

static Graphics2D g2;
static Graphics2D g3; 

Ни в одной из этих переменных нет необходимости. Вы всегда используете объект Graphics, который передается методу рисования.

Прочтите раздел руководства по Swing на Картина на заказ для получения дополнительной информации и лучших примеров того, как структурировать ваш код. Например, вам НЕ следует использовать статические переменные.

Что происходит:

• Вы меняете цвет g2 в слушателе действий.
• Затем вы вызываете repaint();, поэтому вызывается paintComponent(Graphics g).
• В методе paintComponent(Graphics g) вы устанавливаете цвет g2 на c, поэтому вы отменяете изменения, сделанные в слушателе действий.

Вот почему круги в итоге не меняют цвет.

Вы должны установить c = Color.RED; в конструкторе, а не в цикле for, тогда вы можете просто изменить значение c в слушателе действий c = Color.GREEN;.

Также вы устанавливаете c = Color.RED; как в цикле for, так и перед ним, что бесполезно.

Обновлено:
Как предлагает @camickr, ваш код также плохо структурирован.

• Вам не нужны g2 и g3, потому что вы можете рисовать несколько фигур используя тот же объект Graphics g.
• В вашем случае вам даже не нужно вызывать g вне метода paintComponent, потому что вы можете просто изменить переменную Color c, как я сказал выше.
• Вам также не нужны все эти переменные static. Просто сделайте их private, и если вам нужно получить к ним доступ извне класса, вы должны создать несколько getters и setters.

Здесь вы найдете больше о статических переменных и несколько примеров.
Здесь вы найдете больше о геттерах, сеттерах и инкапсуляции.

Пример того, как вы можете изменить свой код:

public class CirclePanel extends JPanel implements ActionListener  {

    private JFrame f;
    private JButton run1, run2, reset, quit;
    private JPanel btnPanel;
    private int NUM;
    private Color c;

    public CirclePanel(){

        color1 = Color.red;
        color2 = Color.black;

        NUM = 5;

        // Setup JFrame and stuff as you were doing.

    }


    @Override
    public void paintComponent(Graphics g){

        super.paintComponent(g);

        g.translate(470, 400);

        for(int i = 0; i < NUM; i++){

            g.setColor(color1);
            g.fillOval(150, 0, 100, 100);

            g.setColor(color2);
            g.fillOval(90, 0, 30, 30);

            g.rotate(2*Math.PI/ NUM);
        }
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        if (e.getSource() == run1) {
            System.out.println("Entered Action Handler");

            color1 = Color.green;

            repaint();
        }
    }

    // GETTERS and SETTERS if needed
}