topic Re: Arrays in SAS Procedures

Arrays

Andrew6 — Sun, 18 Feb 2018 22:12:42 GMT

I am trying to use an array to clean my data, I have tried a few different things and nothing seems to really change what I am seeing, either in the data or the log.

     array optionschar {4}  $ gender
                       Marital_Status
                       Game_Attendance
                       games_attended;
	        do i = 1 to 10;
                       if optionschar{i} = "Male" then optionschar{i} = "M";
                       if optionschar{i} = "m" then optionschar{i} = "M";
                       if optionschar{i} = "Female" then optionschar{i} = "F";
                       if optionschar{i} = "Mar" then optionschar{i} = "M";
                       if optionschar{i} = "Married" then optionschar{i} = "M";
                       if optionschar{i} = "Single" then optionschar{i} = "S";
                       if optionschar{i} = "999" then optionschar{i} = ".";
                       if optionschar{i} = "Four" then optionschar{i} = "4";
        end;
        drop i; 
keep gender Marital_Status Game_Attendance games_attended;
run;

Re: Arrays

novinosrin — Sun, 18 Feb 2018 22:28:51 GMT

why are you looping 10 times when there are only 4 elements in your array. Shouldnt it be

do i=1 to 4;

Re: Arrays

PaigeMiller — Sun, 18 Feb 2018 22:43:47 GMT

This doesn't even seem like a situation where arrays are useful.

You are performing different tasks on each variable in the array. Arrays are useful when you are performing the same task on each variable in the array.

Re: Arrays

novinosrin — Sun, 18 Feb 2018 22:47:33 GMT

use proc format and put function

Re: Arrays

Andrew6 — Sun, 18 Feb 2018 23:02:19 GMT

Thank you both for the insight.

Re: Arrays

Reeza — Mon, 19 Feb 2018 02:43:29 GMT

Check multiple conditions at once using IN instead of the multiple IF statements:

if optionschar{i} = "Male" then optionschar{i} = "M";
if optionschar{i} = "m" then optionschar{i} = "M"

is the same as:

  if optionschar{i} in ("Male", 'm') then optionschar{i} = "M";

And another suggestion, make everything upper case in your comparison to avoid having to check for m, M, Male and male.

if upcase(gender) in ('M', 'MALE') then gender='M';