[3차] 방금그곡 #30
[3차] 방금그곡 #30
Conversation
Summary of ChangesHello @snughnu, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 주어진 음악 정보 목록에서 특정 멜로디를 식별하는 프로그래밍 챌린지에 대한 Swift 솔루션을 도입합니다. 이 솔루션은 시간 계산과 음악 반음 표기법을 효율적으로 처리하여 목표 멜로디를 정확하게 일치시키고, 가장 긴 재생 시간을 가진 일치하는 트랙을 우선적으로 선택합니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| func toMinute(_ time: String) -> Int { | ||
| let t = time.split(separator: ":").map { Int($0)! } | ||
| return t[0] * 60 + t[1] | ||
| } |
There was a problem hiding this comment.
toMinute 함수에서 Int($0)!를 사용하여 강제 언래핑을 하고 있습니다. 이는 입력값이 항상 "HH:MM" 형식의 유효한 정수임을 가정하지만, 만약 그렇지 않은 경우 런타임 오류를 발생시켜 앱을 중단시킬 수 있습니다. compactMap을 사용하여 옵셔널을 안전하게 해제하고, guard 문으로 배열의 크기를 확인하여 코드를 더 견고하게 만드는 것이 좋습니다.
| func toMinute(_ time: String) -> Int { | |
| let t = time.split(separator: ":").map { Int($0)! } | |
| return t[0] * 60 + t[1] | |
| } | |
| func toMinute(_ time: String) -> Int { | |
| let components = time.split(separator: ":").compactMap { Int($0) } | |
| guard components.count == 2 else { return 0 } | |
| return components[0] * 60 + components[1] | |
| } |
| func halfStep(_ s: String) -> String { | ||
| return s | ||
| .replacingOccurrences(of: "C#", with: "c") | ||
| .replacingOccurrences(of: "D#", with: "d") | ||
| .replacingOccurrences(of: "F#", with: "f") | ||
| .replacingOccurrences(of: "G#", with: "g") | ||
| .replacingOccurrences(of: "A#", with: "a") | ||
| .replacingOccurrences(of: "B#", with: "C") | ||
| .replacingOccurrences(of: "E#", with: "F") | ||
| } |
There was a problem hiding this comment.
replacingOccurrences(of:with:)를 연달아 호출하는 방식은 잘 동작하지만, 코드가 길어지고 새로운 치환 규칙을 추가하거나 수정하기 번거로울 수 있습니다. 치환 규칙을 딕셔너리로 정의하고 reduce를 사용하면 코드를 더 간결하고 유지보수하기 쉽게 만들 수 있습니다.
func halfStep(_ s: String) -> String {
let replacements = [
"C#": "c", "D#": "d", "F#": "f", "G#": "g", "A#": "a", "B#": "C", "E#": "F"
]
return replacements.reduce(s) { $0.replacingOccurrences(of: $1.key, with: $1.value) }
}| var played = "" | ||
| for i in 0..<playTime { | ||
| played.append(melody[i % melody.count]) | ||
| } |
There was a problem hiding this comment.
playTime 만큼 반복하면서 한 글자씩 played 문자열을 만드는 방식은 playTime이 클 경우 비효율적일 수 있습니다. Swift의 String은 값 타입이므로 append 할 때마다 새로운 문자열이 생성되고 복사될 수 있기 때문입니다. 멜로디를 필요한 만큼 반복하고 남는 부분을 붙이는 방식으로 더 효율적으로 played 문자열을 생성할 수 있습니다.
let melodyString = String(melody)
let repeatedCount = playTime / melody.count
let remainingCount = playTime % melody.count
let played = String(repeating: melodyString, count: repeatedCount) + melodyString.prefix(remainingCount)
🔗 문제 링크
링크: https://school.programmers.co.kr/learn/courses/30/lessons/17683
난이도: Lv2
✔️ 소요된 시간
1h
📚 새롭게 알게된 내용
계속 34번 틀려서 고민했는데, 레이븐 덕분에 알게됐음
(근데 B#을 C라고 할 수 있는건가)