Почему при изменении окна каждый раз создается новый обьект?

@warrax · Регистрация: 16.10.2013

Author24 — интернет-сервис помощи студентам

сам код:

Кликните здесь для просмотра всего текста

Java

import java.awt.BorderLayout;
import java.awt.Button;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JFrame;
public class SimpleGui3c implements ActionListener {
    JFrame f;
    public static void main(String[] args) {
        SimpleGui3c gui = new SimpleGui3c();
        gui.go();
    }
    private void go() {
        f = new JFrame();
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
        Button btn = new Button("Change color");
        btn.addActionListener(this);
 
        MyDrawPanel drawpanel = new MyDrawPanel();
 
        f.getContentPane().add(BorderLayout.SOUTH, btn);
        f.getContentPane().add(BorderLayout.CENTER, drawpanel);
        f.setSize(300, 300);
        f.setVisible(true);
 
    }
    public void actionPerformed(ActionEvent e) {
        f.repaint();
    }
}
 
import java.awt.Color;
import java.awt.Graphics;
import javax.swing.JPanel;
 
public class MyDrawPanel extends JPanel {
    public MyDrawPanel() {
    }
 
    public void paintComponent(Graphics g) {
        int x = (int) (Math.random() * 250);
        int y = (int) (Math.random() * 250);
        
        int red = (int) (Math.random() * 255);
        int green = (int) (Math.random() * 255);
        int blue = (int) (Math.random() * 255);
        Color startColor = new Color(red, green, blue);
 
        g.setColor(startColor);
        g.fillOval(x, y, 50, 50);
        
    }
}

~~verylazy~~ · 04.11.2013, 18:47

какой объект?

@LanGrande · 04.11.2013, 18:51

Тоже самая проблема и у меня была с кодом)))

А вообще мне кажется,когда ты меняешь размер фрейма вызывается

Java

1	f.repaint();

Который перерисовывает каждый раз кружочек.

Добавлено через 1 минуту

Сообщение от verylazy

какой объект?

Если быстро менять размер появляется много кружков)) Это нормально.В итоге то один

@warrax · 04.11.2013, 21:02 **[ТС]**

Сообщение от verylazy

какой объект?

SimpleGui3c gui = new SimpleGui3c();

@Lancaster · 04.11.2013, 22:14

Как уже сказали выше, что при изменении размера фрейма вызывается метод repaint(), который вызовет все необходимые методы для новой отрисовки(поправьте, если не так) и ваш переопределенный paintComponent() тоже. Т.к. координаты и цвет овала находятся в этом методе, то соответственно, если он будет вызываться несколько раз, то при каждом вызове координаты будут меняться.
Одно из решений может быть в создании класса Oval, которому зададутся все необходимые параметры один раз - при инициализации. А при необходимости их поменять с помощью метода, например, randomize(), который задаст новые координаты.

@warrax · 05.11.2013, 00:08 **[ТС]**

Сообщение от Lancaster

Как уже сказали выше, что при изменении размера фрейма вызывается метод repaint(), который вызовет все необходимые методы для новой отрисовки(поправьте, если не так) и ваш переопределенный paintComponent() тоже. Т.к. координаты и цвет овала находятся в этом методе, то соответственно, если он будет вызываться несколько раз, то при каждом вызове координаты будут меняться.
Одно из решений может быть в создании класса Oval, которому зададутся все необходимые параметры один раз - при инициализации. А при необходимости их поменять с помощью метода, например, randomize(), который задаст новые координаты.

скажите, так правильно будет?

Кликните здесь для просмотра всего текста

Java

import java.awt.BorderLayout;
import java.awt.Button;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JFrame;
 
public class SimpleGui3c implements ActionListener {
    JFrame f;
    MyDrawPanel drawpanel;
 
    public static void main(String[] args) {
        SimpleGui3c gui = new SimpleGui3c();
        gui.go();
    }
 
    private void go() {
        f = new JFrame();
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
        Button btn = new Button("Change color");
        btn.addActionListener(this);
 
        drawpanel = new MyDrawPanel();
        drawpanel.setcolor();
 
        f.getContentPane().add(BorderLayout.SOUTH, btn);
        f.getContentPane().add(BorderLayout.CENTER, drawpanel);
        f.setSize(300, 300);
        f.setVisible(true);
 
    }
 
    public void actionPerformed(ActionEvent e) {
        drawpanel.setcolor();
        f.repaint();
    }
}
 
import java.awt.Color;
import java.awt.Graphics;
 
import javax.swing.JPanel;
 
public class MyDrawPanel extends JPanel {
    Color startColor;
    int x, y, red, green, blue;
 
    public void setcolor() {
        red = (int) (Math.random() * 255);
        green = (int) (Math.random() * 255);
        blue = (int) (Math.random() * 255);
        startColor = new Color(red, green, blue);
    }
 
    public MyDrawPanel() {
        x = (int) (Math.random() * 250);
        y = (int) (Math.random() * 250);
    }
 
    public void paintComponent(Graphics g) {
 
        g.setColor(startColor);
        g.fillOval(x, y, 50, 50);
 
    }
}

фигура стоит на месте, получает цвет и при нажатии меняет. Запутался как с цветом быть и создал метод. Но где именно его вызывать? Я два раза его в коде вставил

@RequiemMass · 05.11.2013, 00:14

А кто нибудь подскажет как от этого избавится?

@Lancaster · 05.11.2013, 00:20

Запутался как с цветом быть и создал метод. Но где именно его вызывать? Я два раза его в коде вставил

Вызывайте там, где его нужно вызвать - а именно, когда необходимо изменить цвет овала.
1-й раз вы его вызываете после создания объекта MyDrawPanel, хотя можно это сделать во время его создания (вызов метода поместить в конструктор).
В другие разы метод вызывается по листенеру.

@warrax · 05.11.2013, 00:37 **[ТС]**

Сообщение от Lancaster

Вызывайте там, где его нужно вызвать - а именно, когда необходимо изменить цвет овала.
1-й раз вы его вызываете после создания объекта MyDrawPanel, хотя можно это сделать во время его создания (вызов метода поместить в конструктор).
В другие разы метод вызывается по листенеру.

Спасибо! Все заработало. Снова туплю на мелочах.

Добавлено через 1 минуту

Сообщение от RequiemMass

А кто нибудь подскажет как от этого избавится?

от чего избавиться? В своем рандоматоре сделайте как сказал Lancaster и все будет как вы хотели. Лично у меня фигура стоит и при изменении окна. При нажатии кнопки меняет цвет.

Добавлено через 6 минут

Сообщение от Lancaster

Вызывайте там, где его нужно вызвать - а именно, когда необходимо изменить цвет овала.
1-й раз вы его вызываете после создания объекта MyDrawPanel, хотя можно это сделать во время его создания (вызов метода поместить в конструктор).
В другие разы метод вызывается по листенеру.

теперь проблемка - как создать несколько обьектов?

@RequiemMass · 05.11.2013, 00:52

warrax, я имею ввиду избавится от мерцания круга, если он 1. Но походу никак, кроме setResizeble(false);

@Lancaster · 05.11.2013, 00:52

Сообщение от warrax

теперь проблемка - как создать несколько обьектов?

Сообщение от Lancaster

Одно из решений может быть в создании класса Oval, которому зададутся все необходимые параметры один раз - при инициализации. А при необходимости их поменять с помощью метода, например, randomize(), который задаст новые координаты.

Как вариант, можно реализовать это так...

Java

public class SimpleGui3c {
 
    private JFrame frame;
 
    public static void main(String[] args) {
        SimpleGui3c gui = new SimpleGui3c();
        gui.go();
    }
 
    private void go() {
        frame = new JFrame();
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
        final MyDrawPanel drawpanel = new MyDrawPanel();
 
        Button btnColor = new Button("Change color");
        btnColor.addActionListener(new ActionListener() {
 
            @Override
            public void actionPerformed(ActionEvent ae) {
                drawpanel.changeColor();
            }
        });
 
        Button btnPosition = new Button("Change position");
        btnPosition.addActionListener(new ActionListener() {
 
            @Override
            public void actionPerformed(ActionEvent ae) {
                drawpanel.changePosition();
            }
        });
 
        JPanel pnlTool = new JPanel(new GridLayout(1, 2));
        pnlTool.add(btnColor);
        pnlTool.add(btnPosition);
 
        frame.getContentPane().add(BorderLayout.SOUTH, pnlTool);
        frame.getContentPane().add(BorderLayout.CENTER, drawpanel);
        frame.setSize(300, 300);
        frame.setVisible(true);
    }
}
 
class MyDrawPanel extends JPanel {
 
    private List<Oval> ovals;
 
    public MyDrawPanel() {
 
        ovals = getRandomOvals();
    }
 
    @Override
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
 
        drawOvals(g);
    }
 
    private void drawOvals(Graphics g) {
        for (Oval oval : ovals) {
            g.setColor(oval.getColor());
            g.fillOval(oval.getX(), oval.getY(), oval.getWidth(), oval.getHeight());
        }
    }
 
    public void changePosition() {
        for (Oval oval : ovals) {
            oval.randomPosition();
        }
        this.repaint();
    }
 
    public void changeColor() {
        for (Oval oval : ovals) {
            oval.randomColor();
        }
        this.repaint();
    }
 
    private List<Oval> getRandomOvals() {
        List<Oval> arrOvals = new ArrayList<>();
 
        for (int i = 0; i < 10; i++) {
            Oval oval = new Oval();
            oval.randomize();
 
            arrOvals.add(oval);
        }
 
        return arrOvals;
    }
}
 
class Oval {
 
    private final int width = 50;
    private final int height = 50;
    private int x;
    private int y;
    private Color color;
 
    public Oval(int x, int y, Color c) {
        this.x = x;
        this.y = y;
        this.color = c;
    }
 
    public Oval() {
    }
 
    public void randomize() {
        randomPosition();
        randomColor();
    }
 
    public void randomPosition() {
        x = (int) (Math.random() * 250);
        y = (int) (Math.random() * 250);
    }
 
    public void randomColor() {
        Random rand = new Random();
        color = new Color(rand.nextInt(255), rand.nextInt(255), rand.nextInt(255));
    }
 
    public int getX() {
        return x;
    }
 
    public void setX(int x) {
        this.x = x;
    }
 
    public int getY() {
        return y;
    }
 
    public void setY(int y) {
        this.y = y;
    }
 
    public Color getColor() {
        return color;
    }
 
    public void setColor(Color color) {
        this.color = color;
    }
 
    public int getWidth() {
        return width;
    }
 
    public int getHeight() {
        return height;
    }
}

~~verylazy~~ Заблокирован
	04.11.2013, 18:47	2
	какой объект? 0

@warrax 17 / 22 / 6 Регистрация: 16.10.2013 Сообщений: 316
	04.11.2013, 21:02 [ТС]	4
	Сообщение от verylazy какой объект? SimpleGui3c gui = new SimpleGui3c(); 0

@Lancaster 63 / 63 / 12 Регистрация: 29.08.2012 Сообщений: 152
	04.11.2013, 22:14	5
	Как уже сказали выше, что при изменении размера фрейма вызывается метод repaint(), который вызовет все необходимые методы для новой отрисовки(поправьте, если не так) и ваш переопределенный paintComponent() тоже. Т.к. координаты и цвет овала находятся в этом методе, то соответственно, если он будет вызываться несколько раз, то при каждом вызове координаты будут меняться. Одно из решений может быть в создании класса Oval, которому зададутся все необходимые параметры один раз - при инициализации. А при необходимости их поменять с помощью метода, например, randomize(), который задаст новые координаты. 1

@RequiemMass 92 / 92 / 18 Регистрация: 06.01.2012 Сообщений: 394
	05.11.2013, 00:14	7
	А кто нибудь подскажет как от этого избавится? 0

@Lancaster 63 / 63 / 12 Регистрация: 29.08.2012 Сообщений: 152
	05.11.2013, 00:20	8
	Запутался как с цветом быть и создал метод. Но где именно его вызывать? Я два раза его в коде вставил Вызывайте там, где его нужно вызвать - а именно, когда необходимо изменить цвет овала. 1-й раз вы его вызываете после создания объекта MyDrawPanel, хотя можно это сделать во время его создания (вызов метода поместить в конструктор). В другие разы метод вызывается по листенеру. 1

@warrax 17 / 22 / 6 Регистрация: 16.10.2013 Сообщений: 316
	05.11.2013, 00:37 [ТС]	9
	Сообщение от Lancaster Вызывайте там, где его нужно вызвать - а именно, когда необходимо изменить цвет овала. 1-й раз вы его вызываете после создания объекта MyDrawPanel, хотя можно это сделать во время его создания (вызов метода поместить в конструктор). В другие разы метод вызывается по листенеру. Спасибо! Все заработало. Снова туплю на мелочах. Добавлено через 1 минуту Сообщение от RequiemMass А кто нибудь подскажет как от этого избавится? от чего избавиться? В своем рандоматоре сделайте как сказал Lancaster и все будет как вы хотели. Лично у меня фигура стоит и при изменении окна. При нажатии кнопки меняет цвет. Добавлено через 6 минут Сообщение от Lancaster Вызывайте там, где его нужно вызвать - а именно, когда необходимо изменить цвет овала. 1-й раз вы его вызываете после создания объекта MyDrawPanel, хотя можно это сделать во время его создания (вызов метода поместить в конструктор). В другие разы метод вызывается по листенеру. теперь проблемка - как создать несколько обьектов? 0

@RequiemMass 92 / 92 / 18 Регистрация: 06.01.2012 Сообщений: 394
	05.11.2013, 00:52	10
	warrax, я имею ввиду избавится от мерцания круга, если он 1. Но походу никак, кроме setResizeble(false); 0