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

rendering 단계에서 문제에 봉착함 - 문제점 review #90

Open
ohoraming opened this issue Apr 18, 2022 · 0 comments
Open

rendering 단계에서 문제에 봉착함 - 문제점 review #90

ohoraming opened this issue Apr 18, 2022 · 0 comments
Assignees
Labels
bug Something isn't working review

Comments

@ohoraming
Copy link
Collaborator

ohoraming commented Apr 18, 2022

Review

Data 가공 단계

프로젝트 초반에 정규표현식을 잘못 짜서 생기는 문제

  • 정규표현식의 통일성 부재 (가독성 감소 문제)
  • tag명, 속성값 그룹마다 trim()(공백 제거)을 했어야 했음
  • undifined, null, 공백, false 등의 예외 처리를 생각했어야 했음
    • function의 경우, undifinedanonymous로 처리하기

rendering 문제

  • membermethod를 후처리하는데서 오는 부담이 커짐

parsing 단계 중의 문제

  • data를 가공하면서 최종 단계에서 object가 아닌 string으로 parsing하면서 data를 사용하기 어렵게 만들어버림
  • 렌더링 단계에서 trimundefined처리를 해버림

해결책

  1. Data 가공

    • 정규표현식으로 표현할 수 있는 것과 javascript 로직을 이용해야하는 부분을 나누어 생각해야 함
    • 정규식으로는 태그와 태그명은 있으나 내용은 있거나 없는 것을 거른 뒤,
      나머지를 js로 처리하기로 함
  2. Render

    • membermethod를 각각 처리해야 함
  3. Parsing

    • 전처리 단계에서 후처리 전까지 필요한 작업을 미리 끝내야 함
@kkn1125 kkn1125 self-assigned this Apr 18, 2022
@ohoraming ohoraming self-assigned this Apr 18, 2022
@kkn1125 kkn1125 added bug Something isn't working review labels Apr 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working review
Projects
None yet
Development

No branches or pull requests

2 participants