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

[Round1] Step3 - sy #997

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

Conversation

jeseoyoung
Copy link

main.swift: step2에서 사용하였던 함수를 삭제하고, step3의 함수를 추가하였습니다.

step3.swift(질문 하나 포함):

  • drawPepero(stickLenth: Int, bodyDecoration: String, topping: String = "", bodyLenth: Int) : 각종 매개변수를 받아 세 메서드를 호출하는 함수
  • drawPeperoStick(stickLength: Int): 길이(Int)에 따른 막대(빼빼로 손잡이)를 그리는 함수
  • drawPeperoBodyAndToppings(bodyDecoration: String, topping: String): 몸통(String), 토핑(String)을 매개변수로 받아 토핑과 몸통을 그리는 함수
  • setPeperoBody(bodyLenth: Int, bodyDecoration: String, topping: String) : 길이(Int)에 따라 몸통을 그리는 함수
    => 해당 함수의 경우 Int parameter 하나만 포함해야 하는 건가요?

@jaemuYeo

Copy link

@jaemuYeo jaemuYeo left a 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()

CodeStarterCamp_Week1/step3.swift Outdated Show resolved Hide resolved
CodeStarterCamp_Week1/step3.swift Outdated Show resolved Hide resolved
CodeStarterCamp_Week1/step3.swift Outdated Show resolved Hide resolved
CodeStarterCamp_Week1/step3.swift Outdated Show resolved Hide resolved
CodeStarterCamp_Week1/step3.swift Outdated Show resolved Hide resolved
Copy link

@jaemuYeo jaemuYeo left a comment

Choose a reason for hiding this comment

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

다시 재요청 주신 부분이 해당 PR맞을까요!!
아직 정보를 출력해주는 부분과 스틱의 위치가 실행예시와 동일하게 프린트 되는 부분이
해결되지 않아 한 번 만 더 수정을 해보고 다음 Round로 넘어가면 좋을 것 같아요.

스크린샷 2024-09-24 오후 4 45 36

실행예시처럼 이러한 정보도 함께 출력되어야합니다.
여러 빼빼로를 생성했을 때 출력 결과를 보기 편하게 개행도 해주어야 하구요 💪

코드 반영해보며 궁금한 내용있으시면 언제든 연락주세요!

CodeStarterCamp_Week1/step3.swift Outdated Show resolved Hide resolved
CodeStarterCamp_Week1/step3.swift Outdated Show resolved Hide resolved
@jeseoyoung
Copy link
Author

넵 해당 PR 맞습니다 감사합니다!

step3.swift

  • drawPepero()에서 빼빼로의 정보를 출력하는 코드를 추가하였습니다.
    결과는 다음과 같습니다!
    image

  • printPeperoBodyAndToppings()의 코드를 변경하였습니다.

  • loopForBodyPrint()의 function명을 printBody()로 변경하였습니다.

function의 적절한 이름을 고르는 것이 조금 어렵게 느껴지네요😔

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.

2 participants