kkn1125 / kkn1125/mkDocumentifyJS

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

Open
#90 0 comments 1 reaction 2 assignees Claimed by @ohoraming View on GitHub
bug review
Dominant language
JavaScript
Stars
2
Forks
1
PR merge metrics
No merged PRs in 30d

Description

# Review

## Data 가공 단계

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

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

## rendering 문제
- `member`와 `method`를 후처리하는데서 오는 부담이 커짐

## parsing 단계 중의 문제
- `data`를 가공하면서 최종 단계에서 `object`가 아닌 `string`으로 `parsing`하면서 `data`를 사용하기 어렵게 만들어버림
- 렌더링 단계에서 `trim`과 `undefined`처리를 해버림

-----

## 해결책

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

3. Render
- `member`와 `method`를 각각 처리해야 함

5. Parsing
- 전처리 단계에서 후처리 전까지 필요한 작업을 미리 끝내야 함

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.