проблемы с кодированием

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

Начнем с самого начала. Вы определили interruptpinnumber, но не используете его в функции attachInterrupt, поскольку объявляете вывод прерывания 1 (цифровой вывод 3), а не 0 в качестве вывода прерывания. вызвать проблемы в вашем коде.

Далее, ваши временные переменные должны быть unsigned long, поскольку функция millis() вернет unsigned long. Также нет необходимости объявлять их изменчивыми, поскольку вы не используете их в процедуре ISR.

Что касается ISR, вы использовали arduino attachInterrupt() и использовали фактический ISR() подпрограмма из библиотек AVR, также без включая это. В ISR вам нужно передать ему вектор, которого нет и который позже будет давать ошибки. Вам также не нужна процедура ISR, если вы использовали функцию Arduino. Вам необходимо убедиться, что то, что вы объявляете как подпрограмму (то есть isr_rpm) в функции attachInterrupt(), которая будет выполняться при возникновении прерывания, действительно существует.

Кроме того, при вычислении прошедшего времени возникают некоторые базовые арифметические проблемы, например, если вы пытаетесь определить разницу.

Как отметил @Majko, и я тоже собирался это сделать, использование побитового AND, & в ваших операторах if() может не быть проблема, но лучше всего сейчас усвоить, что для сравнения следует использовать логическое И &&.

Существует также избыточный оператор if, который я оставлю вам, чтобы найти его, поскольку это ваш код, и вы сможете определить, полезен он или нет. Также вам необходимо следить за своими скобками, так как это тоже приведет к ошибкам, которые вы видите.

Изменить

Думаю, мне нужно уточнить некоторые моменты, так как я думаю, что вы еще больше запутались.

Что касается всей функции Arduino AttachInterrupt, isr_rpm() должен быть функцией, а не определением, я думаю, это и послужило причиной

В ISR вам нужно передать вектор, которого там нет и который в дальнейшем будет давать ошибки. Вам также не нужна процедура ISR, если вы использовали функцию Arduino. Вам необходимо убедиться, что то, что вы объявляете как подпрограмму (то есть isr_rpm) в функции attachInterrupt(), которая будет выполняться при возникновении прерывания, действительно существует.

Этой функцией будет:

void isr_rpm(){

   //код

}

Поскольку @Majko любезно оказал вам услугу и аннотировал ваш код, посмотрите на материалы, связанные с процедурой прерывания (и другие материалы), и сравните их с приведенным ниже руководством по avrfreaks и оригиналом. код, если он у вас есть, проблема заключалась в том, что у вас было слишком много фигурных скобок:

void ISR(){
      buttonState = LOW;
      counter1 = counter1 + 1;
      previousState = HIGH;
    }
    }

должно быть:

ISR(INT0_vect){
      buttonState = LOW;
      counter1 = counter1 + 1;
      previousState = HIGH;
    }

Нет необходимости давать ISR собственное специальное имя, оно взято из более нового кода.

Учебное пособие AVRfreaks, касающееся прерываний и ISR, которое оно даст вам тоже объяснение по поводу нестабильных вещей.

Я не могу больше подчеркнуть этот момент, но просмотрите свой код, аннотированный Маженко, и посмотрите, где вы можете его исправить.

Хорошо, я поклялся, что не буду этого делать, но я переформатировал ваш последний код и собираюсь просмотреть его и аннотировать.

// Включая библиотеки
#include <Wire.h>
#include <LiquidCrystal_I2C.h>
#include <math.h>  /// << ПОЧЕМУ?! Вы им не пользуетесь!

LiquidCrystal_I2C mydisplay(0x27, 16, 2);

// Определить контакты ввода/вывода
#define solenoidpin 9
#define potpin 1
#define ignitioninputpin 2
///#определить прерывание номер 3
#define isr_rpm(INT0_vect)  /// << Чего это надеется достичь?

// Определение переменных
int previousstate;
byte buttonstate;
int counter1;
int start;  ///
int finish; /// << Они должны быть беззнаковыми длинными
int elapsed;///
int RPM;

// Основная настройка
void setup() {
    pinMode(ignitioninputpin, INPUT);
    digitalWrite(ignitioninputpin, HIGH);
    //// AttachInterrupt(3,isr_rpm, FALLING); /// << Это никогда не будет работать с тем #define, на который я указал...
    counter1 = 1;
    mydisplay.init();
    mydisplay.backlight();
    mydisplay.setCursor(0, 1);
    mydisplay.print(" Soleniod Motor ");
    previousstate = HIGH;
    Serial.begin(19200);
    buttonstate = digitalRead(ignitioninputpin);
}

// Основной цикл программы
void loop() {
    buttonstate = digitalRead(ignitioninputpin);
    Serial.println(buttonstate);
    Serial.write("previousstate");
    Serial.println(previousstate);
    Serial.println(counter1);
    float elapsed = start = finish;  /// << Вы уже определили их как целые числа в глобальном
    float RPM = (6000 / elapsed);    /// << объем. Почему вы снова определяете их как числа с плавающей запятой?! Также,
                                     /// << прошедшее = начало = окончание?!?! Я думаю, у вас есть = вместо -
    { /// << Зачем вы тут новый блок начинаете???
        if (buttonstate == LOW && previousstate == HIGH && (counter1 % 2) == 0) {  /// << counter1 % 2 очень неэффективен, поскольку счетчик counter1 подписан.
            digitalWrite(solenoidpin, HIGH);
        }

        if (buttonstate == HIGH && previousstate == LOW && (counter1 % 2) == 0) { /// << предыдущее состояние никогда не меняется с ВЫСОКОГО — это НИКОГДА не запустится!
            digitalWrite(solenoidpin, HIGH);
        }

        if (buttonstate == LOW && previousstate == HIGH && (counter1 % 2) == 0) {  /// << Вы уже проверили это состояние выше с другим результатом. ПОЧЕМУ?!?!
            digitalWrite(solenoidpin, LOW);
        }

        if ((counter1 % 2) == 0) {
            for rpm calcs /// << Это даже не Си!!!
            finish = start;

        start = millis();
        }

        if ((counter1 % 6) == 0) {
            mydisplay.setCursor(0, 0);
            mydisplay.print(" RPM ");
            mydisplay.print(RPM);
            mydisplay.setCursor(0, 1);
            mydisplay.print(" Solenoid Motor ");
        }

        void ISR1(); /// << Это прототип функции, а не ее определение.
        buttonstate = LOW;       /// << Я думаю, они должны быть внутри
        counter1 = counter1 + 1; /// << функция под названием ISR1(), но это
        previousstate = HIGH;    /// << функция не была написана.
    }
}

Вы пытаетесь возиться с написанием процедур обработки прерываний низкого уровня, не имея ни малейшего понятия, как работать с процедурами обработки прерываний низкого уровня. Вернитесь и прочитайте руководства о том, как выполнять внешние прерывания с помощью Arduino. Это очень-очень просто.

В вашей программе есть вещи, которые заставляют меня задуматься: «Что?!?!» и вещи, которых просто не хватает.