inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

mongoDB 기초부터 실무까지(feat. Node.js)

Client 코드 리팩토링 & 마무리

client code refactoring 질문

270

신론

작성한 질문수 3

1

client code에서 리팩토링 파트에서 보니 좀 오히려 프로덕션 레벨 및 좀더 깔끔 한 코드로 바꾸실려고 하는데 오히려 너무 복잡했습니다.  model schema를 만들때 보면, 지금 user schema에는 관게를 안만들고 오히려 blog 와 comment 에만 만드셨는데 좀 쉽게 오히려 user schema에 

comments: [commentSchema],  를 만들어줘도 되지 않았나요?  그리고 Types.objectId만 넣지말고 모든 데이타를 ref 해도 되지 않았을 까요?  아님 필요 한것만이라도요.  지금 comment schema에 user 와 blog는 Types.ObjectId만 있으니 너무 복잡하게 돌아 가는 것 같아  좀 헷갈렸습니다.  

model DBMS/RDBMS schema mongodb mongoose nodejs aws rest-api javascript 데이터 엔지니어링

답변 1

0

김시훈 (도도소프트)

신론님 안녕하세요 :)

신론님 말씀대로 이 부분이 과하게 복잡한게 맞아요! 제일 안좋은 상황을 보여드린거거든요. 여기서 불편함을 많이 느낄수록 좋습니다! 그러면 뒤에서 배우는 내용들이 훨씬 더 잘 와닿으실거에요.

지금즘 보셨텐데 이후에 populate(ref)를 사용해서 코드가 많이 간결해집니다. 그리고 comments:[commentSchema] 이건 내장(nesting)을 말씀하시는듯 한데 이 부분도 바로 이어서 다루어집니다!

다만, 지금 개발하는 API 용도에 맞게 user schema에 comments가 내장되지는 않고 blog schema에 comments가 내장됩니다. 특정 유저가 작성한 모든 comment를 보고 싶은 용도가 있다면 이 경우도 내장을 고려해볼 수는 있는데요. 이런 용도가 있다고 해도 comments를 user에 모두 내장하는건 적합하지 않을 수 있어요. Nesting, Referencing을 언제하면 좋은지도 나중에 다룹니다!

option usecreateindex is not supported 오류 나면은

0

147

1

Schemaless 특성을 재생시 검은화면이 나와요

0

241

1

user 내장 시 ref 필드 필요성

0

240

1

await 관련해서 질문드릴게 있습니다!!

0

293

2

섹션6의 3번째 강의 faker파일이 어디있나요?

0

309

1

$elemMatch 질문이 있습니다.

0

248

1

DB 업데이트에 실패했을 상황에서 대처는 어떻게 할 수 있나요.

2

767

2

블로그에 user 데이터를 추가하는 방법에 대해서

0

238

1

blog 데이터 생성시 user 관련 오류

0

352

1

age 예외 처리를 별도로 하신 이유가 궁금합니다.

2

302

2

delete 메소드가 많은 이유가 궁금합니다.

0

182

1

unique validation 처리

0

297

1

updateMany에서 user._id를 못찾는 상황

1

397

1

find() 가 리턴하는 instance member

1

432

3

몽고 db를 백업방법을 부탁합니다.

0

604

1

인덱스 생성시점

0

713

1

$pull 문법에 대한 질문

0

360

1

comment의 id가 생성되는 시점

1

303

1

리팩토링 강의에서 라우터 리팩토링 전개연산자(...)

1

316

1

session.abortTransaction()에 대한 실제 예시 문의

0

289

1

섹션9 Computed Fields 강의에서, 다음 강의 가기 전 빠진부분

-1

411

1

faker 패키지 취약성 이슈

1

488

1

라우터 리팩토링

0

345

1

섹션4 진행중, mongoose 질문

0

312

1