Есть ли что-то не то с этим кодом

Трудно ответить без какого-либо описания реальной проблемы.

На мой взгляд, с таким большим количеством кода я бы подумал о том, чтобы начать с создания отдельного объявления класса и написать свои определения членов снаружи. Это сделает код более управляемым и облегчит устранение неполадок.

https://www.learncpp.com/cpp-tutorial/class-code-and-header-files/

Кроме того, у вас есть несколько операторов include, по крайней мере, в двух разных местах. Они обычно идут первыми в шеренге вместе с охранниками. Если ваш код состоит из отдельных файлов, пожалуйста, разъясните это читателям.

Вот несколько случайных комментариев от просмотра кода.

Первый вопрос, который больше касается примера, чем самой библиотеки: в стандартных ядрах Arduino нет Serial.printf (). Да, он обеспечивается некоторыми альтернативными ядрами, и жаль, что он недоступен в Arduino, но так оно и есть.

В конструкторе есть if(flowPin==254). Почему такое состояние? flowPin инициализируется нулем, поэтому, казалось бы, условие всегда должно быть ложным.

С точки зрения стиля, вы можете удалить все this-> из методов. В любом случае это неявно.

Ради согласованности имен я бы переименовал retrieve_pin() в get_pin ()и весь метод assign_*() как set_ * ().

Это не полезно, чтобы detachInterrupt() прямо перед attachInterrupt(), поскольку последнее подразумевает первое.

В calculate ()избегайте вызова millis() дважды. У вас нет гарантии , что оба вызова вернут одно и то же значение, а если это не так, то вычисления времени будут отключены. Кроме того, эта функция потерпит неудачу (деление на ноль), если будет вызвана дважды в течение одной и той же миллисекунды.

Нет смысла переводить расход в миллилитры. Вы имеете в виду миллилитры в минуту? В секунду? В любом случае, вы здесь конвертируете скорость в другую единицу, но это все равно скорость потока.

Вычисление Totalmillitres неверно: оно увеличивается на скорость потока. Вместо этого он должен быть увеличен на коэффициент продукта × время. Или еще лучше: вычислите его по количеству импульсов, не проходя через частоту сердечных сокращений. В противном случае вы вычисляете численную производную (чтобы получить скорость) только для того, чтобы интегрировать ее позже (чтобы получить общий объем). Эти дополнительные вычисления не служат никакой другой цели, кроме накопления ошибок округления.

Edit: Размышляя дальше об этой программе, я нашел еще несколько вещей , которые, как мне кажется, можно было бы улучшить. Пара незначительных моментов и один не такой уж незначительный.

Одним из второстепенных моментов является то, что begin() было бы лучшим именем для start (), для согласованности с большинством других библиотек Arduino.

Другое дело, что есть несколько методов, которые не имеют никакого отношения к классу: is_max() и percent(). Это тривиальные вычисления, которые никоим образом не связаны с работой с расходомером. Как пользователь библиотеки, я бы предпочел написать

if (flow.total_ml() >= threshold_volume) ...

чем

if (flow.is_max(threshold_volume)) ...

поскольку первое гораздо яснее и проще для понимания.

Более серьезный момент заключается в обращении с несколькими расходомерами. Первый полностью обрабатывается библиотекой, и это хорошо. Однако для второго пользователь должен предоставить кирпичи самого низкого уровня . Пользователь библиотеки не должен сам справляться с такими низкоуровневые детали. Таким образом, я думаю, что и переменная counter, и функция ISR должны обрабатываться библиотекой как члены потока класс.

Я знаю, что это создает некоторые трудности. Для того чтобы быть присоединенным к прерыванию, ISR должен быть статическим методом класса. Для того чтобы этот метод мог получить доступ к счетчику, этот счетчик должен быть статическим членом. Для того чтобы каждый расходомер имел свой собственный счетчик, мы должны иметь отдельный класс для каждого из них.

Я нашел решение, которое считаю достаточно элегантным: пусть Flow будет шаблоном класса, параметризованным выводом. Это создает отдельный класс для каждого расходомера. Чтобы избежать дублирования кода, эти классы могут быть почти пустыми оболочками, которые делегируют как можно больше обработчику, который является обычным обычным классом.

Вот как может выглядеть этот класс “обработчик”. В нем есть все , кроме ISR:

class FlowMeterHandler {
    // Все здесь закрыто, доступно только классу Flow.
    template<uint8_t pin> friend class Flow;
private:
    void begin(uint8_t pin, void (*isr)()) {
        attachInterrupt(digitalPinToInterrupt(pin), isr, FALLING);
    }
    float flow_rate() {
        uint32_t now = millis();
        if (now - last_millis >= 10) {
            uint32_t this_count;
            noInterrupts();
            this_count = count;
            interrupts();
            last_flow = calibration * (this_count - last_count)
                                    / (now - last_millis);
            last_millis = now;
            last_count = this_count;
        }
        return last_flow;
    }
    // + другие методы...
    volatile uint32_t count = 0;
    uint32_t last_count = 0;
    uint32_t last_millis = 0;
    float last_flow = 0;
    float calibration = 1e3 / 4.5;
};

Я написал методы inline, но вы также можете поместить реализации в отдельный файл .cpp.

А вот и класс шаблона. За исключением ISR, он пересылает все обработчику.

template<uint8_t pin> class Flow {
public:
    static void begin() { handler.begin(pin, isr); }
    static float flow_rate() { return handler.flow_rate(); }
    // другие методы также пересылаются обработчику...
private:
    static FlowMeterHandler handler;
    static void isr() { handler.count++; };
};

template<uint8_t pin> FlowMeterHandler Flow<pin>::handler;

Обратите внимание, что класс не имеет данных экземпляра: все в нем статично.

Вот как это будет использоваться:

Flow<3> f1;
Flow<4> f2;

void setup()
{
    f1.begin();
    f2.begin();
}

void loop()
{
    Serial.print(F("flow rate on 3 = "));
    Serial.println(f1.flow_rate());
    Serial.print(F("flow rate on 4 = "));
    Serial.println(f2.flow_rate());
    delay(1000);
}

Итак, если кому-то интересно, вот что я получил

/*
  это еще предстоит изменить, но происходит из следующих ссылок
  * 
  https://www.bc-robotics.com/tutorials/using-a-flow-sensor-with-arduino/ 
  
  "Они не способны контролировать расход менее 1 литра в минуту или более 30 литров в минуту.
  Датчик рассчитан максимум на 2,0 МПа (290 фунтов на квадратный дюйм) ".
  и   
  каждый поворот означает, что датчик прошел 2,25 мл жидкости

  другой-это 

  https://diyhacking.com/projects/FlowMeterDIY.ino
  "// Датчик расхода с эффектом Холла выдает приблизительно 4,5 импульса в секунду на
   // литр/минута расхода.
  "
  
  затем, наконец, 
  
  https://arduino.stackexchange.com/questions/84578/is-there-anything-wrong-with-this-code
  
  Я не предъявляю никаких личных претензий ни к одному из положений этого кодекса. Это просто моя версия версии Эдгара. 
  Спасибо Эдгар
  * 
  * 
  * 
  * Я использовал start и stop, поскольку они означают начало и остановку процесса. 
  * класс начинается, когда он начинается и не имеет конца isr остается связанным с классом в течение всего времени 
  * и start просто присоединяет прерывание к isr 
  * stop просто вычисляет общую громкость и отсоединяет прерывание.
  * 
  * как скорость потока, связанная со временем 
  * просто установите свое собственное время начала 
  * прочитайте total_ml
  * затем после вашего собственного предопределенного времени
  * прочитайте total_ml еще раз. 
  * и рассчитайте скорость потока исходя из этого.
  */

class FlowMeterHandler {
  // Everything here is private, only accessible to the Flow class.
  template<uint8_t pin> friend class Flow;
private:
  
  void start(uint8_t pin, void (*isr)()) 
  {
    noInterrupts();
    count = 0;
    interrupts();
    attachInterrupt(digitalPinToInterrupt(pin), isr, FALLING);
  }
  
  void stop(uint8_t pin)
  {
    detachInterrupt(digitalPinToInterrupt(pin));
  }
  
  unsigned int count_is()
  {
    unsigned int this_count;
    noInterrupts();
    this_count = count;
    interrupts();
    return this_count;
  }
  
  volatile unsigned int count = 0;
  unsigned int ul_per_tick = 2250;
  
};


template<uint8_t pin> class Flow {
public:
    static void start() { handler.start(pin, isr); }
    static void stop() { handler.stop(pin); }
    static unsigned int total_ul() { return handler.count_is() * handler.ul_per_tick; }
    static unsigned int total_ml() { return ( handler.count_is() * handler.ul_per_tick ) / 1000 ; }
    static void set_ul_per_rotation( unsigned int ul) { handler.ul_per_tick = ul; }
private:
    static FlowMeterHandler handler;
    static void isr() { handler.count++; };
};

template<uint8_t pin> FlowMeterHandler Flow<pin>::handler;

пример ino выглядит так

#include "flow.h"
Flow<3> f1;
Flow<4> f2;

void setup()
{
    f1.start();
    f2.start();
}

void loop()
{
    Serial.print(F("So far volume on 3 = "));
    Serial.println(f1.total_ml());
    Serial.print(F("and the volume on 4 = "));
    Serial.println(f2.total_ml());
    delay(1000);
}

Компилируется чисто на всех моих платах. Бежит ??? Любые другие предложения будут с благодарностью приняты.

Спасибо. Грег