Target practise game

Projects General Guidance
1

I'm looking for some advise in how to improve my program to make it more playable and add some additional features. Basically there are 6 potetiomitors connected to pivoting targets which are flicked back up by six servos once they are knocked down. All Ive managed so far is a buzzer and a count.
Id really like to include a display which i have but haven't got a clue where to start with it. It has the module soldered onto the back and it connects easily in the icc port on the shield I am using. Any help would be greatly appreciated.

Here's my code so far...

#include <Servo.h>

Servo myservo;
Servo myservo1;
Servo myservo2;
Servo myservo3;
Servo myservo4;
Servo myservo5;

int potpin = 0;
int potpin1 = 1;
int potpin2 = 2;
int potpin3 = 3;
int potpin4 = 4;
int potpin5 = 5;

int val;
int val1;
int val2;
int val3;
int val4;
int val5;

const int buzzer = (10);
int count = 0;

void setup() {
myservo.attach(9);
myservo1.attach(8);
myservo2.attach(7);
myservo3.attach(6);
myservo4.attach(5);
myservo5.attach(4);
pinMode(buzzer, OUTPUT);
Serial.begin(9600);

}

void loop() {
Serial.println(count);
//TARGET1//////////////////////////////////
val = analogRead(potpin);
if (val > 500){
count++;
tone(buzzer, 1000);
delay(250);
noTone(buzzer);
delay(100);
myservo.write(0);
delay(300);
myservo.write(60);
delay(15);
}
else
myservo.write(60);
delay(10);
//TARGET2///////////////////////////////////
val1 = analogRead(potpin1);
if (val1 > 450){
count++;
tone(buzzer, 1000);
delay(250);
noTone(buzzer);
delay(100);//
myservo1.write(0);
delay(300);
myservo1.write(60);
delay(15);
}
else
myservo1.write(60);
delay(10);
//TARGET3/////////////////////////////////////
val2 = analogRead(potpin2);
if (val2 > 500){
count++;
tone(buzzer, 1000);
delay(250);
noTone(buzzer);
delay(100);//
myservo2.write(35);
delay(300);
myservo2.write(90);
delay(15);
}
else
myservo2.write(90);
delay(10);
//TARGET4////////////////////////////////////
val3 = analogRead(potpin3);
if (val3 > 600){
count++;
tone(buzzer, 1000);
delay(250);
noTone(buzzer);
delay(100);//
myservo3.write(35);
delay(300);
myservo3.write(90);
delay(15);
}
else
myservo3.write(90);
delay(10);
//TARGET5//////////////////////////////////////
val4 = analogRead(potpin4);
if (val4 < 450){
count++;
tone(buzzer, 1000);
delay(250);
noTone(buzzer);
delay(100);//
myservo4.write(0);
delay(300);
myservo4.write(90);
delay(15);
}
else
myservo4.write(90);
delay(10);
//TARGET6//////////////////////////////////////
val5 = analogRead(potpin5);
if (val5 < 550){
count++;
tone(buzzer, 1000);
delay(250);
noTone(buzzer);
delay(100);//
myservo5.write(0);
delay(300);
myservo5.write(90);
delay(15);
}
else
myservo5.write(90);
delay(10);

}

3

+1 on the use of arrays.

Also, when you are coding, occasionally use CTRL-T. This auto-formats the code making it much easier to read.
Also, when you post code, use code tags as described in the sticky post, How to use this forum - please read.

4

You may also want to try and get rid of those delay() calls, so you can handle multiple targets being hit at the same time (or in very quick succession).
That's definitely going to be a bit harder than the array part - and you'll need some more arrays for that.