ReSQL.ru
     
powered by simpleCommunicator - 2.0.61     © 2026 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Java [игнор отключен] [закрыт для гостей] / рефакторинг
25 сообщений из 107, страница 2 из 5
  2  все
рефакторинг
    #39334993
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
вадязачем городить что-то если можно обойтись простым?

Простым? Прикалываешься? Сравни:

Код: html
1.
2.
3.
<input type="text" style="width: 100%; background: #f1f1f1;" title=" <%= bookKDP.getBookAuthorsWriter().getBook().getTitle() %> " ondblclick="window.open('https://www.amazon.com/dp/<%=bookKDP.getAsin()%>','_blank')" readonly="" value=" <%= bookKDP.getBookAuthorsWriter().getBook().getTitle() %>">

<input type="text" style="width: 100%; background: #f1f1f1;" title="${bookKDP.bookTitle}" ondblclick="window.open('https://www.amazon.com/dp/${bookKDP.asin}>','_blank')" readonly="" value="${bookKDP.bookTitle}">



вадяэто табу придумано непонятно кем и для чего, а потом его теражируют...
Табу придумано тем кто заманался выдирать логику предметной области из представления. И придумано лет 15 назад, если не раньше.
...
Рейтинг: 0 / 0
27.10.2016, 10:01
| Ответить | Цитировать | Написать  
рефакторинг
    #39334994
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
uid uniqueв 8 джаве этот пакет уже влючен, он и пошустрее и thread safe в отличие от Calendar.
Но слегка урезан по сравнению с оригинальным Joda Time.
...
Рейтинг: 0 / 0
27.10.2016, 10:02
| Ответить | Цитировать | Написать  
рефакторинг
    #39334995
Фотография Petro123
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Petro123
Участник
вадя,
Глупо спорить с jstl или скрипплеты.
С тобой не спорят что события в фунеции.
Это тоже самое.
...
Рейтинг: 0 / 0
27.10.2016, 10:03
| Ответить | Цитировать | Написать  
рефакторинг
    #39334996
uid unique
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
uid unique
Участник
uid uniqueBlazkowiczпропущено...

И ещё важный момент. Уверен что в книгде об этом тоже было. Научитесь создавать больше методов. Вот с ходу понять что тут происходит - решительно не возможно. Но если вы сделаете этому куску кода extractMethod и дадите методу внятное объясняющее имя, то это значительно повысит читаемость вашего кода.

У меня тоже вынесло мозг от операций с датой... по моему можно упростить в одну строчку.
Календарь и старые форматтеры желательно заменить joda time , в 8 джаве этот пакет уже влючен, он и пошустрее и thread safe в отличие от Calendar.
Что то подобное слепить можно с джодой:

Код: java
1.
2.
3.
DateTime now = new DateTime(System.currentTimeMillis());
		
someMethod(now.minusDays(2).toDate(), now.plusDays(7).toDate());
...
Рейтинг: 0 / 0
27.10.2016, 10:03
| Ответить | Цитировать | Написать  
рефакторинг
    #39335000
uid unique
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
uid unique
Участник
uid uniqueuid uniqueпропущено...


У меня тоже вынесло мозг от операций с датой... по моему можно упростить в одну строчку.
Календарь и старые форматтеры желательно заменить joda time , в 8 джаве этот пакет уже влючен, он и пошустрее и thread safe в отличие от Calendar.
Что то подобное слепить можно с джодой:

Код: java
1.
2.
3.
DateTime now = new DateTime(System.currentTimeMillis());
		
someMethod(now.minusDays(2).toDate(), now.plusDays(7).toDate());


Еще вдогонку не забудьте про time zone, если она нужна, дата в локальной зоне будет отличаться от GMT (now нужно инициализировать с зоной или сделать шифт, в джоде это есть).
...
Рейтинг: 0 / 0
27.10.2016, 10:06
| Ответить | Цитировать | Написать  
рефакторинг
    #39335009
вадя
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
вадя
Участник
Blazkowiczвадя,

Ну, кто же в 2016м году скриплеты использует? Для доступа-то к свойствам. EL в JSP уже достаточно хорош чтобы всё на нём писать. Скриплеты это табу. Только в исключительных случаях можно применить, если они действительно упрощают решение.
и где же ты в моем примере нашёл скриплет?
только выражение
...
Рейтинг: 0 / 0
27.10.2016, 10:11
| Ответить | Цитировать | Написать  
рефакторинг
    #39335016
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
вадяи где же ты в моем примере нашёл скриплет?
только выражение
Вот:
Код: html
1.
<%= ... %>
...
Рейтинг: 0 / 0
27.10.2016, 10:14
| Ответить | Цитировать | Написать  
рефакторинг
    #39335019
вадя
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
вадя
Участник
Petro123вадя,
Глупо спорить с jstl или скрипплеты.
С тобой не спорят что события в фунеции.
Это тоже самое.

С тобой не спорят что события в фунеции.
Это тоже самое
нука про это попподробнее.
а где должны они быть?
...
Рейтинг: 0 / 0
27.10.2016, 10:16
| Ответить | Цитировать | Написать  
рефакторинг
    #39335022
вадя
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
вадя
Участник
Blazkowicz,

Код: javascript
1.
<%= ... %>


это называется выражение.
...
Рейтинг: 0 / 0
27.10.2016, 10:17
| Ответить | Цитировать | Написать  
рефакторинг
    #39335035
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
вадяBlazkowicz,

Код: javascript
1.
<%= ... %>


это называется выражение.
Это называется путаница в терминах
JSP Scriptlet - который табу, но в крайних случаях бывает полезен именно в JSP
http://docs.oracle.com/javaee/5/tutorial/doc/bnaou.html
JSP Expression - который нафиг не нужен
http://docs.oracle.com/javaee/5/tutorial/doc/bnaov.html
Unified Expression Language - который упрощает JSP код и делает его HTML friendly
http://docs.oracle.com/javaee/5/tutorial/doc/bnahq.html
...
Рейтинг: 0 / 0
27.10.2016, 10:24
| Ответить | Цитировать | Написать  
рефакторинг
    #39335058
вадя
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
вадя
Участник
Blazkowicz,
уж если заниматься буквоедством - то в моём примере JSP Expression
а JSP Scriptlet у ТС.
...
Рейтинг: 0 / 0
27.10.2016, 10:37
| Ответить | Цитировать | Написать  
рефакторинг
    #39335065
вадя
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
вадя
Участник
Blazkowicz,
а вот насчёт
Unified Expression Language - который упрощает JSP код и делает его HTML friendly
вопрос спорный
по сути вводится ещё один язык программирования симбиоз java и HTML разметки
что только усложняет построение этой самой разметки.
по мимо дополнительного кода для его построения и кода на выпонение при формировании страницы.
...
Рейтинг: 0 / 0
27.10.2016, 10:42
| Ответить | Цитировать | Написать  
рефакторинг
    #39335070
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
вадяуж если заниматься буквоедством - то в моём примере JSP Expression
а JSP Scriptlet у ТС.
Продолжаем заниматься ерундой:
In JavaServer Pages (JSP) technology, a scriptlet is a piece of Java-code embedded in the HTML-like JSP code.
...
Рейтинг: 0 / 0
27.10.2016, 10:45
| Ответить | Цитировать | Написать  
рефакторинг
    #39335074
вадя
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
вадя
Участник
Blazkowiczвадяуж если заниматься буквоедством - то в моём примере JSP Expression
а JSP Scriptlet у ТС.
Продолжаем заниматься ерундой:
In JavaServer Pages (JSP) technology, a scriptlet is a piece of Java-code embedded in the HTML-like JSP code.
тогда нахера ты дал три ссылки? сначала разделяешь , а потом объединяешь понятия?
при буквоедстве так быстро переобуваться не стоит
...
Рейтинг: 0 / 0
27.10.2016, 10:50
| Ответить | Цитировать | Написать  
рефакторинг
    #39335090
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
вадятогда нахера ты дал три ссылки?

Чтобы объяснить таким непонятливым, откуда путаница в терминологии.

вадясначала разделяешь , а потом объединяешь понятия?
при буквоедстве так быстро переобуваться не стоит
Я с самого начала сказал про "скриплеты", ты начал буквоедствовать. Твой код это скриплет, который в JEE называют JSP Expression только потому что этот скриплет должен возвращать результат. В обиходе же expression это EL. Не думал что тебе придется такие основы объяснять.
...
Рейтинг: 0 / 0
27.10.2016, 11:00
| Ответить | Цитировать | Написать  
рефакторинг
    #39335100
вадя
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
вадя
Участник
Blazkowicz,
потому что про скриплеты у ТС ты ни слова не сказал , а к моим прицепился,
я просто исправил код HTML на правильный и вместо нескольких выражений оставил одно.
...
Рейтинг: 0 / 0
27.10.2016, 11:08
| Ответить | Цитировать | Написать  
рефакторинг
    #39335108
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
вадяпотому что про скриплеты у ТС ты ни слова не сказал , а к моим прицепился,

И ты посчитал это личным оскорблением. Какой тонкой душевной натуры человек. Ну, извини.

вадяя просто исправил код HTML на правильный и вместо нескольких выражений оставил одно.
Ну, вот, а я добавил, что скриплеты тоже стоило выкинуть, потому что "они зло".
...
Рейтинг: 0 / 0
27.10.2016, 11:14
| Ответить | Цитировать | Написать  
рефакторинг
    #39335127
Фотография Petro123
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Petro123
Участник
вадя С тобой не спорят что события в фунеции.
Это тоже самое
нука про это попподробнее.
а где должны они быть?
не борись с ветряными мельницами.
Я сказал, что Согласен с тобой (не спорю) относительно твоих постов по JS.
Про JSTL толку с тобой говорить, если ты его не писал.
...
Рейтинг: 0 / 0
27.10.2016, 11:24
| Ответить | Цитировать | Написать  
рефакторинг
    #39335153
вадя
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
вадя
Участник
авторНу, вот, а я добавил, что скриплеты тоже стоило выкинуть, потому что "они зло".
спорное утверждение, если учесть сколько дополниельного кода надо добавить дл JSTL .

авторне борись с ветряными мельницами.
Я сказал, что Согласен с тобой (не спорю) относительно твоих постов по JS.
будь точнее в своих выражения.
...
Рейтинг: 0 / 0
27.10.2016, 11:35
| Ответить | Цитировать | Написать  
рефакторинг
    #39335159
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
вадяспорное утверждение, если учесть сколько дополниельного кода надо добавить дл JSTL .

Во-первых ни сколько не надо.
Во-вторых JSTL это не EL.
...
Рейтинг: 0 / 0
27.10.2016, 11:38
| Ответить | Цитировать | Написать  
рефакторинг
    #39335186
Фотография Petro123
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Petro123
Участник
вадяесли учесть сколько дополниельного кода надо добавить дл JSTL
я же говорю, что ни разу не писал:

Код: html
1.
2.
3.
4.
5.
6.
7.
8.
<%@ taglib prefix="c" uri="http://java.sun.com/jsp/jstl/core" %>
<!DOCTYPE .....
...
<div id="links" class="links">
	<c:forEach var="i" begin="1" end="12345" step="1">
              <a href="/media/video/${i}" />.AVI" download>Скачать${i}..., </a>
	</c:forEach>
</div>
...
Рейтинг: 0 / 0
27.10.2016, 11:57
| Ответить | Цитировать | Написать  
рефакторинг
    #39335188
вадя
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
вадя
Участник
Blazkowiczвадяспорное утверждение, если учесть сколько дополниельного кода надо добавить дл JSTL .

Во-первых ни сколько не надо.
Во-вторых JSTL это не EL.
взял пример http://devcolibri.com/1250
нафига ещё разбираться в дополнительном языке??????
когда внизу показано
Код: plaintext
1.
2.
3.
4.
Вы конечно можете использовать java вставки:
<%
    int a = 5;
    int b = a + 10;
%>
просто и наглядно по сравнению с кодом , который описан выше на странице.
даже если посчитать количество строк(не говоря уж о символах ) будет больше.
это если будут платить за количество строк - тогда ещё можно понять.
а такое
Код: plaintext
но если использовать такие вставки то вы не сможете контролировать ход действий или как то их отследить.
голословное утверждение -ход каких действий, отслеживать что??
Код: plaintext
1.
По этому рекомендуется использовать jstl и желательно не использовать какие то вычисления или обработки данных на jsp странице, это нужно выполнять на стороне сервера.
это вообще бред
и это такие выносят табу?????
...
Рейтинг: 0 / 0
27.10.2016, 11:57
| Ответить | Цитировать | Написать  
рефакторинг
    #39335191
Фотография Petro123
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Petro123
Участник
вадяэто вообще бред
тут просто Обычная лень для изучения всего нового. Что в JS, что в Java.
...
Рейтинг: 0 / 0
27.10.2016, 11:59
| Ответить | Цитировать | Написать  
рефакторинг
    #39335204
вадя
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
вадя
Участник
Код: javascript
1.
2.
3.
4.
5.
6.
<div id="links" class="links">
<% int i=0;
    while(++i<1234){
        out.print(String.format("<a href='/media/video/%i.AVI' download>Скачать%i..., </a>",i));
    } %>
</div>


ни чуть не хуже.
если учесть во что транслирется jsp страница .

автортут просто Обычная лень для изучения всего нового. Что в JS, что в Java.
ты вникнись в смысл там написанного...
...
Рейтинг: 0 / 0
27.10.2016, 12:13
| Ответить | Цитировать | Написать  
рефакторинг
    #39335220
Фотография Blazkowicz
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Blazkowicz
Участник
вадяни чуть не хуже.

Если ты не видишь разницы, тогда разговор продолжать бесполезно.

вадяесли учесть во что транслирется jsp страница .

Да пофигу во что транслируется. Это не сервлету потом читать код и вносить в него изменения. Какие-то нелепые нюансы технической реализации никак не влияют на качество кода. Говно код нельзя оправдывать тем что он будет работать, например, в интерпретаторе и всё равно будет медленным.
...
Рейтинг: 0 / 0
27.10.2016, 12:20
| Ответить | Цитировать | Написать  
25 сообщений из 107, страница 2 из 5
  2  все
Форумы / Java [игнор отключен] [закрыт для гостей] / рефакторинг
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение   Полит. конфиден.  
NoSQL.ru       Кролег: Проекты, Новости, Чат       РЕД ФОРУМ       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=59&msg=39335108&tid=2123546]:
 0ms
get settings:
 6ms
get forum list:
 11ms
check forum access:
 2ms
check topic access:
 2ms
track hit:
 57ms
get topic data:
 9ms
get forum data:
 2ms
get page messages:
 45ms
get tp. blocked users:
 2ms
others: 205ms
total:  341ms
созд. / загр.: 341ms
x
x
Закрыть


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