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

[Zioq] Week3 #772

Merged
merged 4 commits into from
Dec 29, 2024
Merged

[Zioq] Week3 #772

merged 4 commits into from
Dec 29, 2024

Conversation

Zioq
Copy link
Contributor

@Zioq Zioq commented Dec 24, 2024

체크 리스트

  • 우측 메뉴에서 PR을 Projects에 추가해주세요.
  • Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 StatusIn Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@Zioq Zioq requested a review from a team as a code owner December 24, 2024 00:55
@github-actions github-actions bot added the js label Dec 24, 2024
@Zioq Zioq requested a review from easyone-jwlee December 24, 2024 00:57
result = (result << 1) | (n & 1); // Shift the result to the left by 1 bit OR it with the least significant bit of n.
n >>= 1; // Shifts the bits of n one place to the right, effectively "removing" the processed LSB.
}
return result >>> 0;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

제가 js를 잘 몰라서 그러는데, >>>를 하고 안 하고 차이가 뭔가요?

Copy link
Contributor Author

@Zioq Zioq Dec 28, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저도 이번 과제는 이해가 잘 되지 않아서 스터디 목적으로 제출한 코드였습니다 :)

>>> 는 JS에서 이런 목적으로 쓰이는 오퍼레이션 이라고 하네요

The >>> operator in JavaScript is the unsigned right shift operator. It shifts the bits of a number to the right by a specified number of positions and fills the leftmost bits with zeros, regardless of the sign of the original number.

Key Points:
It treats the number as an unsigned 32-bit integer.
The result is always non-negative, even if the original number was negative.

let remain = target - nums[i]; // Calculate reamined value to check

// If remained value found in the object, return current index and object's key
if( remain_with_index_obj.hasOwnProperty(remain) ) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저는 보통 in을 많이 썼는데, 실무에서는 이 메소드를 더 많이 쓰나요? ㅎㅎㅎ

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

종종 프로퍼티만을 검사하고 싶을 때 실무에서도 자주 쓰는 메소드 입니다 :) 저도 여러번 사용하기두 하구요 ㅎㅎ

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@obzva @Zioq 제가 예전에 자바스크립트 객체에 특정 속성이 있는지 확인하는 방법을 정리한 포스팅이 있어서 추천드립니다.

TLDR:

객체 자신의 속성 뿐만 아니라 상위 클래스로 부터 상속받은 속성까지 고려하고 싶다면 in 연산자를 사용해야합니다. 상위 클래스로 부터 상속받은 속성을 제외하고 객체 자신이 특정 속성을 가지고 있는지 알고 싶다면 Object.hasOwn() 메서드를 사용하면 됩니다. Object.prototype.hasOwnProperty() 메서드는 몇 가지 문제점이 있어서 사용하는 것을 피해야합니다.

Copy link
Contributor

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

4문제 푸시느라 수고 많으셨습니다! 🙌

Comment on lines +36 to +40
/*



*/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

복잡도 분석을 위한 자리만 만들어두시고 기입을 누락하신 것 같습니다.

let remain = target - nums[i]; // Calculate reamined value to check

// If remained value found in the object, return current index and object's key
if( remain_with_index_obj.hasOwnProperty(remain) ) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@obzva @Zioq 제가 예전에 자바스크립트 객체에 특정 속성이 있는지 확인하는 방법을 정리한 포스팅이 있어서 추천드립니다.

TLDR:

객체 자신의 속성 뿐만 아니라 상위 클래스로 부터 상속받은 속성까지 고려하고 싶다면 in 연산자를 사용해야합니다. 상위 클래스로 부터 상속받은 속성을 제외하고 객체 자신이 특정 속성을 가지고 있는지 알고 싶다면 Object.hasOwn() 메서드를 사용하면 됩니다. Object.prototype.hasOwnProperty() 메서드는 몇 가지 문제점이 있어서 사용하는 것을 피해야합니다.

@SamTheKorean SamTheKorean merged commit 3bdefb1 into DaleStudy:main Dec 29, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

4 participants