Удалено : )

Это сообщение отредактировал(а) Dzo - 14.7.2008, 16:02

По первому куску 

1. Вместо переменной connectionWasClosed наверно лучше помещать закрытие коннекта в блок finally.

2. Навание getByResultingArrayList - неудачное. Это больше напоминает название функции.

Хмм, изменил на resultingArrayList.


Это сообщение отредактировал(а) Dzo - 9.7.2008, 17:24

1). зачем i отдельно?

2). for (int i = 0, n = valuesConsistingArrayList.size(); i < n; i++) { будет работать быстрее, т.к. метод size() вызовется 1 раз

3). Вы на каждый цикл открываете по коннекту и делаете по запросу в базу. Есть такие чудесные вещи как WHERE getByColumnName IN (1, 2, 3, ...). Выполнится в одном запросе

4). геттеры и сеттеры getSomething, setSomething - считаются хорошим стилем для названий методов для переменной something. В результате у вас по конвеншинах получится getGetByColumnName() и setGetByColumnName(). Переменные getSomething/setSomthing/isSomething называть не стоит.

5). Незачем создавать запросы в резалтсет через "+". Если юзер имеет доступ к вводу данных, то ждите SQL инджектов. Для этого у PreparedStatement есть, к примеру, setString()

6). Коннекты надо закрывать в блоке finally

7). Вам жалко пробелов между операторами? (i=0;i<valuesConsistingArrayList.size();i++) читать не очень удобно.

Чтобы не печатать каждый раз int d каждом for loop'е?



Спасибо!



Кстати к вопросу об инджектах? Они тоже возможны, если сайт писан на джаве? Методы защиты стандартные (как на PHP например). 

Вопрос: если я хочу чтобы реализация не зависела от названий элементов (таблиц, столбцов) в базе, или по крайней мере их можно было изменять безболезненно для кода в ОДНОМ месте, что мне стоит попробовать?

JPA\Hibernate

Edit: а от инджектов защищатся не надо, надо грамотно с SQL работать ;)

Это сообщение отредактировал(а) SoulKeeper - 9.7.2008, 17:58

Подробнее? Статью?

А алгоритм сортировки блин не работает : (( Сейчас стал тестить на некоторых вариантаъ не работает : ( Сенодня уже нет сил, затвра добью.

Для начала будет достаточно передавать все параметры через setXXX(). 

Ну а по поводу сортировки - используйте Collections.sort

Это сообщение отредактировал(а) SoulKeeper - 9.7.2008, 20:53

А почему для сортировки не задействовать ORDER BY в запросе?

Разница незначительная, количество элементов в ArrayList хранится в самом объекте. То есть не происходит пересчёта списка каждый раз при вызове size() - это лишь возврат значения закрытой переменной. А чтение кода между тем приобретает дополнительную сложность.

Гениалисимус : )

Открывается соединение, а потому переменной connectionWasClosed(!!) присваивается тру. Такое название только запутывает.