#dev 에러 로그 개선 버그픽스 Option.get 하는 부분 pattern match 하게 변경 구삼이 내준 퀴즈 Option.get은 세 가지로 변경 가능하다.

// 문제 : x가 None일 때 0, Some[String]일 때 문자열의 개수를 리턴하세요.
// 조건 : 패턴매칭 / fold / Option map 세 가지의 방법을 모두 활용해야 함

import scala.util.Random

val x: Option[String] = if (Random.nextBoolean()) Some(Random.nextString(Random.nextInt(1024))) else None

println(x)

val foldResult = x.fold[Int] {
     0
     } { stuff =>
     stuff.length
     }

val patternMatchResult = x match
	case None => 0
	case Some(value) => value.length

val mapOptionResult = x.map[Int](_.length) getOrElse 0

println(s"""
|patternMatch :$patternMatchResult
|fold: $foldResult
|mapOption: $mapOptionResult""")

그리고 wartremover에 나와 있는 가이드는 Option.get 메서드를 사용하는 것을 권장하지 않는다. None일 경우 fail하여 fiber가 죽을 수 있기 때문.

#도메인 관련 ETF > 종목을 여러개 묶어서 주식을 구매하는 경우, (trbc id가)산업이 null일 수 있음.

#로그

ZIO.logWarning(s"CompanyAnalyze#top10CompaniesInTheSameIndustry: No trbc Id for infocode $infocode")

처럼 변경에 취약한 메시지는 권장하지 않음. 또한 외부에서 호출하는 함수의 이름을 로그에 포함시키는 경우 순환참조가 될 수 있음.

#슬랙 링크

1. for (a <- as) yield a 는 a 랑 같습니다. a.map(identity) === a 인거랑 같은 원리

2. 스칼라는 자바보다 타입 추론을 잘 해줘요. Map[String, String](...) 처럼 타입파라미터를 명시해 주는건 간혹 컴파일 실패하면 그때 해주셔도 됩니다.

3. 모듈의 공개 API에 타입 명시하기를 권장하는 거랑 다른 얘기에요.

4. someZIO *> ZIO.succeed(x) 는 [someZIO.as](http://somezio.as/)``(x) 로 줄여 쓰실 수 있어요.

5. ZIO for IntelliJ 플러그인을 설치하시면 이런 경우 개선안을 제안해 줍니다.

@sean과 페어하면서 배운 것 #dev 패턴매칭 스코프 외부에서 데이터가 추가되는 경우 tuple 또는 map을 활용해보기 #도메인 delistdate는 상장폐지된 날짜. null이면 상장폐지 안된 것임 atlasian에 작성할 때 관련 context를 담아서 간결하게 말하기 개발자 외 사람들도 있으니 최대한 코드 정보보단 서비스 실행의 맥락에서 말하기

주의할 점

• 버그를 고치고 잘 되었는지 정상동작하는지 꼭 확인하고 커밋하기
• 커밋을 하고 바로 push하지 않고 꼭 셀프리뷰를 한 이후 push하기

고민할 점

• 더 나은 에러/로그 메시지는 무엇인지?
• 에러와 로그의 레벨은 정확히 어떻게 구분되는 것인지?

좋았던 점 sean이 가르쳐준 내용을 잊지 않게 여러 번 반복해서 알려줌 무엇을 해야 하는지 지속적으로 revoke함

개선할 점

• 무엇을 하고 있는지 확인하면서 코딩하기
• 무엇을 하고 싶은지도 확인하면서 코딩하기