Input wanted

I have ordered and am still waitin for my arduino to come in, and I am new to programming so ignore my ignorance... But I am piecing together some code to spin a motor thru an l293 motor driver with a slight off pulse and serial printing the rpms with an optical sensor. Heres the code. Right now I am just trying to learn, so the code and idea may be out there.

int pin1 = 3;
int pin2 = 4;
int pin3 = 5;
int mc1 = 6;
int mc2 = 7;
int mc3 = 8;
int mc4 = 9;
int mce = 10;


int ledPin = 13;                // IR LED connected to digital pin 13
int statusPin = 12;             // LED connected to digital pin 12
volatile byte rpmcount;
volatile int status;
unsigned int rpm;
unsigned long timeold;

void rpm_fun()
{
   //Each rotation, this interrupt function is run twice, so take that into consideration for
   //calculating RPM
   //Update count
      rpmcount++;

   //Toggle status LED
   if (status == LOW) {
     status = HIGH;
   } else {
     status = LOW;
   }
   digitalWrite(statusPin, status);
 }
void setup(){
  
  Serial.begin(9600);
   //Interrupt 0 is digital pin 2, so that is where the IR detector is connected
   //Triggers on FALLING (change from HIGH to LOW)
   attachInterrupt(0, rpm_fun, FALLING);
   //Turn on IR LED
   pinMode(ledPin, OUTPUT);
   digitalWrite(ledPin, HIGH);
   //Use statusPin to flash along with interrupts
   pinMode(statusPin, OUTPUT);

   rpmcount = 0;
   rpm = 0;
   timeold = 0;
   status = LOW;
  
  pinMode(pin1,INPUT);
  pinMode(pin2,INPUT);
  pinMode(pin3,INPUT);
  pinMode(mc1,OUTPUT);
  pinMode(mc2,OUTPUT);
  pinMode(mc3,OUTPUT);
  pinMode(mc4,OUTPUT);
  pinMode(mce,OUTPUT);
  
  
  digitalWrite(mce, HIGH);
}

void loop() { 
 if(pin1,HIGH) {
   digitalWrite(mc1,HIGH),(mc2,HIGH),delayMicroseconds(50);}
else {
   digitalWrite(mc1,HIGH),(mc2,LOW);
 }
 if(pin2,HIGH) {
   digitalWrite(mc2,HIGH),(mc3,HIGH),delayMicroseconds(50);}
 else {
   digitalWrite(mc2,HIGH),(mc3,LOW);
 }
  if(pin3,HIGH) {
   digitalWrite(mc3,HIGH),(mc4,HIGH),delayMicroseconds(50);}
  else {
   digitalWrite(mc3,HIGH),(mc4,LOW);
  }
 //Update RPM every second
   delay(1000);
   //Don't process interrupts during calculations
   detachInterrupt(0);
   //use two markers to count
   rpm = 30*1000/(millis() - timeold)*rpmcount;
   timeold = millis();
   rpmcount = 0;

   //Write it out to serial port
   Serial.println(rpm,DEC);

   //Restart the interrupt processing
   attachInterrupt(0, rpm_fun, FALLING);
}

if(pin1,HIGH) {

Is perfectly valid C, but will always be true.

Did you intend a "digitalRead" in there?

int pin1 = 3;

"pin1" of what?