променливата numTicks не се увеличава

Имам следната скица и променливата numTicks не се увеличава, скицата се компилира добре към Arduino, но по някаква причина променливата "numTicks" не се увеличава.

/*
 * kegboard-clone-4-KegCop
 * This code is public domain
 *
 * This sketch sends a receives a multibyte String from the iPhone
 * and performs functions on it.
 *
 * This Arduino sketch is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU Public License
 * along with this sketch.  If not, see <http://www.gnu.org/licenses/>.
 * 
 * Examples:
 * http://arduino.cc/en/Tutorial/SerialEvent
 * http://arduino.cc/en/Serial/read
 * http://stackoverflow.com/questions/16532586/arduino-sketch-that-responds-to-certain-commands-how-is-it-done/
 * http://davebmiller.wordpress.com/2011/01/18/arduino-flowmeter/
 * http://forum.arduino.cc/index.php?topic=52003.0
 * http://arduino.cc/en/Reference/AttachInterrupt
 * https://github.com/just-kile/Zapfmaster2000/blob/master/src/zapfmaster2000-zapfkit-avr/draftkitAVR.ino
 *
 * TODO:
 * - eventually get code working with the SF800 flow sensor / flowmeter
 *
 */

// flow_A LED
int led = 4;

// relay_A
const int RELAY_A = A0;

// string / serial event variables
String inputString = ""; // a string to hold incoming data
boolean stringComplete = false; // whether the string is complete
boolean valve_open = false;

// FLOWMETER SHIT
// flowmeter 0 pulse (input) = digital pin 2
// https://github.com/Kegbot/kegboard/blob/master/arduino/kegboard/kegboard_config.h
// which pin to use for reading the sensor? kegboard-mini shield has digital pin 2 allocated
// the SF800 outputs 5400 pulses per litre
// The hall-effect flow sensor (SF800) outputs approximately 5400 pulses per second per litre/minute of flow
// SF800 default (5400 ticks/Liter == 5.4 ticks/mL == 1/5.4 mL/tick)
int flowmeterInterrupt = 0;  // changed from byte
int flowmeterPin = 2;        // changed from byte
int flowmeterPinState = 0;   // variable for storing state of sensor pin
// read RPM
int rpmcount = 0;
int rpm = 0;
unsigned long lastmillis = 0; 

// NEW GLOBALS - 29JUL13
// initial ticks on flow meter
volatile unsigned int numTicks = 0;
// interval for flow meter frequency
int interval = 250;
volatile long previousMillis = 0;

void setup() {
  // initialize serial
//  Serial.flush(); // flush the serial buffer on setup.
  Serial.begin(115200); // open serial port, sets data rate to 9600bps
  Serial.println("Power on test");
  inputString.reserve(200);
  valve_open = false;

  // relay for solenoid cut off valve
  pinMode(RELAY_A, OUTPUT);

  // flowmeter shit
  pinMode(flowmeterPin, INPUT);
  digitalWrite(flowmeterPin, HIGH); // Need to set these HIGH so they won't just tick away

  // The Hall-effect sensor is connected to pin 2 which uses interrupt 0.
  // Configured to trigger on a RISING state change (transition from HIGH
  // state to LOW state)
  attachInterrupt(flowmeterInterrupt, count, RISING);
}

void open_valve() {
  digitalWrite(RELAY_A, HIGH); // turn RELAY_A on
  valve_open = true;
}

void close_valve() {
  digitalWrite(RELAY_A, LOW); // turn RELAY_A off
  valve_open = false;
}

void flow_A_blink() {
  digitalWrite(led, HIGH); // turn the LED on (HIGH is the voltage level)
  delay(1000);              // wait for one second
  digitalWrite(led, LOW);   // turn the LED off by making the voltage LOW
  delay(1000);              // wait for a second
}

void flow_A_blink_stop() {
  digitalWrite(led, LOW);
}

void flow_A_on() {
  digitalWrite(led, HIGH); // turn the LED on (HIGH is the voltage level)
}

void flow_A_off() {
  digitalWrite(led, LOW);   // turn the LED off by making the voltage LOW
}  

// flowmeter shit
void getFlow4() {
//  Serial.println("im here");
//  Serial.println(sensorPin);

  flowmeterPinState = digitalRead(flowmeterPin);

//  Serial.println(sensorPinState);

  volatile unsigned long currentMillis = millis();
  // if the predefined interval has passed
  if(currentMillis - previousMillis > interval) { // Uptade every 1/4 second, this will be equal to reading frecuency (Hz).

    // disconnect flow meter from interrupt
    detachInterrupt(flowmeterInterrupt); // Disable interrupt when calculating
    // check, whether any flow was detected
    if (numTicks >= 0) {
      // start message to computer with tick message symbol
      Serial.print("Ticks:");
      // send amount of ticks for last interval
      Serial.print(numTicks);
    }
    // clean buffer
    Serial.flush();
    // reset amount of ticks
    numTicks = 0;
    // set new start value for interval counter
    previousMillis = currentMillis;
    // reattach interrupt
    attachInterrupt(flowmeterInterrupt, count, RISING);
  }


  if(flowmeterPinState == LOW) {
    flow_A_off();
//    Serial.println("don't blink");
  }
  if(flowmeterPinState == HIGH) {
    flow_A_on();
//    Serial.println("blink damnit");
  }

  if(stringComplete) {
    if(inputString.equals("{close_valve}\n")) {
      //      Serial.println("close vavle.");
      close_valve();
    }
    return;
  }
}
// flow meter interrupt function
void count(){
  numTicks++;
} 

/*
 * Main program loop, runs over and over repeatedly
 */

void loop() {
  if(stringComplete) {
//    Serial.println(inputString);

    if(inputString.equals("{open_valve}\n")) {
//      Serial.println("inputString equates :)");
      open_valve();
    }

    if(inputString.equals("{close_valve}\n")) {
      //      Serial.println("close vavle.");
      close_valve();
    }

  if(valve_open) {
//    Serial.println("valve_open = true");
    inputString = "";
    stringComplete = false;
    while(numTicks <= 1000) {
      getFlow4();
    }
  }

  // clear the string:
  inputString = "";
  stringComplete = false;
  }
//Serial.println("over and over");
}

/*
 SerialEvent occurs whenever a new data comes in the
 hardware serial RX.  This routine is run between each
 time loop() runs, so using delay inside loop can delay
 response.  Multiple bytes of data may be available.
 */

void serialEvent() {
  while(Serial.available()) {
    // get the new byte:
    char inChar = (char)Serial.read();
    // add it to the inputString:
    inputString += inChar;
    // if the incoming character is a newline, set a flag
    // so the main loop can do something about it:
    if (inChar == '\n') {
      stringComplete = true;
    }
    // Serial.println(inputString.length());
  }
}

person ipatch    schedule 29.07.2013    source източник


Отговори (4)


Причината, поради която променливата numTicks не се променя, вероятно се дължи на незадействането на прекъсването. Трябва да поставите точка на прекъсване в count(), за да потвърдите това. След това трябва да разберете защо прекъсването не се задейства както трябва, но това е друг въпрос.

person Fredrik Möller    schedule 29.07.2013
comment
Съжалявам, просто предположих, че е така. - person Fredrik Möller; 30.07.2013

Съжалявам за предишния ми отговор. Не разбрах напълно проблема.

Първо, използвате ли Arduino UNO, ако е така, проверете щифта на платката, обозначен с цифров щифт 2, който е картографиран като "int.0" и се уверете, че линията за прекъсване от разходомера е свързана към този щифт. (вижте: http://arduino.cc/en/Reference/AttachInterrupt).

Според коментарите на Крис по-горе, рутината count() е код, управляван от прекъсване, и изглежда, че е кодиран правилно: numTicks се определя като volatile; и count() НЕ издава I/O команди като printf; и НЕ връща никакви стойности.

Примерният код, който предоставяте, не изолира или подчертава проблема. Бих написал тестова скица, която е просто изпълнение на голи кости на "отваряне" на сензора и след това усещане на прекъсване от разходомера и докладване обратно на конзолата от главния контур. Ако можете да получите код, който открива едно прекъсване от разходомера, за да работи, тогава добавете още код, за да отчетете броя на прекъсванията за една секунда, след това 1/2 секунда и т.н.

И накрая, в предоставения от вас код имате фрагмента:

    if(valve_open) {
         //    Serial.println("valve_open = true");
         inputString = "";
         stringComplete = false;
         while(numTicks <= 1000) {
            getFlow4();
         }
    }

Тъй като numTicks се увеличава от броя на рутинните прекъсвания, по принцип НЕ бих го тествал, освен ако не е приложен някакъв вид сериализация. getFlow4() отделя прекъсването, което е един от начините за сериализиране на достъпа до numTicks. Забележка. на теория кодът може да актуализира numTicks без сериализация, но всяка върната стойност не е непременно точна, тъй като прекъсването може да е задействало и да е увеличило numTicks.

Изглежда, че вашето приложение се интересува от това да знае броя на тиковете в секунда?? в който случай НЕ е нужно да тествате numTicks преди да спрете прекъсванията. Всичко, от което може да се нуждаете, е код, който веднъж на секунда проверява numTicks и ако можете да живеете с отпадане на брояч, тогава нулирайте numTicks, без дори да отделяте прекъсването. Обаче вземането на проби от numTicks е по-скоро като анкета, което прекъсването се опитва да избегне.

Така че, тъй като кодът следи интервалите, тогава разделете интервала на numTicks, за да получите стойност в Hz и не нулирайте numTicks или нулирайте интервала, докато не се доближат до преобръщане.

person JackCColeman    schedule 30.07.2013

Кодът в крайна сметка работи, това беше хардуерен проблем с връзките от разходомера (>.>)

person ipatch    schedule 30.07.2013

Вие всъщност никога не извиквате своя count() метод. Трябва или да вмъкнете numTicks++; в кода, където искате да увеличите броя (препоръчителен начин), или да извикате своя count() метод, където искате да увеличите броя. Тук методът count() е само дефиниран и не се извиква, но би било по-логично просто да увеличите брояча в кода, тъй като това е единственото нещо, което вашият дефиниран метод прави.

person Daniel Underwood    schedule 29.07.2013
comment
какво ще кажете за този ред, attachInterrupt(flowmeterInterrupt, count, RISING); - person ipatch; 29.07.2013
comment
Това е вярно, това ми липсваше. Всичко останало работи ли? Можете да опитате да добавите конзолен изход в метода count(), за да можете да потвърдите, че методът се извиква или не. - person Daniel Underwood; 29.07.2013
comment
@danielu13: Конзолният изход в манипулатора на прекъсвания вероятно не е добра идея - особено ако самият изход е управляван от прекъсване. - person Clifford; 30.07.2013