Форум arduino.ua

Доброго вечора.
Шановна спільнота - прошу вашої допомоги.
Підкажіть , будь- ласка, де шукати помилку...

Коротко - MEGA  2560 -  по таймеру керує:
1. Вкл/Викл помпа, частота 3 min
2. Вкл/Викл реле, частота 140ms
3. Підрахунок RPM по датчику
4. Вимірювання температури двома датчиками
5. Вивід значень в порт, оновлення 1s

Ось код:

[        /*
                      test RPM + temperature
        
            */
           
    #include <OneWire.h>        //
   
    #include <DallasTemperature.h>        //
   
    #define ONE_WIRE_BUS A0        // pin data sensor
   
    OneWire oneWire(ONE_WIRE_BUS);    // creer objects communication OneWire ( themperature)
    DallasTemperature sensors(&oneWire);          // done objects for DallasTemperature

    uint8_t sensor1[8] = { 0x28, 0x5C, 0xC1, 0x05, 0x00, 0x00, 0x00, 0x23};   // adress Sensor 1
    uint8_t sensor2[8] = { 0x28, 0x20, 0xEA, 0x05, 0x00, 0x00, 0x00, 0x4E};   // adress Sensor 2
   
    #define motorPin 46     // pin motor MOSFET/operation
    #define   Vcc_pin 3     // pin +5v Hall Sensor
    #define SIG_pin 2       // pin signal Hall Sensor
   
    #define C1_C_in 22        // C 1 Cold  pin 22 R 1
    #define C1_H_in 23        // C 1 Hot   pin 23 R 2
    #define C1_C_out 24       // C 1 Cold  pin 24 R 3             
    #define C1_H_out 25       // C 1 Hot   pin 25 R 4             
                   
    #define C4_C_in 26        // C 4 Cold  pin 26 R 5
    #define C4_H_in 27        // C 4 Hot   pin 27 R 6
    #define C4_C_out 28       // C 4 Cold  pin 28 R 7
    #define C4_H_out 29       // C 4 Hot   pin 29 R 8

    #define C2_C_in 30        // C 2 Cold  pin 30 R 9
    #define C2_H_in 31        // C 2 Hot   pin 31 R 10
    #define C2_C_out 32       // C 2 Cold  pin 32 R 11
    #define C2_H_out 33       // C 2 Hot   pin 33 R 12

    #define C5_C_in 34        // C 5 Cold  pin 34 R 13
    #define C5_H_in 35        // C 5 Hot   pin 35 R 14
    #define C5_C_out 36       // C 5 Cold  pin 36 R 15
    #define C5_H_out 37       // C 5 Hot   pin 37 R 16

    #define C3_C_in 38        // C 3 Cold  pin 38 R 17
    #define C3_H_in 39        // C 3 Hot   pin 39 R 18
    #define C3_C_out 40       // C 3 Cold  pin 40 R 19
    #define C3_H_out 41       // C 3 Hot   pin 41 R 20

    #define C6_C_in 42        // C 6 Cold  pin 42 R 21
    #define C6_H_in 43        // C 6 Hot   pin 43 R 22
    #define C6_C_out 44       // C 6 Cold  pin 44 R 23
    #define C6_H_out 45       // C 6 Hot   pin 45 R 24

    #define RPM_INTERVAL 1000UL        // update RPM
    #define Pump_INTERVAL 60000UL      // interval ON/OFF pump
    #define Relay_INTERVAL 840UL      // relay operation
    #define SERIAL_SPEED 9600          // Speed serial (com-port)
   
    volatile byte counter;
    unsigned int rpm;
    unsigned long passtime;
   
    void isr()

    {

   //Each rotation, this interrupt function is run twice, so take that into consideration for
   //calculating RPM
   //Update count

           counter++;
    }
   
    void setup() {
   
    Serial.begin(SERIAL_SPEED);
   
                // Hall sensor operation

    pinMode(Vcc_pin, OUTPUT);        // pin Hall sensor output
    digitalWrite(Vcc_pin, HIGH);     // pin Hall sensor VCC +5v
    pinMode(SIG_pin, INPUT);         // pin signal Hall sensor
   
    attachInterrupt(0, isr, RISING); //Interrupts are called on Rise of Input
    rpm = 0;
    counter = 0;
    passtime = 0;                    //Initialise the values
      
            // pump
           
    pinMode(motorPin, OUTPUT);       // motor output
       
            // relay
 
    pinMode(C1_C_in, OUTPUT);      //  pin 22 output:
    pinMode(C1_H_in, OUTPUT);      //  pin 23 output:
    pinMode(C1_C_out, OUTPUT);     //  pin 24 output:
    pinMode(C1_H_out, OUTPUT);     //  pin 25 output:

    pinMode(C4_C_in, OUTPUT);      //  pin 26 output:
    pinMode(C4_H_in, OUTPUT);      //  pin 27 output:
    pinMode(C4_C_out, OUTPUT);     //  pin 28 output:
    pinMode(C4_H_out, OUTPUT);     //  pin 29 output:

    pinMode(C2_C_in, OUTPUT);      //  pin 30 output:
    pinMode(C2_H_in, OUTPUT);      //  pin 31 output:
    pinMode(C2_C_out, OUTPUT);     //  pin 32 output:
    pinMode(C2_H_out, OUTPUT);     //  pin 33 output:

    pinMode(C5_C_in, OUTPUT);      //  pin 34 output:
    pinMode(C5_H_in, OUTPUT);      //  pin 35 output:
    pinMode(C5_C_out, OUTPUT);     //  pin 36 output:
    pinMode(C5_H_out, OUTPUT);     //  pin 37 output:

    pinMode(C3_C_in, OUTPUT);      //  pin 38 output:
    pinMode(C3_H_in, OUTPUT);      //  pin 39 output:
    pinMode(C3_C_out, OUTPUT);     //  pin 40 output:
    pinMode(C3_H_out, OUTPUT);     //  pin 41 output:

    pinMode(C6_C_in, OUTPUT);      //  pin 42 output:
    pinMode(C6_H_in, OUTPUT);      //  pin 43 output:
    pinMode(C6_C_out, OUTPUT);     //  pin 44 output:
    pinMode(C6_H_out, OUTPUT);     //  pin 45 output:

            // Turn relay OFF ;

    digitalWrite(C1_C_in, HIGH);   // R  1 OFF
    digitalWrite(C1_H_in, HIGH);   // R  2 OFF
    digitalWrite(C1_C_out, HIGH);  // R  3 OFF
    digitalWrite(C1_H_out, HIGH);  // R  4 OFF

    digitalWrite(C4_C_in, HIGH);   // R  5 OFF
    digitalWrite(C4_H_in, HIGH);   // R  6 OFF
    digitalWrite(C4_C_out, HIGH);  // R  7 OFF
    digitalWrite(C4_H_out, HIGH);  // R  8 OFF

    digitalWrite(C2_C_in, HIGH);   // R  9 OFF
    digitalWrite(C2_H_in, HIGH);   // R 10 OFF
    digitalWrite(C2_C_out, HIGH);  // R 11 OFF
    digitalWrite(C2_H_out, HIGH);  // R 12 OFF

    digitalWrite(C5_C_in, HIGH);   // R 13 OFF
    digitalWrite(C5_H_in, HIGH);   // R 14 OFF
    digitalWrite(C5_C_out, HIGH);  // R 15 OFF
    digitalWrite(C5_H_out, HIGH);  // R 16 OFF

    digitalWrite(C3_C_in, HIGH);   // R 17 OFF
    digitalWrite(C3_H_in, HIGH);   // R 18 OFF
    digitalWrite(C3_C_out, HIGH);  // R 19 OFF
    digitalWrite(C3_H_out, HIGH);  // R 20 OFF

    digitalWrite(C6_C_in, HIGH);   // R 21 OFF
    digitalWrite(C6_H_in, HIGH);   // R 22 OFF
    digitalWrite(C6_C_out, HIGH);  // R 23 OFF
    digitalWrite(C6_H_out, HIGH);  // R 24 OFF

    }

    void loop() {       
   
                // update RPM;
               
    static unsigned long prevRpmTime  = 0;
    if(millis() - prevRpmTime > RPM_INTERVAL){
    prevRpmTime = millis();

    detachInterrupt(0);                //Interrupts are disabled
    rpm = 60*1000/(millis() - passtime)*counter;
    passtime = millis();
    counter = 0;
    attachInterrupt(0, isr, RISING);   //Restart the interrupt processing

    Serial.print("Reading - RPM = ");
    Serial.println(rpm);
   
    sensors.requestTemperatures();      //temp C print
 
    Serial.print("Sensor 1: ");
    printTemperature(sensor1);
 
    Serial.print("Sensor 2: ");
    printTemperature(sensor2);
   
    Serial.println();
   
        }       
                // time motor
   
    static unsigned long prevPumpTime = 0;
    if(millis() - prevPumpTime > Pump_INTERVAL){
    prevPumpTime = millis();
    digitalWrite(motorPin, HIGH);
    }else{
    digitalWrite(motorPin, LOW);
   
        }       
                // RELAY operation
               
    static unsigned long prevRelayTime = 0;          // save the time of the last switch 
    if(millis() - prevRelayTime > Relay_INTERVAL) {  //check whether the required interval has passed, if so
    prevRelayTime = millis();
        }
                   // C1_H+C3_C - 1
       
    if(millis() - prevRelayTime > 0*Relay_INTERVAL/6&&millis() - prevRelayTime < 1*Relay_INTERVAL/6) {
      
        digitalWrite(C2_C_in, HIGH);      //  30 OFF
        digitalWrite(C2_H_out, HIGH);     //  33 OFF
        digitalWrite(C3_C_out, HIGH);     //  40 OFF
        digitalWrite(C6_H_in, HIGH);      //  43 OFF 
        digitalWrite(C6_H_out, HIGH);     //  45 OFF
      
        digitalWrite(C1_H_in , LOW);      //  23 ON
        digitalWrite(C1_H_out, LOW);      //  25 ON
        digitalWrite(C4_C_out, LOW);      //  28 ON
        digitalWrite(C3_C_in, LOW);       //  38 ON
        digitalWrite(C3_H_out, LOW);      //  41 ON     
        }
                   //  C2_H+C4_C - 2
         
    if(millis() - prevRelayTime > 1*Relay_INTERVAL/6&&millis() - prevRelayTime < 2*Relay_INTERVAL/6) {
      
        digitalWrite(C1_H_in, HIGH);      //  23 OFF
        digitalWrite(C1_H_out, HIGH);     //  25 OFF
        digitalWrite(C4_C_out, HIGH);     //  28 OFF
        digitalWrite(C3_C_in, HIGH);      //  38 OFF 
        digitalWrite(C3_H_out, HIGH);     //  41 OFF         
   
        digitalWrite(C4_C_in, LOW);       //  26 ON
        digitalWrite(C4_H_out, LOW);      //  29 ON
        digitalWrite(C2_H_in, LOW);       //  31 ON
        digitalWrite(C2_H_out, LOW);      //  33 ON
        digitalWrite(C5_C_out, LOW);      //  36 ON     
        }
                  //  C3_H+C5_C - 3
       
  if(millis() - prevRelayTime > 2*Relay_INTERVAL/6&&millis() - prevRelayTime < 3*Relay_INTERVAL/6) {
      
        digitalWrite(C4_C_in, HIGH);      //  26 OFF
        digitalWrite(C4_H_out, HIGH);     //  29 OFF
        digitalWrite(C2_H_in, HIGH);      //  31 OFF
        digitalWrite(C2_H_out, HIGH);     //  33 OFF 
        digitalWrite(C5_C_out, HIGH);     //  36 OFF         
   
        digitalWrite(C5_C_in, LOW);       //  34 ON
        digitalWrite(C5_H_out, LOW);      //  37 ON
        digitalWrite(C3_H_in, LOW);       //  39 ON
        digitalWrite(C3_H_out, LOW);      //  41 ON
        digitalWrite(C6_C_out, LOW);      //  44 ON     
        }
                   // C4_H+C6_C - 4
       
  if(millis() - prevRelayTime > 3*Relay_INTERVAL/6&&millis() - prevRelayTime < 4*Relay_INTERVAL/6) {
      
        digitalWrite(C5_C_in, HIGH);      //  34 OFF
        digitalWrite(C5_H_out, HIGH);     //  37 OFF
        digitalWrite(C3_H_in, HIGH);      //  39 OFF
        digitalWrite(C3_H_out, HIGH);     //  41 OFF 
        digitalWrite(C6_C_out, HIGH);     //  44 OFF         
   
        digitalWrite(C1_C_out, LOW);      //  24 ON
        digitalWrite(C4_H_in, LOW);       //  27 ON
        digitalWrite(C4_H_out, LOW);      //  29 ON
        digitalWrite(C6_C_in, LOW);       //  42 ON
        digitalWrite(C6_H_out, LOW);      //  45 ON     
        }
                   // C5_C+C1_C - 5
       
  if(millis() - prevRelayTime > 4*Relay_INTERVAL/6&&millis() - prevRelayTime < 5*Relay_INTERVAL/6) {
      
        digitalWrite(C1_C_out, HIGH);     //  24 OFF
        digitalWrite(C4_H_in, HIGH);      //  27 OFF
        digitalWrite(C4_H_out, HIGH);     //  29 OFF
        digitalWrite(C6_C_in, HIGH);      //  42 OFF 
        digitalWrite(C6_H_out, HIGH);     //  45 OFF         
         
        digitalWrite(C1_C_in, LOW);       //  22 ON
        digitalWrite(C1_H_out, LOW);      //  25 ON
        digitalWrite(C2_C_out, LOW);      //  32 ON
        digitalWrite(C5_H_in, LOW);       //  35 ON
        digitalWrite(C5_H_out, LOW);      //  37 ON     
        }
                   // C6_H+C2_C - 6
   
  if(millis() - prevRelayTime > 5*Relay_INTERVAL/6&&millis() - prevRelayTime < 6*Relay_INTERVAL/6) {
      
        digitalWrite(C1_C_in, HIGH);      //  22 OFF
        digitalWrite(C1_H_out, HIGH);     //  25 OFF
        digitalWrite(C2_C_out, HIGH);     //  32 OFF
        digitalWrite(C5_H_in, HIGH);      //  35 OFF 
        digitalWrite(C5_H_out, HIGH);     //  37 OFF         
        
        digitalWrite(C2_C_in, LOW);       //  30 ON
        digitalWrite(C2_H_out, LOW);      //  33 ON
        digitalWrite(C3_C_out, LOW);      //  40 ON
        digitalWrite(C6_H_in, LOW);       //  43 ON
        digitalWrite(C6_H_out, LOW);      //  45 ON     
        }
   
    }
       
    void printTemperature(DeviceAddress deviceAddress)
    {
    float tempC = sensors.getTempC(deviceAddress);
    Serial.print(tempC);
    Serial.print((char)176);
    Serial.print("  °C/  ");
    }
]

                      Проблеми

1, При компілюванні - з компа - видає помилку у формулі RPM; - з ноута - все нормально...
2, зависають реле на 5-му "блоці" - // C5_C+C1_C - 5 -далі реле ( всі ) не працюють -висить
3, Виводить дані з оновленням в одну сек - добре, температура -добре, але показники RPM - при нерухомому валі - RPM = 30568
     при обертанні валу рукою - приблизно 1 об/сек - датчик спрацьовує 1 раз - RPM - 60000+

  В яку сторону копати? щоб було нормально...
  Я сам не програміст і не вчусь, просто задумав зробити одну "штукенвію" так що прийшлось...

  Буду дуже вдячний за  будь-яку допомогу.
  На глінтвейн, на здоров"я, буду просто зобовязаний підкинути
  Дякую всім.

Доброго дня, шановна спільнота.
Прошу допомоги в написанні програмок ...
Сам я щойно сів писати малу прогу для керуванням реле. не можу зрозуміти де помилка..

  Modul 2 relay + MAX6675 Switch control themperature
  Written by OT 24/07/2021
  --- thermocuple connection VCC -5V; GRD -GRD; pin SO -4; pin CS -5; pin SCK -6; 
  --- RELAY connection - pin -22,23;
  --- timer (millis)
*/

#include "max6675.h"

int thermoDO = 4;    
int thermoCS = 5;    
int thermoCLK = 6;                        

MAX6675 thermocouple(thermoCLK, thermoCS, thermoDO);

#define PIN_R1 22                  // pin that controls the RELAY 1
#define PIN_R2 23                  // pin that controls the RELAY 2

unsigned long currentTime;         // variable for timer MAX chip to stabilise
unsigned long currentTime2;        // variable for timer Relay1
unsigned long currentTime3;        // variable for timer Relay2

bool RELAYState=0;                 // variable relay 1
bool RELAYState2=0;                // variable relay 2

void setup() 
{
  Serial.begin(9600);
  pinMode(PIN_R1, OUTPUT);          // declare pin 22 as output:
  pinMode(PIN_R2, OUTPUT);          // declare pin 23 as output:

 Serial.println("Temperature");      // display of temperature measurement, wait for MAX chip to stabilize  
                                  
  if (millis() - currentTime > 5000)   
  {
    currentTime = millis();
  }	

}

void loop() {
                                       // basic test, print the current temp

  Serial.print("C = ");
  Serial.println(thermocouple.readCelsius());
  
  if(millis() - currentTime2 > 5000)   //  Checking the time for the relay 1 
  {
    currentTime2 = millis();           //  Change the state of the relay1 to the opposite
	  RELAYState=!RELAYState
	  digitalWrite(PIN_R1, RELAYState);
   
  } 

  if(millis() - currentTime3 > 8000)   //  Checking the time for the relay 2 
  {
    currentTime3 = millis();           //  Change the state of the relay2 to the opposite
	  RELAYState2=!RELAYState2
	  digitalWrite(PIN_R2, RELAYState2);
    
  }   
	                                   // if temprature goes above 480.0C, turn the relay OFF
  if (thermocouple.readCelsius() < 480.00) 
  {
      digitalWrite(PIN_R1, HIGH);          // turn the Relay1 Switch ON 
	    digitalWrite(PIN_R2, HIGH);          // turn the Relay2 Switch ON 
  } else { 
      digitalWrite(PIN_R1, LOW);           // turn the Relay1 Switch OFF
      digitalWrite(PIN_R2, LOW);           // turn the Relay2 Switch OFF 	  
  }
     
}

повідомлення про помилку-

E:ArduinoModul___TC_R millisModul___TempControlModul___TempControl.ino: In function 'void loop()':
Modul___TempControl:52:4: error: expected ';' before 'digitalWrite'
    digitalWrite(PIN_R1, RELAYState);
    ^~~~~~~~~~~~
Modul___TempControl:60:4: error: expected ';' before 'digitalWrite'
    digitalWrite(PIN_R2, RELAYState2);
    ^~~~~~~~~~~~
exit status 1
expected ';' before 'digitalWrite'

Дякую