-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat [#31] 회원가입_프로필 UI 구현 #32
Changes from all commits
316c33a
63b0c69
a5ee717
cd3c6a2
07ae420
bed4e52
05dfd22
ee58880
71dea9f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
// | ||
// String+.swift | ||
// DontBe-iOS | ||
// | ||
// Created by 변희주 on 1/11/24. | ||
// | ||
|
||
extension String { | ||
// 글자가 자음인지 체크 | ||
var isConsonant: Bool { | ||
guard let scalar = UnicodeScalar(self)?.value else { | ||
return false | ||
} | ||
let consonantScalarRange: ClosedRange<UInt32> = 12593...12622 | ||
return consonantScalarRange ~= scalar | ||
} | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 코드 리뷰:
버그 위험점:
개선 제안:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 이 코드 패치에는 크게 문제가 없어 보입니다. 다만 개선을 위해 몇 가지 제안 사항이 있습니다:
이런 면들을 고려하여 코드를 개선하고 안전성을 확보할 수 있을 것입니다. |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,6 +11,7 @@ enum ImageLiterals { | |
enum Common { | ||
static var btnBack: UIImage { .load(name: "btn_back") } | ||
static var logoSymbol: UIImage { .load(name: "logo_symbol") } | ||
static var imgProfile: UIImage { .load(name: "img_profile") } | ||
} | ||
|
||
enum TabBar { | ||
|
@@ -36,7 +37,6 @@ enum ImageLiterals { | |
static var imgTwoTitle: UIImage { .load(name: "title_second") } | ||
static var imgThirdTitle: UIImage { .load(name: "title_third") } | ||
static var imgFourthTitle: UIImage { .load(name: "title_fourth") } | ||
static var imgProfile: UIImage { .load(name: "img_profile") } | ||
} | ||
|
||
enum Login { | ||
|
@@ -52,6 +52,7 @@ enum ImageLiterals { | |
static var btnNecessary: UIImage { .load(name: "btn_necessary") } | ||
static var btnSelect: UIImage { .load(name: "btn_select") } | ||
static var btnView: UIImage { .load(name: "btn_view") } | ||
static var btnPlus: UIImage { .load(name: "btn_plus") } | ||
} | ||
|
||
enum Home { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 이 코드 패치는 이미지 리터럴을 정의하는 열거형에 변경 사항을 추가합니다. 변경 사항:
버그 위험사항: 개선 제안:
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
{ | ||
"images" : [ | ||
{ | ||
"filename" : "btn_plus.png", | ||
"idiom" : "universal", | ||
"scale" : "1x" | ||
}, | ||
{ | ||
"filename" : "btn_plus@2x.png", | ||
"idiom" : "universal", | ||
"scale" : "2x" | ||
}, | ||
{ | ||
"filename" : "btn_plus@3x.png", | ||
"idiom" : "universal", | ||
"scale" : "3x" | ||
} | ||
], | ||
"info" : { | ||
"author" : "xcode", | ||
"version" : 1 | ||
} | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 이 코드 패치는 JSON 포맷으로 작성된 이미지 정보를 포함하고 있습니다. 이미지 파일 이름과 해당 이미지의 idiomatic 속성(일반적인 사용법), scale 속성(해상도 배율) 등의 정보가 포함되어 있습니다. 버전 및 작성자 정보도 포함되어 있습니다. 이 코드 패치에 대한 개선 제안은 다음과 같습니다:
위의 제안사항을 고려하여 코드를 개선할 수 있습니다. |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,12 @@ | ||
// | ||
// UserInfo.swift | ||
// DontBe-iOS | ||
// | ||
// Created by 변희주 on 1/11/24. | ||
// | ||
|
||
import Foundation | ||
|
||
struct UserInfo { | ||
let userNickname: String | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 이 코드 패치에서는 UserInfo라는 구조체가 정의되어 있습니다. 이 구조체는 userNickname이라는 속성을 가지고 있습니다. 버그 위험은 보이지 않으며, 코드 리뷰나 개선 제안은 다음과 같습니다:
개선된 코드: //
// UserInfo.swift
// DontBe-iOS
//
// Created by 변희주 on 2024/01/11.
//
import Foundation
struct UserInfo {
let userNickname: String
} ※ 제 답변은 일반적인 개발 지침에 기반하여 작성된 것입니다. 특정 프로젝트의 요구사항과 코딩 스타일 가이드라인에 따라 조금 다를 수도 있습니다. |
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.
As an AI language model, I don't have real-time information or the ability to predict the future beyond my last update in September 2021. Therefore, I cannot provide specific information about the events of 2023 or 2024. Please refer to reliable news sources or other up-to-date references for the latest information.