Skip to content

[Config-DI] 싱글톤을 보장하는 싱글톤 컨테이너 개발 #45

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
codingbaraGo merged 2 commits into from
Jan 11, 2026
Merged

[Config-DI] 싱글톤을 보장하는 싱글톤 컨테이너 개발 #45

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9c9b668
Merge pull request #43 from codingbaraGo/feat/http/cookie/#41
codingbaraGo Jan 9, 2026
422f02d
refactor(config-di): AppConfig에서 함수 호출시 싱글톤 보장
codingbaraGo Jan 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
211 changes: 143 additions & 68 deletions src/main/java/config/AppConfig.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,127 +11,202 @@
import http.request.InputStreamHttpRequestConverter;
import http.response.HttpResponseBufferedStreamConverter;
import http.response.HttpResponseConverter;
import web.dispatch.argument.ArgumentResolver;
import web.dispatch.Dispatcher;
import web.dispatch.HandlerAdapter;
import web.dispatch.adapter.DefaultHandlerAdapter;
import web.dispatch.adapter.SingleArgHandlerAdapter;
import web.dispatch.argument.ArgumentResolver;
import web.dispatch.argument.resolver.HttpRequestResolver;
import web.dispatch.argument.resolver.QueryParamsResolver;
import web.handler.StaticContentHandler;
import web.handler.WebHandler;
import web.renderer.StaticViewRenderer;
import web.renderer.HttpResponseRenderer;
import web.renderer.StaticViewRenderer;

import java.util.List;

public class AppConfig {
//Http
public HttpRequestConverter httpRequestConverter(){
public class AppConfig extends SingletonContainer {
Comment on lines 23 to +28
Copy link

@github-actions github-actions bot Jan 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

기존 동작과 달라진 점: httpRequestConverter()와 httpResponseConverter()는 단순히 다른 메서드를 호출하기만 하므로, getOrCreate로 감싸지 않았습니다. 이로 인해 매번 같은 객체를 반환하지만 리팩토링 의도와 불일치합니다.

일관성을 위해 명확히 하세요:

  • 옵션 1: 이 두 메서드도 getOrCreate로 감싸기 (싱글톤 명시적 보장)
  • 옵션 2: 이름 변경 또는 문서화: "이 메서드들은 캐시되지 않고 delegation만 수행" 표시


/**
* ===== Http =====
*/
public HttpRequestConverter httpRequestConverter() {
return inputStreamHttpRequestConverter();
}
public HttpResponseConverter httpResponseConverter(){

public HttpResponseConverter httpResponseConverter() {
return httpResponseBufferedStreamConverter();
}

public BufferedReaderHttpRequestConverter httpBufferedReaderRequestConverter(){
return new BufferedReaderHttpRequestConverter();
public BufferedReaderHttpRequestConverter httpBufferedReaderRequestConverter() {
return getOrCreate(
"httpBufferedReaderRequestConverter",
BufferedReaderHttpRequestConverter::new
);
}

public HttpResponseBufferedStreamConverter httpResponseBufferedStreamConverter(){
return new HttpResponseBufferedStreamConverter();
}
public InputStreamHttpRequestConverter inputStreamHttpRequestConverter(){
return new InputStreamHttpRequestConverter();
public HttpResponseBufferedStreamConverter httpResponseBufferedStreamConverter() {
return getOrCreate(
"httpResponseBufferedStreamConverter",
HttpResponseBufferedStreamConverter::new
);
}


//Web
public Dispatcher dispatcher(){
return new Dispatcher(
webHandlerList(),
handlerAdapterList(),
webHandlerResponseHandlerList()
public InputStreamHttpRequestConverter inputStreamHttpRequestConverter() {
return getOrCreate(
"inputStreamHttpRequestConverter",
InputStreamHttpRequestConverter::new
);
}

private List<WebHandler> webHandlerList(){
return List.of(
staticContentHandler(),
registerWithGet(),
registerWithPost()
/**
* ===== Web =====
*/
public Dispatcher dispatcher() {
return getOrCreate(
"dispatcher",
() -> new Dispatcher(
webHandlerList(),
handlerAdapterList(),
webHandlerResponseHandlerList()
Comment on lines 61 to +71
Copy link

@github-actions github-actions bot Jan 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

컬렉션 반환 메서드의 싱글톤 캐싱 주의: webHandlerList() 같은 컬렉션을 반환하는 메서드를 getOrCreate로 캐싱하면, 반환된 List가 변경되지 않는다고 가정합니다.

만약 런타임에 이 리스트를 수정하면 (예: add(), remove()) 싱글톤 인스턴스 자체가 변경됩니다. 이를 방지하려면:

  1. 불변 리스트 반환: Collections.unmodifiableList(...)
  2. 복사본 반환: 매번 새로운 컬렉션 인스턴스 생성
  3. 내부 리스트 접근 금지 (private 필드화)

)
);
}
private RegisterWithGet registerWithGet(){
return new RegisterWithGet();

public List<WebHandler> webHandlerList() {
return getOrCreate(
"webHandlerList",
() -> List.of(
staticContentHandler(),
registerWithGet(),
registerWithPost()
)
);
}
private RegisterWithPost registerWithPost(){
return new RegisterWithPost();

public RegisterWithGet registerWithGet() {
return getOrCreate(
"registerWithGet",
RegisterWithGet::new
);
Comment on lines +78 to +91
Copy link

@github-actions github-actions bot Jan 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

메서드 가시성 변경 (private → public)의 영향: 기존 private 메서드들이 모두 public으로 변경되어, 외부에서 직접 호출할 수 있게 되었습니다.

예를 들어, registerWithGet()을 외부에서 두 번 호출하면:

AppConfig config = new AppConfig();
RegisterWithGet h1 = config.registerWithGet();  // 첫 호출: 캐시 저장
RegisterWithGet h2 = config.registerWithGet();  // 두 번째: 동일 인스턴스 반환

이는 싱글톤 보장 측면에서는 긍정적이지만, 캡슐화를 깨뜨립니다. 필요한 진입점만 public으로 유지하고 나머지는 protected 또는 패키지 private으로 제한하는 것을 검토하세요.

}

private List<HttpResponseRenderer> webHandlerResponseHandlerList(){
return List.of(
staticViewResponseHandler()
public RegisterWithPost registerWithPost() {
return getOrCreate(
"registerWithPost",
RegisterWithPost::new
);
}
private StaticContentHandler staticContentHandler(){
return new StaticContentHandler();

public List<HttpResponseRenderer> webHandlerResponseHandlerList() {
return getOrCreate(
"webHandlerResponseHandlerList",
() -> List.of(
staticViewResponseHandler()
)
);
}
private StaticViewRenderer staticViewResponseHandler(){
return new StaticViewRenderer();

public StaticContentHandler staticContentHandler() {
return getOrCreate(
"staticContentHandler",
StaticContentHandler::new
);
}

public StaticViewRenderer staticViewResponseHandler() {
return getOrCreate(
"staticViewResponseHandler",
StaticViewRenderer::new
);
}

//Adapter
public List<HandlerAdapter> handlerAdapterList(){
return List.of(
singleArgHandlerAdapter(),
defaultHandlerAdapter()
// ===== Adapter =====
public List<HandlerAdapter> handlerAdapterList() {
return getOrCreate(
"handlerAdapterList",
() -> List.of(
singleArgHandlerAdapter(),
defaultHandlerAdapter()
)
);
}

private SingleArgHandlerAdapter singleArgHandlerAdapter(){
return new SingleArgHandlerAdapter(
argumentResolverList()
public SingleArgHandlerAdapter singleArgHandlerAdapter() {
return getOrCreate(
"singleArgHandlerAdapter",
() -> new SingleArgHandlerAdapter(
argumentResolverList()
)
);
}
private DefaultHandlerAdapter defaultHandlerAdapter(){
return new DefaultHandlerAdapter();

public DefaultHandlerAdapter defaultHandlerAdapter() {
return getOrCreate(
Comment on lines +128 to +145
Copy link

@github-actions github-actions bot Jan 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

싱글톤 의존성 그래프 복잡성: 예를 들어, exceptionHandlerMapping()은 serviceExceptionHandler(), errorExceptionHandler(), unhandledErrorHandler()에 의존하는데, 이들이 모두 싱글톤으로 캐싱되므로 의존성 순서가 중요합니다.

현재는 서버 구동 시점에만 로드되므로 문제 없지만, 향후 런타임 재로드나 부분 초기화가 필요하면 의존성 추적이 어려워집니다. 필요시 의존성 명시를 위해 생성자 주입 패턴으로 마이그레이션을 고려하세요.

"defaultHandlerAdapter",
DefaultHandlerAdapter::new
);
}

//Resolver
public List<ArgumentResolver<?>> argumentResolverList(){
return List.of(
httpRequestResolver(),
queryParamsResolver()
// ===== Resolver =====
public List<ArgumentResolver<?>> argumentResolverList() {
return getOrCreate(
"argumentResolverList",
() -> List.of(
httpRequestResolver(),
queryParamsResolver()
)
);
}

private HttpRequestResolver httpRequestResolver(){
return new HttpRequestResolver();
public HttpRequestResolver httpRequestResolver() {
return getOrCreate(
"httpRequestResolver",
HttpRequestResolver::new
);
}
private QueryParamsResolver queryParamsResolver(){
return new QueryParamsResolver();

public QueryParamsResolver queryParamsResolver() {
return getOrCreate(
"queryParamsResolver",
QueryParamsResolver::new
);
}

//Exception
public ExceptionHandlerMapping exceptionHandlerMapping(){
return new ExceptionHandlerMapping(
List.of(
serviceExceptionHandler(),
errorExceptionHandler(),
unhandledErrorHandler()
/**
* ===== Exception =====
*/
public ExceptionHandlerMapping exceptionHandlerMapping() {
return getOrCreate(
"exceptionHandlerMapping",
() -> new ExceptionHandlerMapping(
List.of(
serviceExceptionHandler(),
errorExceptionHandler(),
unhandledErrorHandler()
)
)
);
}

private ServiceExceptionHandler serviceExceptionHandler(){
return new ServiceExceptionHandler();
public ServiceExceptionHandler serviceExceptionHandler() {
return getOrCreate(
"serviceExceptionHandler",
ServiceExceptionHandler::new
);
}
private UnhandledErrorHandler unhandledErrorHandler(){
return new UnhandledErrorHandler();

public UnhandledErrorHandler unhandledErrorHandler() {
return getOrCreate(
"unhandledErrorHandler",
UnhandledErrorHandler::new
);
}
private ErrorExceptionHandler errorExceptionHandler(){
return new ErrorExceptionHandler();

public ErrorExceptionHandler errorExceptionHandler() {
return getOrCreate(
"errorExceptionHandler",
ErrorExceptionHandler::new
);
}
}
21 changes: 21 additions & 0 deletions src/main/java/config/SingletonContainer.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
package config;

import java.util.HashMap;
import java.util.Map;
import java.util.function.Supplier;

public abstract class SingletonContainer {
Copy link

@github-actions github-actions bot Jan 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

정적 맵을 abstract 클래스에서 사용하면 상속 계층 구조 전체에서 공유된다는 점에 주의: 현재 설계에서는 단일 AppConfig만 있으므로 문제가 없지만, 향후 SingletonContainer를 상속받는 다른 설정 클래스가 생기면 모든 싱글톤이 같은 맵에 저장됩니다.

향후 확장성을 위해 다음 중 하나를 고려하세요:

  1. 인스턴스 필드로 변경: 각 설정 클래스가 독립적인 맵을 유지
  2. 생성자 매개변수화: 맵 이름이나 식별자를 받아 구분
  3. AppConfig 전용 구현: SingletonContainer를 상속받지 말고, AppConfig에 직접 구현하여 현재 요구사항에 맞춤

private static final Map<String, Object> singletonMap = new HashMap<>();

@SuppressWarnings("unchecked")
Copy link

@github-actions github-actions bot Jan 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

형변환 경고 억제(@SuppressWarnings)는 타입 안전성을 가릴 수 있습니다: 문자열 기반 key로 인해 런타임 타입 불일치가 발생할 수 있습니다.

예: getOrCreate("httpRequestResolver", SomeOtherClass::new)라고 잘못 호출하면 나중에 캐스트 예외가 발생합니다.

개선 방안: Map<String, Class<?>> 또는 더 타입-안전한 구조를 사용하거나, 최소한 저장된 타입과 요청된 타입이 일치하는지 검증하는 로직을 추가하세요.

public <T> T getOrCreate(String name, Supplier<T> factory) {
Object instance = singletonMap.get(name);
if (instance != null) {
return (T) instance;
}
T created = factory.get();
singletonMap.put(name, created);
return created;
}

}