Соединение данных с базой данных через пул

@-=Angel=- · Регистрация: 08.08.2009

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

Здравствуйте такой вопрос: создаю соединение данных с базой данных через пул. Соединение создалось норм, но почемуто вычитывает из БД непонятно что т.е.

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
List<User> users = new ArrayList<User>();
        Connection connection = null;
        String sql = null;
        try {
            sql = "SELECT UserId, UserName FROM users";
            connection = getConnection();
            Statement stmt = connection.createStatement();
            ResultSet resultSet = stmt.executeQuery(sql);
            while (resultSet.next()){
                users.add(new User(
                        resultSet.getInt("UserId"), 
                        resultSet.getString("UserName"),"1","1","1" 
                        ));
            resultSet.close();
            stmt.close();
        } catch (SQLException e) {
            System.err.println("Query: " + sql + "\n"+e);
}

в этом куске кода непонятно почему выводит на консоль "user" хотя я этого нигде непрописывал и почемуто иногда на resultSet.getString("UserName"),"1","1", "1" кидает exception. Подскажите что тут не так? (Класс User - рабочий и описан норм)

@Deiron · 12.02.2011, 22:27

хм. Не понятно мне, почему Вы эту тему отнесли к Java EE, ну да ладно.
замечания банальны:
1. У цикла while в данном примере нет закрывающей фигурной скобки. Закрывающая скобка цикла while у Вас рядом с catch ()
2. Лучше до того, как делать add, resultSet.getInt("UserId") и resultSet.getString("UserName") записать во временные переменные. Так, на всякий случай. Ну или попробуй вместо resultSet.getString("UserName") поставить временно " ", чтобы узнать, нету ли заглюка чисто на UserName.
3. new User прямо в методе add? Товарищ, это моветон. Не честнее ли было бы создать временную переменную класса User перед добавлением? Опять же, чисто "для красоты".
4. Вижу, Вы ловите только SQLException. по хорошему надо бы еще один catch-блочок, который ловил бы все Exception вообще. А то мало ли?
5. System.err.println("Query: " + sql + "\n"+e) заменить на System.err.println("Query: " + sql + "\n"+e.getMessage()). Вы ведь так самого текста ошибки ни в жизнь не увидите.

@-=Angel=- · 13.02.2011, 01:23 **[ТС]**

Спасибо за замечания, я просто новичек в этом деле... буду учиться)

Новые блоги и статьи Все статьи Все блоги /
Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly 8Observer8 24.01.2026 Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а привычная функция main(). . .	моя боль iceja 24.01.2026 Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .	Модель сукцессии микоризы anaschu 24.01.2026 Решили писать научную статью с неким РОманом	http://iceja.net/ математические сервисы iceja 20.01.2026 Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .
http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь(не выше 3-го порядка) постоянного тока с элементами R, L, C, k(ключ), U, E, J. Программа находит переходные токи и напряжения на элементах схемы классическим методом(1 и 2 з-ны. . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .	Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/

@Deiron 26 / 26 / 9 Регистрация: 25.05.2009 Сообщений: 98
	12.02.2011, 22:27
	хм. Не понятно мне, почему Вы эту тему отнесли к Java EE, ну да ладно. замечания банальны: 1. У цикла while в данном примере нет закрывающей фигурной скобки. Закрывающая скобка цикла while у Вас рядом с catch () 2. Лучше до того, как делать add, resultSet.getInt("UserId") и resultSet.getString("UserName") записать во временные переменные. Так, на всякий случай. Ну или попробуй вместо resultSet.getString("UserName") поставить временно " ", чтобы узнать, нету ли заглюка чисто на UserName. 3. new User прямо в методе add? Товарищ, это моветон. Не честнее ли было бы создать временную переменную класса User перед добавлением? Опять же, чисто "для красоты". 4. Вижу, Вы ловите только SQLException. по хорошему надо бы еще один catch-блочок, который ловил бы все Exception вообще. А то мало ли? 5. System.err.println("Query: " + sql + "\n"+e) заменить на System.err.println("Query: " + sql + "\n"+e.getMessage()). Вы ведь так самого текста ошибки ни в жизнь не увидите. 1

@-=Angel=- 0 / 0 / 0 Регистрация: 08.08.2009 Сообщений: 24
	13.02.2011, 01:23 [ТС]
	Спасибо за замечания, я просто новичек в этом деле... буду учиться) 0