devsam/core
3
6
Fork 1
Code Issues 60 Pull Requests Releases 1 Wiki Activity

DTO 도입 #202

New Issue
Open
opened 2021-12-20 15:41:41 +09:00 by Hide_D · 0 comments
Hide_D commented 2021-12-20 15:41:41 +09:00
Owner
Copy Link
Copy Source

API로 왔다갔다 하는 데이터들이 PHP 단에서 연관 배열로 오고가는데, 키/값이 제대로 설정 되어있는지 제대로 확인되질 않음.

그나마 API의 입력단에서는 Valitron으로 검사하고 있지만, 이것도 API를 만들때마다 수동으로 입력해야 하고, 별로 예쁜 해결책이 아님.

클라이언트 단에서는 TypeScript로 관리하고 있어서 그나마 나은편.

서버 단에서 적어도 통신할때만큼은 DTO를 도입해서 관리하면 나을 것으로 판단...

  • 도입시 고려해야할 것
    • JSON5 호환
    • null checking등 타입 검사를 포함해야함
    • 일관된 ser/des
    • VSCode에서 변수명 추적이 쉽게
    • 정적 분석기에서 경고를 받지 않아야함
  • 여전히 고민해야 할 것
    • KVStorage의 Value역시 JSON인데, 이부분은 어떻게..?
    • General/Nation Table의 Aux column이 기준이 없는데 이부분은 어떻게..? Partial?
API로 왔다갔다 하는 데이터들이 PHP 단에서 연관 배열로 오고가는데, 키/값이 제대로 설정 되어있는지 제대로 확인되질 않음. 그나마 API의 입력단에서는 Valitron으로 검사하고 있지만, 이것도 API를 만들때마다 수동으로 입력해야 하고, 별로 예쁜 해결책이 아님. 클라이언트 단에서는 TypeScript로 관리하고 있어서 그나마 나은편. 서버 단에서 적어도 통신할때만큼은 DTO를 도입해서 관리하면 나을 것으로 판단... - 도입시 고려해야할 것 - [ ] JSON5 호환 - [ ] null checking등 타입 검사를 포함해야함 - [ ] 일관된 ser/des - [ ] VSCode에서 변수명 추적이 쉽게 - [ ] 정적 분석기에서 경고를 받지 않아야함 - 여전히 고민해야 할 것 - KVStorage의 Value역시 JSON인데, 이부분은 어떻게..? - General/Nation Table의 Aux column이 기준이 없는데 이부분은 어떻게..? Partial?
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
UI,UX,Web
게임 룰 변경
게임 엔진 개선
도움 요청
버그
예정 없음
중복
질문
토론
해당 없음
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
62che (유기체) Hide_D (Hide_D) revofpla sars
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: devsam/core#202