형식 맞추기

• 프로그래머라면 형식을 깔끔하게 맞춰 코드를 짜야 한다.
• 팀이 합의하여 규칙을 정하고 모두가 그 규칙을 따라야 한다.

형식을 맞추는 목적

• 코드 형식은 중요하다.
• 의사소통의 일환이다. 의사소통은 전문 개발자의 일차적인 의무다.
• 구현한 코드들이 변경되더라도 개발자의 스타일과 규율은 잘 사라지지 않는다.

적절한 행 길이를 유지하라

• 자바에서 파일 크기는 클래스 크기와 밀접하다.
• 일반적으로 작은 파일이 큰 파일보다 이해하기 쉽다.

신문 기사처럼 작성하라

• 독자는 위에서 아래로 읽는다.
• 메소드 전체 내용을 요약하는 이름을 작성한다.
• 세세한 내용은 또 하나의 메소드로 분리한다.

개념은 빈 행으로 분리하라

• 빈 행은 새로운 개념을 시작한다는 시각적 단서다.
• 다른 개념 사이에 개행이 빠지면 가독성이 현저히 떨어진다.

세로 밀집도

• 서로 밀접한 코드 행은 세로로 가까이 놓여야 한다.

수직 거리

• 같은 파일에 속할 정도로 밀접한 두 개념은 세로 거리로 연관성을 표현한다.
  • 여기서 뜻하는 연관성이란 한 개념을 이해하는데 다른 개념이 중요한 정도다.
  • 연관성이 깊은 두 개념이 멀리 떨어져 있으면 코드를 읽는 사람이 함수 연관관계와 동작 방식을 이해하기 위해 여기저기 찾느라 시간과 노력을 소모한다.

[수직분리]

변수 선언

• 처음으로 사용하기 직전에 선언하며 수직으로 가까운 곳에 위치해야 한다.
• 비공개 함수는 처음으로 호출한 직후에 정의해야 쉽게 눈에 띄어야 한다.

인스턴스 변수

• 클래스 맨 처음에 선언한다.

종속함수

• 한 함수가 다른 함수를 호출한다면 두 함수는 세로로 가까이 배치한다.
• 또한 가능하다면 호출하는 함수를 호출되는 함수보다 먼저 배치해야 자연스럽게 읽힌다.

개념적 유사성

• 개념적인 친화도가 높을수록 코드를 가까이 배치한다.
• 종속적인 관계가 없더라도 기능적으로 유사하다.

public boolean isAdmin() {  
  return StringUtils.equals(RoleCodes.ADMIN, role.getName());  
}

public boolean isManager() {  
  return StringUtils.equals(RoleCodes.MANAGER, role.getName());  
}

public boolean isUser() {  
  return StringUtils.equals(RoleCodes.USER, role.getName());  
}

세로 순서

• 호출되는 순서대로 함수를 배치하면 소스 코드 모듈이 고차원에서 저차원으로 자연스럽게 내려간다.
• 가장 중요한 개념을 표현하고나서 세세한 사항을 표현하면 함수 몇 개만 읽어도 개념을 파악하기 쉬워져서 세세한 사항까지 파고들 필요가 없다.

가로 형식 맞추기

• 행 길이는 어느 정도가 적당할까?
  • 여러 논쟁이 있지만 120자 정도로 제한하는 것을 권고한다고 한다.

가로 공백과 밀집도

• 가로로는 공백을 사용해 밀접한 개념과 느슨한 개념을 표현한다.
  • ex) 할당 연산자 명확, 메소드 파라미터 구분, 연산자 우선순위 강조

가로 정렬

• 정렬을 통해 강조하려고한 진짜 의도가 가려질 수 있다.
• 변수 유형은 무시하고 변수 이름부터 읽게 되버린다?
  • 정렬이 필요할 정도로 목록이 길다면 문제는 목록 길이지 정렬 부족이 아니다.
  • 만약 선언부가 길어진다면 클래스를 쪼개야 한다는 의미다.

들여쓰기

• 들여쓰기가 없다면 인간이 코드를 읽기란 거의 불가능일 것이다.
• 들여쓰기한 파일은 구조가 한눈에 들어온다.
  • ex) 변수, 메소드, 분기문 등..

• 들여쓰기 무시하기

  • 간혹 간단하고 짧은 메소드에서 규칙을 무시하고픈 유혹이 생긴다.

  public String render() { return ""; }
  
  public String render() {   
    return "";   
  }

팀 규칙

• 각자 선호하는 규칙이 있지만 팀에 속한다면 자신이 선호해야할 규칙은 바로 팀 규칙이다.
  • 그래야 스타일이 일관적이고 매끄럽다.
  • 팀 규칙을 정하여 IDE 코드 형식기를 설정하여 모두가 그 규칙을 따라야 한다.
• 좋은 소프트웨어 시스템은 읽기 쉬운 문서로 이뤄진다.

마치며

팀에서 포괄적인 규칙을 정하여 IDE에 적용하여 사용하다가 의아스러운 부분이 있었다.
상세한 부분에 대해서도 팀 내 규칙을 정하여 한 사람이 작성한듯한 가독성 좋은 코드를 작성해야 한다.

적용한 룰에 예외적으로 느껴진 항목

• 테스트케이스 MockMvc 포맷
• 포맷으로 지정한 가로 길이를 조금 초과하는 경우
• 조건문 내 1줄 정지성 키워드(break, return, throw ..) 브레이스 여부

참고문헌 - CleanCode 애자일 소프트웨어 장인 정신

의미있는 이름

• 이름은 패키지, 클래스, 메소드, 변수 등 널리 사용된다.
• 이름을 잘 지으면 여러모로 편하다.

의도를 분명히 밝혀라

• 좋은 이름을 지으려면 시간이 걸리지만 좋은 이름으로 절약하는 시간이 훨씬 더 많다.
• 코드를 보고 더 나은 이름이 떠오르면 즉시 개선하는 것이 좋다.
• 이름에 주석이 필요하다면 의도를 분명히 드러내지 못한 것이다.
  • ex) int d; // 날짜

그릇된 정보를 피하라

• 일반적으로 떠오르는 생각을 왜곡시키지 말라.
• 중의적으로 해석될 수 있는 이름은 지양하라.

• 코드의 일관성을 깨뜨리지 말라.

  ex)
  String codeList; 
  // 실제 데이터를 담는 컨테이너가 List가 아닌데 변수명으로 list를 사용하는 경우
  
  int a = l; 
  if (O == l)
   a = 01;
  else
   l = Ol;
  // 문자 l은 숫자 1처럼 보이고 문자 O는 숫자 0 처럼 보인다.

의미있게 구분하라

• 불용어(noise word)를 피하라.
  • ex) int a1, a2, a3;
• 읽는 사람이 차이를 알수 있도록 이름을 지어라.
• 정확한 개념이 구분이 되지 않는다.
  • ex) ProductInfo? ProductData?

발음하기 쉬운 이름을 사용하라

• 두뇌는 단어를 바탕으로 발음하려고 한다.
  • ex) genymdhms -> generateTimestamp

검색하기 쉬운 이름을 사용하라

• 상수는 검색하기 쉽도록 변수로 정의해서 사용하라.
• 긴 이름이 짧은 이름보다 검색하기 수월하다.
• 이름 길이는 범위 크기에 비례해야 한다.
• 간단한 메서드에서나 짧은 변수명을 사용하라.

인코딩을 피하라

• 불필요한 정신적 부담일 뿐이다. (변수에 부가정보를 덧붙이는 표기. 해독, 발음, 오타)
• 헝가리식 표기법
  • 변수명에 타입을 명시한다. 타입이나 이름이 바뀌더라도 반드시 같이 변하진 않는다.
• 멤버변수 접두어
  • 코드를 읽을수록 접두어/접미어는 관심 밖으로 밀려난다. 구닥다리 코드의 징표일 뿐이다. ex) String m_title, strContent;
• 인터페이스 클래스와 구현 클래스
  • 때론 구분하기 위해서 인코딩이 필요할 때도 있다.
  • ex) 인터페이스 보다는 구현 클래스를 인코딩 하라. ServerManagerImpl, CServerManager (O) / IServerManager (X)

자신의 기억력을 자랑하지 마라

• 독자가 코드를 읽다가 변수명을 변환하게 해서는 안된다.
  • 이미 변수 a와 b가 있다고 c라는 이름을 선택하는 논리는 최악이다.
• 똑똑한 프로그래머와 전문가 프로그래머의 차이점은 명료함이다. 다른 사람들이 잘 알아볼수 있도록 코딩해야 한다.

클래스 이름

• 클래스/객체 이름은 명사/명사구가 적합하다.
  • Manager, Processor, Data, Info와 같은 단어는 피하라.
  • ex) Customer, AddressParser
• 동사는 사용하지 않는다.

메서드 이름

• 메서드 이름은 동사/동사구가 적합하다.
  • ex) deletePage, saveBoard
• 변경자, 접근자, 조건자는 각각 set, get, is로 시작하라.
  • ex) getItem, setName, isStudent

기발한 이름은 피하라

• 구어체/속어는 사용해서 안된다.
• 특정 문화에서만 사용되는 사례도 금한다.
  • ex) kill() -> whack() / abort() -> eatMyShort()

한 개념에 한 단어를 사용하라

• 똑같은 메서드를 클래스마다 get, fetch, retrieve 등 다른 단어로 혼용하면 혼란스럽다.
• 일관성 있는 어휘가 독자에게 좋다.

말장난을 하지 마라

• 한 단어를 두가지 목적으로 사용하지 마라.
• 다른 개념에 같은 단어를 사용한다면 그것은 말장난에 불과하다.

  ex) 
  지금까지 구현한 모든 add 메소드가 기존 값 두개를 더하거나 이어서 새로운 값을 만든다고 가정하자. 
  새롭게 작성하는 메소드는 집합에 값을 하나만 추가한다. 
  일관성 있게 add라는 메소드명을 사용해야 하는가? 아니다. 
  insert나 append 등 다른 이름을 사용하자.

• 최대한 이해하기 쉽게 작성해야 한다.
• 집중적인 탐구가 필요한 코드가 아닌 대충 훑어봐도 이해할 수 있는 코드 작성이 목표다.

해법 영역에서 가져온 이름을 사용하라

• 코드를 읽을 사람도 프로그래머라는 사실을 명심한다.
• 전산용어, 알고리즘 이름, 패턴 이름, 수학용어 등을 사용해도 괜찮다.
• 기술적인 개념에는 기술적인 이름이 가장 적합한 선택이다.
  • ex) queue, stack

문제 영역에서 가져온 이름을 사용하라

• 적절한 용어가 없다면 문제 영역에서 이름을 가져온다.
  • 코드를 유지 보수하는 개발자가 분야 전문가에게 의미를 물어 파악할 수 있다.
• 우수한 개발자와 설계자라면 해법 영역과 문제 영역을 구분할 줄 알아야 한다.

의미 있는 맥락을 추가하라

• 스스로 의미가 분명한 이름은 있다. 하지만 대다수 이름은 그렇지 못하므로 클래스, 함수, 이름 공간에 넣어 맥락을 부여한다.
• 모든 방법이 실패하면 마지막 수단으로 접두어를 붙인다.

  ex)
  firstName, lastName, street, houseNumber, city, state, zipcode 라는 변수가 있다. 
  변수를 훑어보면 주소라는 사실이 금방 알아챈다. 
  하지만 어느 메소드가 state라는 변수 하나만 사용한다면? 
  변수 state가 주소의 일부라는 사실을 금방 알아챌까? 
  addr 접두어를 추가해 addrFirstName, addrLastName, addrState라고 쓰면 맥락이 분명해진다.

불필요한 맥락을 없애라

ex)
 Gas Station Deluxe 라는 어플리케이션을 작성한다고 해서 클래스 이름 앞에 GSD를 붙이지는 말자. GSDAccount, GSDUser ..
 G를 입력하고 자동완성을 누를 경우 모든 클래스가 나타나는 등 효율적이지 못하다.
 개발자를 지원하는 IDE를 방해할 이유가 없다.

마치며

좋은 이름을 선택하려면 설명 능력이 뛰어나야 하고 의미가 뚜렷한 어휘인지 분별할 줄 알아야 한다.
다른 개발자가 반대할까 두려워 이름을 바꾸지 않아서는 안된다. 
나름대로 수정했다가 질책을 받더라도 개선하려는 노력을 중단해서는 안된다. 
그래야 자연스럽게 읽히는 코드를 짜는 데에 더 집중할 수 있다. 
단기적인 효과는 물론 장기적인 이익도 보장한다.

참고문헌 - CleanCode 애자일 소프트웨어 장인 정신

multiple 'X-Frame-Options' headers with conflicting 이슈 해결

동일 도메인에서 <iframe>을 통해 접근하는 경우 X-Frame-Options을 DENY로 설정하면 최신(?) 브라우저에서는 접근할 수 없는 문제가 발생할 수 있다. 이 경우 동일 도메인에서는 <iframe> 접근이 가능하도록 X-Frame-Options를 SAMEORIGIN으로 설정해줘야 한다.

<iframe>을 사용하는 부분이 있어 기존에 아래와 같이 SAMEORIGIN으로 설정해서 잘 사용하고 있었다.

http.headers().addHeaderWriter(new XFrameOptionsHeaderWriter(XFrameOptionsHeaderWriter.XFrameOptionsMode.SAMEORIGIN));

그런데 스프링 시큐리티 버전(3.2.3 -> 4.0.4)을 올리면서 동작에 문제가 생겼다. 브라우저에서 아래와 같은 에러가 발생했다.

이슈

[Chrome]
in a frame because it set multiple 'X-Frame-Options' headers with conflicting values ('DENY, SAMEORIGIN'). Falling back to 'deny'

http 응답 헤더를 보니 DENY와 SAMEORIGIN 설정이 중첩되는 문제가 발생했다.
스프링 시큐리티 버전이 올라가면서 default 값인 DENY에 SAMEORIGIN이 추가되어 충돌난 듯하다.

해결방안

내용을 찾아보니 스프링 시큐리티 4.x 부터 설정하는 방식이 변경되었다.

http.headers().frameOptions().sameOrigin();

위와 같이 설정하니 문제없이 잘 동작하는 것을 확인했다.

비동기 메소드 HttpServletRequest 이슈 해결

@Async로 선언된 비동기 메소드를 호출했다. 메소드는 해당 클래스에 @Autowired된 HttpServletRequest로 부터 정보를 가져오다가 SimpleAsyncUncaughtExceptionHandler.handleUncaughtException() 예외가 발생했다.

No thread-bound request found: Are you referring to request attributesoutside of an actual web request, or processing a request outside of theoriginally receiving thread.

원인은 비동기 처리시 다른 스레드에서 동작하기 때문이다.

해결방안

• request로 부터 필요한 정보를 메소드의 파라미터로 전달한다.
• 다음과 같이 RequestContextHolder에서 현재 HttpServletRequest를 가져온다.

// WebApplicationInitializer 설정에 리스너 추가
servletContext.addListener(new RequestContextListener());

ServletRequestAttributes attr = (ServletRequestAttributes) RequestContextHolder.currentRequestAttributes();
HttpServletRequest request = attr.getRequest();

Tomcat Cookie Domain 이슈 해결

기존 서비스를 Tomcat7에서 Tomcat8로 업그레이드 하는 과정에서 문제가 발생했다. 
원인을 찾아보니 서비스 내 쿠키 생성시 서브 도메인에서도 같이 사용 할 수 있도록 .xxx.com 을 도메인으로 생성하고 있었다.
그런데 Tomcat8에서 도메인이 .(dot)으로 시작하는 쿠키를 생성할 수 없도록 변경되었다. (RFC 6265 Cookie Processor 정책)

해결방안

• 쿠키 도메인을 .(dot)으로 시작하지 않도록 변경한다.
• 기존 LegacyCookieProcessor를 사용하도록 context.xml에 아래와 같이 코드를 추가해줘야 한다.

  <CookieProcessor className="org.apache.tomcat.util.http.LegacyCookieProcessor" />

참조 - https://stackoverflow.com/questions/29608550/tomcat-cookie-domain-validation

Spring @Async

• 메소드를 비동기 처리하면 호출자는 호출한 메소드가 완료될 때 까지 기다릴 필요가 없다.
• @Async를 사용하여 별도의 쓰레드에서 실행시킬 것이다.

Async Configuration

// Java Config
@Configuration
@EnableAsync
public class SpringAsyncConfig {
  ...
}

// XML Config
<task:executor id="myexecutor" pool-size="5"/>
<task:annotation-driven executor="myexecutor"/>

@Async

제약사항

• @Async로 명시된 메소드는 반드시 public으로 선언되어야 한다. 메소드가 public 이어야 프록시가 될 수 있기 때문이다.
• 같은 클래스 내에서 해당 메소드를 호출할 경우 비동기로 작동하지 않는다. 셀프호출은 프록시를 우회하고 해당 메소드를 직접 호출하기 때문이다.

void 반환형

@Async
public void asyncMethodWithVoidReturnType() {
    System.out.println("Execute method asynchronously. "
      + Thread.currentThread().getName());
}

Future 반환형

@Async
public Future<String> asyncMethodWithReturnType() {
    System.out.println("Execute method asynchronously - "
      + Thread.currentThread().getName());
    try {
        Thread.sleep(5000);
        return new AsyncResult<String>("hello world !!!!");
    } catch (InterruptedException e) {
        //
    }

    return null;
}

Executor

• 스프링은 기본값으로 SimpleAsyncTaskExecutor를 사용하여 실제 메소드들을 비동기 실행한다. 기본 설정은 메소드/애플리케이션 레벨로 재정의하여 사용할 수 있다.
• 다음과 같이 설정하면 @Async로 지정된 메소드를 실행하는 기본 실행자가 변경된다. 실행자의 상세 옵션들도 변경할 수 있다.

Method Level Override

• 실행자 빈을 설정하고 @Async 속성에 실행자명을 명시해야 한다.

@Configuration
@EnableAsync
public class SpringAsyncConfig {

    @Bean(name = "threadPoolTaskExecutor")
    public Executor threadPoolTaskExecutor() {
        return new ThreadPoolTaskExecutor();
    }
}

@Async("threadPoolTaskExecutor")
public void asyncMethodWithConfiguredExecutor() {
    System.out.println("Execute method with configured executor - "  + Thread.currentThread().getName());
}

Application Level Override

• AsyncConfigurer 인터페이스를 구현하여 getAsyncExecutor() 메소드를 오버라이드해야 한다.

@Configuration
@EnableAsync
public class SpringAsyncConfig implements AsyncConfigurer {

  @Override
  public Executor getAsyncExecutor() {
    ThreadPoolTaskExecutor executor = new ThreadPoolTaskExecutor();
    executor.setCorePoolSize(5);
    executor.setMaxPoolSize(100);
    executor.setQueueCapacity(100);
    executor.setThreadNamePrefix("TEST_");
    executor.initialize();
    return executor;
  }

}

Exception Handling

• 반환형이 void일 때, 예외는 호출 스레드에 전달되지 않는다. 예외 처리를 위한 설정이 추가적으로 필요하다.

@Configuration
@EnableAsync
public class SpringAsyncConfig implements AsyncConfigurer {

  @Override
  public Executor getAsyncExecutor() {
    ThreadPoolTaskExecutor executor = new ThreadPoolTaskExecutor();
    executor.setCorePoolSize(5);
    executor.setMaxPoolSize(100);
    executor.setQueueCapacity(100);
    executor.setThreadNamePrefix("TEST_");
    executor.initialize();
    return executor;
  }

  @Override
  public AsyncUncaughtExceptionHandler getAsyncUncaughtExceptionHandler() {
    return new SimpleAsyncUncaughtExceptionHandler();
    // Custom 가능
    // return new CustomAsyncExceptionHandler();
  }

}

• AsyncUncaughtExceptionHandler 인터페이스를 구현하여 커스텀 비동기 예외처리자를 만들 수도 있다. handleUncaughtException() 메소드는 잡히지 않은 비동기 예외가 발생할때 호출된다.

public class CustomAsyncExceptionHandler implements AsyncUncaughtExceptionHandler {

    @Override
    public void handleUncaughtException(Throwable throwable, Method method, Object... obj) {
        System.out.println("Exception message : " + throwable.getMessage());
        System.out.println("Method name : " + method.getName());
        for (Object param : obj) {
            System.out.println("Parameter value : " + param);
        }
    }

}

Etc

• Spring Boot 기반의 웹 애플리케이션은 HTTP 요청이 들어왔을 때 내장된 서블릿 컨테이너가 관리하는 독립적인 1개의 Worker 쓰레드에 의해 동기 방식으로 실행된다. 하지만 요청 처리 중 @Async가 명시된 메소드를 호출하면 앞서 등록한ThreadPoolTaskExecutor 빈에 의해 관리되는 또 다른 독립적인 Worker 쓰레드로실행된다. 별도의 쓰레드로 동작하기 때에 원래의 요청 쓰레드는 그대로 다음 문장을 실행하여 HTTP 응답을 마칠 수 있다.

• Async 어노테이션을 사용하는 순간 서로 다른 Thread에서 동작하기 때문에 앞에서 생성한 Transaction이 연장되지 않으므로 주의해야 한다.

레퍼런스 - http://www.baeldung.com/spring-async

@Table(name = "t_user", indexes = { @Index(name = "idx_u_username", columnList = "username") }, 
uniqueConstraints = @UniqueConstraint(name="uk_user", columnNames = {"email", "username" }))

JPA 기본 환경설정

@Configuration
public class DataConfig {

    @Autowired
    private DatabaseProperties databaseProperties;

    @Bean
    public DataSource dataSoure(){
        DriverManagerDataSource dataSource = new DriverManagerDataSource();
        dataSource.setDriverClassName(databaseProperties.getDriver());
        dataSource.setUrl(databaseProperties.getUrl());
        dataSource.setUsername(databaseProperties.getUsername());
        dataSource.setPassword(databaseProperties.getPassword());
        return dataSource;
    }

    @Bean
    public LocalContainerEntityManagerFactoryBean entityManagerFactory(){

        LocalContainerEntityManagerFactoryBean factoryBean = new LocalContainerEntityManagerFactoryBean();
        factoryBean.setDataSource(dataSoure());
        factoryBean.setPackagesToScan("com.example.model");
        factoryBean.setJpaVendorAdapter(new HibernateJpaVendorAdapter());

        // JPA 상세설정
        Properties jpaProperties = new Properties();
        jpaProperties.put(AvailableSettings.SHOW_SQL, true); // SQL 보기
        jpaProperties.put(AvailableSettings.FORMAT_SQL, true); // SQL 포맷팅하기
        jpaProperties.put(AvailableSettings.USE_SQL_COMMENTS, true); // SQL 코멘트 보기
        jpaProperties.put(AvailableSettings.HBM2DDL_AUTO, databaseProperties.getHbm2ddlAuto()); // DDL 자동생성 종류
        jpaProperties.put(AvailableSettings.DIALECT, databaseProperties.getDialect()); // 방언 설정
        jpaProperties.put(AvailableSettings.USE_NEW_ID_GENERATOR_MAPPINGS, true); // 키 생성 전략 사용시 설정
        jpaProperties.put("hibernate.ejb.naming_strategy", "org.hibernate.cfg.ImprovedNamingStrategy"); // 자바의 카멜 표기법을 테이블의 언더스코어 표기법으로 매핑
        factoryBean.setJpaProperties(jpaProperties);

        return factoryBean;
    }
}

Sequence 테이블 생성 DDL

CREATE TABLE MY_SEQUENCE{
  SEQUENCE_NAME VARCHAR(255) NOT NULL,
  NEXT_VAL BIGINT,
  PRIMARY KEY (SEQUENCE_NAME)
}

TABLE 전략 매핑 코드

@Entity
@TableGenerator(
 name = "BOARD_SEQ_GENERATOR",
 table = "MY_SEQUENCE",
 pkColumnValue = "BOARD_SEQ",
 allocationSize = 1)
public class Board {

    @Id
    @GeneratedValue(strategy = GenerationType.TABLE, generator = "BOARD_SEQ_GENERATOR")
    private Long id;
}

@Entity
public class Member {

    @Id
    private String id;

    @Transient
    private String firstName;

    @Transient
    private String lastName;

    @Access(AccessType.PROPERTY)
    public String getFullName(){
        return this.firstName + this.lastName;
    }
 }
// @Id가 필드에 있으므로 필드 접근 방식을 사용하고 getFullName만 프로퍼티를 사용한다.
// Member 엔티티에 fullName 컬럼이 생성되고 firstName + lastName의 결과가 저장된다.

참고서적 - 자바 ORM 표준 JPA 프로그래밍 (김영한)

AES256 암호화시 java.security.InvalidKeyException: Illegal key size 해결 방안

java.security.InvalidKeyException: Illegal key size

JDK 1.8u151 버전? 부터 방법 변경