ReSQL.ru
     
powered by simpleCommunicator - 2.0.59     © 2025 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Java [игнор отключен] [закрыт для гостей] / Controller & Transactions
15 сообщений из 15, страница 1 из 1
Controller & Transactions
    #40068579
Фотография -=Koba=-
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
-=Koba=-
Участник
Есть контроллер

Код: java
1.
2.
3.
4.
5.
6.
7.
8.
class TestController {
    private final TestService testService;
    @PostMapping("/send")
    public void send(@RequestBody TestDto dto) throws JsonProcessingException {
        testService.method1(dto);
        testService.method2(dto);
    }
}



И соотвественно сервис
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
class TestService {
    @Transactional 
    public void method1(TestDto dto) {
...
    }

    @Transactional(propagation = Propagation.REQUIRES_NEW)
    public void method2(TestDto dto) {
...
    }
}



Собственно вопрос
Если в контроллер проходит метод - method1, а method2 падает
Можно ли откатывать и изменения method1?

Читал, что управление транзакция на уровне контроллера плохая идея
Поэтому и возник вопрос

Логику работы методов менять нельзя (из method1 вызвать method2 не могу)
...
Рейтинг: 0 / 0
05.05.2021, 14:33
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068583
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
-=Koba=-,
Теоретически плохо 2 метода в контроллере. Это же уже БЛ?
Но по факту голый контроллер тоже плохо.
Поэтому делай в контроллере если надо.
...
Рейтинг: 0 / 0
05.05.2021, 14:46
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068584
Фотография -=Koba=-
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
-=Koba=-
Участник
Теоретический да... это уже готовая БЛ
И всплыл такой баг

Что изменения должны быть все или ничего
В моем случае, транзакция метода 1 проходит, метод 2 падает

Можете поянсить - Но по факту голый контроллер тоже плохо.
...
Рейтинг: 0 / 0
05.05.2021, 14:48
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068588
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
-=Koba=-,
Еденица работы это вызов POST
Поэтому непонятно, почему ты разбил Одну единицу из двух методов на две транзакции?
Ты должен вызвать всю работу в одной транзакции. Так?
...
Рейтинг: 0 / 0
05.05.2021, 14:56
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068592
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
-=Koba=-,
>Можете поянсить - Но по факту голый контроллер тоже плохо.
= если проект маленький, то вообще можно не разбивать на слои и все делать в контроллерах. Даже ссыль на хибер иметь.
Если большой то контроллер считается ГУИ и там бл не должно быть. Там логика запросов ЮЗВЕРЯ
...
Рейтинг: 0 / 0
05.05.2021, 15:05
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068594
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
-=Koba=-,
>Логику работы методов менять нельзя (из method1 вызвать method2 не могу)
= а из метода3 вызвать 1 и потом 2?
...
Рейтинг: 0 / 0
05.05.2021, 15:08
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068597
Фотография -=Koba=-
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
-=Koba=-
Участник
PetroNotC Sharp
-=Koba=-,
>Логику работы методов менять нельзя (из method1 вызвать method2 не могу)
= а из метода3 вызвать 1 и потом 2?

Ок спасибо за наводку.... подумаю, так можно
...
Рейтинг: 0 / 0
05.05.2021, 15:10
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068635
chpasha
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
chpasha
Участник
со спринговым aop такое может не сработать - метод должен извне вызываться, чтоб его завернуло в траназакцию, async и т.д. - если из метода3 вызвать метод2 или 1, то транзакции не будет у каждого из них, только у метод3. Нужно AspectJ юзать
...
Рейтинг: 0 / 0
05.05.2021, 16:28
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068643
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
chpasha,
Ну, я и подумал что ему не надо делать типа автономной транзакции. Не так часто требуется.
...
Рейтинг: 0 / 0
05.05.2021, 17:22
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068644
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
chpasha,
А вот если понадобится, то я не понял что значит "извне".
Стек вызовов вроде везде одинаков.
...
Рейтинг: 0 / 0
05.05.2021, 17:25
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068649
Stanislav Bashkyrtsev
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Stanislav Bashkyrtsev
Гость
chpasha
со спринговым aop такое может не сработать - метод должен извне вызываться, чтоб его завернуло в траназакцию, async и т.д. - если из метода3 вызвать метод2 или 1, то транзакции не будет у каждого из них, только у метод3. Нужно AspectJ юзать
Ему и не надо чтоб method1 & method2 были обернуты каждый в свою транзакцию. Он же и спрашивает - как сделать одну транзакцию на оба метода. Вариант с method3 предложенный PetroNotC Sharp как раз это и сделает. И есть методы по-проще нежели AspectJ прикручивать, даже если вызов "внутренний".
...
Рейтинг: 0 / 0
05.05.2021, 17:52
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068654
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
Stanislav Bashkyrtsev,
Тут имхо непонятно что он спрашивал.
Как всегда на форуме, три пишем, а семь на ум пошло.
- если нужна была одна, то почему аннотация на новую.
- если две, то с какой целью.
В общем вроде пациент разобрался мирно и без эксцессов.
...
Но есть повод поболтать в рамках инженерии).
...
Рейтинг: 0 / 0
05.05.2021, 18:24
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068658
chpasha
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
chpasha
Участник
Stanislav Bashkyrtsev
Ему и не надо чтоб method1 & method2 были обернуты каждый в свою транзакцию

мне это не очевидно хотя бы потому, что у них propagation разные - я как-то по наивности полагал, что там REQUIRES_NEW не от балды вставлено

Stanislav Bashkyrtsev
И есть методы по-проще нежели AspectJ прикручивать, даже если вызов "внутренний".
Как? Инъекцию менеджера транзакций исключаем, как очевидную


PetroNotC Sharp
А вот если понадобится, то я не понял что значит "извне".
Извне класса, где аннотации объявлены. Вызов service.method1 из контроллера стартует транзакцию (ну в идеале :) ), а вот если внутри самого service вызвать this.method1 откуда-то еще - то нет, поскольку спринг по умолчанию в режиме прокси работает, т.е. делает обертку вокруг класса и перехватывает только внешние вызовы к методам
...
Рейтинг: 0 / 0
05.05.2021, 19:07
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068680
Stanislav Bashkyrtsev
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Stanislav Bashkyrtsev
Гость
chpasha
Stanislav Bashkyrtsev
Ему и не надо чтоб method1 & method2 были обернуты каждый в свою транзакцию

мне это не очевидно хотя бы потому, что у них propagation разные - я как-то по наивности полагал, что там REQUIRES_NEW не от балды вставлено
Мы конечно подождем еще что скажет автор, но это как бы намекает на одну транзакцию:
-=Koba=-Если в контроллер проходит метод - method1, а method2 падает
Можно ли откатывать и изменения method1А propagation поставлен наверно от балды. Он просто еще не разобрался с тем как транзакционность настраивается. Конкретно в данном куске кода мог стоять и обычный REQUIRES .

chpasha
Stanislav Bashkyrtsev
И есть методы по-проще нежели AspectJ прикручивать, даже если вызов "внутренний".
Как? Инъекцию менеджера транзакций исключаем, как очевиднуюПочему мы не рассматриваем очевидные (и правильные) решения? Кроме адекватных решений у спринга есть еще хаки типа self injection, и как вариация на тему - инъекция ApplicationContext'a и явно взять себя из контекста. Ну и конечно еще можно порефакторить код и вынести 2ой метод в какой-то другой класс.
...
Рейтинг: 0 / 0
05.05.2021, 23:36
| Ответить | Цитировать | Написать  
Controller & Transactions
    #40068724
chpasha
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
chpasha
Участник
Stanislav Bashkyrtsev
Почему мы не рассматриваем очевидные (и правильные) решения?
дело не в том, как правильно. вопрос чисто по матчасти - либо есть нормальные способы заставить работать плюшку @Trasnactional "внутри", либо нет.

Stanislav Bashkyrtsev
хаки типа self injection

ну нет, это уже из разряда можно ли съесть лампочку ;)
...
Рейтинг: 0 / 0
06.05.2021, 11:40
| Ответить | Цитировать | Написать  
15 сообщений из 15, страница 1 из 1
Форумы / Java [игнор отключен] [закрыт для гостей] / Controller & Transactions
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение
NoSQL.ru       Новости, Чат       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=59&msg=40068635&tid=2120456]:
 0ms
get settings:
 8ms
get forum list:
 13ms
check forum access:
 3ms
check topic access:
 3ms
track hit:
 46ms
get topic data:
 9ms
get forum data:
 2ms
get page messages:
 47ms
get tp. blocked users:
 1ms
others: 12ms
total:  144ms
созд. / загр.: 144ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]