Your code can be made more readable by consolidating the multiple function calls: if ( index(lowcase(strip(objet)),'bolivie') or index(lowcase(strip(objet)),'chypre') or index(lowcase(strip(objet)),'cyprus') or index(lowcase(strip(objet)),'équateur') or index(lowcase(strip(objet)),'equateur') or index(lowcase(strip(objet)),'éthiopie') or index(lowcase(strip(objet)),'ethiopie') or index(lowcase(strip(objet)),'indo') or index(lowcase(strip(objet)),'kenya') or index(lowcase(strip(objet)),'nigeria') or index(lowcase(strip(objet)),'nigéria') or index(lowcase(strip(objet)),'pakistan') or index(lowcase(strip(objet)),'sao') or index(lowcase(strip(objet)),'sri lanka') or index(lowcase(strip(objet)),'tanzanie') or index(lowcase(strip(objet)),'thai') or index(lowcase(strip(objet)),'tailand') or index(lowcase(strip(objet)),'turquie') or index(lowcase(strip(objet)),'viet') or index(lowcase(strip(objet)),'yemen') or index(lowcase(strip(objet)),'yémen') ) then do; if index(lowcase(strip(objet)),'acc') then do; count_gafi_acc=count_gafi_acc+1; switch = "y"; end; else if index(lowcase(strip(objet)),'refus') then do; count_gafi_refus=count_gafi_refus+1; switch = "y"; end; end; Ron Fehd logical simplicity: know redundancy maven
... View more