인프런 커뮤니티 질문&답변

/**
 * 세션 관리 매니저
 */
@Component
public class SessionManager {
    public static final String SESSION_COOKIE_NAME = "mySessionId";

    private final Map<String, Object> sessionStore = new ConcurrentHashMap<>();

    /**
     * 세션 생성
     *
     * @param value 저장할 값
     * @param resp  HTTP Servlet Response
     */
    public void createSession(
            Object value,
            HttpServletResponse resp
    ) {
        // 세션 ID를 생성하고, 값을 세션에 저장
        String sessionId = UUID.randomUUID().toString();
        sessionStore.put(sessionId, value);

        // 쿠키 생성
        Cookie mySessionCookie = new Cookie(SESSION_COOKIE_NAME, sessionId);
        resp.addCookie(mySessionCookie);
    }

    /**
     * 세션 조회
     *
     * @param req HTTP Servlet Request
     * @return 조회된 세션에 대한 정보(값)
     */
    public Object getSession(
            HttpServletRequest req
    ) {
        return findCookie(req, SESSION_COOKIE_NAME)
                .map(value -> sessionStore.get(value.getValue()))
                .orElse(null);
    }

    /**
     * 세션 만료
     *
     * @param req HTTP Servlet Request
     */
    public void expire(
            HttpServletRequest req
    ) {
        findCookie(req, SESSION_COOKIE_NAME)
                .ifPresent(cookie -> sessionStore.remove(cookie.getValue()));
    }

    /**
     * 쿠키 찾기
     *
     * @param req        HTTP Servlet Request
     * @param cookieName 찾을 쿠키 이름
     * @return 찾은 쿠키
     */
    private Optional<Cookie> findCookie(
            HttpServletRequest req,
            String cookieName
    ) {
        if (req.getCookies() == null) {
            return Optional.empty();
        }

        return Arrays.stream(req.getCookies())
                .filter(cookie -> cookie.getName().equals(cookieName))
                .findAny();
    }
}

지난 강의 시간에 Optional에 대해서 말씀하셨길래 SessionManager에 한 번 적용해 보았습니다.

getSession 같이 외부로 나가는 것이 아닌, 클래스 내부에서만 사용할 수 있는 findCookie에 Optional을 사용하는게 적절한지 의문이 들어 질문남겨봅니다.

Optional을 찾아보니, null을 반환할 수 있는 함수에 대해 사용할 수 있으며, 파라미터보다 반환값으로 사용하는게 맞다고 얘기를 하더군요.

그럼 반환값에 사용하면 위와 같이 ifPresent나 orElse를 이용해 실제 객체 값을 들고 와야하기도 해서, 지금처럼 간단한 로직에서는 굳이 Optional이 아닌, Cookie를 반환해서 if문을 이용해 null체크를 하는게 맞다고 생각은 합니다만..

findCookie는 클래스내부에서 사용하는게 적절하지 않다면, getSession같이 클래스 외부에서 사용할 수 있는 메서드에 대해 반환값을 Optional로 하는 것은 적절한 사용일까요?