s-fam/pyeonhaeng

편행(편의점 할인 행사 앱 프로젝트) v1.0

pyeonhaeng's Issues

PR #23 UIColor extension 관련 추가 의견

Onboarding 화면에 기본 색상을 적용하다가 closed된 23번 PR(UIColor extension관련)에 개인적인 의견이 생겨 글 올립니다.
지난 pr인데 다시 얘기 꺼낸 점 죄송합니다ㅠ
기본 색상은 아래 사진에 보이듯이 네이비 색상에 가까운 색을 말합니다. (색상 코드 : #030026)

▼ 현재 UIColor+.swift의 코드

import UIKit

extension UIColor {

  convenience init(hex: Int, alpha: CGFloat = 1.0) {
    let components = (
      red: CGFloat((hex >> 16) & 0xff) / 255,
      green: CGFloat((hex >> 08) & 0xff) / 255,
      blue: CGFloat((hex >> 00) & 0xff) / 255
    )
    self.init(red: components.red, green: components.green, blue: components.blue, alpha: alpha)
  }
}

// MARK: - Store's SymbolColor
extension UIColor {

  static let cuBackGroundColor = UIColor(hex: 0x51485)
  static let cuFontColor = UIColor(hex: 0x9DC92A)
  static let gsBackGroundColor = UIColor(hex: 0x63514D)
  static let gsFontColor = UIColor(hex: 0x00D7F1)
  static let seBackGroundColor = UIColor(hex: 0xFF8329)
  static let seFontColor = UIColor(hex: 0x005B45)
  static let msBackGroundColor = UIColor(hex: 0x003893)
  static let msFontColor = UIColor(hex: 0xF0F0F0)
  static let emBackGroundColor = UIColor(hex: 0x56555B)
  static let emFontColor = UIColor(hex: 0xFFB41D)
}

• 의견 1)
  기본 색상도 추가해주세요. 기본 색상은 첨부한 첫 번째 사진에 보이듯이 네이비 색상에 가까운 색을 말합니다. (색상 코드 : #030026)
• 의견 2)
  편의점 이름을 풀네임으로 적어주시면 감사하겠습니다. (se -> sevenEleven, ms -> miniStop, em -> emart)
• 의견 3)
  hex 값은 String으로 받는게 맞다고 생각합니다. 이건 제가 23번 pr을 제대로 확인하지 않았던 것 같습니다. 그때 말씀드렸어야 했는데 죄송합니다ㅠ Figma 상에 색상표가 아래 사진 처럼 나와있고 혹여나 나중에 색상이 변경될 시 이 색상표를 토대로 코드도 변경해야할 텐데 바로 String 값을 넣어서 처리하는 게 좋을 것 같아요!
  

Enum Refactoring

건의사항

1. 색상 중복

편의점별 색상 코드가 중복되어있습니다.

1-1. UIColor+.swift

https://github.com/iOS-PPAK/PPAK_CVS/blob/63d576c65a9aee9013b1a10de21f9ea48505c0fc/PPAK_CVS/Configuration/Extensions/UIColor%2B.swift#L23-L35

1-2. CVSType.swift

https://github.com/iOS-PPAK/PPAK_CVS/blob/63d576c65a9aee9013b1a10de21f9ea48505c0fc/PPAK_CVS/Configuration/Constants/Enum/CvsType.swift#L46-L67

편의점 별 색상코드는 UIColor+.swift대신 CVSType.swift에 통합하여 적용하는건 어떤가요? 여러분들은 어떻게 생각하시나요?

2. Codable

CVSType과 EventType Codable을 준수하도록 변경하고 싶어요. Firebase에서 값을 가져올 때 만들어져있는 Enum을 재사용하는 게 좋을 거 같아서요!