Модераторы: LSD, AntonSaburov

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Утечка памяти. 
:(
    Опции темы
LSD
Дата 26.2.2007, 15:49 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Leprechaun Software Developer
****


Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 210
Всего: 538
Да, и на твои обработчики ListDataListener. Вообщем, все что может прямо или косвенно вызываться.


--------------------
Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it.
PM MAIL WWW   Вверх
seacat79
Дата 27.2.2007, 10:50 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 46
Регистрация: 27.2.2007
Где: Украина, Одесса

Репутация: нет
Всего: 1
Выложи текст, который создает CellRender. Скорее всего, ты на каждый запрос генерируешь новый объект
PM MAIL ICQ   Вверх
ShurikA
Дата 27.2.2007, 11:19 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Зануда
***


Профиль
Группа: Завсегдатай
Сообщений: 1364
Регистрация: 29.10.2005
Где: Канада

Репутация: нет
Всего: 3
Цитата(seacat79 @  27.2.2007,  09:50 Найти цитируемый пост)
запрос генерируешь новый объект 

Есть такое дело, ну а как по другому в таком случае?

Код

/*
 * PageListCellRenderer.java
 *
 * Created on December 24, 2006, 10:36 PM
 *
 * To change this template, choose Tools | Template Manager
 * and open the template in the editor.
 */

package apis.DesignCanvas;

import dataclasses.DataClasses.FilledPageDesign;
import java.awt.Color;
import java.awt.Component;
import java.awt.Dimension;
import javax.swing.BorderFactory;
import javax.swing.DefaultListCellRenderer;
import javax.swing.JList;

/**
 *
 * @author ShurikA
 */
public class PageListCellRenderer extends DefaultListCellRenderer{
    private Dimension defDim;
    /** Creates a new instance of PageListCellRenderer */
    public PageListCellRenderer(Dimension d) {
        this.defDim = d;
    }
    
    public Component getListCellRendererComponent(JList list, Object value, int index, boolean isSelected, boolean cellHasFocus) {
        if(value == null)
            return null;
        
        if(!(value instanceof FilledPageDesign))
            return null;
        
        FilledPageDesign pd = (FilledPageDesign) value;
        
        PageStatusIcon icon = new PageStatusIcon(pd, this.defDim);
        icon.setBorder(BorderFactory.createLineBorder(Color.GRAY));
        
        return icon;
    }
}



--------------------
Если долго мучиться, что нибудь получится...
user posted image
PM MAIL WWW ICQ Skype   Вверх
seacat79
Дата 27.2.2007, 11:39 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 46
Регистрация: 27.2.2007
Где: Украина, Одесса

Репутация: нет
Всего: 1
Попробуй сделать следующее:

PageStatusIcon icon = new PageStatusIcon()-> переносишь в класс PageListCellRenderer


а вместо:
Цитата

Код

        PageStatusIcon icon = new PageStatusIcon(pd, this.defDim);
        icon.setBorder(BorderFactory.createLineBorder(Color.GRAY));
        
        return icon;



делаешь что-нибудь типа
Код

        icon.setPD(pd);
        icon.setBorder(BorderFactory.createLineBorder(Color.GRAY));
        
        return icon;
PM MAIL ICQ   Вверх
LSD
Дата 27.2.2007, 11:49 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Leprechaun Software Developer
****


Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 210
Всего: 538
Цитата(ShurikA @  27.2.2007,  11:19 Найти цитируемый пост)
Есть такое дело, ну а как по другому в таком случае?

Создаешь один компонент, когда вызывается getListCellRendererComponent() задаешь для этого компонента нужные свойства (цвет, текст, иконку и т.д.) и возвращаешь его. В самом методе ничего не создаешь.


--------------------
Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it.
PM MAIL WWW   Вверх
ShurikA
Дата 27.2.2007, 13:23 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Зануда
***


Профиль
Группа: Завсегдатай
Сообщений: 1364
Регистрация: 29.10.2005
Где: Канада

Репутация: нет
Всего: 3
Цитата(seacat79 @  27.2.2007,  10:39 Найти цитируемый пост)
делаешь что-нибудь типа

код Java
        icon.setPD(pd);
        icon.setBorder(BorderFactory.createLineBorder(Color.GRAY));
        
        return icon;



попробую попоже, но какая разница, если в теории на предыдущий объект узче никто не указывает?


--------------------
Если долго мучиться, что нибудь получится...
user posted image
PM MAIL WWW ICQ Skype   Вверх
seacat79
Дата 27.2.2007, 13:32 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 46
Регистрация: 27.2.2007
Где: Украина, Одесса

Репутация: нет
Всего: 1
Цитата

попробую попоже, но какая разница, если в теории на предыдущий объект узче никто не указывает?



Разницы нет, кроме того, что:
1) Ты не знаешь когда будет вызван сборщик мусора, и уберет все объекты
2) Есть вероятность того, что сборщик не может удалить объект, так как на него есть ссылки
из других объектов. (Покажи код PageStatusIcon)
PM MAIL ICQ   Вверх
ShurikA
Дата 27.2.2007, 13:45 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Зануда
***


Профиль
Группа: Завсегдатай
Сообщений: 1364
Регистрация: 29.10.2005
Где: Канада

Репутация: нет
Всего: 3
Код

/*
 * PageStatusIcon.java
 *
 * Created on December 24, 2006, 10:48 PM
 *
 * To change this template, choose Tools | Template Manager
 * and open the template in the editor.
 */

package apis.DesignCanvas;

import dataclasses.DataClasses.FilledPageDesign;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.FlowLayout;
import javax.swing.BorderFactory;
import javax.swing.Icon;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JProgressBar;

/**
 *
 * @author ShurikA
 */
public class PageStatusIcon extends JPanel{
    private Icon icon = null;
    private JButton delete = null;
    private JProgressBar pBar;
    private JLabel preview = null;
    private FilledPageDesign pageDesign;
    
    /** Creates a new instance of PageStatusIcon */
    public PageStatusIcon(FilledPageDesign pd, Dimension d) {
        super(null);
        this.pageDesign = pd;
        this.pBar = pd.GetPregressBar();
        Dimension psiDim = new Dimension(d);
        this.setPreferredSize(psiDim);
        this.setMaximumSize(psiDim);
        this.setMinimumSize(psiDim);
        
        if(icon == null)
            preview = new JLabel(new ImageIcon(getClass().getResource("Images/Default Album.png")));
        
        preview.setBounds(10,10,150,100);
        preview.setBorder(BorderFactory.createLineBorder(Color.DARK_GRAY,1));
        add(preview);
        
        int i = pd.GetPageNumber();
        JLabel pageNumber = new JLabel(String.valueOf(i));
        add(pageNumber);
        
        Dimension bDim = new Dimension(90,23);
        delete = new JButton(new ImageIcon(getClass().getResource("Images/Button Delete Normal.png")));
        delete.setRolloverIcon(new ImageIcon(getClass().getResource("Images/Button Delete Over.png")));
        delete.setPressedIcon(new ImageIcon(getClass().getResource("Images/Button Delete Push.png")));
        delete.setActionCommand("delete");
        delete.setPreferredSize(bDim);
        delete.setMinimumSize(bDim);
        delete.setMaximumSize(bDim);
        delete.setBorderPainted(false);
        delete.setContentAreaFilled(false);
        delete.setBounds(psiDim.width-100,10,90,23);
        add(delete);
        
        pBar.setBounds(10,psiDim.height-pBar.getPreferredSize().height-10,psiDim.width-20,pBar.getPreferredSize().height);
        pBar.setStringPainted(true);
        add(pBar);
    }
}


Это сообщение отредактировал(а) ShurikA - 27.2.2007, 13:45


--------------------
Если долго мучиться, что нибудь получится...
user posted image
PM MAIL WWW ICQ Skype   Вверх
COVD
Дата 27.2.2007, 16:07 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1655
Регистрация: 26.7.2005

Репутация: 17
Всего: 43
Почему бы не активизировать лог gc (всего лишь добавить в командной строке параметр для JVM: -Xloggc:) и не посмотреть в динамике. Или в профайлере посмотреть, какие именно обьекты накапливаются. Утечка памяти - это не только OutOfMemory (это очень грубая ошибка проектирования). Проблема может быть также и в том, что gc будет убирать мусор, но вынужден будет включать FullGC, а это замораживает программу на некоторое время.  

Это сообщение отредактировал(а) COVD - 27.2.2007, 16:08
PM MAIL   Вверх
ShurikA
Дата 27.2.2007, 21:40 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Зануда
***


Профиль
Группа: Завсегдатай
Сообщений: 1364
Регистрация: 29.10.2005
Где: Канада

Репутация: нет
Всего: 3
Цитата(LSD @  27.2.2007,  10:49 Найти цитируемый пост)
Создаешь один компонент, когда вызывается getListCellRendererComponent() задаешь для этого компонента нужные свойства (цвет, текст, иконку и т.д.) и возвращаешь его. В самом методе ничего не создаешь. 

Секунду, что то я, по моему не догоняю...
PageListCellRenderer вызывается что бы определить внешний вид ячейки, а при fireContentsChanged эти ячейки просто перерисовываутся, не так ли?


--------------------
Если долго мучиться, что нибудь получится...
user posted image
PM MAIL WWW ICQ Skype   Вверх
LSD
Дата 28.2.2007, 12:42 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Leprechaun Software Developer
****


Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 210
Всего: 538
Цитата(ShurikA @  27.2.2007,  21:40 Найти цитируемый пост)
PageListCellRenderer вызывается что бы определить внешний вид ячейки, а при fireContentsChanged эти ячейки просто перерисовываутся, не так ли?

ListCellRenderer вызывается чтобы отрисовать одну ячейку (т.е. у него будет вызван метод paint())
fireContentsChanged() оповещает всех слушателей (в том числе и JList), что данные изменились. Если надо то JList произведет перерисовку изменившихся элементов.


--------------------
Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it.
PM MAIL WWW   Вверх
Ответ в темуСоздание новой темы Создание опроса
Правила форума "Java"
LSD   AntonSaburov
powerOn   tux
javastic
  • Прежде, чем задать вопрос, прочтите это!
  • Книги по Java собираются здесь.
  • Документация и ресурсы по Java находятся здесь.
  • Используйте теги [code=java][/code] для подсветки кода. Используйтe чекбокс "транслит", если у Вас нет русских шрифтов.
  • Помечайте свой вопрос как решённый, если на него получен ответ. Ссылка "Пометить как решённый" находится над первым постом.
  • Действия модераторов можно обсудить здесь.
  • FAQ раздела лежит здесь.

Если Вам помогли, и атмосфера форума Вам понравилась, то заходите к нам чаще! С уважением, LSD, AntonSaburov, powerOn, tux, javastic.
 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | Java: Общие вопросы | Следующая тема »

Подписаться на тему | Подписка на этот форум | Скачать/Распечатать тему


 




[ Время генерации скрипта: 0.0705 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
 По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.