Viktoriya Kim

Thank you, Fred Kleinschmidt, for the explanation and your advice!
Thank you, Junilu Lacar. Yeah, there is a mistake ))) I should improve my English. =)

1. Define a class variable "circleColor" in the class "MyDrawPanel" and change it in the method "changeColor". Remove the code which changes the color from the method "paintComponent" to fix this problem.
2. In the inner class "ColorListner" of the class "TwoButtons" call the method "changeColor" which changes the variable "circleColor".
Sorry for my English =) I could make some mistakes.

Thank you. I've already found the solution on this forum.
import javax.swing.*; import java.awt.*; import java.awt.event.ActionEvent; import java.awt.event.ActionListener; /** * Created by MS1 on 27.09.2016. */ public class TwoButtons {    JLabel label1;    MyDrawPanel panel;    public static void main(String[] args) {        TwoButtons gui = new TwoButtons();        gui.go();    }    public void go(){        JFrame frame = new JFrame();        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);        JButton labelButton = new JButton("Change label");        labelButton.addActionListener(new LabelListner());        JButton colorButton = new JButton("Change color");        colorButton.addActionListener(new ColorListner());        label1 = new JLabel("I'm label");        panel = new MyDrawPanel();        frame.getContentPane().add(BorderLayout.EAST, labelButton);        frame.getContentPane().add(BorderLayout.SOUTH, colorButton);        frame.getContentPane().add(BorderLayout.WEST, label1);        frame.getContentPane().add(BorderLayout.CENTER, panel);        frame.setSize(500, 300);        frame.setVisible(true);    }    class LabelListner implements ActionListener{        int counter;        public void actionPerformed(ActionEvent event){            counter++;            if (counter%2==0)                label1.setText("Ouch!");            else                label1.setText("Ouch ouch!");        }    }    class ColorListner implements ActionListener{        public void actionPerformed(ActionEvent event){            panel.changeColor();        }    } } public class MyDrawPanel extends JPanel {    GradientPaint circleColor;    public MyDrawPanel(){        Color color1 = getRandomColor();        Color color2 = getRandomColor();        circleColor= new GradientPaint(70, 70, color1, 150, 150, color2);    }    public void paintComponent(Graphics g){        g.fillRect(0, 0, this.getWidth(), this.getHeight());        Graphics2D g2d = (Graphics2D) g;        g2d.setPaint(circleColor);        g2d.fillOval(70, 70, 100, 100);    }    public void changeColor(){        Color color1 = getRandomColor();        Color color2 = getRandomColor();        circleColor= new GradientPaint(70, 70, color1, 150, 150, color2);        this.repaint();    }    Color getRandomColor(){        int red = (int) (Math.random()*255);        int green = (int) (Math.random()*255);        int blue = (int) (Math.random()*255);        Color randomColor = new Color(red, green, blue);        return randomColor;    } }

HI, all. I've started to learn Java and struggled with a task TwoButtons (chapter 12). When I click the button "Change label" it's also changes the color of my circle, it seems that it repaints the JFrame. And when I resize the window, the circle changes the color. I've experimented with the code a little to understand how it works. And I think that if something changes in the window it repaints all the frame. Am I right? Did I something wrong?
Thanks!
import javax.swing.*; import java.awt.*; import java.awt.event.ActionEvent; import java.awt.event.ActionListener; /** * Created by MS1 on 27.09.2016. */ public class TwoButtons {    JLabel label1;    MyDrawPanel panel;    public static void main(String[] args) {        TwoButtons gui = new TwoButtons();        gui.go();    }    public void go(){        JFrame frame = new JFrame();        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);        JButton labelButton = new JButton("Change label");        labelButton.addActionListener(new LabelListner());        JButton colorButton = new JButton("Change color");        colorButton.addActionListener(new ColorListner());        label1 = new JLabel("I'm label");        panel = new MyDrawPanel();        frame.getContentPane().add(BorderLayout.EAST, labelButton);        frame.getContentPane().add(BorderLayout.SOUTH, colorButton);        frame.getContentPane().add(BorderLayout.WEST, label1);        frame.getContentPane().add(BorderLayout.CENTER, panel);        frame.setSize(500, 300);        frame.setVisible(true);    }    class LabelListner implements ActionListener{        int counter;        public void actionPerformed(ActionEvent event){        /*         * it changes the color too... if I remain only code         * "label1.setText("Ouch!");" it changes only for the first time I click the button         */            counter++;            if (counter%2==0)                label1.setText("Ouch!");            else                label1.setText("Ouch ouch!");        }    }    class ColorListner implements ActionListener{        public void actionPerformed(ActionEvent event){            panel.repaint();        }    } } public class MyDrawPanel extends JPanel {    public void paintComponent(Graphics g){        g.fillRect(0, 0, this.getWidth(), this.getHeight());        Graphics2D g2d = (Graphics2D) g;        Color color1 = getRandomColor();        Color color2 = getRandomColor();        GradientPaint gradient = new GradientPaint(70, 70, color1, 150, 150, color2);        g2d.setPaint(gradient);        g2d.fillOval(70, 70, 100, 100);    }    Color getRandomColor(){        int red = (int) (Math.random()*255);        int green = (int) (Math.random()*255);        int blue = (int) (Math.random()*255);        Color randomColor = new Color(red, green, blue);        return randomColor;    } }