Convert continuous numbers to 0 and 1

[Silver77]

This is supposed to convert a number of hours exercised to high and low (1 and 0) but it is not working, what am I doing wrong? BTW this was the exact code given by my professor 

data classdata; 
set classdata; 
if exercise>3.25 then highex=1; 
else if exercise>=0 and exercise<3.25 then highex=0; 
run;

[ballardw]

Doesn't work is awful vague.

Are there errors in the log?: Post the code and log in a code box opened with the "</>" to maintain formatting of error messages.

No output? Post any log in a code box.

Unexpected output? Provide input data in the form of data step code pasted into a code box, the actual results and the expected results. Instructions here: https://communities.sas.com/t5/SAS-Communities-Library/How-to-create-a-data-step-version-of-your-dat... will show how to turn an existing SAS data set into data step code that can be pasted into a forum code box using the "</>" icon or attached as text to show exactly what you have and that we can test code against.

Note: Use of the code construct

data classdata;
   set classdata;

completely replaces the data set on the SET statement. You may have corrupted your data and may have to start from scratch depending on what you have done since the first time you created the Classdata data set.

Especially when testing you should create new data sets so you can recover easily from a logic problem.

If you had run test code prior creating a variable Highex then you may be seeing values from that.

Note: Your values have a "hole" at exactly equal to 3.25 as you are comparing "greater than 3.25" and "less than 3.25". But you do not have any place to handle values of 3.25 if they occur.

[yabwon]

The only thing I could add to @ballardw comment is: strange works for me...

code:

data classdata;
  exercise=.; output;
  exercise=-3; output;
  exercise=0; output;
  exercise=0.25; output;
  exercise=3.2; output;
  exercise=3.25; output;
  exercise=3.5; output;
run;

data classdata; 
set classdata; 
if exercise>3.25 then highex=1; 
else if exercise>=0 and exercise<3.25 then highex=0; 
run;

result:

B-)

[Patrick]

@Silver77 You should always try and formulate if/then/else constructs exhaustive and exclusive.

Exhaustive: Covers all cases. There is always one condition that gets True.

Exclusive: No overlaps. Exactly one condition gets True.

The syntax you shared is not exhaustive. You don't cover cases where Excercise is 3.25, missing or below zero.