OP's code properly displayed using the code button </> and indented for easier reading - and erroneous semi-colons removed
volatile int coins = 0; //counter pin from coin acceptor
int solenoidPin = 12; // output to solenoid1 pin for 1st solenoid
int solenoid2Pin = 13;
void setup()
{
Serial.begin(9600);
pinMode (2,INPUT_PULLUP); //2 is counter pin from coin acceptor
attachInterrupt(0, coinInserted, CHANGE); //coin counterr
pinMode(solenoidPin, OUTPUT); //sets 12th pin as output
pinMode(solenoidPin, OUTPUT);//11th output for sol2
}
void coinInserted()
{
coins=coins+1;
}
void loop()
{
Serial.print(coins)
if( coins == 2)
{
digitalWrite(solenoidPin, HIGH);
delay(1000);
digitalWrite(solenoidPin, LOW);
delay(1000);
coins = 0;
}
else if(coins == 5)
{
digitalWrite(solenoid2Pin, HIGH);
delay(1000);
digitalWrite(solenoid2Pin, LOW);
delay(1000);
}
}
...R