ignatovv / tp_2014_02_java_ignatov

Технопарк@Mail.ru

Курс: Углубленное программирование на Java

Учебный период: весенний семестр 2014г.

##Студент Игнатов Владимир Юрьевич , abbadoh, [email protected]

Подробное о проекте можно прочесть на сайте [Технопарка] [1] [1]:tech-mail.ru

tp_2014_02_java_ignatov's Issues

Все варианты обработки doGet или doPost перевести на switch

Должно получиться в итоге как-то так,
switch:
case "/login":
doLogin(request, response);
break;
case "/auth":
doAuth(request, response);
break;
default:
do404error(...)

Никакой солянки из кода разных уровней абстракции

Не нужно в одном месте делать обработку странички, запрос к базе и логику приложения.
Всё, что с базой - отдельно.
Наполнения контента - отдельно.
Логика - над всем этим.

Пример такого (не совершенного) кода:

if (userExists(con, login, password))
                {
                    HttpSession session = request.getSession();
                    UsersDAO dao = new UsersDAO(con);
                    UserDataSet user = dao.get(login);
                    session.setAttribute("userId", user.getId());
                    session.setAttribute("login", user.getLogin());
                    response.sendRedirect("/userId");
                } else
                {
                    Map<String, Object> pageVariables = new HashMap<>();
                    pageVariables.put("alert", "Wrong username or password");
                    response.getWriter().println(PageGenerator.getPage("authform.tml", pageVariables));
                }

Не нашел использования TransactionExecutor

Не нашел использования PreparedExecutor

FrontEnd не должен ничего знать про базу

Необходимо вынести все методы, связанные с работой с базой данных в отдельный класс - DatabaseServiceImpl.

Например, userExists. Так же boolean методы принято называть с is, чтобы получить вопрос - Is user exists?

Использовать ключевое слово final там, где это рационально

Например,

public class UserDataSet {
    private long id;
    private String login;

Если класс не абстрактный, значит - final.
Если переменная только задается - final.

Нет тестов

Убрать из гит репазитория все .DS_Store