help motor

help with this please code , paragraph Operate OS 4 Engines If Tighten W, w OR 8

#include <AFMotor.h>
AF_DCMotor motor1(1);
AF_DCMotor motor2(2);
AF_DCMotor motor3(3);
AF_DCMotor motor4(4);
byte letra;

void loop()
{
if (Serial.available() > 0)
{
letra = Serial.read();
switch(letra)
{
case '8':
case 'w':
case 'W':
motor1.setSpeed(255);
motor2.setSpeed(255);
motor3.setSpeed(255);
motor4.setSpeed(255);
motor1.run(FORWARD);
motor2.run(FORWARD);
motor3.run(FORWARD);
motor4.run(FORWARD);
break;
}
letra = Serial.read();
letra = 0;
}
}

When there is one byte of data to read, reading two bytes is not good practice. Throwing one of them away is rarely a good idea, either.

The code you incorrectly posted does something. You did not explain what it actually does.

You want the code to do something. You need to describe what it is you want the code to do.

If I Hit W he triggers OS Engines

So does it not work?

If I Hit W he triggers OS Engines

Is that what you expect? If so, what is the problem? If not, what DO you expect?