티스토리 뷰

  1. Jsoup.clean 을 서버를 띄우고 처음 실행시 warming up 으로 인해 많은 시간이 소요된다
       => @PostConstruct 어노테이션을 이용해서 Bean 이 생성된 이후에 Jsoup.clean 을 한번 호출하여 warming up 시켜준다.
  2. 하나의 Input에 대하여 Jsoup.valid 와 Jsoup.clean 을 두번 호출하여 사용한다. 만약 Jsoup.valid 와 Jsoup.clean 이 내부적으로 동일한 메서드와 클래스를 생성해서 사용한다면 불필요하게 두번 생성, 실행되므로 성능개선 포인트가 될 수 있다. 
  3. 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
링크
«   2024/05   »
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
글 보관함