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

[박승훈] 챕터 5: 최신 자바스크립트 문법과 기능 #23

Merged
merged 2 commits into from
Oct 24, 2024

Conversation

Orchemi
Copy link
Contributor

@Orchemi Orchemi commented Oct 18, 2024

No description provided.

Copy link
Member

@hyesungoh hyesungoh left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 👍


> 나의 경험

성능 최적화를 위해 dynamic import를 사용해본 경험이 있어요. 그런데 신규 프로젝트 개발 초기여서 path가 계속 변하는데 dynamic import path는 자동으로 변경되지 않아서 수동으로 path를 변경해주어야 했어요. 이 부분이 조금 불편했던 것 같아요. 하지만 code splitting 학습 할 때 초기 화면에서 필욯나 모듈만을 분리해서 가져오는 것의 이점이 드라마틱하다는 것을 알고 있었기 때문에 최대한 사용을 유지하려고 했어요. 모든 파일을 그렇게 할 필요는 없지만 유지보수에 유리한 선을 잘 정해서 적용해보면 좋을 것 같아요.
Copy link
Member

Choose a reason for hiding this comment

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

오타를 찾앗읍니다 필욯나

Suggested change
성능 최적화를 위해 dynamic import를 사용해본 경험이 있어요. 그런데 신규 프로젝트 개발 초기여서 path가 계속 변하는데 dynamic import path는 자동으로 변경되지 않아서 수동으로 path를 변경해주어야 했어요. 이 부분이 조금 불편했던 것 같아요. 하지만 code splitting 학습 할 때 초기 화면에서 필욯나 모듈만을 분리해서 가져오는 것의 이점이 드라마틱하다는 것을 알고 있었기 때문에 최대한 사용을 유지하려고 했어요. 모든 파일을 그렇게 할 필요는 없지만 유지보수에 유리한 선을 잘 정해서 적용해보면 좋을 것 같아요.
성능 최적화를 위해 dynamic import를 사용해본 경험이 있어요. 그런데 신규 프로젝트 개발 초기여서 path가 계속 변하는데 dynamic import path는 자동으로 변경되지 않아서 수동으로 path를 변경해주어야 했어요. 이 부분이 조금 불편했던 것 같아요. 하지만 code splitting 학습 할 때 초기 화면에서 필요한 모듈만을 분리해서 가져오는 것의 이점이 드라마틱하다는 것을 알고 있었기 때문에 최대한 사용을 유지하려고 했어요. 모든 파일을 그렇게 할 필요는 없지만 유지보수에 유리한 선을 잘 정해서 적용해보면 좋을 것 같아요.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

꼼꼼히 읽어주셨군요. 감사합니다..!

Copy link
Contributor

@sangbooom sangbooom left a comment

Choose a reason for hiding this comment

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

잘 읽었습니다! 💪🏻

@Orchemi Orchemi merged commit 4c2637a into main Oct 24, 2024
1 check passed
@Orchemi Orchemi deleted the 5장/박승훈 branch October 24, 2024 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants