티스토리 뷰
- Jsoup.clean 을 서버를 띄우고 처음 실행시 warming up 으로 인해 많은 시간이 소요된다
=> @PostConstruct 어노테이션을 이용해서 Bean 이 생성된 이후에 Jsoup.clean 을 한번 호출하여 warming up 시켜준다. - 하나의 Input에 대하여 Jsoup.valid 와 Jsoup.clean 을 두번 호출하여 사용한다. 만약 Jsoup.valid 와 Jsoup.clean 이 내부적으로 동일한 메서드와 클래스를 생성해서 사용한다면 불필요하게 두번 생성, 실행되므로 성능개선 포인트가 될 수 있다.
- XssVulnerabilityWhiteListFactory.java 에서 사용하는 SellerWhiteList 클래스를 Spring Factory Bean 으로 구현하여 유연하고 DI 받을 수 있도록 수정.
Jsoup.isValid()
1 2 3 4 5 | public static boolean isValid(String bodyHtml, Whitelist whitelist) { Document dirty = parseBodyFragment(bodyHtml, ""); Cleaner cleaner = new Cleaner(whitelist); return cleaner.isValid(dirty); } | cs |
Jsoup.clean()
1 2 3 4 5 6 | public static String clean(String bodyHtml, String baseUri, Whitelist whitelist) { Document dirty = parseBodyFragment(bodyHtml, baseUri); Cleaner cleaner = new Cleaner(whitelist); Document clean = cleaner.clean(dirty); return clean.body().html(); } | cs |
isValid() 와 clean() 메서드 내부적으로
Document 와 Cleaner 를 생성하여 사용하고 있다.
Cleaner 의 경우 Input 값이 whitelist 이므로 빈이 생성될때 한번 생성하여 계속 사용할 수도 있지만
내부적으로 상태값을 가지고 있을 수도 있기 때문에 성능에 큰 영향을 미치지 않는 다면 메서드가 호출될때 매번 생성하여 사용한다.
Document 는 메서드가 호출될때 한번 생성하여 valid 와 clean 에서 사용하도록 한다.
기억해야 할점은 변하는 것과 변하지 않는 것을 구분하는 것이다.
또한 static 한 메서드를 사용한다 하더라도 내부적으로 어떤 로직으로 처리되는지 확인 한 후에 중복을 제거할 수 있다면 제거하여
시간, 공간 자원을 아낄 수 있다!!
'Code Review' 카테고리의 다른 글
응답 DTO가 변경될때 고려할 사항 (0) | 2018.12.21 |
---|---|
public static final List (0) | 2018.12.21 |
DTO Logging (0) | 2018.12.21 |
공지사항
최근에 올라온 글
최근에 달린 댓글
- Total
- Today
- Yesterday
링크
TAG
- wait()
- reactor
- ConcurrentHashMap
- notify()
- HashMap
- N+1
- getBoolean
- RoutePredication
- mariadb-connector-j
- notifyAll()
- dynamodb
- custom config data convertion
- AbstractMethodError
- DyanomoDB
- mariada-connector
- circurit breaker
- router
- rate limit
- Flux
- spring cloud gateway
- Lazy
- reative
- GlobalFilter
- referencedColumnName
- Seperate Chaining
- ResultSet
- aurora
- MariaDB
- RouteDefinition
- msyql-connector-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 |
글 보관함