Rasberry Pi Controlling Arduino - Reads "Ghost" Input - Doesn't Complete Logic Loop

Question

I am using an Arduino Atmega2560 clone, I have used it many times before with satisfactory results - although I haven't used it with my Rpi before.

My Raspberry Pi is model 4 (B I think?).

#define knap 49
#define motorOutput 8

void setup() {
  // put your setup code here, to run once:
  Serial.begin(9600);
  pinMode(knap, INPUT);
  pinMode(motorOutput, OUTPUT);
  digitalWrite(motorOutput, LOW);
}

void loop() {
  // put your main code here, to run repeatedly:
  int knapSensor = digitalRead(knap);
  Serial.print(knapSensor);
  if (knapSensor == 1) {
    int knapSensor = digitalRead(knap);
    Serial.print("aktiveret");
    digitalWrite(motorOutput, HIGH);
    delay(500);
    digitalWrite(motorOutput, LOW);
  }
}

Now I have double checked, the motor and the button is wired correctly. There is no hardware issue, aside from the cables being sh- quality.

Problem is, whenever I run this program two things will happen;

Either it starts fine by displaying "0000..." on the serial monitor.

Then when I press the button it goes to "AktiveretAktiveretAktiveret..." even when I depress the button.

Or, it will just display "AktiveretAktiveretAktiveret...".

In either case the motor won't work.

Once again, I have done tests where I have removed the power from the breadboard and replaced the voltage pin for the motor with the voltage pin leading from the button.

First thing didn't change the input (i.e still "AktiveretAktiveret,") the latter started the motor.

I.e this isn't about bad wiring.

EDIT:

Added schematics and changed the code

Answer 1

Your button needs a pulldown resistor to work reliably.

See the discussion and solution here

The resistor, with a value of around 10K (but any value between 3K and 20K will probably do), needs to be between the input pin or your Arduino and ground to give it a defined value when the button is released.

Putting a resistor between the Arduino pin and ground will make sure the pin reads "low" when no button is pressed. When the button is pressed, it will be connected directly to +5V and will read high. Only a small current (0.5mA) will flow from +5V to ground through the 10K Ohm resistor in that case. It will not create a short circuit.

Answer 2

What motor are you using? If it works straight from the 5v but not from a GPIO, it could be an issue of insufficient power.

As @StarCat4 mentioned, you also need to add a digitalRead inside your while loop if you want the motor to ever turn off.