nav[aria-label="Primary Navigation"] { padding: 0; & ul { list-style: none; width: 100%; display: flex; flex-direction: row; justify-content: start; align-items: start; gap: 30px; padding: 0; & li { margin: 0; } & ul li { list-style: none; } } }

Help with an if statement

mrosenthal

Hi,

I am probably missing something basic here. I am trying to change a column's values by using a combination of two other column values. I type in the following if statement and the column values do not change. What am I missing here? Thanks for any tips and advice.

Names Default To Here( 1 );
dt = Data Table( "20210811Miflezet_CAPCMP.jmp" );
dt << Begin Data Update;
If( :justice_decision_REC_RILE2 == "reject" & :RILE_Calc2a == "center"
,
	 :Rile_Calc3 == "center"
	 ,
	:justice_decision_REC_RILE2 == "accept" & :RILE_Calc2a == "center"
	,
	 :Rile_Calc3 == "center"
	 ,
	:justice_decision_REC_RILE2 == "reject" & :RILE_Calc2a == "left"
	,
	 :Rile_Calc3 == "right"
	 ,
	:justice_decision_REC_RILE2 == "accept" & :RILE_Calc2a == "left"
	,
	:Rile_Calc3 == "left"
	,
	:justice_decision_REC_RILE2 == "reject" & :RILE_Calc2a == "right"
	,
	:Rile_Calc3 == "left"
	,
	:justice_decision_REC_RILE2 == "accept" & :RILE_Calc2a == "right"
	,
	:Rile_Calc3 == "right"
);
dt << End Data Update;

Accepted answers

Craige_Hales

In your original JSL you probably ( edit: also ) want to change

Rile_Calc3 == "center"&nbsp;//&nbsp;compare.&nbsp;the&nbsp;result&nbsp;of&nbsp;the&nbsp;compare (0 or 1)&nbsp;will&nbsp;be&nbsp;the&nbsp;result&nbsp;of&nbsp;the&nbsp;if(),&nbsp;see&nbsp;below

Rile_Calc3 = "center"&nbsp;//&nbsp;assign&nbsp;value

JSL if statements return a value; you could write it as

:Rile_Calc3 = If( 
   :justice_decision_REC_RILE2 == "reject" & :RILE_Calc2a == "center",  "center",
   :justice_decision_REC_RILE2 == "accept" & :RILE_Calc2a == "center",  "center",
   :justice_decision_REC_RILE2 == "reject" & :RILE_Calc2a == "left",  "right",
   :justice_decision_REC_RILE2 == "accept" & :RILE_Calc2a == "left", "left",
   :justice_decision_REC_RILE2 == "reject" & :RILE_Calc2a == "right", "left",
   :justice_decision_REC_RILE2 == "accept" & :RILE_Calc2a == "right", "right",
   :Rile_Calc3 // or "Unknown" ? depends how you want to handle the unexpected case.
);

If you know there are no unexpected cases, you can omit the final otherwise clause. But the nature of unexpected cases is to show up when least expected.

If Secrets

All comments

jthi

You could use either For Each Row or Set Each Value. Below is an example:

Names Default To Here(1);
dt = Open("$SAMPLE_DATA/Big Class.jmp");
For Each Row(
	If(:sex == "F",
		:age = .
	);
);

dt:weight << Set Each Value(
	If(:sex == "M",
		.,
		:height
	);
);

mrosenthal

Thanks for the answer! As the various scripting guides show, it should be the right thing yet it did not work here. I assume something was wrong with the data table and all the processing I already did on these variables. Eventually, the workaround was to concatenate the two nominal columns to one and then recode the expressions I got there to the desired outcomes. Thanks again.

Craige_Hales

In your original JSL you probably ( edit: also ) want to change

Rile_Calc3 == "center"&nbsp;//&nbsp;compare.&nbsp;the&nbsp;result&nbsp;of&nbsp;the&nbsp;compare (0 or 1)&nbsp;will&nbsp;be&nbsp;the&nbsp;result&nbsp;of&nbsp;the&nbsp;if(),&nbsp;see&nbsp;below

Rile_Calc3 = "center"&nbsp;//&nbsp;assign&nbsp;value

JSL if statements return a value; you could write it as

:Rile_Calc3 = If( 
   :justice_decision_REC_RILE2 == "reject" & :RILE_Calc2a == "center",  "center",
   :justice_decision_REC_RILE2 == "accept" & :RILE_Calc2a == "center",  "center",
   :justice_decision_REC_RILE2 == "reject" & :RILE_Calc2a == "left",  "right",
   :justice_decision_REC_RILE2 == "accept" & :RILE_Calc2a == "left", "left",
   :justice_decision_REC_RILE2 == "reject" & :RILE_Calc2a == "right", "left",
   :justice_decision_REC_RILE2 == "accept" & :RILE_Calc2a == "right", "right",
   :Rile_Calc3 // or "Unknown" ? depends how you want to handle the unexpected case.
);

If you know there are no unexpected cases, you can omit the final otherwise clause. But the nature of unexpected cases is to show up when least expected.

If Secrets

mrosenthal

That actually worked. I did one small tweak using @jthi 's post and did small changes in @Craige_Hales's code. The final format that worked for me was this:

dt = Data Table( "20210812Miflezet_CAPCMP.jmp" );
For Each Row(
 If( 
  :just_dec_rec == "reject" & :RILE_Rec == "center",  :RILE4="center",
   :just_dec_rec == "accept" & :RILE_Rec == "center", :RILE4= "center",
   :just_dec_rec == "reject" & :RILE_Rec == "left", :RILE4= "right",
   :just_dec_rec == "accept" & :RILE_Rec == "left", :RILE4="left",
   :just_dec_rec == "reject" & :RILE_Rec == "right", :RILE4="left",
   :just_dec_rec == "accept" & :RILE_Rec == "right", :RILE4="right"
));