[2주차] 추상 데이터 타입, 컬렉션 (백, 스택, 큐, 연결리스트)#10
Conversation
- 각 자료구조의 API 작성
- dequeue() 실행 후 queue 에 아이템이 남아 있지 않다면 #last 는 undefined 를 가리켜야 한다.
| class Node { | ||
| #item; | ||
|
|
||
| #next; | ||
| } |
There was a problem hiding this comment.
Node 객체를 선언할 때 item과 next를 private으로 선언하셨네요. 하지만 코드를 사용하는 곳을 보면 이렇게 사용하고 있습니다.
this.#first.item = item;그래서 원래 의도와는 다르게 퍼블릭 item속성에 접근하고 있는 것 같습니다. 자바스크립트가 타입이 없다보니 이런 실수가 일어난 것 같네요.
그리고 여기서 Node의 속성들은 private일 필요는 없습니다. 왜냐하면 단순한 자료 구조를 나타내는 객체라서 캡슐화가 필요 없기 때문이에요. Stack은 외부에서 first속성을 건드리면 안되기 떄문에 접근하지 못하도록 private으로 선언했지만, Node는 그렇지 않습니다.
| const persons = createNumberArray(numberOfPersons); | ||
|
|
||
| const queue = new Queue(); | ||
| persons.forEach((person) => queue.enqueue(person)); |
There was a problem hiding this comment.
enqueue의 반환값이 없기도 하고, forEach는 반환값이 없기도하고, forEach 콜백의 반환값은 있어도 버려지기 때문에 화살표 함수를 쓸 때 코드가 더 짧아진다고 하더라도 분명하게 써주는게 좋습니다. 다른 사람이 봤을 떄는 다른 의도가 있는데 잘못쓴건가? 라고 생각도 들 수 있기 떄문에요.
A function to execute for each element in the array. Its return value is discarded. The function is called with the following arguments:
persons.forEach((person) => {
queue.enqueue(person);
});See also
| const solution = (N, M) => { | ||
| const numberOfPersons = N; | ||
| const deathLocation = M; |
There was a problem hiding this comment.
변수명을 더 이해하기 쉬운 이름으로 �변경하셨네요. 파라미터를 다시 변수로 할당할 떄가 있는데 이 떄는 주로 파라미터를 다른 값으로 변경하려는 경우가 많아요. 대부분의 언어들은 예를들어 자바나 코틀린 같은 언어들은 파라미터를 수정할 수 없어요. 그러다 보니 변수를 재할당해서 처리할 떄가 있거든요.
fun solution(N: Number, M: Number): Unit {
N = 3 // Val cannot be reassigned
var value = N
value = 2
}물론 const를 써서 그런 의도가 아님을 밝히셨지만, 혹여나 그런 의도로 사용했나 생각이 들 수 있을 것 같아요. 그래서 여기서는 파라미터의 이름을 직접적으로 바꾸면 더 좋았을 것 같습니다
2 participants
2주차도 화이팅입니다! :)