Skip to content

✨[FEAT]: 필수 구현 과제 1#1

Merged
Roy-wonji merged 3 commits intodevelopfrom
Feature/mission-01
Aug 27, 2025
Merged

✨[FEAT]: 필수 구현 과제 1#1
Roy-wonji merged 3 commits intodevelopfrom
Feature/mission-01

Conversation

@Roy-wonji
Copy link
Contributor

✨ 작업 내용

  • 정답은 랜덤으로 만듭니다.(1에서 9까지의 서로 다른 임의의 수 3자리)

Copy link

@minneee minneee left a comment

Choose a reason for hiding this comment

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

현재 프로젝트에서는 동시성 문제를 제어해야할 상황이 없는 것 같은데 actor와 async/await을 사용한 이유가 있으신가요?

import Foundation
import Combine

actor GameManger {
Copy link

Choose a reason for hiding this comment

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

오타있는 것 같습니다!

@Roy-wonji
Copy link
Contributor Author

현재 프로젝트에서는 동시성 문제를 제어해야할 상황이 없는 것 같은데 actor와 async/await을 사용한 이유가 있으신가요?

차후에 동시성을 사용 할꺼여서 미리 작업을 해놓았습니다

Copy link

@Peter1119 Peter1119 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ~!

Comment on lines +22 to +25
var pool = Array(1...9)
pool.shuffle()
#logDebug("랜덤 숫자", Array(pool.prefix(digits)))
return Array(pool.prefix(digits))

Choose a reason for hiding this comment

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

오 좋네요 ! 저도 수정해야겠어요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Peter1119 그건 지금 단계에서는 요구 사항이 아니어서
정답은 랜덤으로 만듭니다.(1에서 9까지의 서로 다른 임의의 수 3자리)

Choose a reason for hiding this comment

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

앗 그렇네요

Copy link

@Peter1119 Peter1119 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ~!

@Roy-wonji Roy-wonji merged commit 6a4f01f into develop Aug 27, 2025
@Roy-wonji Roy-wonji deleted the Feature/mission-01 branch August 27, 2025 08:55
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