I am currently try to use the web control led code template to control the motor. The setup is to connect the photon with L293D motor module . My logic is 
When I click âonâ in html = the motor moves clockwise & anticlockwise motion interchanging
When I clixk âoffâ in html = it stops
the code is like
// -----------------------------------
// Controlling LEDs over the Internet
// -----------------------------------
// First, let's create our "shorthand" for the pins
// Same as in the Blink an LED example:
// led1 is D0, led2 is D7
int led1 = D2;
int led2 = D3;
// Last time, we only needed to declare pins in the setup function.
// This time, we are also going to register our Spark function
void setup()
{
// Here's the pin configuration, same as last time
pinMode(led1, OUTPUT);
pinMode(led2, OUTPUT);
// We are also going to declare a Spark.function so that we can turn the LED on and off from the cloud.
Spark.function("led",ledToggle);
// This is saying that when we ask the cloud for the function "led", it will employ the function ledToggle() from this app.
// For good measure, let's also make sure both LEDs are off when we start:
digitalWrite(led1, LOW);
digitalWrite(led2, LOW);
}
// Last time, we wanted to continously blink the LED on and off
// Since we're waiting for input through the cloud this time,
// we don't actually need to put anything in the loop
void loop()
{
// Nothing to do here
}
// We're going to have a super cool function now that gets called when a matching API request is sent
// This is the ledToggle function we registered to the "led" Spark.function earlier.
int ledToggle(String command) {
/* Spark.functions always take a string as an argument and return an integer.
Since we can pass a string, it means that we can give the program commands on how the function should be used.
In this case, telling the function "on" will turn the LED on and telling it "off" will turn the LED off.
Then, the function returns a value to us to let us know what happened.
In this case, it will return 1 for the LEDs turning on, 0 for the LEDs turning off,
and 1 if we received a totally bogus command that didn't do anything to the LEDs.
*/
if (command=="on") {
for (int x =0 ; x < 1000 ; x++){
digitalWrite(led1,HIGH);
digitalWrite(led2,LOW);
delay (500);
digitalWrite(led1,LOW);
digitalWrite(led2,HIGH);
delay (500);
}
}
else if (command=="off") {
digitalWrite(led1,LOW);
digitalWrite(led2,LOW);
return ;
}
}
However, when I tried to operate, like first i click on, it can perform correctly with interchanging direction, but i try to click off, it stops for 1 second and continue to start moving interchanging
May I ask whether i can stop the motion by improving the code?
