
Новые сообщения [новые:0]
Дайджест
Горячие темы
Избранное [новые:0]
Форумы
Пользователи
Статистика
Статистика нагрузки
Мод. лог
Поиск
|
|
05.08.2017, 16:06
|
|||
|---|---|---|---|
public static HttpSession session |
|||
|
#18+
Здравствуйте всем! собственно сабж, на сколько безопасна такое объявление? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
06.08.2017, 04:38
|
|||
|---|---|---|---|
public static HttpSession session |
|||
|
#18+
посоны, никаких проблем не будут если я такую переменную объявлю? Код: java 1. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
06.08.2017, 04:46
|
|||
|---|---|---|---|
|
|||
public static HttpSession session |
|||
|
#18+
Музаффар, Да, куча потенциальных проблем. Начиная от глобальной переменной, заканчивая многопоточностью. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
06.08.2017, 10:02
|
|||
|---|---|---|---|
public static HttpSession session |
|||
|
#18+
Музаффар, а чем обычная переменная не устраивает? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
06.08.2017, 10:17
|
|||
|---|---|---|---|
|
|||
public static HttpSession session |
|||
|
#18+
Музаффарна сколько безопасна такое объявление? На сколько объявление само по себе может быть опасным? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
06.08.2017, 10:48
|
|||
|---|---|---|---|
public static HttpSession session |
|||
|
#18+
BlazkowiczМузаффар, Да, куча потенциальных проблем. Начиная от глобальной переменной, заканчивая многопоточностью. т.е. так объявить HttpSession не желательно? maytonМузаффар, а чем обычная переменная не устраивает? мне нужно из класса получить сессию... пока другого пути не нашел... если есть хорошие варианты то был бы благодарен! ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
06.08.2017, 11:00
|
|||
|---|---|---|---|
public static HttpSession session |
|||
|
#18+
МузаффарmaytonМузаффар, а чем обычная переменная не устраивает? мне нужно из класса получить сессию... пока другого пути не нашел... если есть хорошие варианты то был бы благодарен! Я не знаю что вызвало тревогу коллег. Но я предполагаю что наш мир устроен так что обычное веб-приложение обслуживает много HttpSessions. А ты его объявил 1:1 с классом контейнером. Это странно. Вообще если ты пишешь ООП-код - то делай объекты POJO. И как только возникнет задача построения какого-то шаблона (пул или синглтон) то тогда стоит усложнять. А если ты со старта объявил статическую переменную HttpSession то ... у любого делающего code-review возникает крик - "What's the fuck?" и далее тебе придется объяснять - зачем ты это сделал и что имел в виду. Кстати рекомендую тебе установить SonarQube и периодически сканировать свой код на такие ситуации. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
06.08.2017, 11:03
|
|||
|---|---|---|---|
|
|||
public static HttpSession session |
|||
|
#18+
Музаффарт.е. так объявить HttpSession не желательно? не желательно Музаффармне нужно из класса получить сессию... А класс её откуда получает? Почему нужно именно из класса, а не экземпляра. И почему мы до сих пор не используем Spring MVC? Музаффарпока другого пути не нашел... если есть хорошие варианты то был бы благодарен! Хороший вариант, это Spring MVC. Но в качестве костыля можно взять ThreadLocal. Важно только удалять HttpSession оттуда в блоке finally, чтобы пользователи вдруг не начали сессии друг-друга видеть. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
06.08.2017, 11:17
|
|||
|---|---|---|---|
public static HttpSession session |
|||
|
#18+
Музаффармне нужно из класса получить сессию... начни с того, где этот твой класс, и сколько времени живёт его экземпляр. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|
06.08.2017, 14:22
|
|||
|---|---|---|---|
public static HttpSession session |
|||
|
#18+
mayton, хорошо. Blazkowicz, давайте спринг оставим на потом... ну собственно вот код: Код: 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. Код: java 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
|
|
|

start [/forum/topic.php?fid=59&tablet=1&tid=2122688]: |
0ms |
get settings: |
8ms |
get forum list: |
22ms |
check forum access: |
4ms |
check topic access: |
4ms |
track hit: |
80ms |
get topic data: |
11ms |
get forum data: |
3ms |
get page messages: |
58ms |
get tp. blocked users: |
1ms |
| others: | 207ms |
| total: | 398ms |

| 0 / 0 |
