HI,

I am wondering if my approach on programming a trigger and calling a class is the best approach.  This is what I am coding.

I created a class (not static).  In the class I have public member variables that are set from the trigger that intantiates the class.  These variables hold things like Trigger.New, isUpdate, isInsert, etc.  In the trigger I use IF logic to detect the type of trigger even and then instantiate the class and set the member variables accordingly.  Below is an example.  What would your feedback, best practices say about this approach?  Am I on target or are there flaws with this approach and if so what do you suggest?  Thank you in advance for your assistance.

trigger Contact_trigger on Contact ( after insert, after delete, after update, before insert, before update, before delete ) {   
    EmployerGroupPortal egp = new EmployerGroupPortal();
    egp.isInsert = Trigger.isInsert;            
    egp.isUpdate = Trigger.isUpdate;
    egp.isDelete = Trigger.isDelete;
    egp.oldMap = Trigger.oldMap;
    if ( Trigger.isBefore ) {
        if ( Trigger.isInsert || Trigger.isUpdate ) {            
            egp.contacts = Trigger.new;
        }  
        if ( Trigger.isDelete ) { 
            egp.contacts = Trigger.old;
        }
        egp.egpValidation();
    }
}