Estoy codificando una aplicación de utilidad para el ajedrez y tengo un patrón de código repetitivo que sigue repitiéndose en diferentes métodos.
Básicamente, estoy rastreando el color del usuario {white, black} . Si el usuario es white , entonces los movimientos que realiza siguen el patrón de moves[i] % 2 ==0 (dado que un jugador blanco siempre comienza el juego).
if (userColor === "white") { if(length % 2 === 0) { max = white = time - timeA; black = time - timeB; } else { max= black = time - timeA; white = time - timeB; } } else { if(length % 2 === 0) { max = black = time - timeA; white = time - timeB; } else { max = white = time - timeA; black = time - timeB; } }Este es un ejemplo en el que uso el patrón para el color del jugador mencionado anteriormente. ¿Alguien ve una manera en la que podría reducir elegantemente este código?
¿Cómo podría simplificar este fragmento de código? Debe haber una manera ya que aquí hay un patrón de simetría.
He intentado escribir un método que toma el color del usuario junto con max white y black , pero parece que siempre vuelvo al mismo código que he escrito, sin progresar.
Puede usar un operador ternario y eliminar las declaraciones if / else anidadas por completo:
if (userColor === "white") { max = (length % 2 === 0 ? white : black) = time - timeA; black = time - timeB; } else { max = (length % 2 === 0 ? black : white) = time - timeA; white = time - timeB; }En su lugar, preferiría hacer una matriz de white y black , en lugar de variables independientes; luego, puede calcular el índice de destino, asignarlo de manera adecuada y hacer lo mismo para el elemento 1 - index también.
const colors = [white, black]; // always use `colors` now instead of white and black // and make userColor into userColorIndex - an index of 0 for white, 1 for black - or use a boolean `userIsWhite` const evenTurn = length % 2; const newMaxColorWhite = userIsWhite && evenTurn || !userIsWhite && !evenTurn; const index = newMaxColorWhite ? 0 : 1; max = colors[index] = time - timeA; colors[1 - index] = time - timeB; La variable newMaxColorWhite no es necesaria ; puede omitirla por completo y definir el índice en una sola línea con el operador condicional, pero creo que aclara la intención del código.
const index = userIsWhite && evenTurn || !userIsWhite && !evenTurn ? 0 : 1; max = colors[index] = time - timeA; colors[1 - index] = time - timeB;También podrías reemplazar
userIsWhite && evenTurn || !userIsWhite && !evenTurncon
userIsWhite === evenTurnpero eso podría no ser tan comprensible.
esta es una lógica XOR
if ((userColor !== 'white') ^ (length % 2)) { max = black = time - timeA; white = time - timeB; } else { max = white = time - timeA; black = time - timeB; }