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

[Feat] #257 - textField Builder 구현 #274

Merged

Conversation

jeongdung-eo
Copy link
Contributor

@jeongdung-eo jeongdung-eo commented Jun 21, 2024

🌁 Background

  • 중복되는 컴포넌트를 재사용 가능한 디자인 시스템으로 구축하기로 결정하였고, 이를 위해 복잡한 객체 생성 과정을 추상화하고 다양한 클래스로 구현하여 객체 생성을 단순화하고자 빌더 패턴을 적용했습니다.

📱 Screenshot

Simulator Screen Recording - iPhone 13 mini - 2024-06-21 at 16 09 22

👩‍💻 Contents

로그인 뷰에 있는 textField는 빌더 패턴으로 구현하지 않은 이유는 ??!!?!

빌더 패턴은 복잡한 객체의 생성 과정을 추상화하여 객체를 단계적으로 구성할 수 있는 디자인 패턴입니다. 그러나 로그인 뷰처럼 UI가 사용자 상호작용에 따라 동적으로 변경될 수 있는 경우, 빌더 패턴의 제한이 있을 수 있습니다. 따라서 저는 초기에 로그인 뷰의 textField도 빌더 패턴으로 구현하려 했지만, 객체 생성 과정이 유연하게 대응할 수 있는 패턴을 선택해야 한다고 판단하여, 홈과 검색 뷰의 textField만 빌더 패턴으로 구현했슴돵~,~

✅ Testing

#257 풀땡겨 보세염

📝 Review Note

버튼 클릭 시 발생하는 액션과 텍스트의 비동기 로직은 나중에 다시 구현할 예정입니다.

📣 Related Issue

📬 Reference

seongmin221 and others added 25 commits May 13, 2024 23:57
…arding

[Fix] #255 - 온보딩 쪽에 있던 메모리 이슈 해결
[Feat] #263 - feature단위로 ci/cd 도입
[Fix] #260 - 무한 스크롤 과정에서 발생한 메모리 사용량 개선 및 레이아웃 계산 최적화
.build()
}

static func buildSerachTetField() -> UITextField {
Copy link
Contributor

Choose a reason for hiding this comment

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

p2:
여기 오타용 ~~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment on lines 15 to 19
.setLeftViewRect(forInsets: .init(top: 0,
left: 15,
bottom: 0,
right: -15)
)
Copy link
Contributor

Choose a reason for hiding this comment

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

p4:
요거 한 줄로 해도 되지 않을까욤 ??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

@seongmin221 seongmin221 left a comment

Choose a reason for hiding this comment

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

좋습니다 ~~~
비대면 회의 하면서 코드리뷰 같이 해서 더 이상 남길건 없는 것 같아유 ~~

@jeongdung-eo jeongdung-eo merged commit 835bd0b into refactoring/#253-design-system Jun 21, 2024
2 of 3 checks passed
@jeongdung-eo jeongdung-eo deleted the feat/#257-textField-builder branch June 21, 2024 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants