my apex class :
public class copycontact {
    public list<selectoption>account1{set;get;}
    public list<selectoption>account2{set;get;}
    public list<contact>oldcontacts{set;get;}
    public list<opportunity>oldopportinities {set;get;}
    public list<lead>oldleads {set;get;}
    public id selectedaccountId1 {set;get;}
    public id selectedaccountId2 {set;get;}
    public copycontact(ApexPages.StandardController stdController){
        
    }
    public List<SelectOption> getListOfAccounts1()
    {
        List<Account> AccountList1 = [select id,Name from Account] ;
        System.debug('Accounts'+AccountList1.size());
        account1 = new List<SelectOption>();
        account1.add(new SelectOption( ' ' ,'---Select---'));
        for(Account acc : AccountList1 )
        {
            account1.add(new SelectOption(acc.id , acc.Name));
        }
        return account1 ;         
    }
    public List<SelectOption> getListOfAccounts2()
    {
        List<Account> AccountList2 = [select id,Name from Account] ;
        System.debug('Accounts'+AccountList2.size());
        account2 = new List<SelectOption>();
        account2.add(new SelectOption( ' ' ,'---Select---'));
        for(Account acc : AccountList2 )
        {
            account2.add(new SelectOption(acc.id , acc.Name));  
        }
        system.debug('java');
        return account2 ;  
    } 
    public  void clonecontacts(){
        system.debug('gsvdd');
        if(selectedaccountId1!=null){
            oldcontacts=new list<contact>([select id,lastname from contact where accountid=:selectedaccountId1]);
            
            if(oldcontacts.size()==0){
                system.debug('please select create contacts'   );
                ApexPages.addMessage(new ApexPages.Message(ApexPages.Severity.Error,'No contacts to display'));
            }
            else{
                system.debug('oldcontacts'+oldcontacts.size());
                list<contact> newcontacts=new list<contact>();
                list<opportunity> newopportunities=new list<opportunity>();
                set<id> oldcontactid=new set<id>();
                map<id,id> newmap=new map<id,id>();
                for(Contact con:oldcontacts){
                    contact c=new contact();
                    c.lastname=con.LastName;
                    c.AccountId=selectedaccountId2;
                    newcontacts.add(c);
                    oldcontactid.add(con.id);
                }
                insert newcontacts;
                system.debug('newcontacts'+newcontacts.size());
                ApexPages.addMessage(new ApexPages.Message(ApexPages.Severity.CONFIRM,'successfully contacts are copied'));
                for(contact c1:oldcontacts){
                    for(contact c2:newcontacts){
                        if(c1.lastname==c2.LastName){
                            newmap.put(c1.id,c2.id); 
                            system.debug('created map');
                        }
                    }
                }
                
                oldopportinities=[select id,name,copycontact__c,closedate,stagename from opportunity where copyContact__c=:oldcontactid];
                system.debug('oldopportunities'+oldopportinities.size());
                list<lead> newleads=new list<lead>();
                set<id> oldleadids=new set<id>();
                map<id,id> newleadmap=new map<id,id>();
                for(opportunity o:oldopportinities){
                    opportunity op=new opportunity();
                    op.Name=o.name;
                    op.CloseDate=o.CloseDate;
                    op.StageName=o.StageName;
                    op.copyContact__c=newmap.get(o.copyContact__c);
                    newopportunities.add(op);
                    oldleadids.add(o.Id);    
                }
                insert newopportunities;
                system.debug('newopportunity'+newopportunities.size());
                system.debug('oldleadsids'+oldleadids.size());
                ApexPages.addMessage(new ApexPages.Message(ApexPages.Severity.CONFIRM,'successfully opportunities are copied'));
                for(opportunity o1:oldopportinities){
                    for(opportunity o2:newopportunities){
                        if(o2.name==o1.name){
                            newleadmap.put(o1.id,o2.id);
                        }
                    }
                }
                oldleads=[select id,lastname,Company,status,Opportunityleads__c from lead where Opportunityleads__c=:oldleadids];
                system.debug('oldleads'+oldleads.size());
                if(oldleads.size()==0){
                    ApexPages.addMessage(new ApexPages.Message(ApexPages.Severity.error,'no opportunites '));
                }
                else{
                    for(lead le:oldleads){
                        lead l=new lead();
                        l.lastName=le.lastname;
                        l.Company=le.company;
                        l.Status=le.status;
                        l.Opportunityleads__c=newleadmap.get(le.Opportunityleads__c);
                        newleads.add(l);
                    }
                    insert newleads;
                    system.debug('newleads'+newleads.size());
                    ApexPages.addMessage(new ApexPages.Message(ApexPages.Severity.CONFIRM,'successfully leads are copied'));
                }
            }
        }
    }
}

I have a button that I have incorporated some validation rule into for fields that need to be filled in. I am having issue with the fields that are picklists. Below is the code I have but the error I am getting is:<span class="errorStyle">Error: Syntax error. Missing '}'</span>

{!REQUIRESCRIPT("/soap/ajax/42.0/connection.js")} 

var isFilled = {!NOT(ISBLANK( Contact.Start_Date__c  ))};
var isFilled = {!NOT(ISBLANK( Contact.Escort_Day_1__c ))};
var isFilled = {!TEXT( Contact.Preferred_Computer__c  <>  '' ))};
var isFilled = {!TEXT( Contact.Office_Location__c  <>  ''  ))};
var isFilled = {!TEXT( Contact.Riskonnect_Department__c  <>  ''  ))};
var isFilled = {!TEXT( Contact.Accounting_Department__c  <>  '' ))};

if( isFilled){var contactRec = new sforce.SObject( "Contact" );
var records = sforce.connection.query("SELECT Id, Onboarding_Process_Builder_Trigger__c FROM Contact Where id = '{!Contact.Id}'");
var contactRec = records.getArray('records')[0];  



    contactRec.Onboarding_Process_Builder_Trigger__c = {!Contact.Onboarding_Process_Builder_Trigger__c} + 1; 

 sforce.connection.update([contactRec]); 
    window.location.reload();}

else {alert( "Employment Section is required." );}