Getting Global Variable Value

[an93]

Hello,

I am having some trouble with getting the value of global variable being set, in functions. Below is the flow screenshot;

Function 4 goes:
global.set("TRIGGER",msg.payload);
var res = global.get('LENGTH');
var res1 = global.get('CABLE_DESCRIPTION');
var res2 = global.get('FAULT_STRING');
var res3 = global.get('TRIGGER')
if(res3===true){
msg.payload = "res"+"res1"+"res2";
}
else{
return msg[null];
}

Function 1 till 3 remain same with change in definition of global variable name.
global.set("LENGTH",msg.payload);
return msg;

What i am trying to achieve here if trigger goes high then capture the data and display it !!

Any help/recommendation would be appreciated !

[Paul-Reed]

@an93

var res = global.get('LENGTH');
var res1 = global.get('CABLE_DESCRIPTION');
var res2 = global.get('FAULT_STRING');
var res3 = global.get('TRIGGER')
if(res3===true){
msg.payload = "res"+"res1"+"res2";
}

You are trying to construct the msg.payload from strings, and not the variables that you created earlier...
example - res is a variable, "res" is a string

[Colin]

@an93

return msg[null];

What does that line do? I don't think it is valid javascript, but perhaps it is some subtle syntax that I have not met so far. I wonder whether you meant to say return null

[an93]

@Colin, yes that's a typo.

[an93]

@Paul-Reed, how do i call the global variable then,
supposedly saying
For Ex:
global.set('LENGTH',msg.payload);
return msg

Is it supposed to be
var res = global.get('LENGTH');

[Paul-Reed]

@an93

how do i call the global variable then,

You are calling the global variables correctly, but when you are trying to create the msg.payload you are referring to the variables as strings, because you have added speech marks around the variable's names.
Are res, res1 and res2 numbers because you seem to be adding (+) their values together.
If res, res1 and res2 are string values, then maybe && instead of +

[an93]

Yes res and res1 are numbers, res 3 is string, tried changing it but no luck yet

global.set('TRIGGER',msg.payload);

var res = global.get('LENGTH');
var res1 = global.get('CABLE_DESCRIPTION');
var res2 = global.get('FAULT_STRING');
var res3 = global.get('TRIGGER')
//var rest = msg.payload;

if(res3===false)
{
// msg.payload = "how are you";
msg.payload = res;
}
else{
return msg;
}

[Paul-Reed]

@an93

var res2 = global.get('FAULT_STRING');

Is res2 a string (it's called FAULT_STRING?) if so, it's like trying to add values like;

3 + 4 + donut = ???

[an93]

global.set('TRIGGER',msg.payload);

var res = global.get('LENGTH');
var res1 = global.get('CABLE_DESCRIPTION');
var res2 = global.get('FAULT_STRING');
var trig = global.get('TRIGGER')

if(trig===false)
{

msg.payload = res;
}
else{
return msg;
}

res is float which should "var" as datatype
res 1 and 2 are strings for which i get your point about my datatype being wrong

[Paul-Reed]

@an93

else{
return msg;
}

What is the msg - it's not defined.

Bedtime for me

[an93]

its supposed to be null if the condition does not meet

Bedtime @5 on friday. LAME !!!!!!

[Paul-Reed]

@an93

Bedtime @5 on friday. LAME !!!!!!

In UK - 23:10hrs....

[an93]

Alright in the mean time i will try >

[an93]

@Colin how can we return the message with combination of String and variable together ?
var res = global.get('LENGTH');
var res1 = global.get('CABLE_DESCRIPTION');
var res2 = global.get('FAULT_STRING');
var trig = global.get('TRIGGER')
if(trig===false)
{
msg.payload = res;
return msg;
}
else
{
return msg[null];
}

[knolleary]

Can you clarify exactly what you want the function to return? You are getting four values from context. Describe in words what you want the result to be.

[an93]

@knolleary
Got it, that helps. Thank you ! Reached my max for today but that helped.

[knolleary]

Right at the start of this thread, you had the code:

global.set("TRIGGER",msg.payload);
var res = global.get('LENGTH');
var res1 = global.get('CABLE_DESCRIPTION');
var res2 = global.get('FAULT_STRING');
var res3 = global.get('TRIGGER')
if(res3===true){
msg.payload = "res"+"res1"+"res2";
}
else{
return msg[null];
}

As Paul pointed out, the line:

msg.payload = "res"+"res1"+"res2";

Will result in msg.payload having the value "resres1res2". If you meant to join those three variables together, you should do:

msg.payload = res+res1+res2;

You will also need a return msg at the end so it returns the message.

@an93

The return should be message with individual values.

That doesn't help explain what format you want it to be in. I have assumed you want it to return a message with the three values joined together as a string.

If instead you want it to return a message with an object payload with the three separate values in it, you could do:

msg.payload = {
   res: res,
   res1: res1,
   res2: res2
}

You would then be able to reference the individual properties later in the flow as msg.payload.res and msg.payload.res1 etc.