As much as I see you are repeating same lines few times except the COMPUTE argument. you can define:       %macro comp(arg);                 compute &arg;                      If _name_='Application'   then call define(_col_,'format','comma20.0');               If _name_='CreditAR'   then call define(_col_,'format','percent8.1');               if _name_= 'JudgmentalAR' then call define(_col_,'format','percent8.1');               if _name_= 'Referral % ' then call define(_col_,'format','percent8.1');           endcomp;      %mend common_lines;   Then change your code to:          Proc report data=final  contents="   " nowd split='\'             .... enter your code here upto ...                   %comp(year2015);           %comp(year2016);           %comp( Num_of_Variance);           %comp(mature2015);           %comp( Num_of_Variance2);         RUN;            ... View more

1)   In your code you wrote:   call symput('read'||left(count),file_name); call symput('dset'||left(count),substr(file_name,118,7));   it means that your macro variables will be:     &read1 , &read2 , ...     &dset1 , &dset2 , ...   Yout wrote:    %put &read;    %put &dset but there are no macro variables &read &dset  but &read1 &dset1 etc.   2) there is no error in:  proc import datafile="&&read&i"   I do see error in:   out=%scan(&dset,&i) dbms=dlm replace; I suppose you meant to write:  out=%scan(&&dset.&i) dbms=dlm replace;          ... View more