[Team07 - BMO & ZG] 기본 UI 및 Entity 구현 #12

BMO5 · 2021-05-07T08:03:05Z

작업 내용

기본 UI작업
DTO 설계
TableView 뼈대 구성
CALayer를 통해 볼카운트 구현

고민과 해결

CALayer를 통해 어떡하면 잘 그릴 수 있을지 고민하였습니다. 좌표를 일일이 찍고, 이를 확인하면서 하는게 맞는지 의구심 들긴 합니다!

- 깃 커밋 컨벤션 추가

- 브랜치 전략 추가

- 팀원 추가

Player, Players 등의 모델의 계층 추상화 issue: #4

[iOS] 기본 UI 작업, DTO 작업

ChocOZerO

기본 UI만 맞추기 위해 대략적인 객체 흐름을 잡기위해 그런거 같긴한데, 생성자로도 외부에서도 프로퍼티들을 접근하는 방법이 없어서 감이 잡히지 않네요.

ChocOZerO · 2021-05-10T13:45:07Z

iOS/baseball/baseball/Controller/GameSelectViewController.swift

+        super.init(coder: coder)
+    }
+
+    override func loadView() {


loadView의 라이프사이클을 다시 확인해보세요.
아래 코드중에 loadView타이밍에 처리해야되는 코드는 없어보입니다.

ChocOZerO · 2021-05-10T13:49:23Z

iOS/baseball/baseball/Controller/ScoreViewController.swift

+    private var tempInt: Int = 0 //뷰가 추가되는 효과를 보기 위한 임시 변수
+
+    func isAttack(myTeam: MyTeam) -> Bool {
+        return myTeam.Value == InningInfo().attackTeam


InningInfo 가 struct던데 비교할때마다 매번 객체를 생성해서 비교하나요?

ChocOZerO · 2021-05-10T13:54:27Z

iOS/baseball/baseball/Model/Entity/InningScore.swift

+
+import Foundation
+
+struct InningScore: Decodable {


프로퍼티가 전부 private이고 init시점에도 값을 못 넣는데 이 객체가 원하는 의도를 잘 수행할지 의심이 되네요

ChocOZerO · 2021-05-10T13:55:34Z

iOS/baseball/baseball/Model/Entity/MyTeam.swift

+    case Home
+    case Away
+
+    var Value: String {


calculated property도 프로퍼티이기 때문에 대문자로 시작하는건 일반적인 컨벤션에 어긋납니다

ChocOZerO · 2021-05-10T13:56:24Z

iOS/baseball/baseball/Model/Entity/Pitcher.swift

+
+import Foundation
+
+struct Pitcher:Decodable {


이것도 모두 감춰져있어서 어떤 용도로 사용하는지 감이 안오네요

ChocOZerO · 2021-05-10T14:04:04Z

iOS/baseball/baseball/View/MatchUpCell.swift

+    //storyBoard!!
+    required init?(coder: NSCoder) {
+        super.init(coder: coder)
+        let tapGesture = UITapGestureRecognizer(target: self, action: #selector(didTapCell(_:)))


didTapCell 뒤의 (_:) 는 삭제해도 될거 같은데 확인해보세요

ChocOZerO · 2021-05-10T14:07:01Z

iOS/baseball/baseball/View/MatchUpCell.swift

+    private var delegate: GameSelectViewControllerManageable!
+
+    override init(frame: CGRect) {
+        super.init(frame: frame)


같은 initializer인데 이 방식에선 tapGesture가 필요 없나요?

ChocOZerO · 2021-05-10T14:08:31Z

iOS/baseball/baseball/View/MatchUpCell.swift

+    @IBOutlet weak var awayLabel: UILabel!
+    @IBOutlet weak var gameNumber: UILabel!
+
+    private var delegate: GameSelectViewControllerManageable!


initialize 시점에 할당되지 않는 프로퍼티엔 강제 언래핑 방식을 사용하지 않는 것을 추천합니다. set 함수가 불리기전에 다른 함수가 불려서 delegate에 접근하게된다면 앱 크래시가 유발되기 때문입니다.

ChocOZerO · 2021-05-10T14:09:44Z

iOS/baseball/baseball/View/PitchingHistoryDataSource.swift

+
+class PitchingHistoryDataSource {
+
+    private var dataSource: UITableViewDiffableDataSource<String, String>!


여기에서도 setupDataSource가 호출되기 전에 applySnapshot 이 불리면 앱 크래시를 유발합니다. 순서를 맞춰야 동작하는 방식은 위험합니다.

커스텀 셀 생성

게임 진행 페이지 볼 카운트 셀 생성

Feature/6/game view

Be/feature/#12/data jdbc dependency : Spring Data JDBC의존성 추가 및 스키마 작업

Temporary branch merge

* Update README.md * Update README.md * Update README.md * docs: FE 초기 폴더구조 * feat: 리액트 기본 환경 설정 * feat:[#3] teamSelect 폴더구성 및 Component 제작 중 * chore * feat : [#5] teamScore 폴더 제작 및 TeamScore Component 제작 완성 * feat: [#3] TeamSelect 초기 버전 제작 완성 및 추가 설정 - Styled-components의 ThemeProvider 적용 - 디렉토리 구조 일부 변경 * chore: .DS_Store 제거 * feat: [#11] GamePlay 컴포넌트 구조 설계 완료 - GamePlay 컴포넌트 CSS 정의 (프로토타입) - GamePlay 컴포넌트를 구성하는 모든 파일 생성 - 기능은 없음 (파일만 생성함) - App.js의 GlobalStyle 수정 - 전역으로 수직 / 수평 가운데 정렬 * feat: [#12] GamePlay - GameScore 컴포넌트 완성 (프로토타입) * feat: [#14] GamePlay - MatchScreen 컴포넌트 완성 (프로토타입) * feat: SituationScreen 프로토타입 완성 및 router 적용 - react router dom 활용 - 존재하지 않는 페이지 임시 생성 * chore: 일부 폴더 구조 변경 (TeamScore, TeamSelect) * refactor: [#24] 전체적인 CSS 보정 및 GamePlay 부분 전반적으로 수정 * feat: [#26] GamePlay - BattleGround 프로토타입 완성 * feat: [#30] Proxy 설정 및 TeamSelect API 적용 작업 중.. * feat: [#28] PlayerListPopup 프로토타입 완성 및 일부 네이밍 변경 - 일부 네이밍 변경 - TeamScore -> TeamScorePopup * style: [#28] PlayerListPopup의 Style 변경 & theme.js 일부 속성 제거 - PlayerListPopup의 StyledPlayerListPopup의 CSS 변경 * feat:[#30] API 적용 * chore * feat: [#33] 재사용 되는 PopupFrame 생성 (초기버전 & 활용) - 이 PopupFrame 컴포넌트를 활용하여 PlayerListPopup & TeamScorePopup 컴포넌트 디자인 변경 - TeamScorePopup는 테이블과 전체적 스타일 변경 필요 * feat: [#35] GamePlay BattleGround Animation 추가 * feat:[#35] 애니메이션 수정 * feat: TeamScorePopup 디자인 전체적 변경 및 임시 팝업 기능 * feat: [#40] GamePlay Component API 조작중 * feat: [#41] Popup 컴포넌트들 애니메이션 적용 및 Context 생성 & 일부 구조 변경 * feat: ([#40] 과는 거리가 먼) OAuth를 위한 Login 관련 컴포넌트 생성+ - 일부 Context에 useReducer로 적용 * feat: [#45] GameScore & MatchScreen 데이터 활용하여 표시 - 라이언 아이콘의 State도 GamePlayProvider로 이동. * feat: Oauth Login 구현 * feat: [#45] Ball, Strike, Out Count 컨트롤 (Pitch 버튼 클릭 시) - 초기 타입 * feat: [#45] BattleGround 컴포넌트 데이터 핸들링 (1) - BattleGround - GameScore: 득점 시 점수 변화 - Round: Round 상태 변화 - MatchScreen: 상황에 따라 선수 데이터 변경 - 대체적으로 GamePlayProvider와 BattleGround, GamePlay가 주로 수정되었습니다 * feat: table API 적용 * feat: 안타 or 아웃에 따라 PUT 요청, 점수판 관련 API 요청은 진행중 * chore * chore: Add background Co-authored-by: dudn1933 <[email protected]> Co-authored-by: leehangeul <[email protected]>

kihyuk-sung and others added 12 commits May 3, 2021 14:40

docs: Add git commit convention

11dfe27

- 깃 커밋 컨벤션 추가

docs: Add branch strategy

af35331

- 브랜치 전략 추가

docs: Add teamates

dc890fd

- 팀원 추가

style: 기본UI 구성

0deece4

feat: 모델 추상화

420a1f9

Player, Players 등의 모델의 계층 추상화 issue: #4

feat: 네트워크 get함수 기본 구현

587025d

feat: Create TableView DataSource

2f8e41f

feature: DTO 기본 구성

7f61060

feature: Draw BallCount in InningInfoView

17e858e

style: 점수화면 수정

2dcad6f

fixed gitignore

a0ae3ea

Merge pull request #14 from PizzaCola-K/feature/basicUI

ae7f26e

[iOS] 기본 UI 작업, DTO 작업

kihyuk-sung requested a review from ChocOZerO May 7, 2021 09:30

kihyuk-sung added the review-iOS label May 7, 2021

kihyuk-sung assigned BMO5, Settpark and ChocOZerO May 7, 2021

okstring pushed a commit that referenced this pull request May 10, 2021

[#12] Fix: 테이블뷰 셀 xib에서 스택뷰를 사용하여 오토레이아웃 정상적으로 작동

e7eb995

ChocOZerO reviewed May 10, 2021

View reviewed changes

kyupid pushed a commit that referenced this pull request May 11, 2021

[#12] feat: Create PlayerInformationCell

91e0224

커스텀 셀 생성

kyupid pushed a commit that referenced this pull request May 11, 2021

[#12] feat: Create PlayScreen BallCountCell

8332745

게임 진행 페이지 볼 카운트 셀 생성

sphilee pushed a commit that referenced this pull request May 12, 2021

Merge pull request #12 from Polynomeer/feature/6/game-view

152e914

Feature/6/game view

ksundong pushed a commit that referenced this pull request May 16, 2021

[#12] feat : MySQL DB연동을 위한 설정 추가

23c978d

ksundong pushed a commit that referenced this pull request May 16, 2021

[#12] feat : DB 스키마 스크립트 작성

18d49ae

ksundong pushed a commit that referenced this pull request May 16, 2021

[#12] feat : 앱 실행했을 때 schema파일을 실행하도록 설정 추가

1b11a78

ksundong pushed a commit that referenced this pull request May 16, 2021

Merge pull request #15 from Malloc72P/BE/feature/#12/DataJdbcDependency

a3791fb

Be/feature/#12/data jdbc dependency : Spring Data JDBC의존성 추가 및 스키마 작업

HongzCloud pushed a commit to HongzCloud/baseball that referenced this pull request May 18, 2021

Merge pull request codesquad-members-2021#12 from hiro032/temp

3230349

Temporary branch merge

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Team07 - BMO & ZG] 기본 UI 및 Entity 구현 #12

[Team07 - BMO & ZG] 기본 UI 및 Entity 구현 #12

BMO5 commented May 7, 2021

ChocOZerO left a comment

ChocOZerO May 10, 2021

ChocOZerO May 10, 2021

ChocOZerO May 10, 2021

ChocOZerO May 10, 2021

ChocOZerO May 10, 2021

ChocOZerO May 10, 2021

ChocOZerO May 10, 2021

ChocOZerO May 10, 2021

ChocOZerO May 10, 2021


		class PitchingHistoryDataSource {

		private var dataSource: UITableViewDiffableDataSource<String, String>!


		import Foundation

		struct Pitcher:Decodable {

[Team07 - BMO & ZG] 기본 UI 및 Entity 구현 #12

Are you sure you want to change the base?

[Team07 - BMO & ZG] 기본 UI 및 Entity 구현 #12

Conversation

BMO5 commented May 7, 2021

작업 내용

고민과 해결

ChocOZerO 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