How to design custom action method in Selenium Grid architecture - selenium

Question

I have a very basic question related to how to design method for executing selenium GRID.
In the current implementation of selenium framework in my project, we have created an action class which includes all selenium WebElelement actions in a static format.
For sequential script execution, there is no issue. But for parallel script execution, I heard that we can't design a method as static as only one copy will be created. Then, how to write custom action method which we can use in other classes.
Could you please advise on this.
Current Implementation:
public class ActionUtil{
public static void selectByVisibleText(WebElement element, String visibleText, String elementName)
{
try {
Select oSelect = new Select(element);
oSelect.selectByVisibleText(text);
log.info(text + " text is selected on " + elementName);
} catch (Exception e) {
log.error("selectByVisibleText action failed.Exception occured :" + e.toString());
}
}
}
Use of 'selectByVisibleText' static method in other page classes:
public void selectMemorableQuestion1(String question) {
ActionUtil.selectByVisibleText(memorableQuestion1, question, "memorableQuestion1");
}

Answer 1

If You're trying to make parallel test run, and use methods in that way avoid static methods.
You need to add the synchronized modifier if you are working with objects that require concurrent access.
You may have concurrency issue (and being in non thread-safe situation) as soon as the code of your static method modify static variables.
So bottom line use synchronized modifier, avoid using static modifier due to thread-safe issues.
public class ActionUtil{
public synchronized void selectByVisibleText(WebElement element, String visibleText, String elementName)
{
try {
Select oSelect = new Select(element);
oSelect.selectByVisibleText(text);
log.info(text + " text is selected on " + elementName);
} catch (Exception e) {
log.error("selectByVisibleText action failed.Exception occured :" + e.toString());
}
}
so call would be:
ActionUtil.selectByVisibleText(...);