| Цитата |
Сразу видно PHP'ешника по полету
|
Да Вы правы, вот переучиваюсь  поразительно что так быстро раскусили, наверное сами на PHP когда-то кодили ;)
| Цитата |
1) Если ближе к теме, то инициализацию драйвера JDBC надо выносить в метод init() сервлета, не дело при каждом запросе создавать новый коннект к БД, который как известно ресурс исчерпаемый и драгоценный. При запросе надо только проверять живо ли соединение или нет.
|
Благодарю за критику
метод init() в сервлете это тот который вызывается автоматически?
сервлет должен выглдять так?
| Код |
import java.util.*;
import java.io.*;
import javax.servlet.*;
import javax.servlet.http.*;
import MySQL.*;
public class indexServlet extends HttpServlet {
private String getpage;
//Подключиться к БД
private MySQLConnect MySQLConnectObj = new MySQLConnect();
private MySQLQuery MySQLQueryObj = new MySQLQuery();
public void init(){
MySQLConnectObj.DownloadDriver(); //загрука драйвера
MySQLConnectObj.Connected(); //подключение к MySQL
//Передача утсновленного соединения с БД объекту "запросы"
MySQLQueryObj.setConnection(MySQLConnectObj.GetConnection());
}
protected void doGet(HttpServletRequest request, HttpServletResponse response)
throws ServletException, IOException, NullPointerException
{
//получить параметр GET "выбор пользователя"
getpage = request.getParameter("page");
//Проверка установки текущего соединения с БД
if(MySQLConnectObj.GetConnection() != null){
Vector VectorPage = new Vector(); //для массива объектов из БД
VectorPage = MySQLQueryObj.SelectAllField("up_menu");//выборка по БД
}
//При отсуствии соединения установить его
else if(MySQLConnectObj.GetConnection() == null){
init() ;
}
//когда отключаться от БД? после каждой обработки запроса пользователем
//или после разрушения сессии?
// MySQLConnectObj.DisConnected();
/*
//Разобрать вектор
GenPageMySQL PageObj = new GenPageMySQL();
for(int i=0; i<VectorUpMenuObj.size(); ++i)
{
// PageObj = (PageObj)VectorUpMenuObj.elementAt(i);
request.setAttribute("up_menu_theme",VectorUpMenuObj.elementAt(i));
RequestDispatcher Dispatcher = getServletContext().getRequestDispatcher("/WEB-INF/jsp/index.jsp");
Dispatcher.forward(request, response);
}
*/
return;
}
}
|
| Цитата |
2) Далее после запроса положено закрывать ResultSet, а перед скроллированием записей в ResultSet неплохо бы убедиться, что он не пуст. ResultSet.next() при пустом запросе запросто может сгенерировать Exception.
|
Так в классе отвечающем за запросы MySQLQuery устанлена обработка исключительных ситуаций, или кроме всего прочего надобно делать проверку наличия выборки по БД?
if(resultQuery == null) { return VectorPageObj; }
package MySQL;
/*
* To change this template, choose Tools | Templates
* and open the template in the editor.
*/
| Код |
import java.util.*;
import java.sql.Connection;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
/**
*
* @author initmax
*/
public class MySQLQuery {
private Connection CurrentConnect; //Объект типа подключение для работы с БД
private Vector VectorPageObj = new Vector(); //создать объект типа вектор "динамическое" создание массива
public MySQLQuery() {}
//Конструктор принимает текущее установленное соединение с БД
public MySQLQuery(Connection CurrentConnectBase)
{
CurrentConnect = CurrentConnectBase;
}
//*****************************************************************
//Метод принимает имя таблицы и возвращает полную по ней выборку
public Vector SelectAllField(String NameTable)
{
try
{
//Выполним запрос в БД по имени таблицы, переданой методу
Statement st = CurrentConnect.createStatement();
String query = ("select * from "+NameTable);
ResultSet resultQuery = null;
resultQuery = st.executeQuery(query);
//проверим есть ли данные по запросу
if(resultQuery == null) { return VectorPageObj; }
//Пройдём в цикле по результату выполненого массива
//Создадим массив "Vector" объектов с данными страницы
while (resultQuery.next())
{
GenPageMySQL PageObj = new GenPageMySQL();
PageObj.setId(resultQuery.getInt("id"));
PageObj.setTheme(resultQuery.getString("theme"));
PageObj.setPage(resultQuery.getString("page"));
VectorPageObj.add(PageObj); //добавим в конец массива заполненый объект
}
}
catch (SQLException e) {
e.printStackTrace();
}
return VectorPageObj;
}
/**
*@set the CurrentConnect
*/
public void setConnection(Connection CurrentConnectBase) {
CurrentConnect = CurrentConnectBase;
}
}
|
Так или я что-то не понял?
| Цитата |
3) Зачем вы создаете перед запросом Vector? Он же все равно создается внутри класса MySQLQuery?
|
Да он создаётся внутри класса,но объявлен как private, поэтому метод возвращает Vector
ссылка на который инициализируется в indexServlet
VectorPage = MySQLQueryObj.SelectAllField("up_menu");
или Вы полагаете более верным решением создать в классе MySQLQuery метод GetVector?
| Цитата |
4) Далее согласно стандартному стилю именований методов в Java если имя метода начинается с глагола, то его надо писать маленькими буквами. Например не SelectAllFields(), а selectAllFields()
|
Учту, Благодарю!
| Цитата |
Ну это все относится к стилю, а не к сути. По сути вы устанавливаете соединение с БД, но не проверяете живо оно или нет, просто выдаете printStackTrace() и идете дальше. Ну вот поэтому и получаете NullPointerException. А соединение с БД вы (как уже говорил выше) устанавливаете при каждом запросе и через несколько запросов ресурсы коннектов БД заканчиваются.
|
Делать так как указал выше в классе public class indexServlet?
|
мдяяя... вам точно заняться нечем ... 