Hello, The first parenthesis is not closed. I think you should remove the opening parentheis after the "or" : or index(lowcase(.... ... View more

Hello, Please define "does not work". Can you post an extract of the log ? After the loop you have a "do" that has no corresponding then or else. ... View more

@Shmuel 's answer.   Given that macro are not needed, there are several reasons your macro doesn't work :   - You use macro %if...%then...%else but should use regular if...then...else as the macro versions only make text comparisons. So the condition severity_&i = "WARNING"  will compare two strings (for &i.=2, for instance),  severity_2 (wo quotes) and "WARNING" (with quotes) and will never be true   - You repeat the test for each value of &i. so that only the last test will influence the final value of invalid_data. You should generate a series of tests separated by "or".   - You forgot a semi-colon after %then data_invalid = "Y"    Also, you can use a simple %do loop : %do i=1 %to 3; ... %end; ... View more

Hello,   As @Kurt_Bremser said, the macro language is not designed to manipulate data and perform calculations though you can use some data step functions through %sysfunc.   As shown by @PeterClemmensen's answer, this can lead to rather complex expressions (3 %sysfunc calls) thus making debugging more difficult.   Using data steps will provide more readable and reliable code.   If you need your value in the form of a macro variable, you still can export the result with a call symput (with maxim 28 in mind).   %let reporting_dt=20161231; data _NULL_; reporting_dt=input("&reporting_dt",yymmdd8.); reporting_dt_new=intnx('month', reporting_dt,12,'E'); call symputx("reporting_dt_new", reporting_dt_new); call symputx("reporting_dt_new_fmt", put(reporting_dt_new,yymmddn8.)); run; %put &=reporting_dt_new; %put &=reporting_dt_new_fmt; ... View more

Hello,   What is the following instruction used for ?   %let cntr = %sysfunc(compress(&cntry.));   If you want to remove leading and trailing blanks, the following will be enough : %let cntr=&cntry.; By using compress, you also remove blanks in the country name so "COSTA RICA", for instance, will become "COSTARICA".   Try the following code and look at the log : data _NULL_; call symput("cntry"," COSTA RICA "); run; %put ***&=cntry.***; %let cntr = %sysfunc(compress(&cntry.)); %put ***&=cntr.***; %let cntr2=&cntry.; %put ***&=cntr2.***; Note that it would not have been necessary if symputx had been used instead of symput. ... View more

Hello,   In this case, if first and last are kept character, SAS will do an implicit conversion and issue a note, so @Shmuel's code will also give the wanted result, though in a less "clean" manner.   85 data have; 86 first="0"; 87 last="10"; 88 do _n_=first to last; 89 put _n_=; 90 end; 91 run; NOTE: Character values have been converted to numeric values at the places given by: (Line):(Column). 88:12 88:21 _N_=0 _N_=1 _N_=2 _N_=3 _N_=4 _N_=5 _N_=6 _N_=7 _N_=8 _N_=9 _N_=10 NOTE: The data set WORK.HAVE has 1 observations and 2 variables. NOTE: DATA statement used (Total process time): real time 0.00 seconds cpu time 0.00 seconds ... View more