Замечания по коду

@dimashnurenko · Регистрация: 29.06.2013

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

Здравствуйте! Можете посмотреть код, и написать замечания по нему...Я на Java пишу недавно.Хотелось бы выслушать замечания по коду или рекомендации...

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
public class EntityDaoImpl<T,K extends Serializable> implements EntityDao<T,K>{
 
    @Autowired
    private SessionFactory sessionFactory;
    private Session session;
    private Class<T> type;
 
    public EntityDaoImpl(Class<T> type){
        this.type = type;
    }
 
    @Override
    public K addEntity(T entity) throws ServiceExecption {
        K quantity = null;
        try{
            session = (Session) sessionFactory.openSession().beginTransaction();
            quantity = (K)session.save(entity);
            session.getTransaction().commit();
        }catch (Exception e){
            if(session!=null){
                session.getTransaction().rollback();
                session.close();
            }
            throw new ServiceExecption("Can't add entity",e);
        }
        return quantity;
    }
 
    @Override
    public T getEntityById(K id) throws ServiceExecption {
        T element = null;
        try{
            session = (Session) sessionFactory.openSession().beginTransaction();
            element =(T) session.get(type,id);
            session.getTransaction().commit();
        }catch (Exception e){
            if(session!=null){
                session.getTransaction().rollback();
                session.close();
            }
            throw new ServiceExecption("Can't find entity",e);
        }
        return element;
    }
 
    @Override
    public void updateEntity(T entity) throws ServiceExecption {
        try{
            session = (Session) sessionFactory.openSession().beginTransaction();
            session.update(entity);
            session.getTransaction().commit();
        }catch (Exception e){
            if(session!=null){
                session.getTransaction().rollback();
                session.close();
            }
            throw new ServiceExecption("Can't update entity",e);
        }
    }
 
    @Override
    public void deleteEntity(T entity) throws ServiceExecption {
        try{
            session = (Session) sessionFactory.openSession().beginTransaction();
            session.delete(entity);
            session.getTransaction().commit();
        }catch (Exception e){
            if(session!=null){
                session.getTransaction().rollback();
                session.close();
            }
            throw new ServiceExecption("Can't delete entity",e);
        }
    }
}

KuKu · 09.04.2014, 11:22

Управление транзакциями в DAO слой не надо совать. Да и код дублируется:

Java
1
2
3
4
5
6
7
8
9
10
11
try{
            session = (Session) sessionFactory.openSession().beginTransaction();
            ...
            session.getTransaction().commit();
        }catch (Exception e){
            if(session!=null){
                session.getTransaction().rollback();
                session.close();
            }
            ...
        }

Используйте @Transactional или tx:advice для управление транзакциями.

@KEKCoGEN · 11.04.2014, 17:38

Если использовать CrudRepository то код можно свести примерно к такому виду

Java
1
public interface MyClassRepository extends Repository<MyClass, Long> {  }

Спринг дата сам создаст все основные методы такие как save, find, findBy, etc

Задачи такого типа имеет смысл решать самому только если на то есть действительно веская причина.

вот тут можно почитать

http://docs.spring.io/spring-d... ories.html

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++ 8Observer8 17.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip	Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая ссылка» (hard link),. . .	[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .	делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .