Using Rotary Encoder with Arduino

you need to specify the type of currentStateCLK before using it. something like:

Int currentStateCLK;

Thank you.

But it is defined:

// Include the Servo Library
#include <Servo.h>

// Rotary Encoder Inputs
#define inputCLK 4
#define inputDT 5

// Create a Servo object
Servo myservo;

int counter = 0;
int currentStateCLK;
int previousStateCLK;

void setup() {

// Set encoder pins as inputs
pinMode (inputCLK,INPUT);
pinMode (inputDT,INPUT);

// Setup Serial Monitor
Serial.begin (9600);

// Attach servo on pin 9 to the servo object
myservo.attach(9);

// Read the initial state of inputCLK
// Assign to previousStateCLK variable
previousStateCLK = digitalRead(inputCLK);

}

Any other idea?

Thanks.

It is probably a typo before the 'if' statement. I am just guessing, as I do not see the relevant code.

This is the whole code.

I don't see the typo, but maybe you can discover something.

Thank you for your time and help.

/*Rotary Encoder with Servo Motor Demo
rot-encode-servo-demo.ino
Demonstrates operation of Rotary Encoder
Positions Servo Motor
Displays results on Serial Monitor
DroneBot Workshop 2019
https://dronebotworkshop.com
*/

// Include the Servo Library
#include <Servo.h>

// Rotary Encoder Inputs
#define inputCLK 4
#define inputDT 5

// Create a Servo object
Servo myservo;

int counter = 0;
int currentStateCLK;
int previousStateCLK;

void setup() {

// Set encoder pins as inputs
pinMode (inputCLK,INPUT);
pinMode (inputDT,INPUT);

// Setup Serial Monitor
Serial.begin (9600);

// Attach servo on pin 9 to the servo object
myservo.attach(9);

// Read the initial state of inputCLK
// Assign to previousStateCLK variable
previousStateCLK = digitalRead(inputCLK);

}

void loop()

// Read the current state of inputCLK
currentStateCLK = digitalRead(inputCLK);

// If the previous and the current state of the inputCLK are different then a pulse has occured
if (currentStateCLK != previousStateCLK){

// If the inputDT state is different than the inputCLK state then
// the encoder is rotating counterclockwise
if (digitalRead(inputDT) != currentStateCLK) {
counter --;
if (counter<0){
counter=0;
}

} else {
// Encoder is rotating clockwise
counter ++;
if (counter>180){
counter=180;
}

}

// Move the servo
myservo.write(counter);

Serial.print("Position: ");
Serial.println(counter);
}
// Update previousStateCLK with the current state
previousStateCLK = currentStateCLK;
}

OK 

Right after 'void loop() there should be the opening brace, like this '{'

Just add it on after void loop() and  it will compile!

That's it.

Thank you so much.

Now it works.

I'm happy.

Yvonne

glad to have helped.

I need some help with the basic encoder code got it working with one encoder but when I try to add a second encoder I can't get it to work.