O.K. i'm building a robot for school with my arduino leonard and I need some help with my sketch...

What do you need help with?

Whenever I put my hand in front of the IR sensor the servos move but I didn't mean for it to do that. Since I am not that advanced i'm trying to make it go forward until it sees something, turn right, and if it can go there it goes. If not it turns left and tries to go that way. If it cant do that i simply pick it up and move it.tototo30

it would probably be helpful if i knew what was happening in each line so i can see whats happening in each step

One rather troubling part of your code is this

void loop()

{

  cont1.write(135);

  cont2.write(45);

  delay(500);

it is part of the main loop and has no control around it. This means it will be executed every time through the loop no matter the other conditions

and this part

  if(analogRead(sensePin)>tenin)

  {

    cont1.write(45);

    cont2.write(135);

  }

will be immediately over ridden by the above code as there is no delay or any other control code to prevent it therefor making it redundant.

for good form (Not sure if yours will compile or if it is just a typo) you should not do this

else(analogRead(sensePin)<twenin);

but instead should write else if(analogRead(sensePin)<twenin);

As Patrick stated, if you can add comments for what the variables are supposed to be (I am assuming sensing an object less or greater in distance from a specified amount but that's guessing) but comment in plain English rather than electrically

So if you agree, apply the changes needed to address the comments I made, add some comments and re post your observations, well go from there

Peter

What do you mean "happening in each line" sorry if that's something really basic but I'm kind of a newbie

So did you mean to take those out or edit those because here is the edited version:

#include<Servo.h>

Servo cont1;

Servo cont2;

//Servo head;

int sensePin = A0;

int tenin = 230;

int twenin = 100;

void setup()

{

  cont1.attach(10);

  cont2.attach(11);

  //head.write(90);

  //head.attach(9);

  //pinMode(9 OUTPUT);

}

void loop()

{

  cont1.write(135);

  cont2.write(45);

  delay(500);

  if(analogRead(sensePin)<tenin)

  {

    cont1.write(92);

    cont2.write(92);

    delay(5000);

    cont1.write(45);

    cont2.write(135);

    delay(500);

    //head.write(90);

  }

  //head.write(0);

  else(analogRead(sensePin)<twenin);

    {

      cont1.write(180);

      cont2.write(180);

      delay(500);

    }

  if(analogRead(sensePin)>tenin)

  {

    cont1.write(45);

    cont2.write(135);

  }

}

if you meant to edit it let me know because all it does right now is continually turn right :/Robert Peter Oakes

the logic I can understand but does not make sense.

What is (In plane English, not code) the logic your trying to code.

what is the purpose of the two variables

int tenin = 230;

int twenin = 100;

and when reading the sensor I am assuming your getting an analog signal from something, what is the something as most sensors I have seen that measure distance are ultrasonic and return a pulse you have to time?

what servo does what, I am assuming one is speed and one is steering, if not then what?

Peter

I am using an infrared distance sensor and i get numbers for it instead of a time. What i'm trying make it do is: Go forward until it sees something.

It turns right and gets a reading.

If it can go right, it goes right.

If you can help me with that it would be great.

Patrick