-
Notifications
You must be signed in to change notification settings - Fork 253
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
[Round1] Step3 - sy #997
base: ss_20_sy
Are you sure you want to change the base?
[Round1] Step3 - sy #997
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Step3도 고생많으셨습니다!!
다만 실행예시와 같이 정보
가 출력되고 있지 않군요 🤔
다음 재요청에 해당 부분도 추가해보면 좋을 것 같습니다!
그 외 몇몇 코멘트 남겨두었습니다!
순서도
순서도에 격자가 있어 화살표의 흐름을 잘 파악하기가 힘들어요 ㅠㅠ
draw.io를 활용해 격자를 없애서 작성해보시면 좋을 것 같습니다! 💪
순서도에는 지난 스텝2처럼 간결하게 작성해주셔도 좋아요!
만약 함수를 작성한다면 매개변수는 생략하셔도 좋습니다~
Ex. 빼뺴로그리기 실행 -> 몸통 셋팅... 또는 drawPepero() -> setPeperoBody()
c322801
to
2551ac6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2551ac6
to
2867a14
Compare
main.swift: step2에서 사용하였던 함수를 삭제하고, step3의 함수를 추가하였습니다.
step3.swift(질문 하나 포함):
=> 해당 함수의 경우 Int parameter 하나만 포함해야 하는 건가요?
@jaemuYeo