topic Re: My code doesn't want to work inside a DO loop- not accessing value? in SAS Programming

My code doesn't want to work inside a DO loop- not accessing value?

HB — Wed, 06 Sep 2017 21:25:41 GMT

Hey,

I have code like:

%let year = 15;

DATA my_new_file_&year;                           
   SET source_file&year;
calculate stuff;
make things;
run;

proc freq data = my_new_file_&year;
	tables variable;
	format;
	title "&year stuff";
run;

that works fine and is very useful

When i want to do 16 (or 17 or 18) I copy and past the block of code and make

%let year = 16;

or whatever year I want to run.

What I want to do is something like

%macro my_yearly_stuff; 
%do myyear = 15 %to 18;

%let year = %myyear
code block- all my working statements calling &year here

%end; 
%mend my_yearly_stuff;

Run all the statement in a loop that iterates from 15 to 18 and executes the code block for each year. But when I do that I get nothing. No error. No warning. Just nothing. The code is echoed in the log window but nothing happens.

What am i doing wrong? How do i access the value of myyear and get it into the code?

Re: My code doesn't want to work inside a DO loop- not accessing value?

art297 — Wed, 06 Sep 2017 21:39:14 GMT

Change the %myyear to &myyear and make sure to end the statement with a semi-colon

Art, CEO, AnalystFinder.com

Re: My code doesn't want to work inside a DO loop- not accessing value?

Reeza — Wed, 06 Sep 2017 22:26:46 GMT

%macro my_yearly_stuff; 
%do myyear = 15 %to 18;
%let year = %myyear -> not needed, use the myYear macro variable directly. 
code block- all my working statements calling &myYear here

%end; 
%mend my_yearly_stuff;

Note the small changes needed.

Re: My code doesn't want to work inside a DO loop- not accessing value?

Tom — Thu, 07 Sep 2017 04:45:51 GMT

If it does absolutely nothing then perhaps it is because you never called it?

Also you are referencing an undefined macro where I think you want to reference an macro variable.

If your going to the trouble of making a macro you might as well use some parameters.

%macro my_yearly_stuff(startyr,endyr); 
%local year;
%do year = &startyr %to &endyr;

... code block- all my working statements calling &year here

%end; 
%mend my_yearly_stuff;

%my_yearly_stuff(15,18);

Re: My code doesn't want to work inside a DO loop- not accessing value?

ShiroAmada — Thu, 07 Sep 2017 04:54:43 GMT

%macro my_yearly_stuff;
%do myyear = 15 %to 18;
*%let year = %myyear; *Not required (IMO);
code block- all my working statements calling &myyear. here

%end;
%mend my_yearly_stuff;

Re: My code doesn't want to work inside a DO loop- not accessing value?

HB — Thu, 07 Sep 2017 15:18:30 GMT

We have a winner.

I wasn't calling it.

I had

%mend my_yearly_stuff;

which is exactly what the example showed.

I changed that to

%mend;
%my_yearly_stuff;

and it runs.

Re: My code doesn't want to work inside a DO loop- not accessing value?

ballardw — Thu, 07 Sep 2017 15:37:18 GMT

The code

%mend my_yearly_stuff;

tells us two things, one it is the end of a macro definition and (if use properly) that the macro it is ending is my_yearly_stuff. If you have any text other than the macro it is ending you will get a warning in the log.

Better would be:

%mend my_yearly_stuff;

%my_yearly_stuff;