Arduino Uno & Stepper-Shield

[Michael H. (m0ddmylife)]

Hallo. Habe ein klitzekleines Problem mit meiner Programmierung.

Die Digitaleingänge werden nicht abgefragt. zumindest wird nix 
angenommen von denen.

wenn mir jemand einen "wink mit dem Zaunpfahl" geben könnte währ ich 
sehr dankbar.

/*Schrittmotorsteuerung für eine Barndormontierung

   Frequenz einstellbar und über SerialPort veränderbar
   
   by Michael H
   
   version 0.8b  
*/
boolean dirPinx     = 3;
boolean stepperPinx = 2;
boolean dirPiny     = 7;
boolean stepperPiny = 6;
boolean x_ein_dir   = 10; //Motor X Richtung
boolean en          = 12; //Motorfreigabe
boolean x_4         = 11; //Motor X 4Hz
int x_sensor        = A0; //X-Speed Poti
int x_sensor_val    = 0;  //X-Speed Speicher
boolean x           = 0;
int d_x             = true;
boolean enable      = false;
boolean dir_old     = false;
int serInp          = 0;

void setup() 
{
  Serial.begin(9600);
  pinMode(dirPinx, OUTPUT);
  pinMode(stepperPinx, OUTPUT);
  pinMode(dirPiny, OUTPUT);
  pinMode(stepperPiny, OUTPUT);
  pinMode(x_ein_dir, INPUT);
  pinMode(en, INPUT);
  pinMode(x_4, INPUT);

//interne Pullups:
    digitalWrite(x_ein_dir, HIGH);
    digitalWrite(en, HIGH);        
    digitalWrite(x_4, HIGH);
    
// Y-Achse auf 0
    digitalWrite(dirPiny, LOW);
    digitalWrite(stepperPiny, LOW);
}


void step(boolean dir)
{
  
  if(dir != dir_old)
  {
   digitalWrite(dirPinx,dir);
   delay(50);
  }

    digitalWrite(stepperPinx, HIGH);
    delayMicroseconds(x_sensor_val);
    digitalWrite(stepperPinx, LOW);
    delayMicroseconds(x_sensor_val);
    dir_old= dir;
    return;
}

void loop(){
  
  x = digitalRead(x_4);
  
    if (Serial.available()>0)
     { serInp = Serial.read();
       Serial.flush();
     }
 
    if( x = LOW )
     {
      x_sensor_val = 125;
     }
    else x_sensor_val = serInp;

  enable = digitalRead(en);

    if (enable = true)
     {
      d_x = digitalRead(x_ein_dir);
       if(d_x = true)
       {
         step(true);  
       }
       else
       {
         step(false);
       }
   }


}

mfg
Michael

[Electronics'nStuff (Gast)]

Die Schalter sind aktiv-low?

[EGS (Gast)]

Electronics'nStuff schrieb:
> Die Schalter sind aktiv-low?

Wie kommst du darauf? Wenn du wegen:

digitalWrite(x_ein_dir, HIGH);

das schliesst, nein. der Befehl aktiviert den internen Pullup an dem 
Pin.

[EGS (Gast)]

Michael Hakenkötter schrieb:
> if( x = LOW )

könnte dein Fehler sein, hab ich in nem anderen Post leider auch 
vergessen zu schreiben, denn:

Vergleich auf Wert:

 x == y (x is equal to y)

Wert setzen auf Y

 x = Y

Du fragst die Werte nicht du setzt diese in deinem If-Anweisungen

MfG EGS

[Michael H. (m0ddmylife)]

Wow danke! ich hab mir einen Wolf gesucht!!!

manchmal übersieht man die einfachsten dinge ;)

Schönes Wochenende euch !

Michael

[Karl H. (kbuchegg)]

Michael Hakenkötter schrieb:
> Wow danke! ich hab mir einen Wolf gesucht!!!
>
> manchmal übersieht man die einfachsten dinge ;)

Es ist eigentlich ganz einfach.
Spar dir solche Sachen

    if (enable == true)

und dann kannst du da auch keinen Fehler machen.
Wenn enable sowieso schon ein boolscher Wert ist, dann brauchst du den 
nicht mit true zu vergleichen. Denn da kann nur dann als Ergebnis des 
ganzen Ausdrucks true rauskommen, wenn enable auch schon true war

    if (enable)

macht also ganz genau das gleiche. Nur ohne Potential für Tippfehler.

Leider weiß ich nicht, wie HIGH und LOW beim Arduino definiert ist. Aber 
wenn die Italiener nicht ganz bescheuert waren, dann haben sie LOW als 0 
und HIGH als 1 definiert. Wodurch man anstatt

    if( x == HIGH )

genausogut

    if( x )

schreiben kann, und anstelle von

    if( x = LOW )

kann man

    if( !x )

schreiben und hat wieder die Tippfehler-Gefahr gebannt.
Nennt man die Variable dann nicht einfach nur x, sondern gibt ihr einen 
sprechenden Namen, dann wird das ganze meistens sogar noch 
übersichtlicher als mit den ganzen Vergleichen.

[Michael H. (m0ddmylife)]

Ok danke für deinen tipp! ich werd es mal ausprobieren :)