hufscheer / spectator-server

7 stars 0 forks source link

[FEAT] 출전 선수 정보 변경 #214

Closed Jin409 closed 1 month ago

Jin409 commented 2 months ago

🌍 이슈 번호

📝 구현 내용

🍀 확인해야 할 부분

이전 데이터 관련

기존에는 출전 / 선발 / 후보를 구분하지 않고 뛰고 있는 사람만 라인업에 올렸기 때문에 기존 데이터베이스에 있던 라인업 선수들은 모두 is_playing 을 true 로 갖게 하도록 했는데 어떻게 생각하시는지?

선발 / 후보 변경 시에 선수의 출전 정보 변경

슬랙에 코멘트 남긴 것처럼 선수의 출전 정보를 저장하는 시점이 두개가 있다고 생각했어

1-1. 라인업 선수가 선발이 될 때마다 isPlaying 을 true 로 바꾸고 후보가 될 때마다 false 로 바꾼다. 그리고 교체 타임라인에서 선발이었던 선수가 나갈 때마다 false 로 바꾼다. 1-2. 게임이 시작하는 시점에 (게임 시작 상태변경 타임라인이 등록되는 경우) 라인업 선수 중 선발인 선수는 모두 isPlaying 을 true 로 바꾼다. 그리고 교체 타임라인에서 선발이었던 선수가 나갈 때마다 false 로 바꾼다.

이 pr 에서는 1-1 로 구현하기는 했어. 왜냐하면 1-2 의 경우에는 상태변경 타임라인이 등록되고 만약 롤백되는 상황이 일어나게 되면 변경 지점이 너무 많아져서 복잡해지지 않을까? 하는 생각이 들었어. 게임 시작 전까지는 사실상 출전 == 선발이니까 괜찮지 않을까? 하는 생각이었어 이거에 대해 어떻게 생각하는지 궁금해.

라인업 선발/후보 변경 시점 관련

슬랙에서 라인업 정보를 변경하는 데에 제한을 둬야 하지 않겠냐고 이야기했던 것도 같은 맥락인데, 경기 시작 이후에 선수의 선발/후보 정보를 변경하게 되면 선발 != 출전이 아니게 되니까 라인업을 더 이상 건드리면 안되지 않을까? 하는 생각이 들었어. 그런데 이렇게 되면 만약 주장이 교체되어 나간다면.. 주장 변경을 못하게 되네

😵‍💫 다른 사람들의 의견이 필요한 것 같아