сервер на java

[Lemon2009]

Создавал сервер (сокеты) на java.
Все работает.
Но не долго.

Запущен на FreeBSD.

Проблема в том, что программа через некоторое время после запуска подключает клиенты, но не отвечает.
И размер используемой памяти постоянно растет (хоть и пробовал не раз оптимизировать код)

[alexinspir]

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

Создавал сервер (сокеты) на java.
Все работает.
Но не долго.

Запущен на FreeBSD.

Проблема в том, что программа через некоторое время после запуска подключает клиенты, но не отвечает.
И размер используемой памяти постоянно растет (хоть и пробовал не раз оптимизировать код)

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

[Lemon2009]

я вот такой код сделал:

public synchronized ResultSet query(String sql) throws SQLException {
        return conn.createStatement().executeQuery(sql);
    }

    public synchronized int queryUpdate(String sql) throws SQLException {
        return conn.createStatement().executeUpdate(sql);
    }

при каждом обращение к базе обращаюсь по ним (query, queryUpdate).

Это нормально? или стоит создать только один statement и с него только обращаться?

мож это и есть утечка.

[Lemon2009]

еще вот такой:

public synchronized int addMessage(String UIN, String message) {        
        try {
            UIN = Utils.escapeString(UIN);
            message = Utils.escapeString(message);
            ResultSet result = query("SELECT `uin` FROM `cms_bot_bots` WHERE `online`=1");
            int i = 0;
            while (result.next()) {
                conn.createStatement().executeUpdate("INSERT INTO `cms_bot_msg` ( `id` , `uin` , `msg` , `time` , `bot` , `read` ) VALUES ('"+Long.toString(System.currentTimeMillis() + (long)i)+"', '"+UIN+"', '"+message+"', '"+Utils.getUnixTimeString()+"', '"+result.getString("uin")+"', '0')");
                i++;
            }
            result.close();
            return conn.createStatement().executeUpdate("INSERT INTO `cms_cli_msg` ( `port` , `time` , `unix_time` , `user` , `msg` ) VALUES ('"+SERVER_PORT+"', '0000-00-00 00:00:00', '"+Utils.getUnixTimeString()+"', '"+UIN+"', '"+message+"')");
        } catch (SQLException ex) {
            Log.addToErrorLog(ex);
            return 0;
        }
    }

[Lemon2009]

и еще вопрос. правильно ли использовать внутри цикла исключения, не тормозят ли они.

public void run() {
        try {
            while(true) {
                ResultSet r = query("SELECT * FROM `cms_bot_users` WHERE `chat`=0");
                while(r.next()) {

                }
                Thread.sleep(10);
            }
        } catch (InterruptedException ex) {
            ex.printStackTrace();
        } catch (SQLException ex) {
            ex.printStackTrace();
        }
    }

мож какой то другой проавильный вариант есть?

[alexinspir]

бегло осмотрел, ничего критического при этом не обнаружил.
Однако, зачем использовать для бота базу данных? чем обычные коллекции и стеки не устраивают?
Возможно я не знаю всех ваших причин, но тем не менее я считаю что это очень плохое архитектурное решение.

[Lemon2009]

я сделал клиент под icq-чат. если я понятно выразился ))
чтобы без icq в чате общаться. уже под готовый проект, который на базе