While loop refusues to stop

[Idahowalker]

@bheeduino

delay(6900);

those might mess up the receiving of data. What happens when you try?

Using delay is another thing you might want to get rid of and can be worked on when you try your code from post#41.

And I'd only include one read cmd statement or you risk the cmd being overwritten.

What happens when you send 3 commands in 6900m Seconds? Do they all get registered or do none of the get registered as being received?

[bheeduino]

@Idahowalker

those might mess up the receiving of data.

okay I'll eliminate it, try and update by tomorrow.

@Idahowalker

I'd only include one read cmd statement

It was supposed to be a joke. I do know 1 is enough

[baljo]

@bheeduino

if(cmd == "front" || cmd == "frontfront" ) { 
    while(staus==2 ){

And, as has been mentioned before, be sure you spell status correctly, in the while comparison you have written staus

[bheeduino]

OK Update time:
putting cmd = BTSerial.readString(); in the middle of the while loop, it stops now
but the servos have some type of delay after the two for() function. Can't seem to do anything with it. Here's the brand new code. It compiles this time

#include <SoftwareSerial.h> 

#include <Servo.h> 
Servo rightleg; 
Servo leftleg;

int bluetoothTx = 5; 
int bluetoothRx = 6; 
int pos = 0;
SoftwareSerial BTSerial(bluetoothTx, bluetoothRx);
int status = 2;
void setup()
{
  rightleg.attach(9); 
  leftleg.attach(10);
  
  Serial.begin(9600);

  
  BTSerial.begin(9600);
}

void loop(){
String cmd;
cmd = BTSerial.readString() ;

  if(BTSerial.available()==0){

  }
    
  
  if(cmd == "front" || cmd == "frontfront" ) { 
    while(status==2 ){
      cmd = BTSerial.readString() ;
      if(BTSerial.available() != 0){
    Serial.println(cmd);
      }
   for (pos = 0; pos <= 45; pos += 2) { 
    
    rightleg.write(pos); 
    leftleg.write(pos);
  
delay(10);
                           
  }
  
  for (pos = 45; pos >= 0; pos -= 2) { 
    rightleg.write(pos); 
    leftleg.write(pos); 
  

     delay(10);         
                           
  }
  
 if (cmd == "stop" && status != 0 || cmd == "stopstop" && status != 0){
status = 0;


}
if (cmd == "front" && status != 2 || cmd == "frontfront" && status != 2) {
  status = 2;
}
    }
  
  }
  
  if (cmd == "front" && status != 2 || cmd == "frontfront" && status != 2) {
  status = 2;
}
  
Serial.println(cmd);
  }

[Idahowalker]

One delay you got is 2(delay(10) * the number of loop iterations) and another delay is Serial.println(cmd).

[StefanL38]

well done posting the code as a code-section.
There is another thing that you should follow.

press Ctrl-T

for autoformatting the indentions of the code
Your code with improper indentions

After

pressing Ctrl-T

the autoformatted code looks like this

The closing brace is in the same collum as the "i" of if in the same columm as the "f" of the for

[StefanL38]

@bheeduino

but the servos have some type of delay after the two for() function. Can't seem to do anything with it.

until now your style of learning programming seems to be head fast forward
trying different things until it works.

This means your knowledge about the functions that you use is rather small.
If you look up the documentaion of the function readString()

Serial.readString() | Arduino Reference

You can read there that the function readString() has a timeout
calling readStr() wais for a complete string to be received until the line-ending character is received or if the timeout-waitingtime is over

Your coe will aöways react immidiately after you have learned

• what non-blocking timing is and how it works
• what state-machine are and how state-machines work
• writing your customised serial receive-function derived from robin2 serial input basics

You can invest 100 hours into trying this, trying that
and after these 100 hour still not really knowing how to code it
or you can invest 30 hours of time to learn the things mentioned above and have a quick-responsive code

it is up to you to decide which way to go

Example-code for timing based on millis() easier to understand through the use of example-numbers / avoiding delay()

non-blocking timing. Execute code only from time to time. There are a lot of different ways to learn programming. This thread wants to add another approach that is different to the yet existing ones. UPDATE 06.01.2023 if you are mainly interested in applying non-blocking timing you can do a quick read of this short tutorial / demonstration If you are intersted in understanding the details how it works go on reading here. I use an everyday analogon in this post to explain the basic principle …

A Demo-Code explaining the switch-case state-machine and how to do things (almost) in parallel

Hi everybody, if you look at the end of this first posting you see a list of links that show where this tutorial was used as a link. Go through this list to find additional information on how to apply this code This democode requires only the pure microcontroller without any hardware. You just open the serial monitor to watch the behaviour of the code. From time to time I enjoy writing demo-codes that want to explain programming-functionalities. There are a lot of different approachs to expl…

Serial Input Basics - updated

Updated Version Please note that this is a revised version of the advice in this earlier Thread which has become very long. As far as possible I have kept the code examples identical or simplifed them slightly. It should not be necessary to refer to the older Thread, but feel free to do so. Contents The following sections are in this Tutorial Introduction Serial data is slow by Arduino standards Example 1 - Receiving single characters Why code is organized into functions Exampl…

best regards Stefan

[bheeduino]

@StefanL38 Thank you for typing out that beautiful paragraph but I'm probably not going to do that because i need a lot of time to actually figure out the walking sequence and "engineer" the physical legs. I'll get back to you guys.
Thanks!