using interrupts

int coinPulseCount;
int newCoinInserted;
int coinPulse;
byte cmd = 0;
int opCountPin = 3;
volatile unsigned long pulseTime;





void setup() {
  // put your setup code here, to run once:
Serial.begin(9600);
Serial.print("INPUT on PIN");
Serial.print(opCountPin);
Serial.println();
pinMode(opCountPin, INPUT);
attachInterrupt(1, coinpulse, RISING);
}

void loop() {
  // put your main code here, to run repeatedly:
if(coinPulse >0 && millis()-pulseTime > 200)
{
  newCoinInserted = coinPulseCount ;
  coinPulseCount = 0;
}
switch(newCoinInserted)
{
  case 1:
 Serial.print("1 rupee");
 newCoinInserted = 0;
 break;
 
 case 2:
 Serial.print("2 rupee");
 newCoinInserted = 0;
 break;

 case 3:
 Serial.print("3 is inserted");
 newCoinInserted = 0;
 break;

 case 4:
 Serial.print("4 rupee inserted");
 newCoinInserted = 0;
 break;

 case 5:
 Serial.print("5 rupee inserted");
 newCoinInserted = 0;
 break;

 case 6:
 Serial.print("6 rupee inserted");
 newCoinInserted = 0;
 break;
}
}

void coinpulse()
{
    coinPulseCount++;
    pulseTime = millis();
}

can someone please look at the code and check if i have anything wrong