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

몇 가지 실수들.. #2

Open
zeliard opened this issue Oct 26, 2014 · 2 comments
Open

몇 가지 실수들.. #2

zeliard opened this issue Oct 26, 2014 · 2 comments
Assignees

Comments

@zeliard
Copy link

zeliard commented Oct 26, 2014

대충 보다 보니 실수들이 보이는데...
이런거는 습관적인거라.. 좋은 습관을 의도적으로 들이는 편이 좋음.

예를 들어,

  1. 자식 클래스의 소멸자는 virtual.
  2. 구조체나 클래스 선언할 때, typedef는 C++에서 쓰지 않음.
    (클래스 내에서 typedef로 타입 이름 바꿀때는 주로 using 사용)
  3. 클래스 내부 벡터 같은 컬렉션에 대한 포인터를 클래스 외주로 넘겨주는 것들은 최대한 지양. (이건 99.9% 구조적인 문제임)
@clogic29
Copy link
Contributor

1, 2 수정했습니다!
그런데 3번의 경우엔 해결하는 방법으로 어떤 것들이 있을까요?
vectorItem* getter(index)이런식으로 구현해서 index에 해당하는 item이 없으면 null을 반환하는 방법을 생각해봤는데 이건 어떨까요??

일일히 읽고 이슈 남겨주셔서 감사합니다!

@zeliard
Copy link
Author

zeliard commented Oct 28, 2014

벡터 아이템을 외부에서 순회해야 되는 경우라면, 벡터의 위치가 잘못된 것이거나.. 아예 백터 아이템을 들고 있는 객체에 일을 맡기는 방식이 되어야 함.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants