A introdução de virtual threads (#195) causou um bug interessante: como as threads de JogadorConectado não são threads propriamente ditas, a VM não espera elas encerrarem pra dar shutdown, fazendo com que o soft shutdown (#185) falhe (o servidor cai imeditamente ao finalizar a thread que escuta conexões).
Para corrigir isso, foi preciso manter uma collection com as threads de JogadorConectado em execução (o que, por sua vez, exigiu que fosse implementado um callback naquela classe para que MiniTrucoServer saiba quando um jogador finaliza e tire ele da collection). Quando o listen é interrompido, a thread que escuta as conexões (a única coisa mantendo a JVM no ar nessa hora) espera que não haja mais nenhum jogador conectado antes de encerrar.
Com essa coleção em mãos, ficou fácil implementar dois outros itens de #38: um jeito fácil de ver quantos jogadores estão conectados (bastou logar um evento quando a coleção é modificada: "Jogadores conectados: <tamanho da coleção>", que é facilmente grepável) e um limite do número de jogadores conectados (quando uma conexão entra e a coleção já está no limite daquele tamanho, manda uma notificação toast e desconecta).
Poderia fazer o máximo ser alterável dinamicamente, mas vamos pensar MVP aqui.
chesterbr
commented
1 year ago
A introdução de virtual threads (#195) causou um bug interessante: como as threads de
JogadorConectadonão são threads propriamente ditas, a VM não espera elas encerrarem pra dar shutdown, fazendo com que o soft shutdown (#185) falhe (o servidor cai imeditamente ao finalizar a thread que escuta conexões).Para corrigir isso, foi preciso manter uma collection com as threads de
JogadorConectadoem execução (o que, por sua vez, exigiu que fosse implementado um callback naquela classe para queMiniTrucoServersaiba quando um jogador finaliza e tire ele da collection). Quando o listen é interrompido, a thread que escuta as conexões (a única coisa mantendo a JVM no ar nessa hora) espera que não haja mais nenhum jogador conectado antes de encerrar.Com essa coleção em mãos, ficou fácil implementar dois outros itens de #38: um jeito fácil de ver quantos jogadores estão conectados (bastou logar um evento quando a coleção é modificada: "Jogadores conectados: <tamanho da coleção>", que é facilmente
grepável) e um limite do número de jogadores conectados (quando uma conexão entra e a coleção já está no limite daquele tamanho, manda uma notificação toast e desconecta).Poderia fazer o máximo ser alterável dinamicamente, mas vamos pensar MVP aqui.