Oi, Emerson
A função "movimentoplayer1" está repetida
Como deveria ser:
function movimentoplayer1(){
if (keyIsDown(UP_ARROW)) {
yPLAYER1 -= 10;
}
if (keyIsDown(DOWN_ARROW)) {
yPLAYER1 += 10;
}
}
ou forma mais compacta
function movimentoplayer1(){
if (keyIsDown(UP_ARROW)) yPLAYER1 -= 10;
if (keyIsDown(DOWN_ARROW)) yPLAYER1 += 10;
}
Opcional! Para limitar o valor de yPLAYER1 e evitar que a raquete suma da tela, use a função "constrain()"
function movimentoplayer1(){
if (keyIsDown(UP_ARROW)) yPLAYER1 -= 10;
if (keyIsDown(DOWN_ARROW)) yPLAYER1 += 10;
yPLAYER1 = constrain(yPLAYER1, 0, 310);
}