제로베이스-한달한권 클린코드 DAY12

2022-01-29

제로베이스-한달한권 클린코드 DAY12

Chapter.13, 14

Chapter 13 - 동시성

01. 동시성 프로그래밍이란

동시성 프로그래밍

어플리케이션을 효율적으로 실행하기 위해 멀티코어를 온전히 활용하도록 구현하는 방식

  • 외부 서비스의 응답을 기다리면서 아무일도 하지 않으면 CPU 사이클이 낭비된다. 낭비되는 자원을 줄이기 위한 구현방식

클라이언트가 아닌 어플리케이션 관점에서 봐야한다

  • 내 어플리케이션의 효율성을 높여야 한다.

  • 더불어 내 어플리케이션이 동작하는 머신의 환경이 효율적으로 돌아가도록 내 어플리케이션에 메모리 누수나 자원이 낭비되지 않도록 신경쓴다.

02. 동시성 프로그래밍이 필요한 이유

동시성 프로그래밍의 미신과 오해(1)

  • 동시성은 항상 성능을 높여준다 (X)

  • 동시성은 때로 성능을 높여준다 (O)

  • 대기시간이 아주 길어 여러 스레드가 프로세서를 공유할 수 있거나, 여러 프로세서가 동시에 처리할 독립적인 계산이 충분히 많은 경우에만 성능이 높아진다.

  • 예시: 웹 브라우저에서 여러 가지 이미지 리소스들을 불러와 다운로드할 때

  • 예시 상황 : Servlet

Java Servlet 동시성 구현

  • 요청이 들어오면 Thread Pool에 있는 Thread가 서블릿의 service() 메서드를 호출한다.
  • service의 doGet(), doPost()에서 요청에 대한 처리를 하도록 구현한다.
public class SimpleHttpServlet extends HttpServlet { // Not thread safe, static protected static List list = new ArrayList(); // Not thread safe protected Map map = new HashMap(); // Thread safe to access object, not thread safe to reassign variable protected Map map = new ConcurrentHashMap(); // Thread safe to access object (immutable), not thread safe to reassign variable protected String aString = "a string value"; protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException { // Not thread safe, unless the singleton is 100% thread safe. SomeClass.getSomeStaticSingleton(); // Thread safe, locally instantiated, and nver escapes method. Set set = new HashSet(); } }
  1. 해당 서블릿에 여러 쓰레드가 접근할 수 있기 때문에 멤버변수들은 Not Thread Safe하다.
  2. doGet(), doPost()는 하나의 쓰레드 영역에서 실행되기 때문에 로컬 변수들은 Thread Safe하다.

동시성 프로그래밍의 미신과 오해(2)

  • 동시성을 구현해도 설계는 변하지 않는다(X)

  • 동시성을 구현하면 설계를 바꿔야 한다(O)

  • 단일 스레스 시스템과 다중 스레드 시스템은 설계가 판이하게 다르다.

  • '무엇'과 '언제'를 분리하면 시스템의 구조가 크게 달라진다.

동시성 프로그래밍의 미신과 오해(3)

  • Web이나 EJB와 같은 컨테이너를 사용한다면 동시성을 이해할 필요가 없다.(X)

  • 컨테이너를 사용해도 동시성을 이해해야 한다(O)

  • 어플리케이션이 컨테이너를 통해 멀티 쓰레드를 사용하는 것이기 때문에 컨테이너의 동작을 이해해야 한다.

  • 동시 수정, 데드락 같은 문제를 피할 수 있는지를 알아야 한다.

03. 안전한 동시성 프로그래밍 규칙

단일 책임 원칙(SRP) 설계

동시성 관련 코드는 다른 코드와 분리하라

  • 동시성 코드는 독자적인 개발, 변경, 조율 주기가 있다.
  • 동시성 코드에는 독자적인 난관이 있다. 다른 코드에서 겪는 난관과 다르며 훨씬 어렵다.
  • 잘못 구현한 동시성 코드는 별의별 방식으로 실패한다. 주변에 있는 다른 코드가 발복을 잡지 않더라도 동시성 하나만으로도 충분히 어렵다.

자료 범위를 제한하라

공유 자료를 최대한 줄여라

  • 동시 수정 문제를 피하기 위해 객체를 사용하는 코드 내 임계영역을 synchronized 키워드로 보호하라
  • 보호할 임계영역을 빼먹거나, 모든 임계영역을 보호했는지 확인하느라 수고가 드므로 임계 영역의 수를 최소화 해야 한다.

자료 사본을 사용하라

공유 자료를 줄이려면, 최대한 공유하지 않는 방법이 제일 좋다

  • 객체를 복사해 읽기 전용으로 사용한다.
  • 각 스레드가 객체를 복사해 사용한 후 한 스레드가 해당 사본에서 결과를 가져온다.
  • 사본을 사용하는 방식으로 내부 잠금을 없애 수행 시간을 절약하는 것이 사본 생성과 가비지 컬렉션에 드는 부하를 상쇄할 가능성이 크다.

Thread는 가능한 독립적으로 구현하라

다른 스레드와 자료를 공유하지 않는다

  • 서블릿처럼 각 Thread는 클라이언트 요청 하나를 처리한다.
  • 모든 정보는 비공유 출처(client의 request)에서 가져오며 로컬 변수에 저장한다.
  • 각 서블릿은 마치 자신이 독자적인 시스템에서 동작하는 양 요청을 처리한다.

라이브러리를 이해하라

java.util.concurrent 패키지를 익혀라

  • Thread Safe 한 컬렉션을 사용한다.
    • ConcurrentHashMap, AtomicLong
  • 서로 무관한 작업을 수행할 때는 executor 프레임워크를 사용한다.
  • 가능하다면 Thread가 Blocking되지 않는 방법을 사용한다.

동기화하는 메서드 사이에 존재하는 의존성을 이해하라

공유 객체 하나에는 메서드 하나만 사용하라

  • 클라이언트에서 잠금 - 클라이언트에서 첫 번째 메서드를 호출하기 전에 서버를 잠근다. 마지막 메서드를 호출할 때까지 잠금을 유지한다.
  • 서버에서 잠금 - 서버에다 "서버를 잠그고 모든 메서드를 호출한 후 잠금을 해제하는" 메서드를 구현한다. 클라이언트는 이 메서드를 호출하기만 하면 된다.
  • 연결(Adapter)서버 - 잠금을 수행하는 중간 단계를 생성한다. '서버에서 잠금'방식과 유사하지만 원래 서버는 변경하지 않는다.

문제 상황: 공유 자원 접근

Client

// 공유 자료 IntegerIterator iterator = new IntegerIterator(); // Thread에서 호출되는 코드 while(iterator.hasNext()) { int nextValue = iterator.next(); }
  • Thread1이 hasNext() true를 받았으나 잠시 다른 일에 선점당한다.
  • Thread2가 hasNext() true를 받고 next()를 호출한다.
  • Thread1이 실행을 재개하여 next()를 또 호출한다.

Server

public class IntegerIterator implements Iterator<Integer> { private Integer nextValue = 0; public synchronized boolean hasNext() { return nextValue < 100000; } public synchronized Integer next() { if(nextValue == 100000) throw new IteratorPastEndException(); return nextValue++; } public synchronized Integer getNextValue() { return nextValue; } }

Client Based Lock

Client

// 공유 자료 IntegerIterator iterator = new IntegerIterator(); // Thread에서 호출되는 코드 while(iterator.hasNext()) { int nextValue; synchronized (iterator) { if(!iterator.hasNext()) break; nextValue = iterator.next(); } doSometingWith(nextValue); }
  • 클라이언트에서 락을 걸어 Thread Safe하게 블록을 보호한다.
  • 자원을 사용하는 클라이언트마다 위의 처리를 해줘야 한다. 비효율적이다.

Server Based Lock

Client

// 공유 자료 IntegerIteratorServerLocked iterator = new IntegerIteratorServerLocked(); // Thread에서 호출되는 코드 while(true) { Integer nextValue = iterator.getNextOrNull(); if(nextValue == null) break; }

Server

public class IntegerIteratorServerLocked { private Integer nextValue = 0; public synchronized Integer getNextOrNull() { if(nextValue < 100000) return nextValue++; else return null; } }
  • 서버에서 두 개로 나눠졌던 동작을 하나로 합쳐 락을 건다.
  • 클라이언트에서는 보호된 메서드를 호출하기만 하면 된다.

Adapter Based Lock

Client

// 공유 자료 ThreadSafeIntegerIterator iterator = new ThreadSafeIntegerIterator(); // Thread에서 호출되는 코드 while(true) { Integer nextValue = iterator.getNextOrNull(); if(nextValue == null) break; }

Adapter

public class ThreadSafeIntegerIterator { private IntegerIterator iterator = new IntegerIterator(); public synchronized Integer getNextOrNull() { if(iterator.hasNext()) return iterator.next(); return null; } }
  • 어댑터에서 서버에서 두 개로 나눠졌던 동작을 하나로 합쳐 락을 건다.
  • 클라이언트에서는 보호된 메서드를 호출하기만 하면 된다.
  • 서버의 코드가 외부 코드라서 수정할 수 없을 때 우리 코드에서 Adapter를 만들어 사용한다.

04. 동시성 테스트 방법

동시성 코드를 테스트 해야 한다

테스트를 했다고 동시성 코드가 100% 올바르다고 증명하기는 불가능하다

하지만 충분한 테스트는 위험을 낮춘다

  • 문제를 노출하는 테스트 케이스를 작성하라
  • 프로그램의 설정과 시스템 설정과 부하를 바꿔가며 자주 돌려라
  • 테스트가 실패하면 원인을 추적하라
  • 다시 돌렸더니 통과한다는 이유로 그냥 넘어가면 절대로 안된다.

코드에 보조 코드를 넣어 돌려라

드물게 발생하는 오류를 자주 발생시키도록 보조 코드를 추가한다

  • 코드에 wait(), sleep(), yield(), priority() 함수를 추가해 직접 구현한다.
  • 보조코드를 넣어주는 도구를 사용해 테스트한다.
    • 다양한 위치에 ThreadJigglePoint.jiggle()을 추가해 무작위로 sleep(), yield()가 호출되도록 한다.
  • 테스트 환경에서 보조 코드를 돌려본다.

동시성 코드를 실제 환경이나 테스트 환경에서 돌려본다

다양한 요청과 상황에 동시성 코드가 정상적으로 동작하는지 확인한다

  • 배포하기 전에 테스트 환경에서 충분히 오랜시간 검증한다.
  • 동시성 코드를 배포한 후에 모니터링을 통해 문제가 발생하는지 지켜본다.

Chapter 14 - 점진적인 개선

01. 책의 예제

명령형 인수 구문 분석기

code-init.webp

코드 초안

  • 모든 로직이 하나의 클래스에 들어가있다.
  • 처음부터 지저분한 코드를 짜려는 생각은 없었고, 코드를 어느 정도 손봤지만 새로운 인수 유형이 들어오면서 재앙이 시작됐다.
  • 이제는 개선해야 할 때라는 걸 깨닫고, 변경 전후 시스템이 동일 하게 돌아간다는 사실을 확인하기 위해 테스트들을 작성해뒀다.
  • 자잘하게 점진적으로 개선해나갔다.

code-complete.webp

코드 완성본

  • Args 클래스에서 코드 중복을 최소화하고, ArgsException 클래스를 분리했다. ArgumentMarshaler 클래스를 통해 여러 인수에 대한 추후 확장성을 만들어 냈다.
  • 코드만 분리해도 설계까 좋아진다. 관심사를 분리하면 코드를 이해하고 보수하기 훨씬 쉬워진다.

02. 점진적으로 개선하기

  1. 코드가 나빠지고 있음을 느꼈을 때 기능을 추가하지 않고 개선을 시작한다.

  2. 테스트 코드를 작성한다.

  • 변경을 가한 후에도 시스템이 변경 전과 똑같이 돌아가야 한다.
  • 테스트 코드가 없다면 작성하고, 코드를 수정하기 전 상태에서 모든 테스트가 통과해야 한다.
  1. 점진적으로 개선한다.
  • 책임에 따라 클래스를 나누고, 코드를 옮긴다. 테스트가 깨지지 않도록 확인하며 자잘한 변경을 조금씩 진행한다.

프로그램을 망치는 가장 좋은 방법 중 하나는 개선이라는 이름 아래 구조를 크게 뒤집는 행위다.

제로베이스-한달한권 클린코드 DAY11

제로베이스-한달한권 클린코드 DAY11

Chapter.12

제로베이스-한달한권 클린코드 DAY13

제로베이스-한달한권 클린코드 DAY13

Chapter.17