I'm trying to create a etch a sketch-esque mouse with one rotary encoder for the x axis and another for the y axis. I've seen it be done whit potentiometers but I cant find anything about it being done with rotary encoders.
My problem is that I'm pretty new at this and I'm not that good at it yet...
I found this code for reading the encoder, http://www.kevindarrah.com/download/arduino_code/Encoder.pde . But how can I implement this code in to a mouse to get it to move? Any tips?
But how can I implement this code in to a mouse to get it to move? Any tips?
On which Arduino? What have YOU tried?
Keep in mind that the Mouse library only deals with relative movement of the mouse. And only for certain Arduinos.
Leonardo
This is where I am at the moment
#include <Mouse.h>
//////////////
int val;
int encoder0PinA = 3;
int encoder0PinB = 4;
int encoder0Pos = 0;
int encoder0PinALast = LOW;
int n = LOW;
void setup(){
pinMode(1, INPUT); //mouse.begin and LMB
pinMode(2, INPUT); // axis
pinMode(5, INPUT); // mouse movment x1/x10
//////////////
pinMode (encoder0PinA,INPUT);
pinMode (encoder0PinB,INPUT);
Serial.begin (9600);
}
void loop(){
//Mus av/på
if(digitalRead (1) == HIGH) {
Mouse.begin();
}
if(digitalRead (1) == LOW) {
Mouse.end();
}
if (digitalRead(1) == HIGH) {
Mouse.press(MOUSE_LEFT);
}
if (digitalRead(1) == LOW) {
Mouse.release(MOUSE_LEFT);
}
n = digitalRead(encoder0PinA);
if ((encoder0PinALast == LOW) && (n == HIGH)) {
if (digitalRead(encoder0PinB) == LOW) {
encoder0Pos--;
} else {
encoder0Pos++;
}
}
encoder0PinALast = n;
if (digitalRead(5) == HIGH); {
if (digitalRead(2) == HIGH);{
if (encoder0Pos--); {
Mouse.move(+1,0,0);
}
if (encoder0Pos--); {
Mouse.move(-1, 0,0);
}
}
if (digitalRead(2) == LOW);{
if (encoder0Pos--); {
Mouse.move(0,+1,0);
}
if (encoder0Pos--); {
Mouse.move(0, -1,0);
}
}
}
if (digitalRead(5) == LOW); {
if (digitalRead(2) == HIGH);{
if (encoder0Pos--); {
Mouse.move(+10,0,0);
}
if (encoder0Pos--); {
Mouse.move(-10, 0,0);
}
}
if (digitalRead(2) == LOW);{
if (encoder0Pos--); {
Mouse.move(0,+10,0);
}
if (encoder0Pos--); {
Mouse.move(0, -10,0);
}
}
}
}
I feel like I'm slowly moving forward with this project 
The new idea is to use this thing to drag stuff more precise on the screen and, well... Lets face it , I'm also gonna play a bunch of old games!
#include <Mouse.h>
// Encoder
int val;
int encoder0PinA = 3;
int encoder0PinB = 4;
int encoder0Pos = 0;
int encoder0PinALast = LOW;
int n = LOW;
void setup(){
pinMode(1, INPUT); // x
pinMode(2, INPUT); // y
pinMode(5, INPUT); // x1 x10 mouse cursour movement
// Encoder setup
pinMode (encoder0PinA,INPUT);
pinMode (encoder0PinB,INPUT);
Serial.begin (9600);
}
void loop(){
// x/y, lmb and mouse.begin
if(digitalRead(1) == HIGH && digitalRead(2) == LOW);{ // x axis, mouse.begin, mouse.press
Mouse.begin();
Mouse.press(MOUSE_LEFT);
}
if(digitalRead(1) == LOW && digitalRead(2) == HIGH);{ // y axis, mouse.begin, mouse.press
Mouse.begin();
Mouse.press(MOUSE_LEFT);
}
if(digitalRead(1) == LOW && digitalRead(2) == LOW);{ // off
Mouse.end();
Mouse.release(MOUSE_LEFT);
}
// Encoder
n = digitalRead(encoder0PinA);
if ((encoder0PinALast == LOW) && (n == HIGH)) {
if (digitalRead(encoder0PinB) == LOW) {
encoder0Pos--;
} else {
encoder0Pos++;
}
}
encoder0PinALast = n;
// Cursour movment
if (digitalRead(5) == HIGH); { // x1 cursour movement
if (digitalRead(1) == HIGH);{ // x+
if (encoder0Pos--); {
Mouse.move(+1,0,0);
}
if (encoder0Pos--); { // x-
Mouse.move(-1, 0,0);
}
}
if (digitalRead(2) == HIGH);{ // y+
if (encoder0Pos--); {
Mouse.move(0,+1,0);
}
if (encoder0Pos--); { // y-
Mouse.move(0, -1,0);
}
}
}
if (digitalRead(5) == LOW); { // x10 cursour movment
if (digitalRead(1) == HIGH);{ // x+
if (encoder0Pos--); {
Mouse.move(+10,0,0);
}
if (encoder0Pos--); { // x-
Mouse.move(-10, 0,0);
}
}
if (digitalRead(2) == HIGH);{ // y+
if (encoder0Pos--); {
Mouse.move(0,+10,0);
}
if (encoder0Pos--); { // y-
Mouse.move(0, -10,0);
}
}
}
}
if(digitalRead(1) == HIGH && digitalRead(2) == LOW);{ // x axis, mouse.begin, mouse.press
Properly formatting this statement shows:
if(digitalRead(1) == HIGH && digitalRead(2) == LOW)
; // Do nothing
{
// x axis, mouse.begin, mouse.press
Is that what you want?
It is generally not a good idea to use pin 1 (or 0) for digital input as it is used by the hardware serial interface which you may need to use for debugging to print to the Serial monitor.
UKHeliBob:
It is generally not a good idea to use pin 1 (or 0) for digital input as it is used by the hardware serial interface which you may need to use for debugging to print to the Serial monitor.
On the Leonardo?
On the Leonardo?
Leonardo. Well, yes, you have a point......
Of course, should you need to use Serial1 for something then it is not a good idea to use pins 0 and 1 on the Leonardo, but that is a little more obscure, I agree.
PaulS:
if(digitalRead(1) == HIGH && digitalRead(2) == LOW);{ // x axis, mouse.begin, mouse.pressProperly formatting this statement shows:
if(digitalRead(1) == HIGH && digitalRead(2) == LOW)
; // Do nothing
{
// x axis, mouse.begin, mouse.press
Is that what you want?
I dont understand, wont it do anything? 
I dont understand, wont it do anything?
The body of the if statement is the ;. That is a no-operation. It does nothing.
The useless if statement is then followed by a bunch of code in useless curly braces.
If you delete the ;, the story is completely different.
PaulS:
The body of the if statement is the ;. That is a no-operation. It does nothing.The useless if statement is then followed by a bunch of code in useless curly braces.
If you delete the ;, the story is completely different.
Oh, I see. So i should take the away all ; before { in the if statements like this?
if(digitalRead(1) == HIGH && digitalRead(2) == LOW){ // x axis, mouse.begin, mouse.press
Mouse.begin();
Mouse.press(MOUSE_LEFT);
UKHeliBob:
Leonardo. Well, yes, you have a point......Of course, should you need to use Serial1 for something then it is not a good idea to use pins 0 and 1 on the Leonardo, but that is a little more obscure, I agree.
I guess I can change it just in case i would need it in the future!
I also just found a litte little problem of me typing encoder0Pos-- insted of encoder0Pos++ on the x+ part 
Oh, I see. So i should take the away all ; before { in the if statements like this?
If you want the code to work, yes.
Fredemo:
Oh, I see. So i should take the away all ; before { in the if statements like this?if(digitalRead(1) == HIGH && digitalRead(2) == LOW){ // x axis, mouse.begin, mouse.press
Mouse.begin();
Mouse.press(MOUSE_LEFT);
If you want to make PaulS happy, you should do it like this:
if(digitalRead(1) == HIGH && digitalRead(2) == LOW)
{
// x axis, mouse.begin, mouse.press
Mouse.begin();
Mouse.press(MOUSE_LEFT);
// other conditional code
}
You get to make myself and other other programmers, with a good sense of style, happy at the same time.
It's also easy to find matching braces this way.
DuaneDegn:
If you want to make PaulS happy, you should do it like this:You get to make myself and other other programmers, with a good sense of style, happy at the same time.
It's also easy to find matching braces this way.
Got it!
#include <Mouse.h>
// Encoder
int val;
int encoder0PinA = 3;
int encoder0PinB = 4;
int encoder0Pos = 0;
int encoder0PinALast = LOW;
int n = LOW;
void setup(){
pinMode(5, INPUT); // x1 x10 mouse cursour movement
pinMode(6, INPUT); // x
pinMode(7, INPUT); // y
// Encoder setup
pinMode (encoder0PinA,INPUT);
pinMode (encoder0PinB,INPUT);
Serial.begin (9600);
}
void loop(){
// x/y, lmb and mouse.begin
if(digitalRead(6) == HIGH && digitalRead(7) == LOW)
{
// x axis, mouse.begin, mouse.press
Mouse.begin();
Mouse.press(MOUSE_LEFT);
}
if(digitalRead(6) == LOW && digitalRead(7) == HIGH)
{
// y axis, mouse.begin, mouse.press
Mouse.begin();
Mouse.press(MOUSE_LEFT);
}
if(digitalRead(6) == LOW && digitalRead(7) == LOW)
{
// off
Mouse.release(MOUSE_LEFT);
Mouse.end();
}
// Encoder
n = digitalRead(encoder0PinA);
if ((encoder0PinALast == LOW) && (n == HIGH))
{
if (digitalRead(encoder0PinB) == LOW)
{
encoder0Pos--;
}
else
{
encoder0Pos++;
}
}
encoder0PinALast = n;
// Cursour movment
if (digitalRead(5) == HIGH)
{
// x1 cursour movement
if (digitalRead(6) == HIGH)
{
// x+
if (encoder0Pos++)
{
Mouse.move(+1,0,0);
}
if (encoder0Pos--)
{
// x-
Mouse.move(-1, 0,0);
}
}
if (digitalRead(7) == HIGH)
{
// y+
if (encoder0Pos++)
{
Mouse.move(0,+1,0);
}
if (encoder0Pos--)
{
// y-
Mouse.move(0, -1,0);
}
}
}
if (digitalRead(5) == LOW)
{
// x10 cursour movment
if (digitalRead(6) == HIGH)
{
// x+
if (encoder0Pos++)
{
Mouse.move(+10,0,0);
}
if (encoder0Pos--)
{
// x-
Mouse.move(-10, 0,0);
}
}
if (digitalRead(7) == HIGH)
{
// y+
if (encoder0Pos++)
{
Mouse.move(0,+10,0);
}
if (encoder0Pos--)
{
// y-
Mouse.move(0, -10,0);
}
}
}
}
I've hopefully made it more pretty now, fixed my little encoder0Pos++ mistake and I'm no longer using pin 1 
I've tried to build it now and it doesn't work! The cursor just keep stuttering from left and right.
I used a light version of my code to test it out.
#include <Mouse.h>
// Encoder
int val;
int encoder0PinA = 3;
int encoder0PinB = 4;
int encoder0Pos = 0;
int encoder0PinALast = LOW;
int n = LOW;
void setup(){
pinMode(5, INPUT); // x1 x10 mouse cursour movement
pinMode(6, INPUT); // x
pinMode(7, INPUT); // y
// Encoder setup
pinMode (encoder0PinA,INPUT);
pinMode (encoder0PinB,INPUT);
Serial.begin (9600);
Mouse.begin();
}
void loop(){
// Encoder
n = digitalRead(encoder0PinA);
if ((encoder0PinALast == LOW) && (n == HIGH))
{
if (digitalRead(encoder0PinB) == LOW)
{
encoder0Pos--;
}
else
{
encoder0Pos++;
}
}
encoder0PinALast = n;
if (encoder0Pos--) {
Mouse.move (-10, 0, 0);
}
if (encoder0Pos++) {
Mouse.move (+10, 0, 0);
}
}
In some rare occasions when I turn the encoder it will move the way I turn it and after it goes back to stutter from left to right (about 1cm).
Is the code alright? Have I done something wrong with the hardware?
Edit: I can't see the picture I posted so I'm gonna post the link here http://postimg.org/image/va9rakg6r/.
if (encoder0Pos--) {
Mouse.move (-10, 0, 0);
}
if (encoder0Pos++) {
Mouse.move (+10, 0, 0);
}
What are you trying to do here? Unconditionally decrementing and incrementing encoder0Pos does not make sense to me.
PaulS:
What are you trying to do here? Unconditionally decrementing and incrementing encoder0Pos does not make sense to me.
I'm trying to get the mouse cursor to move as I turn the rotary encoder
Then, rather than incrementing and decrementing the encoder position, just move the mouse.
If that is too much trouble, you still need to make sure that the encoder position is positive before moving the mouse and decrementing, or is negative before moving the mouse and incrementing.
PaulS:
Then, rather than incrementing and decrementing the encoder position, just move the mouse.If that is too much trouble, you still need to make sure that the encoder position is positive before moving the mouse and decrementing, or is negative before moving the mouse and incrementing.
Do you got any examples or know where I can find out more about how I would achieve that? I've tried replacing the encoder0Pos with Mouse.move and some other stuff like a new encoder code.
Remember I'm new at this and I appreciate everything that help me learn more.
I finaly got it to work!