Breaking from a for loop in a separate loop

Using Arduino Programming Questions
#1 
#define ONE_WIRE_BUS 13                         
#include <OneWire.h>                                           
#include <DallasTemperature.h>                                 
OneWire oneWire(ONE_WIRE_BUS);
DallasTemperature sensors(&oneWire);                       
int count = 0;
unsigned long pHigh;
unsigned long pLow;
unsigned long interval;
unsigned long RPS;
unsigned long total;
unsigned long start_time;
int choke_on = 8;
int choke_off = 9;
int start_ctrl = 10;
int val;
int average;
int pulsePin = 5;
int button1 = 14;
void setup(){
  pinMode (pulsePin, INPUT);
  pinMode (choke_on, OUTPUT);
  pinMode (choke_off, OUTPUT);
  Serial.begin(9600);
  sensors.begin();
}


void loop(){
  if (button1==HIGH) {
    starter_engaged();
  }
}
  void starter_engaged() {                                                                                              //                                                     
  start_time = millis(); 
  total = 0;
  for (int i = 0; i < 5; )
  {
    pHigh = pulseIn(pulsePin, HIGH, 100000);
    pLow = pulseIn(pulsePin, LOW, 100000);

    unsigned long interval = pHigh + pLow;
      if ( pHigh > 0 && pLow > 0 && interval > 6000 )
    {
      Serial.print(pHigh); 
      Serial.print("\t:\t"); 
      Serial.print(pLow);
      RPS = (1000000UL / interval);
      Serial.print("\tRPS: "); 
      Serial.println(RPS);
      total = total + RPS;
      i++;
    }
    else if ((millis()-start_time) > 3300) {  // Test if 3.3 seconds has passed since the starter was engaged
    digitalWrite(start_ctrl, LOW);
    Serial.println  ("STARTER OFF");
    disengage_starter_timeout();                                                 
    }
  average = total / 5;
  Serial.print("                               Average RPS: "); 
  Serial.println(average); 
  if (average >= 25) {
 
    digitalWrite(start_ctrl, LOW);
    Serial.println  ("STARTER OFF");
    Serial.print    ("RPS: ");
    Serial.println  (RPS);
    Serial.println  ("ENGINE RUNNING");// Send "Engine Running" message after engine has started
    delay(10);
    disengage_starter();             // Go to disengage_starter after engine is running
  }
// else {
//    starter_engaged();
}
}
void disengage_starter() {                                                                                               // disengage_starter()
  sensors.requestTemperatures();                    // Read the temperature
  val = (sensors.getTempCByIndex(0));               // Store temp

  if (val < 0) {

    delay(100);
    Serial.println ("WARM UP ACTIVE");
    delay(10);
    Serial.print    ("WAIT : ");
    Serial.println  (val);
    disengage_starter();                          //  repeat until 0 degrees

  } else if (val >= 0) {

    digitalWrite(choke_on, LOW);
    digitalWrite(choke_off, HIGH);
    delay(500);
    digitalWrite(choke_off, LOW);
    Serial.println  ("CHOKE OFF");
    delay(10);
    loop();
  }
}
void disengage_starter_timeout() {                                                                                //disengage_starter_timeout()
  digitalWrite(start_ctrl, LOW);                      // Disengage the starter
  Serial.println  ("ENGINE START UNSUCESSFUL");
  delay(1300);
  starter_REngaged();                                 // Restart engine
}
void starter_REngaged() {                                                                                             // starter_REngaged()
  if (count < 3) {
    count++;                                          // Increment the count
    Serial.println  ("RESTARTING");
    delay(1200);
    Serial.print    ("ATTEMPT No: ");
    Serial.println  (count);
    starter_engaged();

  } else {
    Serial.println  ("EXCEEDED 3 ATTEMPTS!!! CHECK SYSTEM!!!");
    count = 0;                                        //Reset counter
    loop();
  }
}

Now the conditions in the for loop in starter_engaged() requires 5 sets of pulses to exit. My concern is that if for some reason the pulses arent read then the condition is not met, so if that is the case can i break; from the for loop in subsequent loops such as starter_REngaged()?

#2

I cant post the whole code because its too large

So create a small but complete example program that demonstrates the problem you are trying to solve

#3

ok so i done a quick edit on the original code

#4

Did you try with external interrupts?
Every time in for loop check for condition that you mention. If its true, trigger external interrupt that will be your desured function.

#5

UKHeliBob:
So create a small but complete example program that demonstrates the problem you are trying to solve

You code does not compile and is thus incomplete.

#6

Ok re-edited edit, has compiled and included more of what code im running.

I have looked at breaking in subsequent loops but this will not compile as the call to break is not inside the particular loop.

It works fine when pulses are present but if not it will continue with the code and when it gets back to the main loop, the for loop is still executing.

#7

The pulseIn() function returns 0 if no pulses are detected within the timeout period. Use that info to terminate the loop and do something else.

#8 
  if (button1 = HIGH)

Before you do anything else fix this and paste the fixed code in a new post. DO NOT update previously pasted code and thus invalidate subsequent replies relating to it

#9

OK sorry just trying to make it easier, but obviously not.

#10

andyroid:
Now the conditions in the for loop in starter_engaged() requires 5 sets of pulses to exit. My concern is that if for some reason the pulses arent read then the condition is not met, so if that is the case can i break;

Don't use a FOR loop if the number of iterations is outside your control. Just use an IF clause and a variable that counts things. Allow loop() to do the repetition.

You should probably save the value of millis() when the sequence starts and call things off if it is taking too long.

...R

#11