tomatophobia / allowance-chart Goto Github PK

현재 거래내역 입력을 수동으로 할지 자동으로 할지, 입력한다면 포맷 어떻게 될지, 수수료는 명시적으로 표시되는지 아닌지 정보가 없기 때문에 예측에 의해 도메인 모델이 형성되고 있다. 따라서 정확한 유즈케이스를 하나 보고 가는 것이 좋다

1. 스크롤 내려와 나오는 것 그냥 보이도록 변경
2. 길게 말로 쓰는 대신 아이콘 사용
3. 링크드인 링크도 있으면 좋을 듯

usd -> 소수점 2자리,
krw -> 소수점 없음 등

지금은 test 감지를 못함..

로컬 환경에서 테스트는 minio를 사용하면 좋을 듯

Asset을 사용하는 의미가 없어지고 Account에 의존하기 때문에 테스트도 불편해진다

고려해야 할 점: 수익률을 보수적으로 잡기 위해 매수 시 평단가를 올림했는데 이 때문에 계좌 총 평가액이 매수 후 올라감

Originally posted by @tomatophobia in #15 (comment)

실제 매매 후 시간이 지나고 거래내역이 들어와서인지 왜인지는 모르겠지만 하루에 매도와 매수가 동시에 일어나는 경우 매도 거래내역이 먼저 들어오고 그 다음 매수 거래내역이 들어오는 경우가 있다. 이 경우 증권의 갯수가 음수가 되므로 에러가 발생해버린다

예) PBW 매도 다음에 매수가 들어옴

거래일,거래구분,통화,거래금액,질권일,입금환율,종목코드,수량,유가잔고,국내세,제미납금,외화결제금액,거래상대명
순번,적요명,환전,환전금액,상환금액,출금환율,종목명,단가,수수료,현지세,연체/신용이자,외화예수금,원화예수금
2021.1.13,해외증권장내매매,USD,122.7,,,PBW,1,-1,,,,
1,현금매도,,,,,Powershares Wilderh Clean En,122.7,0.1,0.01,,"1,065.16",10
2021.1.13,해외증권장내매매,USD,,,,PBW,1,0,,,,
2,현금매수,,,,,Powershares Wilderh Clean En,122.68,,,,"1,065.16",10
2021.1.13,해외증권장내매매,USD,247.64,,,PBW,1,1,,,,
3,현금매수,,,,,Powershares Wilderh Clean En,124.96,0.2,,,817.32,10

파일 업로드 후 서버가 받는 것까지

http4s/http4s#2314 해당 이슈가 처리되면 해결될 수도 있음.

대표적으로 현재 DataTables 라이브러리를 webjar를 통해 사용하고 있는데 해당 라이브러리 내부의 이미지를 라이브러리 코드에서 로드할 때 버전을 뺀 url로 리소스를 요청하는데 해당 요청이 실패함. #62

spring에서는 webjar-locator를 사용한다고 함.

DaishinEntry, TransactionRecord 에서 잘못 사용하고 있음

못 찾겠다 꾀꼬리..

처음에는 주식 매수 후 평단가가 올라가는 상황을 괜찮다고 생각했는데 이를 적절히 테스트할 상황이 생각나지 않음. 생각해보니 상황이 애매해서 그런 것 같음.

따라서 매수 할 때 평단가를 구하기 위해 주식들을 합쳐서 관리하지 않고 매수한 것들을 따로 관리한다.

단 이렇게 할 경우 수익금 계산에서 선입선출법으로 계산하게 된다. 양도소득세 계산시 증권사에 따라 이동평균법, 선입선출법이 따로 들어간다. 어떻게 할까나...? 일단 양도소득세 자체를 증권 계좌에서 내는 것은 아니니까 이를 제외하고 생각해도 될 것 같긴하다.

Originally posted by @tomatophobia in #27 (comment)

중심이 되는 도메인 모델들을 기획하고 엉성하게 엮어본다

EventSourceAccount 파일이 너무 비대해지는 것도 방지하고 reduce 로직도 분리해서 볼 수 있다. 분리한 로직에 대해서는 따로 unit 테스트를 만든다.

https://scalalandio.github.io/chimney/transformers/customizing-transformers.html#transforming-coproducts

side effect가 있으므로 Instant.now() 삭제하는 것이 좋을 듯

test, test-coverage 등등 옛날에 라우가 했던 것들 참고

근데 생각해보니까 거래 비용은 증권사의 수수료 + 세금인데 이 때 세금이 증권사랑 관련 없고 시장과 관련이 있음. 지금 코스피, 미국 시장에 투자하고 있어서 수수료 계산할 때 결국 마켓에서 비롯되는 세금, 계좌에서 비롯되는 수수료를 분리해야 한다.

Originally posted by @tomatophobia in #35 (comment)

슬랙에 질문 올린 것

안녕하세요! wartremover를 켜고 scala로 코딩 중에 있습니다. tapir를 이용해 엔드포인트를 만드는 중 multipartBody를 넣어주는 부분에서 JavaSerializable로 타입 인퍼런스가 되는 것 때문에 에러가 발생하고 있는데요. 되도록이면 suppress warning을 안쓰고 싶은 마음에 여러가지 수정을 해보았는데요. (case class 뒤에 extends Product with Serializable 을 붙여본다던가..) 정확한 원인을 이해하지 못해 헛다리 짚고 있는 느낌입니다. 😅 (sttp, tapir의 코드를 제가 잘 이해하지 못한 점도 있을 것 같습니다.)
혹시 wartremover 쓰시는 분께서는 Serializable, JavaSerializable 옵션 등을 이 경우 어떻게 하시는 편이신가요? 고쳐야 하는 부분인지 false positive 정도로 생각하고 무시해도 되는 것인지 확신이 들지 않아 질문드립니다!

final case class NameWithFile(name: String, file: Part[TapirFile])

val someEndpoint = 
    endpoint.post
      .in("post-file")
      .in(multipartBody[NameWithFile]) // [wartremover:JavaSerializable] Inferred type containing Serializable: sttp.model.Part[Comparable[_ >: java.io.File with String <: java.io.Serializable] with java.io.Serializable]

구삼:
아마 MultipartCodec 인스턴스를 유도하는 매크로 때문이 아닐까 싶은데요, 저라면

1. suppress warning 했을 때 정상작동하는지부터 보고
2. 작동을 하면 NameWithFile의 암시적 MultipartCodec 인스턴스를 직접 만들어 보거나
3. MultipartCodecDerivation 코드를 복붙해서 조금씩 고쳐보면서 실제 원인을 파악하거나 할 것 같네요.
   서버에서만 쓰실거면 TapirFile 타입 앨리어스 대신 java.io.File 이나 java.io.InputStream 등을 직접 쓰는 것도 고려해 보세요.

도메인 모델을 전 계층에서 사용하면 변경 요구사항이 있을 때 영향을 받는 곳이 너무 많아진다. 추후 필요하다고 판단될 경우 계층별로 적절한 모델을 만들고 중간 매핑을 해야 한다.

Equity 엔티티가 Account들을 여러 개 보관한다

현재 테스트 코드로만 실행하였고 실제 앱 실행은 안하고 있음

aecor의 Enriched와 비슷한 기능 있을 것 같은데 찾아보기

투자하는 국가별로 분류한 투자현황을 보고 싶다.

시장 정보를 추가하고 시장 정보 내부에 국가를 넣을지 아니면 시장과 국가를 따로 다 추가할지

해당 주식이 어떤 시장에 상장되어 있는지, 주식을 표현하는 symbol, 상장된 국가?도 필요하면 넣기