계산기 [STEP 3] Ari #112

leeari95 · 2021-11-17T09:25:11Z

@lina0322

엘림 안녕하세요!!!

약속했던 기한보다 반나절[?] 일찍 보내드렸...는데 충분히 고민해봤다는 판단에 PR을 보내드립니다. 😆
이 외에 제가 놓친 부분이 있다면 번거롭더라도 한 번만 더 짚어주신다면 감사하겠습니다. 🙏🏻

항상 감사합니다! 엘림 🥰

고민했던 것

자동 스크롤이 제대로 동작하지 않는 경우
스크롤 뷰의 콘텐츠 오프셋 설정시 레이아웃을 바로 업데이트 할 수 있도록 하였습니다. 이유는 레이아웃 업데이트가 예약이 되어있는 상태에서 y값 좌표를 계산하여 오프셋설정을 하게된다면 업데이트 이전의 레이아웃으로 y값으로 계산하기 때문에 layoutIfNeeded() 메소드를 호출하여 레이아웃 업데이트가 필요하였습니다.
```
private func scrollToBottom(_ view: UIScrollView) {
    calculatorScrollView.layoutIfNeeded()
    let bottomOffset = CGPoint(x: 0, y: calculatorScrollView.contentSize.height - calculatorScrollView.frame.height)
    view.setContentOffset(bottomOffset, animated: false)
}
```
layoutIfNeeded() 메소드를 추가해주고 난 후
계산식을 스택뷰에 추가할 때 속성값을 어떻게 설정해줘야하는지 고민해보았습니다. 기존 스토리보드에 설정되어있는 스택뷰의 속성값과 동일하도록 구현해보았습니다.
```
formulaStackView.axis = .horizontal
formulaStackView.alignment = .fill
formulaStackView.distribution = .fill
formulaStackView.spacing = 8
```
프로젝트 요구사항 외 추가 설계한 부분
- 계산이 끝나고 난 후의 상황
  - 스택뷰에 새롭게 업데이트하여 이어서 계산할 수 있도록 추가 설계
  - CE를 누른다면 나왔던 연산결과와 쌓여있던 계산식이 모두 사라질 수 있게 설계
- NaN이 발생하고 난 후의 상황
  - AC, CE 버튼만 활성화하여 연산을 초기화 후 다시 계산을 시작할 수 있도록 설계
- 첫번째 계산식이 올라갈 때의 상황
  - 피연산자만 올라갈 수 있도록 설계
계산이 끝나고나서 추가적으로 발생할 수 있는 예외 핸들링
- '=' 연산자를 클릭 시 연산자버튼 혹은 AC, CE 버튼을 제외하고 나머지 버튼들은 동작하지 않도록 구성
- NumberFormatter로 인해 쉼표가 추가되어있는 문자열을 replacingOccurrences를 이용하여 제거하는 메소드를 구현
- 계산이 되었는지 안되었는지 상태를 나타내는 프로퍼티(hasCalculated)를 추가하여 다양하게 활용.
코드 내부 가독성을 위해 고민했던 것
- ViewController를 extension을 활용하여 분리해보았습니다. 주석예약어 // MARK: 도 같이 활용해보았습니다.
- 계속 옵셔널 바인딩 처리가 필요한 Label.text는 연산프로퍼티를 활용하여 코드 내부를 개선해보았습니다.
- 가독성을 위해 += 연산자 사용을 하지않았습니다. 풀어서 쓰는게 좀더 가독성이 좋을 때가 있다는 생각이 들었습니다.
- StackView 내부에 있는 UILabel의 text를 joined하는 메서드를 UIStackView를 extension하여 프로퍼티(toString)로 만들어주었습니다.
  사용하면 stackView.toString 와 같은 결과가 되서 좀더 직관적일 수 있도록 구현해주었습니다.
- 중복, 반복되는 부분은 메소드로 분리해주는 리팩토링을 진행했습니다.
- 조건문을 변수로 만들어주어 메소드 내부 가독성을 향상시켰습니다.

궁금했던 것 / 조언을 얻고 싶은 부분

layoutIfNeeded() 메소드를 즉시 값이 변경되어야하는 애니메이션에서 많이 사용된다고 알고있는데, 그런 경우가 지금 계산기 프로젝트의 자동스크롤 말고 또 어떤 어떤 경우가 있는지 궁금합니다. 또 setNeedsLayout() 메소드는 언제 사용되어야 적절한지가 궁금합니다.
다양한 예외 상황들을 구현해주느라 IBAction 메소드 내부가 굉장히 지저분한 것 처럼 보이는데.. 착각인걸까요...??? 만약 개선할 수 있다면 조언을 얻고 싶습니다.
보통 숫자의 포맷을 정할 때는 NumberFormatter를 자주 활용하는 편인가요? 아니면 다른 타입도 있는 걸까요? 엘림은 비슷한 타입을 찾으려면 어떻게 찾아보시는지... 궁금합니다!
엘림만의 디버깅 팁이 궁금합니다. 이번에 디버깅을 배우게 되면서... 정복해보고 싶은 마음이 샘솟네요...!!! 전 아직 n, po 말고 더 상세하게 사용해보지도 못했고, 또 어떻게 연습해봐야할지 감이 잘 안오네요. 현업에 계신 엘림이 자주 사용하는 명령어, 혹은 디버깅을 연습하기 위해서 해보면 좋을만할 것들을 추천받고 싶어요.
스택뷰 내부에 있는 Label을 꺼내오려고 이중 forEach를 사용하고 있는데, 더 개선해볼 수 있을 지... 조언이 필요합니다.

이 외에 부족한 점이 보이신다면 말씀해주세요! 감사합니다! 🙇🏻‍♀️

- 숫자입력이 있을 때 숫자를 "0"으로 변경하도록 기능 추가

- 연산자 레이블의 변화가 존재할 때 문자열을 삭제하도록 구현

- 점이 이미 Label에 포함된 경우 더이상 추가하지 않도록 예외사항 구현

- 숫자가 0인 경우 버튼이 작동하지 않도록 구현

- AC Button 클릭 시 Formula Stack View가 비워지도록 기능 구현

- Label의 속성을 설정해주는 setUpFormulaLabel 메소드 추가

- y값 계산을 위해 stackView 추가 후 scrollView의 layout을 즉시 업데이트 하도록 layoutIfNeeded() 호출 - y좌표 계산식을 수정함

- scrollView 안에 추가되어있던 stackView 내부에 Label을 String으로 반환하는 메소드 구현 - 연산 결과를 NumberFormatter로 설정하는 메소드 구현

…로 수정 #5 - UIStackView의 extension으로 확장하고 toString 프로퍼티로 구현

lina0322

아리! 또 반가워요~ 여기까지 달려오느라 고생이 많았어요.
짧은 시간이었지만 많은 발전들을 보여주었네요!

이번 스텝도 고생 많았답니다~
코멘트 보면서 조금 더 다듬어나가 볼까요~?

layoutIfNeeded()

많이 쓰여요. 정말 많이 쓰입니다, 생각보다.
그렇다고 뭐 guard처럼 많이 쓰인다는건 아니지만... 그래도 생각보다 자주 쓰게 됩니다.
다만, layoutIfNeeded가 어떤것인지 명확하게 이해하고 사용했으면 좋겠어요.
아마 뒤쪽에 수업에서 한번 더 다뤄지긴 할테지만, 지금 궁금해졌다면 한번 찾아보고 테스트 해보는 것을 추천드려요.

메소드 내부가 굉장히 지저분한 것 처럼 보이는데

평소 아리답지 않게... 저도 조금 지저분하다고 느꼈어요.
각각 다른 경우라서 guard를 따로쓴 것 같은데, return 값이 같다면 묶어줘도 상관 없는 것 같아보였어요.
굳이 나눌 정도로 논리적 흐름이 다르다면, 함수로 분리되었어야 하지 않았을까 하는 생각이 드네요..
이 부분 저도 좀 아쉬웠습니다🥲

NumberFormatter

저는 그냥 NumberFormatter 사용합니다. 😉

디버깅 팁

저는 사실 UI 확인할때 하이라키 많이 키고, lldb를 많이 쓰는 편은 아닌거같아요!ㅎㅎ
이 부분에 있어서 저는 팁이 없지만, 아마 제 동기인 그린에게 물어보면 팁을 가득 줄거에요
그는 lldb를 좋아하거든요... 용기내여 그에게 디엠 한 번 해보길...😉

스택뷰 내부에 있는 Label을 꺼내오려고 이중 forEach를 사용하고 있는데, 더 개선해볼 수 있을 지... 조언이 필요합니다.

우리 이러지말고 계산 내역을 스택뷰에 추가할 때, 각각 레이블 두개가 아니라 커스텀 뷰 한 묶음으로 만들기에 도전해볼까요...!!!!!💪

요번에도 언제든 궁금한게 생기면 DM 쏴주세욥!

lina0322 · 2021-11-18T12:12:41Z

Calculator/Calculator/Controller/ViewController.swift

@@ -6,13 +6,264 @@

 import UIKit

+// MARK: Property and LifeCycle


주석의 위치가 다소 애매하다고 생각되어요. 아래 Extension의 Mark들과 같은 위치에 두려고 한 것 같은데, 안쪽에 넣는것이 더 좋지 않을까요?
이 클래스에 대한 설명으로 착각할 수 있겠어요.🥲

lina0322 · 2021-11-18T12:16:12Z

Calculator/Calculator/Controller/ViewController.swift

+ currentOperand != "0"
+ }
+
+ var hasCalculated = false


밖에서 접근할 일이 있나요?👀

lina0322 · 2021-11-18T12:17:02Z

Calculator/Calculator/Controller/ViewController.swift

+ removeFormulaView()
+ }
+}
+// MARK: IBAction method


보통 MARK 위아래로 줄바꿈을 하는 것 같아요.
아래쪽은 팀, 사람마다 다르지만 위쪽은 다들 하는 것 같은?!

lina0322 · 2021-11-18T12:17:35Z

Calculator/Calculator/Controller/ViewController.swift

+ }
+
+ var hasCalculated = false
+
 override func viewDidLoad() {
 super.viewDidLoad()


왜 super을 호출해야할까요?🤔
없으면 동작하지 않나요?!

viewDidLoad 메소드 내부에 기존 스토리보드에서 추가되어있는 스택뷰와 레이블들을 세팅해주는 메소드를 추가해주었습니다.
즉 viewDidLoad 후에 어떠한 작업을 해주어야하므로 해당 코드를 실행해주려면 super를 호출해야 합니다.
호출하지 않는다면 아직 viewDidLoad의 설정이 끝나지 않은 상태에서 어떤 작업이 이루어지므로 버그 혹은 오동작이 발생할 수 있는 것으로 알고있습니다.

lina0322 · 2021-11-18T12:18:27Z

Calculator/Calculator/Controller/ViewController.swift

+ guard hasCalculated == false else {
+ return
+ }
+ guard let newOperand = sender.titleLabel?.text else {
+ return
+ }
+ guard isNotZero || newOperand != "00" else {
+ return
+ }
+ guard isNotZero else {
+ currentOperand = newOperand
+ return
+ }
+ guard let doubleValue = Double(removeComma(currentOperand) + newOperand) else {
+ return
+ }
+ currentOperand = setUpNumberFormat(for: doubleValue)


전부 각각 guard를 사용해야하나요?😥

lina0322 · 2021-11-18T12:27:58Z

Calculator/Calculator/Controller/ViewController.swift

+ private func removeComma(_ value: String) -> String {
+ guard value.contains(",") else {
+ return value
+ }
+ return value.replacingOccurrences(of: ",", with: "")
+ }


저는 사실 replacingOccurrences만으로 충분히 동작할 것 같은데, 따로 이렇게 만들어준 이유가 있겠죠..?!

따로 쉼표가 문자열에 포함되어있지 않다면 실행하지 않도록 하고 싶었어요!

lina0322 · 2021-11-18T12:28:55Z

Calculator/Calculator/Controller/ViewController.swift

+ return value.description
+ }
+ guard formatterNumber.count < 26 else {
+ return formatterNumber.map{ $0.description }[0]


인덱스 0에 접근 하는 것이, 언제나 항상 안전할 수 있을까요?🥲
좀 더 안전하게 배열의 첫번째 값을 꺼내올 수는 없을지요!

lina0322 · 2021-11-18T12:30:00Z

Calculator/Calculator/Controller/ViewController.swift

+ guard let input = label?.text else {
+ return
+ }
+ inputValues.append(input.replacingOccurrences(of: ",", with: ""))


removeComma 웨 않쒀여...? 진짜 궁그매서..

extension UIStackView 부분이라서 바로 removeComma를 가져올 수가 없었어요 😇
그래서 그냥 replacingOccurrences메소드를 사용해주었답니다.

lina0322 · 2021-11-18T12:31:09Z

Calculator/Calculator/Model/CalculatorError.swift

 case .wrongOperator:
 return "잘못된 연산자입니다."
 case .wrongOperand:
 return "잘못된 숫자입니다."
+ case .notNumber:
+ return "NaN"


오... 뭔가 description이 느낌이 다르네요🤔
위에서 localizedDescription을 쓰던데, 이 Error 타입에 적용시켜보는 것은 어떨까요?

lina0322 · 2021-11-18T12:32:25Z

Calculator/Calculator/Model/Formula.swift

@@ -38,6 +38,9 @@ struct Formula {
 if operators.isEmpty == false {
 operators.clear()
 }
+ guard currentOperand.isNaN == false else {
+ throw CalculatorError.notNumber


여기서 봤을땐 흐름상 notNumber보다는 isNaN이 좀 더 명확한 표현이 될수도 있겠다싶네요.
물론 같은 뜻이지만, notNumber이면 int가 아니라 string이야, 혹은 숫자가 아니라 글자가 들어왔어 라는 느낌이 될 것 같아서요..!😀

lina0322 · 2021-11-18T12:49:29Z

아, 아리 추가로
지금 요렇게 스크롤하면 가려지는 부분이 있는데, 이 부분도 수정해볼 수 있을까요?
가능하다면 하고, 어렵다면 지나가도 좋습니다!😉
(하지만 아리는 할 수 있다고 생각해욯ㅎㅎㅎㅎㅎ)

~~그리고 너무 빨리 끝나서 심심하면... 저 계산기 네모 버튼들을 동그라미로 만들기 추천드립니다...ㅎㅎㅎㅎㅎㅎ~~

- UIView extension으로 IBInspectable을 활용하여 cornerRadius 프로퍼티 추가 - 모든 버튼의 cornerRadius 값을 추가

- extension UIView 삭제 - calculatorButtons 배열 추가 - viewWillLayoutSubviews 메소드에 버튼들의 cornerRadius 값을 설정하는 로직 추가

- FormulaStackView 클래스 추가하여 addFormulaStackView 메소드 로직 수정 - extension UIStackView에 내부 로직 수정

- 프로퍼티 isNotCalculated, hasDotNotIncluded, hasMinusNotIncluded 추가 - clearEntryButtonTapped 내부 잘못된 로직 수정

leeari95 · 2021-11-19T07:54:04Z

안녕하세요. 엘림! 피드백 보고 To do List 작성하여 한번 개선해보았습니다.

개선하면서 궁금했던 것들도 생겨서 추가로 적어보았습니다. 덕분에 코드가 좀더 간결해진 것 같아요. 감사합니다🥰

수정사항

주석 위치와 줄바꿈을 수정
스크롤바가 보이지 않도록 수정
놓쳤던 접근제어 추가하여 수정
같은 return을 하는 guard문을 합쳐서 최소화
메소드 내에 중복되는 부분 확인 후 함수로 분리 작업
toggle을 삭제하고 직접 대입해주는 방식으로 수정
Error case(notNumber) 리네이밍 후 LocalizedError 프로토콜 채택 후 적용
- 활용할 수 있는 프로퍼티들을 최대한 활용해보았습니다.
계산기 버튼의 형태를 원형으로 수정
- IBInspectable으로 처리했다가 하드코딩 같다고 느껴져서 viewWillLayoutSubviews를 활용하여 Button의 값을 설정해주었습니다.
커스텀 뷰를 만들어서 ViewController 내부 코드 개선

궁금했던 것 / 조언을 얻고 싶은 부분

이렇게 개선을 하여도 여전히 IBAction부분이 뚱뚱하다고 느껴지네요.
이런 경우 역할을 분담하여 타입을 만들어 주는 것이 좋을까요?
예를 들어 사용자의 입력값을 검사하는 validation 같은 타입을 만들어서 새롭게 로직을 짜야할 것 같다는 생각이 들었어요.
- 그러나 Bool을 리턴하는 프로퍼티들이 다 ViewController에 관련된 값들을 가지고 반환을 하기 때문에 어떻게 건들여야할까.. 하는 막막함이....🥲
커스텀뷰를 처음 만들어 보는거라 약간 어려움이 있었는데 잘 만든게 맞는지 모르겠네요 🥲

- OSLog를 import 하여 extension 추가 - 에러를 처리하는 부분에 print문이 아니라 os_log문으로 수정

lina0322

아리~ 역시 제가 생각했던 것보다 잘해주었네요!🙏🙏

좀 더 생각해봤으면 하는 부분만 코멘트로 달았고, 이번 프로젝트는 여기까지 진행하고
뒤 프로젝트에서 다른 캠퍼와 합치면서 여러가지로 다시 고민해보는 것도 좋을 것 같아요.

이번 프로젝트 너무 고생많았고, 계속 발전하는 모습 또 고민하는 모습 보기 좋았습니다👍👍
지금처럼 언제든 편하게 궁금한거 있을때 DM주세요!
(바빠서 빨리 대답 못해줘도 기다려주는 아리 최고...🙏)

다음 프로젝트도 화이팅!

lina0322 · 2021-11-22T09:56:50Z

Calculator/Calculator/Controller/ViewController.swift

+ hasCalculated = true
+ return
+ case .queueNotFound:
+ os_log(.error, log: .error, "%@", error.errorDescription ?? error.localizedDescription)


ㅎㅎ log는 어떤가요? 불편하진 않나요...?🤤

print문 보다 쓰기가 조금 어렵지만 그에 비해 좋은 장점들이 있습니다.

위 방식처럼 print문을 상당히 쉽게 대체할 수 있다.

콘솔 앱과 로그를 함께 사용하면 보다 효율적인 방식으로 문제를 디버깅하는데 도움이 될 수 있다.

성능 오버헤드가 낮고 나중에 검색할 수 있도록 장치에 보관된다.

위와 같은 장점 때문에라도 OSLog를 사용하면 더 좋을 것 같다는 생각이 들었습니다! 😄

lina0322 · 2021-11-22T09:57:34Z

Calculator/Calculator/Extension/OSLog+extension.swift

+import OSLog
+
+extension OSLog {
+ private static var subsystem = Bundle.main.bundleIdentifier!


되도록 강제언래핑은 피해주세요!🙏

lina0322 · 2021-11-22T09:58:09Z

Calculator/Calculator/Model/CalculatorError.swift

+ default:
+ return "알 수 없는 에러가 발생했습니다."


lina0322 · 2021-11-22T09:58:17Z

Calculator/Calculator/View/FormulaStackView.swift

+import UIKit
+
+class FormulaStackView: UIStackView {
+ private(set) var element = [String]()


lina0322 · 2021-11-22T09:58:32Z

Calculator/Calculator/View/FormulaStackView.swift

+ override init(frame: CGRect) {
+ super.init(frame: frame)
+ loadView()
+ }
+
+ required init(coder: NSCoder) {
+ super.init(coder: coder)
+ loadView()
+ }


두 init은 어떤 차이가 있을까요?

override 이니셜라이저는 상속한 부모의 이니셜라이저를 재정의한 것이고,
required는 필수로 정의해야하는 이니셜라이저 입니다 😁

coder와 frame의 차이는 이미 알고 있는거라서 말씀 안해주신건가유?😉

헉.. 중요한 부분이였네요. 찾아서 아래와 같이 정리해보았습니다. 😊

init(frame:)

코드로 UIView Class의 View 인스턴스를 만들기 위해 지정된 이니셜라이저.
Frame rectangle을 가지고 구현하고자 하는 뷰의 중심과 경계선을 지정해준다.
스토리보드, xib, nib 같은 인터페이스 빌더를 사용하지 않고 코드로 UIView Calss의 View object를 만들기 위해 지정된 이니셜라이저다.

init(coder:)

기본적으로 storyboard나 xib를 활용하면 별도의 코딩 없이 앱의 속성을 수정할 수가 있는데 이것을 가능하게 해주는 과정을 unarchiving이라고 한다. Interface builder는 코드가 아니기 때문에 앱을 컴파일 하는 시점에서 컴파일러가 인식할 수 없고 이를 코드로 변환해주는 unarchiving 과정이 필요하다는 것이다.
이 과정에서 init?(coder:)가 사용된다.
파라미터 coder를 통해 NSCoder 타입의 객체가 전달되는 것이고 전달된 NSCoder 타입의 객체가 decoding되어 초기화된 후 컴파일 할 수 있게 decoding된 자기자신(self)을 반환하는 작업이라고 보면 될 것 같다.

내가 구성한 View의 상태를 앱의 disk에 저장하는 과정을 serialize라고 한다. deserialize는 반대로 disk에 저장된 상태를 불러오는 작업이라고 볼 수 있다.
NSCoding이라는 프로토콜을 통해 이 serialize와 deserialize 작업이 가능해진다.

따라서 init(coder:)의 용도는 아래와 같다.

Storyboard라는 인터페이스 빌더를 사용하여 뷰의 상태를 수정할 경우 serialization 작업을 Xcode가 init(coder:)를 호출하여 앱 내 뷰 작업을 저장하고 불러오는 작업을 해준다.

UIView 선언부를 보면 NSCoding 프로토콜을 채택하고 있는데 NSCoding 선언부를 살펴보면 실패가능한 이니셜라이저를 작성하도록 되어있다.
프로토콜을 준수하는 클래스에서 프로토콜에서 요구하는 이니셜라이저 요구사항을 구현하려면 required 키워드가 붙어야 한다. 따라서 이를 상속받은 FormulaStackView와 같은 클래스에서는 스토리보드르 사용하고 있지 않지만 init?(coder:)를 구현해줘야 하고 앞에 꼭 required를 붙여주어야 한다.

lina0322 · 2021-11-22T10:02:43Z

Calculator/Calculator/Controller/ViewController.swift

+ override func viewWillLayoutSubviews() {
+ super.viewWillLayoutSubviews()
+ calculatorButtons.forEach { button in
+ button.layer.cornerRadius = button.layer.frame.size.width / 2
+ }
+ }


요 함수가 언제 불리는지 좀 더 찾아보면 좋을 것 같아요!🙏🙏

뷰가 SubView의 레이아웃을 변경하기 직전에 호출되는 메서드입니다!

viewWillLayoutSubviews에 브레이크 포인트 잡아보거나, log 혹은 print 찍어보세요🙏

leeari95 added 30 commits November 16, 2021 01:27

feat: 추가 operand, operator Label, setUpFormulaLabel 메소드 #5

0b5cfc0

feat: 숫자 버튼 터치 시 Label에 실시간으로 반영하는 기능 추가 #5

b700d55

feat: 숫자입력이 없을 때 연산자 버튼 터치 시 Label에 반영되는 기능 추가 #5

5df02a4

refactor: 계산되었는지 확인하는 연산 프로퍼티 추가 #5

1b87026

feat: clearButtonTapped IBAction 메서드 추가 #5

d70f260

- 숫자입력이 있을 때 숫자를 "0"으로 변경하도록 기능 추가

refactor: "0", "00" 버튼을 눌렀을 때의 예외사항 조건문 수정 #5

a0f7f1e

refactor: CE 버튼을 눌렀을 때의 예외사항 수정 #5

5cb66cb

- 연산자 레이블의 변화가 존재할 때 문자열을 삭제하도록 구현

feat: AC 버튼의 IBAction 메소드 추가 #5

867feb8

feat: 점(.) 버튼을 클릭했을 때 Label에 반영되도록 기능 추가 #5

8e373f0

- 점이 이미 Label에 포함된 경우 더이상 추가하지 않도록 예외사항 구현

refactor: CE 버튼 네이밍 수정 #5

31c543b

feat: plus/minus(⁺⁄₋) 버튼을 추가 #5

ef59b50

- 숫자가 0인 경우 버튼이 작동하지 않도록 구현

feat: Formula Stack View 초기 설정 메소드 추가 #5

b4154c6

- AC Button 클릭 시 Formula Stack View가 비워지도록 기능 구현

feat: 연산자 버튼 클릭 시 Formula Stack View에 추가되는 기능 추가 #5

0868c64

feat: Scroll View에 내용이 계속 추가 될 때 하단 스크롤로 이동하도록 구현 #5

4102a6a

refactor: stackView의 aligment 수정, removeFormulaView 내부 수정 #5

7a86ad5

refactor: scrollToBottom 메소드 내부 오입력 수정 #5

e11d90e

refactor: 스크롤뷰와 스택뷰의 프로퍼티명 수정 #5

726fe25

refactor: 계산스택이 없는 상태일 때 숫자만 추가되도록 수정 #5

01a726d

- Label의 속성을 설정해주는 setUpFormulaLabel 메소드 추가

refactor: 자동 스크롤 기능이 제대로 동작하지 않는 부분 수정 #5

1976739

- y값 계산을 위해 stackView 추가 후 scrollView의 layout을 즉시 업데이트 하도록 layoutIfNeeded() 호출 - y좌표 계산식을 수정함

refactor: 곱셈, 나눗셈 연산자의 rawValue를 버튼 Label 텍스트와 동일하게 수정 #5

0065d1d

refactor: CalculatorError 파일 이름 수정과 에러 case 추가 #5

ef700c1

refactor: 빼기 연산자의 rawValue를 버튼의 titleLabel text와 동일하게 수정 #5

2eeb13a

refactor: 연산자 버튼 터치 시 이전에 반영된 연산자가 scrollView에 쌓이도록 수정 #5

861c2ae

feat: '=' 버튼을 터치 시 연산 결과가 나오도록 기능 구현 #5

650ec58

- scrollView 안에 추가되어있던 stackView 내부에 Label을 String으로 반환하는 메소드 구현 - 연산 결과를 NumberFormatter로 설정하는 메소드 구현

refactor: IBAction 메소드를 extension을 활용하여 분리 #5

ef49fd3

refactor: Label을 초기화 해줬던 하드코딩 부분 수정 #5

3f30647

refactor: isCalculated 연산자의 이름을 isNotZero로 수정 #5

7f62501

refactor: stackView의 Label을 String으로 반환하는 메소드를 extension UIStackView으…

6c29b6c

…로 수정 #5 - UIStackView의 extension으로 확장하고 toString 프로퍼티로 구현

refactor: Formula Stack View 관련 값들을 extension을 활용하여 분리 #5

3faabac

refactor: Label을 초기화 해주는 메소드를 extension으로 분리 #5

c71dd2a

leeari95 added 4 commits November 17, 2021 16:51

fix: formula 내부에 queue가 비어있어서 계산을 할 수가 없는 예외 case 추가 #5

d5d2e1a

refactor: reduce를 제거하고 replacingOccurrences 메소드로 수정 #5

118b46f

feat: 사용자가 보는 화면 숫자에 콤마(,)가 표시되도록 구현 #5

7bd4c94

docs: README.md 내부 STEP2, STEP3 내용 추가

73d9348

leeari95 force-pushed the 4-leeari95 branch from 5b65f91 to 73d9348 Compare November 17, 2021 09:35

lina0322 requested changes Nov 18, 2021

View reviewed changes

leeari95 added 13 commits November 18, 2021 23:15

style: 주석 위치 수정, 줄바꿈 추가 #6

6969a09

style: 계산기 내역이 늘어날 때 생기는 스크롤바 제거 #6

03e30b1

refactor: hasCalculated 프로퍼티에 접근제어 추가 #6

5425113

refactor: 같은 return문의 guard문들을 하나로 합쳐주어 수정 #6

581be15

refactor: operatorButtonTapped 내부 메소드로 분리, 동일한 로직의 guard문 합쳐서 수정 #6

a6b7ca4

refactor: operatorButtonTapped 메서드 내에 중복되는 부분 메소드로 분리 #6

688d1fa

refactor: hasCalculated을 toggle하는 부분을 직접 대입해주는 것으로 수정 #6

cdfd79d

refactor: notNumber를 isNaN으로 리네이밍 #6

eefc854

refactor: LocalizedError 프로토콜 채택 후 프로퍼티 추가 #6

740a5a5

feat: 계산기 버튼의 형태를 원형으로 스타일 수정 #6

f848ed6

- UIView extension으로 IBInspectable을 활용하여 cornerRadius 프로퍼티 추가 - 모든 버튼의 cornerRadius 값을 추가

refactor: 버튼의 cornerRadius 설정을 IBInspectable이 아니라 코드로 수정 #6

d081036

- extension UIView 삭제 - calculatorButtons 배열 추가 - viewWillLayoutSubviews 메소드에 버튼들의 cornerRadius 값을 설정하는 로직 추가

feat: 커스텀 뷰 FormulaStackView를 추가 #6

6276b23

- FormulaStackView 클래스 추가하여 addFormulaStackView 메소드 로직 수정 - extension UIStackView에 내부 로직 수정

refactor: 버튼 제어를 위한 프로퍼티 추가 후 IBAction 내부 개선 #6

bc282af

- 프로퍼티 isNotCalculated, hasDotNotIncluded, hasMinusNotIncluded 추가 - clearEntryButtonTapped 내부 잘못된 로직 수정

leeari95 requested a review from lina0322 November 19, 2021 07:59

refactor: LocalizedError 채택후 추가한 잘못된 프로퍼티명 수정 #6

cd37d21

leeari95 force-pushed the 4-leeari95 branch from 6b8e90a to cd37d21 Compare November 19, 2021 08:15

leeari95 added 3 commits November 20, 2021 00:21

chore: 인터페이스 빌더로 cornerRadius를 주었던 부분 삭제 #6

a032712

docs: README.md Trouble Shooting, PR 후 개선사항, 키워드 추가

3b237e1

refactor: print문이 아니라 log를 출력하도록 수정 #6

086331b

- OSLog를 import 하여 extension 추가 - 에러를 처리하는 부분에 print문이 아니라 os_log문으로 수정

lina0322 approved these changes Nov 22, 2021

View reviewed changes

lina0322 merged commit c9234de into yagom-academy:4-leeari95 Nov 22, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

계산기 [STEP 3] Ari #112

계산기 [STEP 3] Ari #112

leeari95 commented Nov 17, 2021 •

edited

Loading

lina0322 left a comment

lina0322 Nov 18, 2021 •

edited

Loading

lina0322 Nov 18, 2021

lina0322 Nov 18, 2021

lina0322 Nov 18, 2021

leeari95 Nov 19, 2021

lina0322 Nov 18, 2021

lina0322 Nov 18, 2021

leeari95 Nov 19, 2021

lina0322 Nov 18, 2021

lina0322 Nov 18, 2021

leeari95 Nov 19, 2021 •

edited

Loading

lina0322 Nov 18, 2021

lina0322 Nov 18, 2021

lina0322 commented Nov 18, 2021

leeari95 commented Nov 19, 2021

lina0322 left a comment

lina0322 Nov 22, 2021

leeari95 Nov 22, 2021

lina0322 Nov 22, 2021

lina0322 Nov 22, 2021

lina0322 Nov 22, 2021

lina0322 Nov 22, 2021

leeari95 Nov 22, 2021

lina0322 Nov 22, 2021

leeari95 Nov 23, 2021

lina0322 Nov 22, 2021

leeari95 Nov 22, 2021

lina0322 Nov 22, 2021

		@@ -6,13 +6,264 @@

		import UIKit

		// MARK: Property and LifeCycle

		default:
		return "알 수 없는 에러가 발생했습니다."

계산기 [STEP 3] Ari #112

계산기 [STEP 3] Ari #112

Conversation

leeari95 commented Nov 17, 2021 • edited Loading

고민했던 것

궁금했던 것 / 조언을 얻고 싶은 부분

lina0322 left a comment

Choose a reason for hiding this comment

layoutIfNeeded()

메소드 내부가 굉장히 지저분한 것 처럼 보이는데

NumberFormatter

디버깅 팁

스택뷰 내부에 있는 Label을 꺼내오려고 이중 forEach를 사용하고 있는데, 더 개선해볼 수 있을 지... 조언이 필요합니다.

lina0322 Nov 18, 2021 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

leeari95 Nov 19, 2021 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

lina0322 commented Nov 18, 2021

leeari95 commented Nov 19, 2021

수정사항

궁금했던 것 / 조언을 얻고 싶은 부분

lina0322 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

init(frame:)

init(coder:)

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

leeari95 commented Nov 17, 2021 •

edited

Loading

lina0322 Nov 18, 2021 •

edited

Loading

leeari95 Nov 19, 2021 •

edited

Loading