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;     }

switch (state){ ... case 11: if (something){ state = 12; // waiting state timer = 10000; // how long to wait } break; case 12: timer -= fperiod; if (timer > 0) break; // Here do things that are meant to happed when the timer expires. // Should generally only be a change of state, just to keep things // obvious to future-you