프로필 사진을 삭제하려고 changeProfileImage에 nil을 넣었는데, 오류가 뜹니다. 유즈케이스를 보니까 User.profileImage를 다시 nil로 만드는 방법이 없는것 같습니다.

프로필 관리 화면에서 프로필 사진을 삭제하는 기능을 구현해야 할 것 같은데, 이러려면 새로운 유즈케이스가 필요할 것 같습니다! 혹은 현재 상태로 다른 방법이 있다면 알려주시면 감사하겠습니다.

Description

lofi 기준으로 하나의 화면에 나타내었던 가입 코드, 닉네임 화면 분할

Sketch Screenshot

To-do

• [x] 가입코드 화면과 닉네임 화면 분할
• [x] navigation 연결
• [x] textfield가 focused 됨에 따라 키보드 스타일 변경
• [x] 가입 코드 textfield underbar 색깔 변화 및 포커싱
• [x] 닉네임 textfield underbar 포커싱

Etc

작업사항

related to #17

• underline이 들어간 textfield 작성
• underline을 만들어주는 함수는 view의 extension으로 작성
• textfield 코드 간결화를 위해 makeTextField function으로 분리
• button은 공통 component가 나오기 전까지 임시
• 올바른 code를 입력했는지 검사하는 로직 구현
• UserDefaults를 통해 회원가입 이미 한 유저인지 정보 저장

리뷰포인트

• private으로 선언한 makeTextField function의 구현방식이 옳은지 확인 부탁드립니다.
• 위의 makeTextField function에 optional한 function을 pass 하도록 구현하고 사용해봤는데 해당 부분이 올바르게 작성되었는지 확인 부탁드립니다

다음으로 진행될 작업

• layout 위치 조정
• 및 #17 작업 리스트

이미지

질문

• optional function은 어떻게 unwrapping해서 사용할까요?
• onAppear를 사용했을 때 focusState가 작동하지 않는 문제가 있어 asyncAfter로 시간차를 주었습니다. Help
• 레이아웃 구성하는 부분에서 항상 어려움이 있습니다. 내일 도움 주시길 바랍니다

기타

References

optional function을 parameter로 넘기고 사용하는 방법 https://stackoverflow.com/questions/24395853/how-does-one-make-an-optional-closure-in-swift

focusState https://www.raywenderlich.com/31569019-focus-management-in-swiftui-getting-started

작업사항

사용법

ProfileImage(User(id: "1", nickname: "아보", profileImage: nil), diameter: 100)

매개변수로 User 타입과 프로필 사이즈를 주면 됨

리뷰포인트

• User와 프로필 사진 크기를 매개변수로 받음
• User.profileImage가 nil이면 텍스트를, 주소가 있으면 사진을 불러옴
• 텍스트 크기는 프로필 크기에 따라 변하도록 만듦

다음에 할 작업

• 기본 프로필의 색상 변경
• 기본 프로필의 텍스트 폰트 변경

매 뷰마다 파이어베이스에서 유저 정보를 불러와서 url을 통해 이미지를 다운받는 것은 뭔가 안 좋을 것 같습니다. 유저 정보를 어떻게 관리해야될지 도움을 요청합니다!

• 어플 실행시 유저 정보를 불러와서 닉네임과 프사를 (어플이 켜져있는 동안만) 저장해두고, 다른 뷰에서 갖다 쓰는지,
• 아니면 닉네임과 프사는 어차피 프로필 관리 화면에서만 바꿀 수 있으니까 기기에 저장해두고 프로필 관리 화면에서 변경시 기기에 저장된 것을 업데이트함과 동시에 파이어베이스에도 업로드 해야 하는지,
• 아니면 파이어베이스가 알아서 캐시까지 다 해주는지(캐시로 해줘도 이미지가 url이라 의미 없을것이라 예상).

백엔드?에서 데이터를 가져와서 쓰는건 처음이라 너무 어렵네요ㅠㅠ 도와주세요!

Description

탭뷰의 첫번째 화면인 "택시팟" UI를 구현할 예정입니다.

To-do

• [ ] LazyVStack로 현재 생성되어 있는 모든 택시팟 채팅방 CellView를 로딩하여 보여준다.
• [ ] ScrollView 밑에 LazyVStack를 구현하여 채팅방 CellView를 스크롤하여 볼 수 있도록 한다.
• [ ] CellView를 날짜별로 Daily 구별하여 보여주어야 한다.
• [ ] 택시팟 CellView를 선택하면 '아보'가 구현한 택시팟 정보 View와 연동 예정
• [ ] "+" 버튼을 UI 오른쪽 상단에 구현하여, '호종E'가 만든 택시팟 생성 View와 연동 예정
• [ ] "날짜 선택" 버튼을 UI에 구현하여, '레이몬드'가 만든 half-modal 달력과 연동 예정

Etc

1. LazyVStack / VStack / List 생각해보기 Loading할 데이터가 많으면 LazyVStack이 필수겠지만, 이번 프로젝트에서는 VStack또는 List로 구현해도 될 것같은 생각이 든다. 확장성 및 사용량을 고려하여 LazyVStack로 미리 구현해보는 것도 좋을지도. (이전 프로젝트때는 LazyVStack으로 Dumy데이터 셀을 불러와서 구현하였음.)

Description

Xcode SwiftLint 적용

To-do

• [x] (개인 별) mac os swiftLint 설치
• [x] (호종이 작업) xcode swiftLint run script 추가
• [x] 요셉
• [x] 아보
• [x] 조이
• [x] 제리
• [x] 레이몬드

Etc

Mac os에 SwiftLint 설치하는 법

1. Mac os 에 home brew 패키지 매니저 설치하기 Home brew 설치 가이드 사이트
2. 터미널에 다음 명령어 입력

brew install swiftlint

3. 각자 mac 에 homebrew 로 swiftLint 설치 후 task 완료 바랍니다.

작업사항

ProfileView에서 프로필 사진 수정할때, 노란색 버튼 영역을 터치하였을때에도, 프로필 사진 수정이 가능하게끔 Fix

리뷰포인트

"프로필 사진 변경한 경우", "프로필 사진이 있는 경우", "프로필 사진이 없는 경우"에 모두 적용되도록 ZStack을 Button_label 라벨안에 넣어 Image("profileImage_plus")을 적용하였습니다.

var profileImageEditButton: some View {
    Button {
        showActionSheet.toggle()
    } label: {
        ZStack {
            if let newImage = selectedImage { // 프로필 사진을 변경한 경우
                Image(uiImage: newImage)
                    .profileCircle(profileSize)
            } else if let imageURL = imageContainer { // 프로필 사진이 있는 경우
                WebImage(url: URL(string: imageURL))
                    .profileCircle(profileSize)
            } else { // 프로필 사진이 없는 경우
                textProfile(profileSize)
            }
            Image("profileImage_plus")
                .resizable()
                .frame(width: 48, height: 48)
                .opacity(0.8)
        }
    }
}

마이팟에서 참가한 택시파티를 떠나고 MyPartyViewModel의 myPartyList를 변경해줘야 하는데 현재는 삭제요청이 성공하면 다시 파티를 불러오고 있습니다. (관련 커밋 : 730aa33)

func leaveMyParty(user: User, party: TaxiParty) {
         myPartyUseCase.leaveTaxiParty(party, user: user) { [weak self] error in
             guard let self = self else {
                 print(error ?? "")
                 return
             }
             print("Leave Party, user:\(user.id), party:\(party.id)")
             self.getMyParties(user: user)
         }
     }

이때 서버에서 다시 불러오지 않고 MyPartyViewModel의 myPartyList에서 직접 제거해주는 것이 더 좋을까요?

private func deletePartyInList(party: TaxiParty) {
         if let index = self.myPartyList.firstIndex(of: party) {
             self.myPartyList.remove(at: index)
         }
     }

• 택시팟에서 생성하거나 참가하기 전까지 내 택시팟은 추가되지 않음
• 다시 불러오는 것보다 효율적인 것 같음

이런 이유 때문에 직접 제거해주는 것이 좋을 것 같다가도 서버에서 불러와서 맞춰주는 것이 좋을 것 같아서 헷갈려요.. 어떤 방법이 좋을지 의견 부탁드립니다!

선행 작업

• [x] #38
• [x] #69

Description

• 앱의 전체 사이클에서 지속될 유저 데이터 관리
• 닉네임 변경, 프로필 사진 변경 등의 유즈케이스

질문

• 파일 이름과 파일 위치가 어떻게 되어야 할지
• 제가 할 수 있을까요? ㅠㅠ

To-do

• [x] 각 유즈케이스 호출 메서드 작성
• [x] User가 옵셔널일 때

Etc

• 전 못할 것 같아요

접근 제어

접근지정자란 public, private 과 같은 키워드로, 클래스, 구조체, 프로퍼티 등의 은닉화 를 위해 사용됩니다.

우리가 항상 SwitUI View 구조체를 만들 때, @State 프로퍼티에 private 접근 지정자를 작성하라는 이유는, 해당 State 값이 작성된 View 내부에서만 사용가능하기 때문입니다. 즉 외부에서 접근조차 하지 못하게 만들어서 개발자의 실수로 생길 수 있는 오류를 미연에 방지하기 위함이기도 합니다.

또한 블로그 글 에 따르면, 접근 제어는 다음과 같은 장점들을 가집니다

• 접근 지정자의 작성은 그 자체로 communication 의 의미를 가진다. 개발자가 private 으로 지정한 인스턴스 메소드나 프로퍼티는, 절대 override 할 수 없고 내부 로직만을 위한 프로퍼티임을 의미한다. 다른 개발자들은 암시적으로 메소드나 프로퍼티의 사용 범위를 알 수 있다. 신중하게 선택된 하나의 단어가 코드를 문서화 시키는 것이다.
• 상수와 변수 (let, var) 또한 그 자체로 communication 의미를 가진다고 생각합니다. 작성하는 개발자는 아무 생각 없이 var 와 let 을 혼용해서 쓸 수 있지만, 그것을 리뷰하는 다른 개발자들은 해당 프로퍼티가 변경의 여지가 있는지 없는지 나머지 코드를 보면서 스스로 판단해야 합니다.
• swift performance 가 향상된다. Method dispatch 라는 흥미로운 주제에 관해 이야기하는데, 관심 있으시면 읽어보세요

다음과 같은 클래스가 있다고 가정해봅시다.

class TestClass {
  func getComputedValue(value: Int) -> Int {
    let processedValue: Int = valueProcessing(value)
    return computeValue(processedValue)
  }

  func valueProcessing(value: Int) -> Int {
   //..,
  }
  
  func computeValue(value: Int) -> Int {
    //...
  }
}

• 위 클래스는 Int 데이터를 받아서 특정한 전처리를 한다음, 전처리한 데이터에 특정 연산을 거치고 유저에게 최종적으로 값을 반환합니다.
• 해당 클래스를작성한 개발자의 의도는 유저가 getComputedValue 만 호출해서 원하는 동작을 수행하는 것입니다.
• 그러나 다른 개발자들은valueProcessing, computeValue 메소드에도 접근할 수 있는데, 이 자체로 혼란을 줄 수 있습니다.
• 해당 메소드들은 어떤 용도에 사용하는지 직접 생각해야하고, 최악의 경우 한 명 한 명 직접 찾아와서 물어볼 수도 있겠죠.
• 하지만 처음 작성할 때부터 private 메소드로 작성했다면, 이런 상황은 미연에 방지할 수 있습니다.

접근 제어의 역할은 이렇게 중요하고, 다음과 같이 View 내부적인 연산을 위해 필요한 프로퍼티들도 되도록이면 접근제어를 세심히 고려해주시기 바랍니다.

struct TestView: View {
  private let day: [String] = ["월", "화", "수", ....]
  var body: some View {
    //...
  }
}

커밋할 때 불필요한 파일은 제외하기

이번 프로젝트를 하면서 가장 많이 나온 상황은, 직접 만든 View 를 테스트하기 위해 최상위 View에 직접 만든 뷰를 작성한 것입니다. 또한 여러가지 이유로 다른 파일들을 수정할 수 있습니다. 다른 클래스나 구조체 등을 테스트하기 위해 직접 코드를 수정해볼 수 있습니다. 그러나 단순 테스트를 위해 다른 파일을 수정했으면, 다음 커밋에서 해당 변경사항들은 모두 제외해야합니다. 후에 충돌의 원인이 될 수도 있고, 리뷰하는 사람도 의아합니다.

커밋 전 항상 변경사항을 찬찬히 살펴보는 시간을 가지고 원할한 협업을 하도록 합시다.

@pis3120 @SH0123 @Chaeho-Min @YoonyoungL @jsyworks

Description

현재는 단순히 메시지 배열이 존재하고, 리스트 형태로 한 메시지당 단순한 하나의 message row 로 표현하는 등 많은 메시지가 쌓일 경우 메시지의 가독성이 줄어든다는 단점이 있습니다. 따라서 K사의 메신저처럼 한 사람이 연속적으로 보내는 메시지의 경우 합쳐주거나, 날짜가 바뀔 때 날짜 표시를 해주는 등의 추가 작업이 필요할 것 같습니다! 해당 알고리즘의 요구사항에 대해 팀원들 간 회의가 필요해 보입니다!

Etc

메시지 최적화에 대해선 별 생각이 없었는데, 메시지 pagination 등 이제 좀 더 기술적으로 어려운 부분까지 구현해야 한다고 봅니다. 앱이 성공할지도 미지수고 얼마나 많은 유저가 사용할진 모르겠지만, 시간이 지날수록 앱이 점점 느려지는 것을 방지하려면 이제 좀 더 deep dive 해야 한다고 봅니다.

@Chaeho-Min @YoonyoungL @jsyworks @SH0123 @pis3120