Einfache if-Abfrage

Hallo erstmals

und zwar ich habe eine einfache If Abfrage die nicht ganz so funktioniert wie sie soll.

In der Variable StandortErde steht ob ein Eingang HIGH oder LOW ist.

Wenn der Befehl "stepper.step(1);" nicht in der If Schleife steht funktioniert er.
Wenn der Eingang auf HIGH ist passiert garnichts.
Doch beim Wechsel von HIGH auf LOW macht der Schrittmotor 1 Schritt.

Die Else Verwzeigung ist nur dazu da den Schrittmotor abzuschalten.
Diese funktioniert einwandfrei.

if (StandortErde == HIGH){
stepper.step(1);
delay(100);
zaehler = zaehler + 1;
}
else{
digitalWrite(8,LOW);
digitalWrite(9,LOW);
digitalWrite(10,LOW);
digitalWrite(11,LOW);
zaehler = 0;
}

Bitte den gesamten Code, und zwar in Code-Tags (#-Zeichen). :wink:

Also der Motor hängt an pin 8,9,10,11

und der Taster an Pin 4

#include <Time.h>
#include <DCF77.h>
#include <Utils.h>
#include <Stepper.h>

// change this to the number of steps on your motor


// create an instance of the stepper class, specifying
// the number of steps of the motor and the pins it's
// attached to
Stepper stepper(100, 8, 9, 10, 11);

boolean debug = true;
int serspeed = 9600 ;
int StandortErde = 0;
int zaehler = 0;


#define DCF_PIN 2	         // Connection pin to DCF 77 device
#define DCF_INTERRUPT 0		 // Interrupt number associated with pin

time_t time;
DCF77 DCF = DCF77(DCF_PIN,DCF_INTERRUPT);


void setup()
{

  Serial.begin(serspeed);
  stepper.setSpeed(1);

  if(debug)
  {
    Serial.println("Debugging aktiviert");
  }
  DCF.Start();
}


void loop(){

  time_t DCFtime = DCF.getTime(); // Check if new DCF77 time is available

    if ((DCFtime!=0) && (debug = true))
  {
    Serial.println("Time is updated");
    setTime(DCFtime);
  }	
  digitalClockDisplay();  



  if(debug = true){

    //  Serial.print("Zähler: ");
    //Serial.println(zaehler);
    Serial.print("Standort: "); 
    Serial.println(digitalRead(4));  
  }


  StandortErde = digitalRead(4);   


  if (StandortErde == HIGH){
    stepper.setSpeed(1); 
    stepper.step(1); 
    zaehler = zaehler + 1;
  }
  else{ 
    digitalWrite(8,LOW);
    digitalWrite(9,LOW);
    digitalWrite(10,LOW);
    digitalWrite(11,LOW);
    zaehler = 0;
  }

  delay(1000);  


}


void digitalClockDisplay(){
  // digital clock display of the time
  Serial.print(hour());
  printDigits(minute());
  printDigits(second());
  Serial.print(" ");
  Serial.print(day());
  Serial.print(" ");
  Serial.print(month());
  Serial.print(" ");
  Serial.print(year()); 
  Serial.println(); 
}

void printDigits(int digits){
  // utility function for digital clock display: prints preceding colon and leading 0
  Serial.print(":");
  if(digits < 10)
    Serial.print('0');
  Serial.print(digits);
}

Ich stelle mal einfach einpinMode(4, INPUT); in den Raum, besser noch in die setup(). :smiley:

[edit] Pull-Down-Widerstand nicht vergessen!

Ich habe einen Pull up Widerstand verwendet.

Funktioniert auch.

stock95:
Ich habe einen Pull up Widerstand verwendet.

Funktioniert auch.

Wenn du in der if-Anweisung auf HIGH prüfst, der Pin aber per Pullup permanent auf HIGH gezogen wird, so läuft dein Motor die ganze Zeit. Oder hast du deinen Code auf die Abfrage nach LOW-Pegel umgestellt? Interner oder externer Pullup verwendet?

Oh entschuldige ich habe mich verschaut.

Es ist ein externer Pulldown Widerstand

Edit:

Mir ist aufgefallen wenn ich den Taster auslasse dass der Schrittmotor 1 Schritt macht und danach wieder aufhört.

  delay(1000);

Wenn Du Deinen Taster länger als ein Sekunde drückst, wird sich der Motor langsam bewegen.

Nimm den delay() raus und der Taster funktioniert wahrscheinlich wie gewünscht.

Nein tut er nicht.

ich habe ihn schon länger als 10 Sekunden gedrückt

Du hast doch eine Menge Debug-Ausgaben eingebaut, deren Ergebnis wir aber nicht kennen.
Hängt das ganze evtl. für 2 Minuten in   DCF.getTime();  bis die DCF-Uhr ein Signal hat?

Du solltest einen guten Wissens-Vorsprung haben gegen unsere Rate-Versuche.

Übrigens, nur am Rande:

  if(debug = true){

sorgt dafür, dass debug eingeschaltet wird. Aber das willst du ja sowieso :wink:

kleiner Tipp, schalte doch einfach erstmal alle anderen Fehlerquellen aus und lass nur das Abfragen Deines Tasters und den "if-Block" in der loop(). Dann wirst Du sehen, ob Du bei gedrücktem Taster jede Sekunde einen Step bekommst.
Im übrigen ist es sicher nicht notwendig jede Sekunde die DCF-Zeit zu prüfen.

Nein es verhält sich genauso wie vorher.

wie sieht der abgespeckte Test-Sketch denn nun aus?

#include <Stepper.h>

Stepper stepper(100, 8, 9, 10, 11);


void setup(){
    stepper.setSpeed(1);
  
}


void loop(){
  
   if (digitalRead(4) == HIGH){
    stepper.setSpeed(1); 
    stepper.step(1); 
  }
  else{ 
    digitalWrite(8,LOW);
    digitalWrite(9,LOW);
    digitalWrite(10,LOW);
    digitalWrite(11,LOW);
    
  }

  delay(1000); 
  
  
  
  
  
  
  
  
  
  
}
  delay(1000);

Nimm das delay() raus, das erschwert Dir nur die Arbeit beim Debuggen, da Du immer eine Sekunde auf jeden Step warten musst. Dein Motor macht eine hundertstel Umdrehung pro Sekunden, Du wirst also mit dem delay() kaum eine Aktivität registrieren, selbst wenn sie stattfindet.

immer noch das gleiche verhalten

Dann vereinfachen wir mal radikal:

#include <Stepper.h>

Stepper stepper(100, 8, 9, 10, 11);


void setup(){
    stepper.setSpeed(1);
  
}

void loop(){
  stepper.step(1);
}

Dreht sich der Motor mit diesem Sketch?

Eine Frage. Die Pins 8,9,10 und 11 sind ja die Anschlüsse für den Stepper.
Ich verstehe, das der Stepper einen Schritt machen soll. Aber warum setzt Du die Pins 8,9,10 und 11 in jedem anderen Fall auf LOW? Damit pfuscht Du im schlimmsten Fall der Stepper-Lib ins Handwerk.
Also, welchen Sinn hat der else-Zweig in deinem if-Konstrukt?

Ansonsten schliesse ich mich erstmal "pylon" an. Funktioniert denn der Stepper überhaupt wie gewünscht?

Ja der Sketch von pylon funktioniert..

Ich muss die ausgänge auf low setzen damit ich den schrittmotor abschalten kann, da er sonst heiß wird.

Dann haben wir hier Testcode für den Taster. Funktioniert dieser auch?

void setup() {
  Serial.begin(9600);
}


void loop() {
  if (digitalRead(4) == HIGH) {
    Serial.println("Taster gedrückt");
  }
}