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

[Invidam] Week 02 Solution #GoLang #60

Merged
merged 1 commit into from
May 10, 2024
Merged

[Invidam] Week 02 Solution #GoLang #60

merged 1 commit into from
May 10, 2024

Conversation

Invidam
Copy link
Contributor

@Invidam Invidam commented May 9, 2024

2주차 풀이입니다!

1주차와 달라진 점

Important

문제 풀이와 설명 모두 하나의 파일에 올렸어요!

원래는 코드를 올려야 한다고 생각해 .go를 고집했는데, 패키지나 메인함수가 없어 실행할 수 없는 코드를 올리는 게 큰 의미가 없는 것 같아 문서를 올렸어요.

- ~~.go
+ ~~.go.md

4번째 문제를 풀 때, 릿코드의 문서화 기능을 발견하여 이전 문제들과는 퀄리티가 좀 다릅니다..ㅠ

2주차 후기

  • 아직은 난이도가 쉬운 것 같아, 테스트 코드도 적어보고 영어 문서도 적어보며 여러 시도를 했었습니다!
  • 여러분들 덕분에 md 문법도 많이 배우네요~
  • 고 언어 문법이 재밌지만 어렵네요 ㅎ

Copy link
Member

@sounmind sounmind left a comment

Choose a reason for hiding this comment

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

와, GO 언어는 모르지만 엄청 정성스럽게 문서화하셨네요 멋져요 👍

Copy link
Contributor

@SamTheKorean SamTheKorean left a comment

Choose a reason for hiding this comment

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

leetcode에서 풀이 올릴 때 쓰는 템플릿 쓰신건가요? 깔끔하게 정리해주셔서 공부가 많이 됐습니다!

@SamTheKorean SamTheKorean merged commit e819d57 into DaleStudy:main May 10, 2024
1 check passed
Copy link
Contributor

@dev-jonghoonpark dev-jonghoonpark left a comment

Choose a reason for hiding this comment

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

다양한 접근들을 해보신게 느껴지네요
고생하셨습니다..! 👍

(n = list's size)
# Code
```
const visited = -10001
Copy link
Contributor

Choose a reason for hiding this comment

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

input 값 범위를 사용해서 푸신게 재밌는 접근인것 같습니다 👍

# Code
```
func mergeTwoListsV1(list1 *ListNode, list2 *ListNode) *ListNode {
if list1 == nil && list2 == nil {
Copy link
Contributor

Choose a reason for hiding this comment

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

코드 부분이 인덴트가 4칸일 때가 있고 8칸일때가 있어서 통일해주면 좋지 않을까 싶네요...!

(찾아보니깐 4칸이 권장인것 같네요.)

```


type Stack[T any] struct {
Copy link
Contributor

Choose a reason for hiding this comment

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

Go는 기본적으로 Stack 과 같은 자료구조를 제공해주지 않나보군요...?!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 ㅠ 다른 풀이들을 보면 별도 자료구조를 구현하지 않고 배열과 내장함수들만 이용하는 경우도 있더라구요.

@DaleSeo DaleSeo added this to the week2 milestone May 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants