|
|
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
Tsyklop, Ваша проблема это следствие того что вы берете Connection из DataSource в одном месте, а закрываете совершенно в другом. Хороший код должен следовать правилу - взял, закрыл в этом же методе в блоке finally: Код: java 1. 2. 3. У вас этого нет. DataSource у вас один единственный. А вы его зачем-то помещаете в кучу разных переменных без особой на то надобности. И соответсвенно путаетесь. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 13:00 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
TsyklopЯ сейчас все переделываю. А вот если бы инфраструктура работы с БД была бы отделена от бизнес-транзакций, то переделывали бы только инфрастуктуру а не всё. - Вы можете методы своего Server выделить в интерфейс? - Вы можете написать Proxy с этим интерфейсом, так чтобы InvocationHandler, который бы для всех таких методов создавал Database, потом делегировал бы вызов настоящему объекту Server, и после этого бы уже закрывал ваш Database? - Вы можете передать Database из InvocationHanlder-а в Server через параметры или ThreadLocal? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 13:03 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
TsyklopСессия разрушается через 30 минут бездействия. это параметр и его можно менять самому. TsyklopЕсли юзер зашел на страницу а сессии нет нет. Сессия всегда есть. Только будет новая и в сесии не будет объекта User TsyklopДаже если сессия есть то все равно выполняется проверка хэша авторизации который в куках я спросил зачем тебе DAO который работает с базой!!!!! ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 13:05 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
TsyklopТо бишь для каждого объекта брался свой коннекшн из пула - это не есть правильно. кто сказал такую ересь? У MS рекомендации - на одну форму-окно один коннект. Т.е. если 5 форм висят в памяти то будет 5 коннектов. И нет проблем. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 13:08 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
TsyklopТо бишь для каждого объекта брался свой коннекшн из пула - это не есть правильно попытайся обосновать почему это не правильно ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 13:11 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
наконец то мы добрались до сути всех топиков Tsyklop. У него было 3 коннекта на 3 класса и он решил что это непорядок. Надо объединить 3 класса. Перфекционист - что возьмёшь)) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 13:11 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
Petro123TsyklopСессия разрушается через 30 минут бездействия. это параметр и его можно менять самому.причём менять динамически в коде, продлять на некоторое время, к примеру 5мин, и можно поставить слушатель окончания сессии - если сессия закончилась на сервере - с помощью ws послать сообщение клиенту(браузеру) и он перейдёт на страницу входа. т.е. скроет от посторонних всё что юзер делал. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 13:12 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
вадя, разумеется. Ты видишь его фильтр выше который вызывается на каждый F5 странички. Разве не видно, что нафига там запрос в базу. Это ведь тормоза! ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 13:15 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
Petro123У него было 3 коннекта на 3 класса и он решил что это непорядок просто каша в голове от всего на свете, плюс отсутствие минимальных базовых знаний. Это нормально. Ненормально упрямство и нежелание вникнуть в советы. Могу только присоединиться к совету о прочтении книжек Мартина и Фаулера. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 13:17 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
chpashaНенормально упрямство и нежелание вникнуть в советы по моему опыту, упрямцы такого рода (задиристые) не становятся профи в программистах. Удачи аффтару! ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 13:20 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
TsyklopПроверить реально ли юзер авторизован. Сессия разрушается через 30 минут бездействия. Если юзер зашел на страницу а сессии нет то нужно проверить юзера и заполнить объект данными. Даже если сессия есть то все равно выполняется проверка хэша авторизации который в куках. Если юзер не авторизован или хэш не валидный его выбрасывает на главную. Прошу прощения, но зачем?! Чем вам стандартные библиотеки/фреймворки/приложения не угодили? Тем более стандартный сервер приложений/сервлетов все это умеет "из коробки" ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 13:45 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
mad_nazgulПрошу прощения, но зачем?! Чем вам стандартные библиотеки/фреймворки/приложения не угодили? ему некогда с ними разбираться ТС - теперь ты видишь, к чему приводят повторы? Страниц столько же, одни и те же вопросы обсасываются снова и снова ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 13:50 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
mad_nazgulПрошу прощения, но зачем?! Чем вам стандартные библиотеки/фреймворки/приложения не угодили? Тем более стандартный сервер приложений/сервлетов все это умеет "из коробки" читайте первый пост темы. Там я об этом написал. Что он умеет из коробки? проверить если ли юзер в бд? проверить авторизован ли он? и т.д. Если реально такое есть, без фреймворков, то покажите пожалуйста. очень интересно. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 14:13 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
chpashaPetro123У него было 3 коннекта на 3 класса и он решил что это непорядок просто каша в голове от всего на свете, плюс отсутствие минимальных базовых знаний. Это нормально. Ненормально упрямство и нежелание вникнуть в советы. Могу только присоединиться к совету о прочтении книжек Мартина и Фаулера. Каких базовых знаний не хватает? огласите список пожалуйста. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 14:14 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
BlazkowiczTsyklopЯ сейчас все переделываю. А вот если бы инфраструктура работы с БД была бы отделена от бизнес-транзакций, то переделывали бы только инфрастуктуру а не всё. - Вы можете методы своего Server выделить в интерфейс? - Вы можете написать Proxy с этим интерфейсом, так чтобы InvocationHandler, который бы для всех таких методов создавал Database, потом делегировал бы вызов настоящему объекту Server, и после этого бы уже закрывал ваш Database? - Вы можете передать Database из InvocationHanlder-а в Server через параметры или ThreadLocal? в Server нет методов. это просто контроллер который передаёт запрос в модель - то бишь создает объект модели и передаёт в неё все данные. Вот мой Server: Код: 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. Controller: Код: 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. 76. 77. 78. 79. 80. Далее все передаётся в модель. Пример метода login: Код: 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. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 14:19 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
TsyklopКаких базовых знаний не хватает? огласите список пожалуйста как писать хороший код (Clean code , Robert Martin) и как организовывать архитектуру приложения (Patterns of Enterprise Application Architecture, Martin Fowler) . Наверное имеет смысл именно в таком порядке ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 14:24 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
Petro123У MS рекомендации - на одну форму-окно один коннект. Т.е. если 5 форм висят в памяти то будет 5 коннектов. И нет проблем. В формах работают по-очереди. А здесь 3 класса могут сохранять все одновременно - если будут 3 разных коннекта, они не будут видеть изменений сделанных из 2-х других классов, при должном умении можно самого себя задедлочить, не ясно как передавать FK между ними и в каком порядке коммитить эти транзакции чтобы не упасть, а самое главное нельзя будет реализовать транзакционное изменение данных хотя бы 2-мя этими классами, если данные меняются ими 2-мя без распределенных транзакций. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 14:49 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
Локшин МаркВ формах работают по-очереди.это что то новое. Расшифруйте что за очередь? ГУИ? Дак могут и потоки быть. Локшин МаркА здесь 3 класса могут сохранять все одновременно - если будут 3 разных коннекта, они не будут видеть изменений сделанных из 2-х других классов конечно. Т.к. у него рукописный ОРМ. Зачем? Вместо ОРМ всё пустить в один ворота-коннект? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 14:52 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
Локшин Маркесли будут 3 разных коннекта, они не будут видеть изменений сделанных из 2-х других классов а вы знаете что в пуле тоже не видны изменения? ))) По вашему убрать пул? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 14:53 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
Локшин Марки в каком порядке коммитить эти транзакции чтобы не упасть вы не читали тред - у него автокоммит ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 14:55 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
Petro123Расшифруйте что за очередь? ГУИ? Дак могут и потоки быть. Это означает, что данные меняются формами независимо друг от друга. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 14:55 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
Petro123а вы знаете что в пуле тоже не видны изменения? ))) По вашему убрать пул? Не нужно в каждый менеджер совать свой коннекшен, не важно откуда его брать - напрямую с драйвера или с коннекшен пула. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 14:58 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
Локшин МаркPetro123Расшифруйте что за очередь? ГУИ? Дак могут и потоки быть. Это означает, что данные меняются формами независимо друг от друга. как захочу. Хочу передам коннект. Захочу создам новый. Вы чего испугались то? Я не пойму? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 14:58 |
|
||
|
Объединение sql запросов в один класс - упрощение (Обновлено)
|
|||
|---|---|---|---|
|
#18+
Локшин МаркНе нужно в каждый менеджерслово менеджер придумали вы. И детские страхи только у вас. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2018, 15:00 |
|
||
|
|

start [/forum/topic.php?fid=59&msg=39632548&tid=2122109]: |
0ms |
get settings: |
9ms |
get forum list: |
15ms |
check forum access: |
4ms |
check topic access: |
4ms |
track hit: |
152ms |
get topic data: |
10ms |
get forum data: |
3ms |
get page messages: |
51ms |
get tp. blocked users: |
1ms |
| others: | 248ms |
| total: | 497ms |

| 0 / 0 |

Извините, этот баннер — требование Роскомнадзора для исполнения 152 ФЗ.
«На сайте осуществляется обработка файлов cookie, необходимых для работы сайта, а также для анализа использования сайта и улучшения предоставляемых сервисов с использованием метрической программы Яндекс.Метрика. Продолжая использовать сайт, вы даёте согласие с использованием данных технологий».
... ля, ля, ля ...