Code Review
외부 라이브러리 사용시 성능 개선
소농배
2018. 12. 21. 09:23
- 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 한 메서드를 사용한다 하더라도 내부적으로 어떤 로직으로 처리되는지 확인 한 후에 중복을 제거할 수 있다면 제거하여
시간, 공간 자원을 아낄 수 있다!!