Переменная 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() НЕ выдает команды ввода-вывода, такие как 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, чтобы получить значение в Гц, и не обнуляйте 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