Skip to content
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.

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

[로운] 4단계 - HTTP 웹 서버 구현 미션 제출합니다. #200

Open
wants to merge 3 commits into
base: lowoon
Choose a base branch
from

Conversation

lowoon
Copy link

@lowoon lowoon commented Nov 16, 2020

안녕하세요! 4단계 제출합니다!
미션 진행하면서 static 파일들을 core에 두는게 맞는지 궁금했고, was같은 경우는 api를 바라보는 module을 한가지 더 만들어서 모아두는 것이 맞는지 궁금했습니다!
감사합니다!

@lowoon lowoon changed the title Lowoon [로운] 4단계 - HTTP 웹 서버 리팩토링 미션 제출합니다. Nov 16, 2020
Copy link

@jeonghoon1107 jeonghoon1107 left a comment

Choose a reason for hiding this comment

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

안녕하세요 로운!
간단한 피드백을 남겼으니 확인해주세요.

implementation project(':module-common')

implementation 'org.springframework:spring-core:5.2.6.RELEASE'
implementation 'com.github.jknack:handlebars:4.2.0'

Choose a reason for hiding this comment

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

gradle.properties를 활용해보면 좋을 것 같아요.

Copy link
Author

Choose a reason for hiding this comment

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

제가 찾은 방법은 버전관리를 gradle.properties로 하는 것이었는데 혹시 다르게 활용할 수 있는 부분들이 있을까요?

@@ -1,4 +1,4 @@
package domain.user.model;
package user;

public class User {
Copy link

@jeonghoon1107 jeonghoon1107 Nov 20, 2020

Choose a reason for hiding this comment

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

User와 Session, Cookie 등이 common 에 같이 있는 모습인데 애플리케이션에서 필요한 것들은 api 모듈로 웹 서버모듈은 프레임워크 모듈로 기준을 잡아 진행해보면 어떨까요?

간단한 기준을 드리면 작성한 웹서버 모듈을 이용해 하나의 애플리케이션을 만드는 구조로 가면 어떨까요?
간단한 예로 우리가 스프링 프레임워크를 사용한다고 해서 스프링 프레임워크에 우리가 서비스하고 싶은 user 등의 코드를 넣을 수 없듯이 웹서버 모듈은 애플리케이션을 개발하기 위한 코어의 역할로만 구분하면 좋을 것 같아요.

Copy link
Author

@lowoon lowoon Nov 25, 2020

Choose a reason for hiding this comment

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

applicationframework로 분리하였습니다.
그렇게 하다보니 webserver에서 Controller를 알게 되면 순환구조가 생기게 되어서 Reflection을 활용하도록 수정하였습니다.
현재는 Controller만 찾아서 등록을 하고 있어서 constructor가 2개씩 있는데 ServiceBean으로 등록하고 DI해줄 수 있도록 계속해서 수정하겠습니다.

@lowoon lowoon changed the title [로운] 4단계 - HTTP 웹 서버 리팩토링 미션 제출합니다. [로운] 4단계 - HTTP 웹 서버 구현 미션 제출합니다. Nov 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants