cancel
Showing results for
Did you mean:
Resolver I

## Risk matrix grid - can I simplify what I have created?

So, I have setup a matrix grid and though it works I feel there has to an easier way than what I've done.

So, each of my risks has a likelihood and impact score. These numbers then represent a grid position 11, 12, 13, 14, 15, 21,22....up to 55!

Each label box on my grid has its own variable which I assign when the screen is loaded.  Obviously this has to go through 75+ if statements even if a number a grid is matched (this being the need to possible having more than one value in a grid box).

Has anyone looked at this before or got a smarter method?

``````//Get the grid position based on their scores.
Set(varInherent, varRecord.'Inherent Risk Likelihood Score'.Value & varRecord.'Inherent Risk Impact Score'.Value);
Set(varResidual, varRecord.'Residual Risk Likelihood Score'.Value & varRecord.'Inherent Risk Impact Score'.Value);
Set(varTarget, varRecord.'Target Risk Likelihood Score'.Value & varRecord.'Target Risk Impact Score'.Value);

//If there is no score then set a variable to display notice box for no score set
If(varTarget = "" || varTarget = "00", Set(varNoTarget, true),  false);
If(varInherent = "" || varInherent = "00", Set(varNoInherent, true), false);
If(varResidual = "" || varResidual = "00", Set(varNoResidual, true), false);

//Work through each grid item and decide if it has a risk value.
Set(varGrid11, "");
If(varInherent = "11", Set(varGrid11, "I "),varGrid11);
If(varResidual = "11", Set(varGrid11, varGrid11 & "R "), varGrid11);
If(varTarget = "11", Set(varGrid11, varGrid11 & "T "), varGrid11);

Set(varGrid12, "");
If(varInherent = "12", Set(varGrid12,"I "),varGrid12);
If(varResidual = "12",Set(varGrid12,varGrid12 & "R "),varGrid12);
If(varTarget = "12",Set(varGrid12,varGrid12 & "T "),varGrid12);

Set(varGrid13,"");
If(varInherent = "13",Set(varGrid13,"I "),varGrid13);
If(varResidual = "13",Set(varGrid13,varGrid13 & "R "),varGrid13);
If(varTarget = "13",Set(varGrid13,varGrid13 & "T "),varGrid13);``````

1 ACCEPTED SOLUTION

Accepted Solutions
Super User III

Hi @Coopedup ,
First one can be improved as below - the rest would just be a little window dressing

``````Set(varNoTarget, varTarget = "" || varTarget = "00");
Set(varNoInherent, varInherent = "" || varInherent = "00");
Set(varNoResidual,  varResidual = "" || varResidual = "00");``````

Please click Accept as solution if my post helped you solve your issue. This will help others find it more readily. It also closes the item. If the content was useful in other ways, please consider giving it Thumbs Up.

2 REPLIES 2
Super User III

Hi @Coopedup ,
First one can be improved as below - the rest would just be a little window dressing

``````Set(varNoTarget, varTarget = "" || varTarget = "00");
Set(varNoInherent, varInherent = "" || varInherent = "00");
Set(varNoResidual,  varResidual = "" || varResidual = "00");``````

Please click Accept as solution if my post helped you solve your issue. This will help others find it more readily. It also closes the item. If the content was useful in other ways, please consider giving it Thumbs Up.

Super User III

Hi @Coopedup ,

Just checking if you got the result you were looking for on this thread. Happy to help further if not.

Please click Accept as solution if my post helped you solve your issue. This will help others find it more readily. It also closes the item. If the content was useful in other ways, please consider giving it Thumbs Up.

Announcements