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

[Algorithm] 선택 정렬 #195

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[Algorithm] 선택 정렬 #195

wants to merge 1 commit into from

Conversation

jaminleee
Copy link
Collaborator

Summary

Check List

  • I created file and check spell again
  • I linked File to README.md

Copy link
Owner

@da-in da-in left a comment

Choose a reason for hiding this comment

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

  • 장단점 설명 부분 문장 간격 줄여주세요!
  • 불안정 정렬이 뭐에요? 등의 우리 컨벤션에 맞지 않는 문장들 수정해주세요.
  • 기존 문서 36번째 줄에 '주어진 배열 안에서 교환(swap)을 통해 ,' 등과 같이 불필요한 기호, 문법이나 문장들이 이상한 곳이 많은 것 같습니다. 전체적으로 재검토 해주시면 감사합니다.
  • 장점 세 번째 줄의 다른 메모리 공간을 필요로 하지 않는다. => 제자리 정렬과 같은 문장 기술도 저희 컨벤션과는 맞지 않는 것 같습니다.

@@ -7,6 +7,16 @@

Copy link
Owner

Choose a reason for hiding this comment

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

이 위에 백문이 불여일견! ~자 지워주시면 감사드립니당👍🏻

@da-in da-in mentioned this pull request Nov 8, 2023
2 tasks
Copy link
Collaborator

@lalabulla lalabulla left a comment

Choose a reason for hiding this comment

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

전체적으로 문장 수정 및 정리가 필요한 것 같아요! 눈에 잘 안들어와요ㅠㅠ

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

Successfully merging this pull request may close these issues.

3 participants