0-9194 / 0-9194/mmorpg-servers

World - Positioning of characters translated to tiles instead of absolute positions

Abierto
#14 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Performance World Server
Lenguaje dominante
JavaScript
Estrellas
1
Forks
0
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Actually the positioning of characters are in absolute pixels. Any value between the boundaries of the room is valid.

This leads to some positional bugs, since the server and client relies on multiples of pixels (actually 16), if a player happens to be spawned on a number that do not respect this tile size, the validation of movement on server and the tile arrival script on client will not work as expected.

The idea is to, instead of save the absolute pixel value of X and Y, save the tile position (multiple of tile size) on each axis.
For example:

Today, to represent a 320 x 160 position, we save the value as is.
With this new strategy, we save a value representing the multiples. If tile size is 16, than we would save 20 x 10.

Another improvement will be that the movement validation on server and tile arrival on client will need less overhead.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Encuentra dónde se almacenan y actualizan las posiciones de los personajes en el código del servidor, probablemente en una clase de jugador o entidad. Comprende los scripts actuales de validación de movimiento y llegada a casillas. El cambio implica convertir coordenadas absolutas de píxeles a coordenadas basadas en casillas (múltiplos del tamaño de la casilla, por ejemplo, 16). Prueba asegurándote de que la validación del servidor y los scripts del cliente funcionen correctamente con el nuevo sistema de coordenadas.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
javascript, node.js
Área
backend, game-dev
Tipo de issue
Refactorización
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.