Снова. Снова этот repaint()

@vlodis · Регистрация: 29.05.2015

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

Всем привет. Сколько дней уже пытаюсь, нечего не выходит. Метод repaint не срабатывает. Как я понимаю, ошибка появляется в методе redraw класса draw. Ниже приведу укороченный код программы.
Надеюсь кто-нибудь поможет мне. Заранее спасибо.

Самый главный файл:

Java

public class main {
    private static int star[][] = new int[50][2];
    private static int colstar = 0;
    public static void main(String[] args) {
        new window().window();
        new tick(star, colstar);
    }
}

Window:

Java

public class window extends JFrame{
    public void window(){
        setTitle("Mega Stars");
        setBounds(100,100,500,500);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setResizable(false);
        setBackground(new Color(50,50,50));
        JPanel panel = new draw();
        add( panel );
        setVisible(true);
    }
}

Tick:

Java

public class tick{
    private int[][] star;
    private int colstar;
    private Timer t;
    public tick(int[][] starIn, int colstarIn){
        star = starIn;
        colstar = colstarIn;
        
        t = new Timer(1000, new timer());
        t.start();
    }
    void draw(){
        new draw().redraw(star, colstar);
    }
    class timer implements ActionListener{
        @Override
        public void actionPerformed(ActionEvent e) {
            draw();
        }
    }
}

Draw:

Java

public class draw extends JPanel{
    private int[][] draw;
    private int coldraw;
    private Thread call;
    
    private int i = 0;
    
    public void redraw(int star[][], int colstar){
        draw = star;
        coldraw = colstar;
        repaint();
        System.out.println("redraw");
    }
    public void paintComponent(Graphics g){
        super.paintComponent(g);
        g.clearRect(0, 0, 500, 500);
 
        g.setColor(new Color(255,255,255,200));
        g.fillRect(i, 15, 3, 3);
        System.out.println("Paint");
    }
}

Сейчас в консоль выводит:

Paint Paint Paint redraw redraw redraw ...

Если бы работал repaint выводило бы что-то наподобие этого:

Paint Paint Paint redraw Paint redraw Paint redraw Paint ...

@Welemir1 · 29.05.2015, 20:23

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

new window().window();

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

@vlodis · 29.05.2015, 20:41 **[ТС]**

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

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

Спасибо за такое замечание. Исправил все классы, кроме класса draw, так как если переименовать, возникает ошибка при присваивании класса под тип JPanel.
Но ошибка с repaint еще остается)

Pablito · 29.05.2015, 21:04

new draw().redraw(star, colstar);
Тут создается типа панель из класса draw и что дальше? Она никуда не добавляется.
Вообще код очень стремный, его надо стереть и переписать с нуля, по-другому. Но перед этом почитать пару уроков по компонентам и ООП.

@vlodis · 29.05.2015, 21:13 **[ТС]**

Сообщение от Паблито

new draw().redraw(star, colstar);
Тут создается типа панель из класса draw и что дальше? Она никуда не добавляется.
Вообще код очень стремный, его надо стереть и переписать с нуля, по-другому. Но перед этом почитать пару уроков по компонентам и ООП.

Так внутри new draw().redraw(star, colstar) происходит вызов repaint, который в свою очередь вызовет paintComponent, или я чего-то не понимаю?
Да, то что код стремный это точно, но я только начинаю программировать на java, так что потом подучусь и всё исправлю.

Pablito · 29.05.2015, 21:21

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

вызов repaint, который в свою очередь вызовет paintComponent

я не уверен что будет вызов, панель создана, но никуда не добавлена
она сама по себе, нет смысла на ней что-то перерисовывать

@vlodis · 29.05.2015, 21:33 **[ТС]**

Сообщение от Паблито

я не уверен что будет вызов, панель создана, но никуда не добавлена
она сама по себе, нет смысла на ней что-то перерисовывать

В классе window есть вот это -

Java

1 2	JPanel panel = new draw(); add( panel );

window вызывается раньше класса draw, и по моему, добавляет панель, возможно я ошибаюсь.
Но 1 раз программа рисует мой квадрат, он не рисует последующие разы..

Pablito · 29.05.2015, 21:43

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

window вызывается раньше класса draw, и по моему, добавляет панель, возможно я ошибаюсь

добавляет
а new draw().redraw(star, colstar);
куда добавляет?

вобщем я немного переписал код, получилось не идеально, но на форуме есть человек, который любит рефакторить мой код - он зайдет и доведет его до идеала

Java

public class Trololo {
 
    private static int star[][] = new int[50][2];
    private static int colstar = 0;
 
    public static void main(String[] args) {
        new Window(star, colstar);
    }
}
 
class Window extends JFrame {
    Window(int[][] starIn, int colstarIn) {
        setTitle("Mega Stars");
        setBounds(100, 100, 500, 500);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setResizable(false);
        setBackground(new Color(50, 50, 50));
        add(new MyCoolPanel(starIn, colstarIn));
        setVisible(true);
    }
}
 
class MyCoolPanel extends JPanel {
 
    private int[][] starIn;
    private int colstarIn;
    private int i = 0;
    private final Color myColor = new Color(255, 255, 255, 200);
 
    MyCoolPanel(int[][] starIn, int colstarIn) {
        this.starIn = starIn;
        this.colstarIn = colstarIn;
 
        new Timer(1000, new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                System.out.println("timer tick");
                repaint();
            }
        }).start();
    }
 
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        g.clearRect(0, 0, 500, 500);
        g.setColor(myColor);
        g.fillRect(i, 15, 3, 3);
        System.out.println("Paint");
    }
}

я там все классы в один файл собрал что бы легче было тестить, их потом можно разнести по разным

@vlodis · 29.05.2015, 21:57 **[ТС]**

Сообщение от Паблито

добавляет
а new draw().redraw(star, colstar);
куда добавляет?

вобщем я немного переписал код, получилось не идеально, но на форуме есть человек, который любит рефакторить мой код - он зайдет и доведет его до идеала

Java

public class Trololo {
 
    private static int star[][] = new int[50][2];
    private static int colstar = 0;
 
    public static void main(String[] args) {
        new Window(star, colstar);
    }
}
 
class Window extends JFrame {
    Window(int[][] starIn, int colstarIn) {
        setTitle("Mega Stars");
        setBounds(100, 100, 500, 500);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setResizable(false);
        setBackground(new Color(50, 50, 50));
        add(new MyCoolPanel(starIn, colstarIn));
        setVisible(true);
    }
}
 
class MyCoolPanel extends JPanel {
 
    private int[][] starIn;
    private int colstarIn;
    private int i = 0;
    private final Color myColor = new Color(255, 255, 255, 200);
 
    MyCoolPanel(int[][] starIn, int colstarIn) {
        this.starIn = starIn;
        this.colstarIn = colstarIn;
 
        new Timer(1000, new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                System.out.println("timer tick");
                repaint();
            }
        }).start();
    }
 
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        g.clearRect(0, 0, 500, 500);
        g.setColor(myColor);
        g.fillRect(i, 15, 3, 3);
        System.out.println("Paint");
    }
}

я там все классы в один файл собрал что бы легче было тестить, их потом можно разнести по разным

Ааа.... теперь все понятно.
Но вот только мне желательно нужен таймер вне класса унаследованного от JPanel или JFrame. Таймер будет вызывать еще кучу других методов и делать это из JPanel мне кажется не лучшей идеей. Как быть?

Pablito · 29.05.2015, 22:18

как вариант, хотя можно сделать кучей способов, смотря что вообще в конце-концов планируется

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

Java

public class Trololo {
 
    private static int star[][] = new int[50][2];
    private static int colstar = 0;
 
    public static void main(String[] args) {
        Window w = new Window(star, colstar);
 
        new Timer(1000, new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                System.out.println("timer tick");
                w.repaint();
            }
        }).start();
    }
}
 
class Window extends JFrame {
    Window(int[][] starIn, int colstarIn) {
        setTitle("Mega Stars");
        setBounds(100, 100, 500, 500);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setResizable(false);
        setBackground(new Color(50, 50, 50));
        add(new MyCoolPanel(starIn, colstarIn));
        setVisible(true);
    }
}
 
class MyCoolPanel extends JPanel {
 
    private int[][] starIn;
    private int colstarIn;
    private int i = 0;
    private final Color myColor = new Color(255, 255, 255, 200);
 
    MyCoolPanel(int[][] starIn, int colstarIn) {
        this.starIn = starIn;
        this.colstarIn = colstarIn;
    }
 
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        g.clearRect(0, 0, 500, 500);
        g.setColor(myColor);
        g.fillRect(i, 15, 3, 3);
        System.out.println("Paint");
    }
}

@vlodis · 29.05.2015, 22:38 **[ТС]**

Сообщение от Паблито

как вариант, хотя можно сделать кучей способов, смотря что вообще в конце-концов планируется

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

Java

public class Trololo {
 
    private static int star[][] = new int[50][2];
    private static int colstar = 0;
 
    public static void main(String[] args) {
        Window w = new Window(star, colstar);
 
        new Timer(1000, new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                System.out.println("timer tick");
                w.repaint();
            }
        }).start();
    }
}
 
class Window extends JFrame {
    Window(int[][] starIn, int colstarIn) {
        setTitle("Mega Stars");
        setBounds(100, 100, 500, 500);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setResizable(false);
        setBackground(new Color(50, 50, 50));
        add(new MyCoolPanel(starIn, colstarIn));
        setVisible(true);
    }
}
 
class MyCoolPanel extends JPanel {
 
    private int[][] starIn;
    private int colstarIn;
    private int i = 0;
    private final Color myColor = new Color(255, 255, 255, 200);
 
    MyCoolPanel(int[][] starIn, int colstarIn) {
        this.starIn = starIn;
        this.colstarIn = colstarIn;
    }
 
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        g.clearRect(0, 0, 500, 500);
        g.setColor(myColor);
        g.fillRect(i, 15, 3, 3);
        System.out.println("Paint");
    }
}

Черт возьми, замечательное решение.)
Вроде разобрался. Спасибо.

Новые блоги и статьи
Что лучше использовать при создании класса в Java: сеттеры или конструктор? Alexander-7 08.01.2025 Вопрос подробнее: На вопрос: «Когда одновременно создаются конструктор и сеттеры в классе – это нормально?» куратор уточнил: «Ваш класс может вообще не иметь сеттеров, а только конструктор и геттеры. . .	Как работать с GraphQL на TypeScript InfoMaster 08.01.2025 Введение в GraphQL и TypeScript В современной разработке веб-приложений GraphQL стал мощным инструментом для создания гибких и эффективных API. В сочетании с TypeScript, эта технология. . .	Счётчик на базе сумматоров + регистров и генератора сигналов согласования. Hrethgir 07.01.2025 Создан с целью проверки скорости асинхронной логики: ранее описанного сумматора и предополагаемых fast регистров. Регистры созданы на базе ранее описанного, предполагаемого fast триггера. То-есть. . .	Как перейти с Options API на Composition API в Vue.js BasicMan 06.01.2025 Почему переход на Composition API актуален В мире современной веб-разработки фреймворк Vue. js продолжает эволюционировать, предлагая разработчикам все более совершенные инструменты для создания. . .	Архитектура современных процессоров inter-admin 06.01.2025 Процессор (центральный процессор, ЦП) является основным вычислительным устройством компьютера, которое выполняет обработку данных и управляет работой всех остальных компонентов системы. Архитектура. . .	История создания реляционной модели баз данных, правила Кодда Programming 06.01.2025 Предпосылки создания реляционной модели В конце 1960-х годов компьютерная индустрия столкнулась с серьезными проблемами в области управления данными. Существовавшие на тот момент модели данных -. . .
Полезные поделки на Arduino, которые можно сделать самому raxper 06.01.2025 Arduino как платформа для творчества Arduino представляет собой удивительную платформу для технического творчества, которая открывает безграничные возможности для создания уникальных проектов. Эта. . .	Подборка решений задач на Python IT_Exp 06.01.2025 Целью данной подборки является предоставление возможности ознакомиться с различными задачами и их решениями на Python, что может быть полезно как для начинающих, так и для опытных программистов. . . .	С чего начать программировать микроконтроллеры raxper 06.01.2025 Введение в мир микроконтроллеров Микроконтроллеры стали неотъемлемой частью современного мира, окружая нас повсюду: от простых бытовых приборов до сложных промышленных систем. Эти маленькие. . .	Из чего собрать игровой компьютер inter-admin 06.01.2025 Сборка игрового компьютера требует особого внимания к выбору комплектующих и их совместимости. Правильно собранный игровой ПК не только обеспечивает комфортный геймплей в современных играх, но и. . .	Обновление сайта www.historian.by Reglage 05.01.2025 Обещал подвести итоги 2024 года для сайта. Однако начну с того, что изменилось за неделю. Добавил краткий урок по последовательности действий при анализе вредоносных файлов и значительно улучшил урок. . .	Как использовать GraphQL в C# с HotChocolate Programming 05.01.2025 GraphQL — это современный подход к разработке API, который позволяет клиентам запрашивать только те данные, которые им необходимы. Это делает взаимодействие с API более гибким и эффективным по. . .

@Welemir1 Автоматизируй это! 7210 / 4705 / 1224 Регистрация: 30.03.2015 Сообщений: 13,412 Записей в блоге: 29
	29.05.2015, 20:23	2
	Сообщение от vlodis new window().window(); зачем городить? Просто конструктор нельзя было использовать? и такое использование методов у тебя не один раз в коде. Имя класса должно быть с большой буквы и нежелательно заводить переменные не отличимые от имени класса. 0

@vlodis 0 / 0 / 0 Регистрация: 29.05.2015 Сообщений: 6
	29.05.2015, 20:41 [ТС]	3
	Сообщение от Welemir1 зачем городить? Просто конструктор нельзя было использовать? и такое использование методов у тебя не один раз в коде. Имя класса должно быть с большой буквы и нежелательно заводить переменные не отличимые от имени класса. Спасибо за такое замечание. Исправил все классы, кроме класса draw, так как если переименовать, возникает ошибка при присваивании класса под тип JPanel. Но ошибка с repaint еще остается) 0

Pablito 2884 / 2296 / 769 Регистрация: 12.05.2014 Сообщений: 7,978
	29.05.2015, 21:04	4
	new draw().redraw(star, colstar); Тут создается типа панель из класса draw и что дальше? Она никуда не добавляется. Вообще код очень стремный, его надо стереть и переписать с нуля, по-другому. Но перед этом почитать пару уроков по компонентам и ООП. 0

@vlodis 0 / 0 / 0 Регистрация: 29.05.2015 Сообщений: 6
	29.05.2015, 21:13 [ТС]	5
	Сообщение от Паблито new draw().redraw(star, colstar); Тут создается типа панель из класса draw и что дальше? Она никуда не добавляется. Вообще код очень стремный, его надо стереть и переписать с нуля, по-другому. Но перед этом почитать пару уроков по компонентам и ООП. Так внутри new draw().redraw(star, colstar) происходит вызов repaint, который в свою очередь вызовет paintComponent, или я чего-то не понимаю? Да, то что код стремный это точно, но я только начинаю программировать на java, так что потом подучусь и всё исправлю. 0

Pablito 2884 / 2296 / 769 Регистрация: 12.05.2014 Сообщений: 7,978
	29.05.2015, 21:21	6
	Сообщение от vlodis вызов repaint, который в свою очередь вызовет paintComponent я не уверен что будет вызов, панель создана, но никуда не добавлена она сама по себе, нет смысла на ней что-то перерисовывать 0

Снова. Снова этот repaint()

Решение