Am I doing this right? (component programming)

component palletchange "Rotates pallets all day";
pin in bit tableIsUp;        //HW pin
pin in bit tableIsDown;        //HW pin
pin in bit tableIsCW;        //HW pin
pin in bit tableIsCCW;        //HW pin
pin in bit tableReady;        //SW pin
pin in bit tableRotate;        //SW pin
pin in bit enable;            //SW pin - Net to "machine-is-enabled"

pin out bit tableUp;
pin out bit tableDown;
pin out bit tableRotateCW;
pin out bit tableRotateCCW;
pin out bit tableAirblast;
pin out bit startHydPump;
pin out bit rotateDone;                            //1 if rotation completed successfully
pin out bit tableInMachine;                        //Return which  table is under the spindle

license "GPL";
author "D.E";


function _ nofp;
;;

bool tableIsReady;                                //Set 1 if tableReady button is pressed
FUNCTION(_) {
    if(enable){                                    
//-------------Check state of tableReady button-------------------------------
        if(tableReady && !tableIsReady){
            tableIsReady = 1;                    //Enable tableIsReady (physical buttonpress)
            while(tableReady){}                    //Halt program from continuing while button is pressed
        }
        else if(tableReady && tableIsReady){
            tableIsReady = 0;                    //Disable tableIsReady (physical button was pressed again)
            while(tableReady){}                    //Halt program from continuing while button is pressed
        }    
        
//-------------Send error message and halt program----------------------------    
        if(tableRotate && !tableReady){            //If a rotation is requested but table is not set to ready
            rtapi_print_msg(RTAPI_MSG_ERR, "Pallet not ready!");
            //exit program somehow?
        }
        
//-------------Everything is ok, rotate pallet--------------------------------            
        if(tableRotate){                        //Table rotate pin is high
            startHydPump =1;                    //Start hyd pump
            tableUp =1;                            //Lift table
            tableAirblast =1;                    //Turn on table airblast
            while(!tableIsUp){}                    //Do nothing while table is lifting (should have timeout..)
            if(tableIsCW){
                tableRotateCCW =1;                //Rotate table CCW
                while(!tableIsCCW){}            //Wait
                for(int i=0; i<1000; i++){}        //Wait 1 sec to let table reach rotation stop
                tableRotateCCW =0;                //Turn off output
                
            }
            else if{tableIsCCW){
                tableRotateCW =1;                //Rotate table CCW
                while(!tableIsCW){}                //Wait
                for(int i=0; i<1000; i++){}        //Wait 1 sec to let table reach rotation stop
                tableRotateCW =0;                //Turn off output    
            }
            tableUp =0;                            //Turn off output
            tableDown =1;                        //Lower table
            while(!tableIsDown){}                //Do nothing while table is lowering (should have timeout..)
            for(int i=0; i<1000; i++){}            //Wait 1 sec to let table settle
            tableDown =0;                        //Turn off output
            startHydPump =0;                    //Turn off output
            tableAirblast =0;                    //Turn off output
            tableIsReady = 0;                    //Set table to not ready
        }
    }
}

variable double timer;
variable float on-delay = 1.0;
//--------------------------------

//--------------------------------
     timer += fperiod;

     if(timer >= on_delay) {
        //Do something now
        timer = 0.0;
    }