c++ 코드리뷰? 질문입니다

Question

안녕하세요 c&#43;&#43;을 공부하고 있는 고2입니다. 원래 알고리즘 파다가 템플릿이랑 클래스 써보다가 모던 c&#43;&#43;에 관심이 생겨서 공부하면서 만들어보고 있는데 제가 실수한 거라던가, 안좋은 습관, 성능을 하락시키는 코드, 등이 있는지 있다면 무엇인지 궁금해서 질문합니다.https://github.com/jonghyuk0/HeapHeader ↑힙을 구현한 클래스 입니다. 아마 클래스 기본기능과 템플릿 알아보려 만든거라 크게 문제가 있을 것 같진 않네요.https://github.com/jonghyuk0/SentenceHeader↑문자열을 구현한 클래스입니다. 아마 이코드에서 문제가 좀 있을거 같은데 우측값 레퍼런스등등을 공부하며 만들었습니다.우측값 레퍼런스는 아직도 이해가 어렵네요 ㅜ.위 코드에서 안전하지 않은 부분이나 안좋은 코드등이 있다면 알려주시면 감사하겠습니다.

스토리 · Answer

밑에 코드만 봐서 제 기준으로 꼽자면.sentence 클래스라는 이름임에도, 굳이 멤버변수에 sentence_라고 붙일 이유는 없어 보입니다.explicit를 붙이는 이유도 궁금합니다.this->를 쓴 곳도 있고 안쓴 곳도 있는 곳,memory_capacity 혹은 길이가 음수가 될 일이 없다면 자료형의 부호 타입 지정도 필요해 보입니다get_len()과 length()가 어떤 일을 하는 지 모호합니다.operator[]의 버퍼 언더/오버플로우 처리를 하는 부분이 없습니다.우측값으로 복사할 때 기존 값에 대한 메모리 해제 부분이 없습니다. 헤더 가드로 iostream의 중복 참조를 막았지만, 필요하다면 파일 분리(cpp)를 하는 게 맞을 수도 있습니다.operator 처리도 위험해 보입니다. sen은 지역 변수 이기 때문.자세히는 보진 못 했고, 눈에 띄는 것만 봤기 때문에 틀릴 수도 있습니다.

c++ 코드리뷰? 질문입니다

1개의 답변

#oop 목록