Conversation
opficdev
commented
Feb 27, 2026
- closed 앱델리의 print 문을 Logger로 대체한다 #138
Summary of ChangesHello, 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! 이 풀 리퀘스트는 AppDelegate 파일의 로깅 방식을 표준화하여 애플리케이션의 디버깅 및 유지보수성을 향상시키는 것을 목표로 합니다. 기존의 print 문을 전용 Logger 인스턴스로 교체함으로써, 로그 메시지의 구조화 및 분류가 용이해지고, 불필요한 Combine 임포트를 제거하여 코드의 간결성을 높였습니다. Highlights
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
|
| didRegisterForRemoteNotificationsWithDeviceToken deviceToken: Data | ||
| ) { | ||
| print("APNs token: \(deviceToken.map { String(format: "%02.2hhx", $0) }.joined())") | ||
| logger.info("APNs token: \(deviceToken.map { String(format: "%02.2hhx", $0) }.joined())") |
There was a problem hiding this comment.
Data를 hex string으로 변환하는 로직을 Data의 extension으로 분리하여 재사용성과 가독성을 높이는 것을 제안합니다.
예를 들어, 다음과 같은 extension을 추가할 수 있습니다.
extension Data {
var hexString: String {
map { String(format: "%02.2hhx", $0) }.joined()
}
}이렇게 하면 코드가 더 간결해지고 다른 곳에서도 이 로직을 쉽게 재사용할 수 있습니다.
| logger.info("APNs token: \(deviceToken.map { String(format: "%02.2hhx", $0) }.joined())") | |
| logger.info("APNs token: \(deviceToken.hexString)") |