Motors are not working in else part

Other Hardware Motors, Mechanics, Power and CNC
1

#include <SPI.h>
#include <nRF24L01.h>
#include <RF24.h>
#include <AFMotor.h>
AF_DCMotor motor1(1);
AF_DCMotor motor2(2);
AF_DCMotor motor3(3);
AF_DCMotor motor4(4);
RF24 radio(10,11);
const byte address[6] = "00001";

void setup() {
// put your setup code here, to run once:
radio.begin();
radio.openReadingPipe(0,address);
radio.setPALevel(RF24_PA_MAX);
radio.setDataRate(RF24_250KBPS);
radio.startListening();
}

void loop() {
if (radio.available()) {
char text[32] = "";
radio.read(&text, sizeof(text));
String transData = String(text);
if (transData == "nrftest") {
motor1.setSpeed(225/3);
motor2.setSpeed(225/3);
motor3.setSpeed(225/3);
motor4.setSpeed(225/3);
motor1.run(FORWARD);
motor2.run(FORWARD);
motor3.run(FORWARD);
motor4.run(FORWARD);
delay(4000);
motor1.run(RELEASE);
motor2.run(RELEASE);
motor3.run(RELEASE);
motor4.run(RELEASE);
}
}
else
{
motor1.setSpeed(225);
motor2.setSpeed(225);
motor3.setSpeed(225);
motor4.setSpeed(225);
motor1.run(FORWARD);
motor2.run(FORWARD);
motor3.run(FORWARD);
motor4.run(FORWARD);
delay(4000);
motor1.run(RELEASE);
motor2.run(RELEASE);
motor3.run(RELEASE);
motor4.run(RELEASE);
}
}

2

Here is loop() with proper indentation to show the code blocks (and posted in the proper code tags):

void loop() {
  if (radio.available()) {
    char text[32] = "";
    radio.read(&text, sizeof(text));
    String transData = String(text);
    if (transData == "nrftest") {
      motor1.setSpeed(225 / 3);
      motor2.setSpeed(225 / 3);
      motor3.setSpeed(225 / 3);
      motor4.setSpeed(225 / 3);
      motor1.run(FORWARD);
      motor2.run(FORWARD);
      motor3.run(FORWARD);
      motor4.run(FORWARD);
      delay(4000);
      motor1.run(RELEASE);
      motor2.run(RELEASE);
      motor3.run(RELEASE);
      motor4.run(RELEASE);
    }
  }
  else
  {
    motor1.setSpeed(225);
    motor2.setSpeed(225);
    motor3.setSpeed(225);
    motor4.setSpeed(225);
    motor1.run(FORWARD);
    motor2.run(FORWARD);
    motor3.run(FORWARD);
    motor4.run(FORWARD);
    delay(4000);
    motor1.run(RELEASE);
    motor2.run(RELEASE);
    motor3.run(RELEASE);
    motor4.run(RELEASE);
  }
}

Did you intend to nest the code blocks in this way? The 'else' depends on 'radio.available()'.

3

To which if() should be related this else?

first:

or second ?

4

Thank you