Skip to content

[kimyoung] WEEK 01 Solutions #321

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

Merged
merged 6 commits into from
Aug 17, 2024
Merged

[kimyoung] WEEK 01 Solutions #321

merged 6 commits into from
Aug 17, 2024

Conversation

@kim-young
Copy link
Contributor Author

ProjectsIteration 설정을 하고 싶은데 제가 온보딩을 참석 못해서 권한이 없는거 같네요 😂
권한 주시면 제가 설정하도록 하겠습니다!

@leokim0922
Copy link
Contributor

안녕하세요 :) 혹시 인비테이션 수락을 하셨을까요? 아래 메세지를 한 번 확인해보시겠어요?

Discord Message

@kim-young
Copy link
Contributor Author

안녕하세요 :) 혹시 인비테이션 수락을 하셨을까요? 아래 메세지를 한 번 확인해보시겠어요?

Discord Message

image

제가 온보딩을 참석 못해서 인비테이션이 안왔나봐요 😂
@DaleSeo @SamTheKorean 혹시 리더분들께서 도와주실수 있으신가요?

@DaleSeo
Copy link
Member

DaleSeo commented Aug 14, 2024

@kim-young 님, 안녕하세요. 온보딩을 참석하신 분을 최종 멤버로 확정하고 스터디 2기를 진행한다고 공지드렸는데 혹시 못 보셨을까요? 온보딩 때 다룬 내용이 상당히 많은데 제대로 활동이 가능하실지 모르겠네요? 😓

@kim-young
Copy link
Contributor Author

안녕하세요 @DaleSeo 님 답장 주셔서 감사합니다! 그리고 온보딩 참석하지 못해서 죄송해요. 제가 깃헙을 이런 식으로 깊이 있게 사용해보는게 처음인지라 공지 및 시간을 제대로 숙지를 못했네요 100% 제 불찰입니다 😭 일단 제공해주신 documentation 및 눈치로 1주차 문제를 4/5 PR 섭밋 중에 있긴한데 혹시 진행이 어려울까요? 가능하다면 함께하고 싶습니다!

@kim-young kim-young marked this pull request as ready for review August 15, 2024 14:42
@DaleSeo
Copy link
Member

DaleSeo commented Aug 16, 2024

@kim-young 님, 제가 대화를 좀 나누고 싶어서 디스코드로 DM 들렸는데 응답이 없으시네요. 운영진의 입장을 여기에 간단히 말씀드리면 좋은 스터디의 경험을 위해서 온보딩을 매우 중요하게 생각하고 있습니다. 확인하신 기여 가이드는 온보딩의 극히 일부 내용이라서 놓치신 부분이 상당히 많으신데요. 다음 모임에 참석 가능으로 표시해주셨는데, 혹시 30분 일찍 참석하실 수 있다면 제가 빠르게 온보딩을 해드릴 수 있을 것 같은데 어떠신가요?

@kim-young
Copy link
Contributor Author

@kim-young 님, 제가 대화를 좀 나누고 싶어서 디스코드로 DM 들렸는데 응답이 없으시네요. 운영진의 입장을 여기에 간단히 말씀드리면 좋은 스터디의 경험을 위해서 온보딩을 매우 중요하게 생각하고 있습니다. 확인하신 기여 가이드는 온보딩의 극히 일부 내용이라서 놓치신 부분이 상당히 많으신데요. 다음 모임에 참석 가능으로 표시해주셨는데, 혹시 30분 일찍 참석하실 수 있다면 제가 빠르게 온보딩을 해드릴 수 있을 것 같은데 어떠신가요?

@DaleSeo 님, 그렇게까지 해주신다니 너무 감사합니다 ㅠㅠ 그룹을 운영하는데 있어서 이런 예외상황을 두는게 쉽지 않다는걸 알고 있는데 이런 배울수 있는 기회 주셔서 너무 감사해요. 제가 그럼 토요일에 8시반까지 오도록 하겠습니다!

@@ -0,0 +1,16 @@
var hammingWeight = function (n) {
let bitVal = n.toString(2); // convert input value to bit
Copy link
Contributor

Choose a reason for hiding this comment

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

저를 포함해서 다른 분들도 pythonic한 풀이를 많이 사용하셨는데, 비트연산을 활용한 Brian Kernighan's Algorithm도 참고해보시면 좋을 것 같습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

리뷰 해주셔서 감사합니다! 저도 스스로 문제들 풀어보고 (PR 섭밋 후) 달레님께서 제공해주신 풀이들을 봤는데, 말씀 해주신 알고리즘을 접할수 있었습니다! 따로 체크인은 안했지만 릿코드에서 다시 한번 풀어봤는데 다른 접근 방식이 있다는게 신기하더라고요 👍

for (let i = 0; i < s.length; i++) {
let left = i,
right = i; // odd length substrings
helper(s, left, right);
Copy link
Contributor

Choose a reason for hiding this comment

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

이건 알고리즘과는 관련 없는 이야기지만, helper 보다는 해당 함수의 역할을 나타내는 이름이 가독성을 위해 좋을 것 같습니다

Copy link
Contributor Author

Choose a reason for hiding this comment

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

저도 모르게 helper 로 쓰는게 습관이 되었네요. 좋은 지적해주셔서 감사합니다!

@leokim0922 leokim0922 merged commit 226a07a into DaleStudy:main Aug 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

5 participants